paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-03 13:33:24 +00:00
Code Issues Packages Projects Releases Wiki Activity

message_fetch: Remove no longer needed check for home filter.

Browse Source 
`Filter.public_terms` will return the same result even if the
narrow is combined feed. Since we are no longer passing
empty narrow parameters for combined feed, this condition is
not required.
This commit is contained in:
Aman Agrawal
2024-11-18 16:36:17 +05:30
committed by Tim Abbott
parent 09c9db7b18
commit 1954855b6a
1 changed files with 1 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
web/src/message_fetch.ts
View File

@@ -331,16 +331,7 @@ export function load_messages(opts: MessageFetchOptions, attempt = 1): void {
blueslip.error("Message list data is undefined!");
}
// This block is a hack; structurally, we want to set
// data.narrow = opts.msg_list.data.filter.public_terms()
//
// But support for the all_messages_data sharing of data with
// the combined feed view and the (hacky) page_params.narrow feature
// requires a somewhat ugly bundle of conditionals.
let narrow_data: NarrowTerm[] = [];
if (!msg_list_data.filter.is_in_home()) {
narrow_data = msg_list_data.filter.public_terms();
}
let narrow_data = msg_list_data.filter.public_terms();
if (page_params.narrow !== undefined) {
narrow_data = [...narrow_data, ...page_params.narrow];
}