From a866a0c38e4266b62dc1e4da41f278166457125f Mon Sep 17 00:00:00 2001 From: Deimos Date: Thu, 22 Nov 2018 17:14:45 -0700 Subject: [PATCH] Deleted post cleanup: check is_deleted This shouldn't make much difference in practice since deleted_time should be updated automatically, but it's probably safest to make sure that the posts are still deleted before cleaning them up, in case the deleted_time ends up staying set somehow. --- tildes/scripts/clean_private_data.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tildes/scripts/clean_private_data.py b/tildes/scripts/clean_private_data.py index 4e857ca..4360de0 100644 --- a/tildes/scripts/clean_private_data.py +++ b/tildes/scripts/clean_private_data.py @@ -87,6 +87,7 @@ class DataCleaner: updated = ( self.db_session.query(Comment) .filter( + Comment.is_deleted == True, # noqa Comment.deleted_time <= self.retention_cutoff, # type: ignore Comment.user_id != 0, ) @@ -113,6 +114,7 @@ class DataCleaner: updated = ( self.db_session.query(Topic) .filter( + Topic.is_deleted == True, # noqa Topic.deleted_time <= self.retention_cutoff, # type: ignore Topic.user_id != 0, )