Merge pull request #12183 from nextcloud/backport/12178/stable29

[stable29] fix(messagesStore): compare message ids as numeric values
nextcloud · Apr 22, 2024 · 08f3bca · 08f3bca
2 parents 778f2ab + a7afcfb
commit 08f3bca
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/store/messagesStore.js b/src/store/messagesStore.js
@@ -504,15 +504,15 @@ const mutations = {
 			return
 		}
 
-		const messagesToRemove = messageIds.sort().reverse().slice(199)
+		const messagesToRemove = messageIds.sort((a, b) => b - a).slice(199)
 		const newFirstKnown = messagesToRemove.shift()
 
 		messagesToRemove.forEach((messageId) => {
 			Vue.delete(state.messages[token], messageId)
 		})
 
-		if (state.firstKnown[token] && messagesToRemove.includes(state.firstKnown[token])) {
-			Vue.set(state.firstKnown, token, newFirstKnown)
+		if (state.firstKnown[token] && messagesToRemove.includes(state.firstKnown[token].toString())) {
+			Vue.set(state.firstKnown, token, +newFirstKnown)
 		}
 	},
 }