refactor: streamline comment deletion process and enhance database transaction handling

Author: dewanakl

app/Controllers/Api/CommentController.php

@@ -122,27 +122,15 @@ public function destroy(string $id): JsonResponse
         }
 
         $comment = $this->comment->getByOwnId(Auth::id(), $id);
-
         if (!$comment->exist()) {
             return $this->json->errorNotFound();
         }
 
-        try {
-            $status = DB::transaction(function (LikeContract $like) use ($comment): int {
-                $like->deleteByCommentID($comment->uuid);
-                $this->comment->deleteByParentID($comment->uuid);
-
-                return $comment->destroy();
-            });
-
-            if ($status === 1) {
+        if ($this->comment->deleteAllByParentID(Auth::id(), $comment->uuid)) {
                 return $this->json->successStatusTrue();
             }
 
             return $this->json->errorServer();
-        } catch (Throwable) {
-            return $this->json->errorServer();
-        }
     }
 
     #[UuidMiddleware]

app/Repositories/CommentContract.php

@@ -11,6 +11,7 @@ public function getAll(int $user_id, bool $is_admin, string $user_name, int $lim
     public function count(int $user_id): int;
     public function getByUuid(int $user_id, string $uuid): Model;
     public function getByOwnId(int $user_id, string $own_id): Model;
+    public function deleteAllByParentID(int $userId, string $parentId): bool;
     public function deleteByParentID(string $uuid): int;
     public function countCommentByUserID(int $id): int;
     public function countPresenceByUserID(int $id): Model;

app/Repositories/CommentRepositories.php

@@ -123,6 +123,44 @@ public function getByOwnId(int $user_id, string $own_id): Model
             ->first();
     }
 
+    public function deleteAllByParentID(int $userId, string $parentId): bool
+    {
+        $commentUuids = [$parentId];
+        $nextParents = [$parentId];
+
+        while (count($nextParents) > 0) {
+            $comments = Comment::where('user_id', $userId)
+                ->whereIn('parent_id', $nextParents)
+                ->select('uuid')
+                ->get();
+
+            $newUuids = [];
+            foreach ($comments as $comment) {
+                $commentUuids[] = $comment->uuid;
+                $newUuids[] = $comment->uuid;
+            }
+
+            $nextParents = $newUuids;
+        }
+
+        return DB::transaction(function () use ($commentUuids, $userId): bool {
+
+            Like::where('user_id', $userId)
+                ->whereIn('comment_id', $commentUuids)
+                ->delete();
+
+            $deletedComments = Comment::where('user_id', $userId)
+                ->whereIn('uuid', $commentUuids)
+                ->delete();
+
+            if (count($commentUuids) === $deletedComments) {
+                return true;
+            }
+
+            throw new Exception('Uncompleted deletion: comments=' . strval($deletedComments));
+        });
+    }
+
     public function deleteByParentID(string $uuid): int
     {
         return Comment::where('parent_id', $uuid)->delete();