feat: updated soft delete functionality

Author: Alam-2U

forum/backends/mongodb/comments.py

@@ -415,7 +415,7 @@ def soft_delete(self, comment_id: str, user_id: str) -> int:
 
         # Update Elasticsearch index
         if result.modified_count > 0:
-            from forum.search.es import ElasticsearchDocumentBackend
+            from forum.search.es import ElasticsearchDocumentBackend  # pylint: disable=import-outside-toplevel
 
             es_backend = ElasticsearchDocumentBackend()
             es_backend.update_document(
@@ -441,7 +441,7 @@ def restore(self, comment_id: str) -> int:
 
         # Update Elasticsearch index
         if result.modified_count > 0:
-            from forum.search.es import ElasticsearchDocumentBackend
+            from forum.search.es import ElasticsearchDocumentBackend  # pylint: disable=import-outside-toplevel
 
             es_backend = ElasticsearchDocumentBackend()
             es_backend.update_document(

forum/backends/mongodb/threads.py

@@ -24,8 +24,6 @@ def delete(self, _id: str) -> int:
         get_handler_by_name("comment_thread_deleted").send(
             sender=self.__class__, comment_thread_id=_id
         )
-        from forum.backends.mongodb.users import Users
-
         Users().delete_read_state_by_thread_id(_id)
         return result
 
@@ -323,29 +321,27 @@ def soft_delete(self, thread_id: str, user_id: str) -> int:
 
         # Cascade soft delete to all comments of this thread
         if result.modified_count > 0:
-            from forum.backends.mongodb.comments import Comment
+            from forum.backends.mongodb.comments import Comment  # pylint: disable=import-outside-toplevel,cyclic-import
             Comment().soft_delete_by_thread(thread_id, user_id)
 
         # Delete read states from user model
         if result.modified_count > 0:
-            from forum.backends.mongodb.users import Users
             Users().delete_read_state_by_thread_id(thread_id)
 
         # Update Elasticsearch index to reflect the soft delete
         if result.modified_count > 0:
             try:
-                from forum.search.es import ElasticsearchDocumentBackend
-
+                from forum.search.es import ElasticsearchDocumentBackend  # pylint: disable=import-outside-toplevel
                 es_backend = ElasticsearchDocumentBackend()
                 es_backend.update_document(
                     index_name=self.index_name,
                     doc_id=thread_id,
                     update_data={"is_deleted": True},
                 )
-            except Exception as e:
+            except Exception as e:  # pylint: disable=broad-exception-caught
                 # Log the error but don't fail the soft delete operation
                 # Elasticsearch updates are not critical for the soft delete to succeed
-                import logging
+                import logging  # pylint: disable=import-outside-toplevel
                 log = logging.getLogger(__name__)
                 log.warning(f"Failed to update Elasticsearch for thread {thread_id}: {e}")
 
@@ -367,18 +363,17 @@ def restore(self, thread_id: str) -> int:
         # Update Elasticsearch index to reflect the restore
         if result.modified_count > 0:
             try:
-                from forum.search.es import ElasticsearchDocumentBackend
-
+                from forum.search.es import ElasticsearchDocumentBackend  # pylint: disable=import-outside-toplevel
                 es_backend = ElasticsearchDocumentBackend()
                 es_backend.update_document(
                     index_name=self.index_name,
                     doc_id=thread_id,
                     update_data={"is_deleted": False},
                 )
-            except Exception as e:
+            except Exception as e:  # pylint: disable=broad-exception-caught
                 # Log the error but don't fail the restore operation
                 # Elasticsearch updates are not critical for the restore to succeed
-                import logging
+                import logging  # pylint: disable=import-outside-toplevel
                 log = logging.getLogger(__name__)
                 log.warning(f"Failed to update Elasticsearch for thread {thread_id}: {e}")
 

forum/backends/mysql/api.py

@@ -2305,7 +2305,7 @@ def soft_delete_thread(thread_id: str, user_id: str) -> int:
                     defaults={'username': f"user_{user_id}"}
                 )
             result = thread.soft_delete(user)
-            
+
             if result:
                 # Cascade soft delete to all comments in this thread
                 Comment.objects.filter(
@@ -2316,10 +2316,10 @@ def soft_delete_thread(thread_id: str, user_id: str) -> int:
                     deleted_at=timezone.now(),
                     deleted_by=user
                 )
-                
+
                 # Hard delete subscriptions for this thread
                 MySQLBackend.delete_subscriptions_of_a_thread(thread_id)
-            
+
             return 1 if result else 0
         except ObjectDoesNotExist:
             return 0