Skip to content

Commit e55265e

Browse files
authored
Merge pull request #1666 from mercihabam/fix-flag
fix(frontend): message store fails to remove non-existent rows from combined lists on update
2 parents e896383 + 0805360 commit e55265e

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

modules/core/js_modules/Hm_MessagesStore.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ class Hm_MessagesStore {
104104
if (messagesReadyCB) {
105105
messagesReadyCB(this);
106106
}
107-
}, this);
107+
});
108108

109109
return this;
110110
}
@@ -235,12 +235,13 @@ class Hm_MessagesStore {
235235

236236
let store = this;
237237
return this.getRequestConfigs().map((config) => {
238+
const initialConfig = Object.assign([], config);
238239
return new Promise((resolve, reject) => {
239240
Hm_Ajax.request(
240241
config,
241242
(response) => {
242243
if (response) {
243-
response.sourceId = store.hashObject(config);
244+
response.sourceId = store.hashObject(initialConfig); // Do not use this config object because the request appends a "hm_page_key" entry, which would change the hash
244245
resolve(response);
245246
}
246247
},

0 commit comments

Comments
 (0)