From b63b789f7796d08ed7f598ef29d50443f9ff68fe Mon Sep 17 00:00:00 2001
From: Jens Langhammer <jens.langhammer@beryju.org>
Date: Fri, 12 Mar 2021 12:41:17 +0100
Subject: [PATCH] web: prevent duplicate messages

---
 web/src/elements/messages/MessageContainer.ts | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/web/src/elements/messages/MessageContainer.ts b/web/src/elements/messages/MessageContainer.ts
index 394219dc7..3ed15286a 100644
--- a/web/src/elements/messages/MessageContainer.ts
+++ b/web/src/elements/messages/MessageContainer.ts
@@ -8,7 +8,7 @@ export function showMessage(message: APIMessage): void {
     if (!container) {
         throw new Error("failed to find message container");
     }
-    container.messages.push(message);
+    container.addMessage(message);
     container.requestUpdate();
 }
 
@@ -34,6 +34,14 @@ export class MessageContainer extends LitElement {
         }
     }
 
+    // add a new message, but only if the message isn't currently shown.
+    addMessage(message: APIMessage): void {
+        const matchingMessages = this.messages.filter(m => m.message == message.message);
+        if (matchingMessages.length < 1) {
+            this.messages.push(message);
+        }
+    }
+
     connect(): void {
         if (navigator.webdriver) return;
         const wsUrl = `${window.location.protocol.replace("http", "ws")}//${
@@ -60,7 +68,7 @@ export class MessageContainer extends LitElement {
         });
         this.messageSocket.addEventListener("message", (e) => {
             const data = JSON.parse(e.data);
-            this.messages.push(data);
+            this.addMessage(data);
             this.requestUpdate();
         });
         this.messageSocket.addEventListener("error", (e) => {
@@ -76,8 +84,8 @@ export class MessageContainer extends LitElement {
                     .message=${m}
                     .onRemove=${(m: APIMessage) => {
                         this.messages = this.messages.filter((v) => v !== m);
-                            this.requestUpdate();
-                        }}>
+                        this.requestUpdate();
+                    }}>
                     </ak-message>`;
                 })}
         </ul>`;