From e2016bc67d6814303616c43f5ac13c43f280787f Mon Sep 17 00:00:00 2001 From: Hypolite Petovan Date: Sat, 27 Nov 2021 17:05:58 -0500 Subject: [PATCH] Fix Photo delete endpoint - Set the request variable type - Fix the condition on a nonexistant item field --- src/Module/Api/Friendica/Photo/Delete.php | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/Module/Api/Friendica/Photo/Delete.php b/src/Module/Api/Friendica/Photo/Delete.php index 1857de5c71..d4288b88be 100644 --- a/src/Module/Api/Friendica/Photo/Delete.php +++ b/src/Module/Api/Friendica/Photo/Delete.php @@ -21,7 +21,6 @@ namespace Friendica\Module\Api\Friendica\Photo; -use Friendica\DI; use Friendica\Model\Item; use Friendica\Model\Photo; use Friendica\Module\BaseApi; @@ -35,11 +34,10 @@ class Delete extends BaseApi { protected function rawContent(array $request = []) { - self::checkAllowedScope(self::SCOPE_WRITE); $uid = self::getCurrentUserID(); $request = self::getRequest([ - 'photo_id' => null, // Photo id + 'photo_id' => '', // Photo id ], $request); // do several checks on input parameters @@ -60,7 +58,7 @@ class Delete extends BaseApi if ($result) { // function for setting the items to "deleted = 1" which ensures that comments, likes etc. are not shown anymore // to the user and the contacts of the users (drop_items() do all the necessary magic to avoid orphans in database and federate deletion) - $condition = ['uid' => $uid, 'resource-id' => $request['photo_id'], 'type' => 'photo']; + $condition = ['uid' => $uid, 'resource-id' => $request['photo_id'], 'post-type' => Item::PT_IMAGE, 'origin' => true]; Item::deleteForUser($condition, $uid); $result = ['result' => 'deleted', 'message' => 'photo with id `' . $request['photo_id'] . '` has been deleted from server.'];