delete all locks for post for delete by post id

ushahidi · Sep 19, 2024 · ace4450 · ace4450
1 parent 2f64bca
commit ace4450
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/Ushahidi/Modules/V5/Actions/Post/Handlers/UpdatePostLockCommandHandler.php b/src/Ushahidi/Modules/V5/Actions/Post/Handlers/UpdatePostLockCommandHandler.php
@@ -43,7 +43,7 @@ private function updatePostLock($post_id)
         // if user can break the lock "has admin role" then release the old lock if it from other user
         // If the lock is inactive and the lock is not for current user release old one and create new one
 
-        $this->setCurrentPostLock($post_id);
+        $this->getCurrentPostLock($post_id);
 
         if ($this->current_post_lock) {
             if (($this->lockIsBreakable()) || (!$this->currentLockIsActive() && !$this->userOwnsCurrentLock())) {
@@ -76,7 +76,7 @@ private function userOwnsCurrentLock()
         $user = Auth::user();
         return intval($user->id) === intval($this->current_post_lock->user_id);
     }
-    private function setCurrentPostLock($post_id): void
+    private function getCurrentPostLock($post_id): void
     {
         try {
             $this->current_post_lock = $this->post_lock_repository->findByPostId($post_id);

diff --git a/src/Ushahidi/Modules/V5/Repository/Post/EloquentPostLockRepository.php b/src/Ushahidi/Modules/V5/Repository/Post/EloquentPostLockRepository.php
@@ -38,7 +38,10 @@ public function deleteById(int $id): void
     }
     public function deleteByPostId(int $post_id): void
     {
-        $this->findByPostId($post_id)->delete();
+        $post_locks = PostLock::where('post_id', '=', $post_id)->get();
+        foreach ($post_locks as $post_lock) {
+            $post_lock->delete();
+        }
     }
     public function deleteByUserId(int $user_id): void
     {