PB-984: avoid duplicate errors

src/store/modules/ui.store.js

@@ -389,12 +389,18 @@ export default {
         setShowDisclaimer({ commit }, { showDisclaimer, dispatcher }) {
             commit('setShowDisclaimer', { showDisclaimer, dispatcher })
         },
-        addErrors({ commit }, { errors, dispatcher }) {
-            if (
-                errors instanceof Array &&
-                errors.filter((error) => error instanceof ErrorMessage).length === errors.length
-            ) {
-                commit('addErrors', { errors, dispatcher })
+        addErrors({ commit, state }, { errors, dispatcher }) {
+            if (errors instanceof Array && errors.every((error) => error instanceof ErrorMessage)) {
+                errors = errors.filter(
+                    (error) =>
+                        // we only add the errors that are not existing within the store
+                        ![...state.errors].some((otherError) => {
+                            error.isEquals(otherError)
+                        })
+                )
+                if (errors.length > 0) {
+                    commit('addErrors', { errors, dispatcher })
+                }
             } else {
                 throw new Error(
                     `Error ${errors} dispatched by ${dispatcher} is not of type ErrorMessage, or not an Array of ErrorMessages`
@@ -412,13 +418,22 @@ export default {
                 commit('removeError', { error, dispatcher })
             }
         },
-        addWarnings({ commit }, { warnings, dispatcher }) {
+        addWarnings({ commit, state }, { warnings, dispatcher }) {
             if (
                 warnings instanceof Array &&
-                warnings.filter((warning) => warning instanceof WarningMessage).length ===
-                    warnings.length
+                warnings.every((warning) => warning instanceof WarningMessage)
             ) {
-                commit('addWarnings', { warnings, dispatcher })
+                warnings = warnings.filter(
+                    (warning) =>
+                        // we only add the warnings that are not existing within the store
+                        ![...state.warnings].some((otherWarning) => {
+                            warning.isEquals(otherWarning)
+                        })
+                )
+                log.error(warnings)
+                if (warnings.length > 0) {
+                    commit('addWarnings', { warnings, dispatcher })
+                }
             } else {
                 throw new Error(
                     `Warning ${warnings} dispatched by ${dispatcher} is not of type WarningMessage, or not an Array of WarningMessages`

src/utils/ErrorMessage.class.js

@@ -6,6 +6,15 @@ export default class ErrorMessage {
      */
     constructor(msg, params = null) {
         this.msg = msg
-        this.params = params
+        this.params = params ?? {}
+    }
+
+    isEquals(object) {
+        return (
+            object instanceof ErrorMessage &&
+            object.msg === this.msg &&
+            Object.keys(this.params).length === Object.keys(object.params).length &&
+            Object.keys(this.params).every((key) => this.params[key] === object.params[key])
+        )
     }
 }

src/utils/WarningMessage.class.js

@@ -6,6 +6,15 @@ export default class WarningMessage {
      */
     constructor(msg, params = null) {
         this.msg = msg
-        this.params = params
+        this.params = params ?? {}
+    }
+
+    isEquals(object) {
+        return (
+            object instanceof WarningMessage &&
+            object.msg === this.msg &&
+            Object.keys(this.params).length === Object.keys(object.params).length &&
+            Object.keys(this.params).every((key) => this.params[key] === object.params[key])
+        )
     }
 }