message_edit: len(changed_messages) is always > 0.

It is only ever added to, and starts as a one-element list.
This commit is contained in:
Alex Vandiver 2023-09-25 20:56:20 +00:00
parent 68be070306
commit db3bcb8889
1 changed files with 2 additions and 6 deletions

View File

@ -996,7 +996,7 @@ def do_update_message(
send_event(user_profile.realm, event, users_to_be_notified) send_event(user_profile.realm, event, users_to_be_notified)
resolved_topic_message_id = None resolved_topic_message_id = None
if topic_name is not None and content is None and len(changed_messages) > 0: if topic_name is not None and content is None:
# When stream is changed and topic is marked as resolved or unresolved # When stream is changed and topic is marked as resolved or unresolved
# in the same API request, resolved or unresolved notification should # in the same API request, resolved or unresolved notification should
# be sent to "new_stream". # be sent to "new_stream".
@ -1014,11 +1014,7 @@ def do_update_message(
changed_messages=changed_messages, changed_messages=changed_messages,
) )
if ( if (new_stream is not None or topic_name is not None) and stream_being_edited is not None:
len(changed_messages) > 0
and (new_stream is not None or topic_name is not None)
and stream_being_edited is not None
):
# Notify users that the topic was moved. # Notify users that the topic was moved.
changed_messages_count = len(changed_messages) changed_messages_count = len(changed_messages)