From 4bd7ec7c3699b08655fb3d6ae2a00a19c8a086db Mon Sep 17 00:00:00 2001 From: Abhijeet Prasad Bodas Date: Thu, 29 Apr 2021 21:48:21 +0530 Subject: [PATCH] message lists: Rename update_items_for_muting to include `topic`. This change will help avoid confusion with a similar method which will be added in further commits for user muting. --- frontend_tests/node_tests/message_list_data.js | 2 +- static/js/message_list.js | 2 +- static/js/message_list_data.js | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/frontend_tests/node_tests/message_list_data.js b/frontend_tests/node_tests/message_list_data.js index f045fad074..da38bbc198 100644 --- a/frontend_tests/node_tests/message_list_data.js +++ b/frontend_tests/node_tests/message_list_data.js @@ -74,7 +74,7 @@ run_test("basics", () => { mld.remove([50]); assert_contents(mld, [10, 15, 20, 25, 30, 35, 40, 45, 60, 70]); - mld.update_items_for_muting(); + mld.update_items_for_topic_muting(); assert_contents(mld, [10, 15, 20, 25, 30, 35, 40, 45, 60, 70]); mld.reset_select_to_closest(); diff --git a/static/js/message_list.js b/static/js/message_list.js index 70db0f5aec..f18c9f119d 100644 --- a/static/js/message_list.js +++ b/static/js/message_list.js @@ -384,7 +384,7 @@ export class MessageList { } update_topic_muting_and_rerender() { - this.data.update_items_for_muting(); + this.data.update_items_for_topic_muting(); if (this.data.excludes_muted_topics) { this.rerender(); } diff --git a/static/js/message_list_data.js b/static/js/message_list_data.js index 58fce13be4..e490e7f1e1 100644 --- a/static/js/message_list_data.js +++ b/static/js/message_list_data.js @@ -186,7 +186,7 @@ export class MessageListData { return this.messages_filtered_for_topic_mutes(messages); } - update_items_for_muting() { + update_items_for_topic_muting() { if (!this.excludes_muted_topics) { return; }