From 2bc6d52c722061a8f8d6493f96c1771c2edde08b Mon Sep 17 00:00:00 2001 From: Mateusz Mandera Date: Tue, 28 May 2019 11:40:56 +0200 Subject: [PATCH] retention: Fix name of move_attachment_message_to_archive_by_message. The first instance of the word "message" should be in plural. We rename to move_attachment_message_to_archive_by_message. --- zerver/lib/retention.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/zerver/lib/retention.py b/zerver/lib/retention.py index fc1084abb8..974225288a 100644 --- a/zerver/lib/retention.py +++ b/zerver/lib/retention.py @@ -147,7 +147,7 @@ def archive_messages() -> None: move_expired_to_archive() clean_expired() -def move_attachment_message_to_archive_by_message(message_ids: List[int]) -> None: +def move_attachment_messages_to_archive_by_message(message_ids: List[int]) -> None: # Move attachments messages relation table data to archive. id_list = ', '.join(str(message_id) for message_id in message_ids) @@ -189,7 +189,7 @@ def move_messages_to_archive(message_ids: List[int]) -> None: [ArchivedAttachment(**attachment) for attachment in attachments] ) - move_attachment_message_to_archive_by_message(message_ids) + move_attachment_messages_to_archive_by_message(message_ids) # Remove data from main tables Message.objects.filter(id__in=message_ids).delete()