Sort text replacements better to avoid overlaps, fixes #334

Wunderfitz · Feb 10, 2021 · 4435828 · 4435828
1 parent fc2c53f
commit 4435828
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/qml/js/functions.js b/qml/js/functions.js
@@ -235,7 +235,15 @@ function enhanceHtmlEntities(simpleText) {
     return simpleText.replace(ampRegExp, "&amp;").replace(ltRegExp, "&lt;").replace(gtRegExp, "&gt;");//.replace(rawNewLineRegExp, "<br>");
 }
 
-function messageInsertionSorter(a, b) { return (b.offset+b.removeLength) - (a.offset+a.removeLength) }
+function messageInsertionSorter(a, b) {
+    if ((b.offset + b.removeLength) > (a.offset + a.removeLength)) {
+        return 1;
+    }
+    if ((b.offset + b.removeLength) < (a.offset + a.removeLength)) {
+        return -1;
+    }
+    return b.offset - a.offset;
+}
 
 function enhanceMessageText(formattedText, ignoreEntities) {