From a7450b4056e314e24e1c59c90d1491bb0f84388f Mon Sep 17 00:00:00 2001
From: rldhont <rldhont@gmail.com>
Date: Fri, 1 Dec 2023 12:13:42 +0100
Subject: [PATCH] Bugfix JS: Catch error from GetLegendGraphics

---
 assets/src/modules/WMS.js              |  5 ++++-
 assets/src/modules/action/Symbology.js | 13 +++++++------
 2 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/assets/src/modules/WMS.js b/assets/src/modules/WMS.js
index 049b60bc69..ca1a9965c3 100644
--- a/assets/src/modules/WMS.js
+++ b/assets/src/modules/WMS.js
@@ -50,6 +50,9 @@ export default class WMS {
                 ...options
             })
         });
-        return response.json();
+        if (response.ok) {
+            return response.json();
+        }
+        throw new Error(response.text);
     }
 }
diff --git a/assets/src/modules/action/Symbology.js b/assets/src/modules/action/Symbology.js
index 695bf0aa61..574811ed55 100644
--- a/assets/src/modules/action/Symbology.js
+++ b/assets/src/modules/action/Symbology.js
@@ -27,13 +27,14 @@ export async function updateLayerTreeLayersSymbology(treeLayers) {
         STYLES: wmsStyles,
     };
 
-    const response = await wms.getLegendGraphic(wmsParams);
-    for (const node of response.nodes) {
-        // If the layer has no symbology, there is no type property
-        if (node.hasOwnProperty('type')) {
-            treeLayersByName[node.name].symbology = node;
+    await wms.getLegendGraphic(wmsParams).then((response) => {
+        for (const node of response.nodes) {
+            // If the layer has no symbology, there is no type property
+            if (node.hasOwnProperty('type')) {
+                treeLayersByName[node.name].symbology = node;
+            }
         }
-    }
+    }).catch(console.error);
     return treeLayers;
 }