From eef688e89f9a2de7f76b9c59ca3602ae495f52a0 Mon Sep 17 00:00:00 2001
From: Olli Kauppinen <olli.kauppinen@nls.fi>
Date: Fri, 23 Aug 2024 14:21:21 +0300
Subject: [PATCH 1/2] update logging

---
 .../plugins/sotka/SotkaStatisticalDatasourcePlugin.java       | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/service-statistics-sotka/src/main/java/fi/nls/oskari/control/statistics/plugins/sotka/SotkaStatisticalDatasourcePlugin.java b/service-statistics-sotka/src/main/java/fi/nls/oskari/control/statistics/plugins/sotka/SotkaStatisticalDatasourcePlugin.java
index decf45dc8..9b0e9b422 100755
--- a/service-statistics-sotka/src/main/java/fi/nls/oskari/control/statistics/plugins/sotka/SotkaStatisticalDatasourcePlugin.java
+++ b/service-statistics-sotka/src/main/java/fi/nls/oskari/control/statistics/plugins/sotka/SotkaStatisticalDatasourcePlugin.java
@@ -45,13 +45,15 @@ public void update() {
             JSONArray responseJSON = new JSONArray(data);
             SotkaIndicatorParser parser = new SotkaIndicatorParser(config);
             LOG.info("Parsing indicator response of length: " + responseJSON.length());
+            int parsedIndicators = 0;
             for (int i = 0; i < responseJSON.length(); i++) {
                 StatisticalIndicator indicator = parser.parse(responseJSON.getJSONObject(i), sotkaToLayerMappings);
                 if(indicator != null) {
+                    parsedIndicators++;
                     onIndicatorProcessed(indicator);
                 }
             }
-            LOG.info("Parsed indicator response.");
+            LOG.info("Updated datasource:", config.getUrl(), "with parsed", parsedIndicators, "of", responseJSON.length(), "indicators.");
         } catch (JSONException e) {
             LOG.error("Error in mapping Sotka Indicators response to Oskari model: " + e.getMessage(), e);
         }

From 3b217df40160399fd2328771b9938394549c415d Mon Sep 17 00:00:00 2001
From: Olli Kauppinen <olli.kauppinen@nls.fi>
Date: Fri, 23 Aug 2024 14:22:10 +0300
Subject: [PATCH 2/2] catch all exceptions

---
 .../statistics/plugins/sotka/parser/SotkaIndicatorParser.java  | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/service-statistics-sotka/src/main/java/fi/nls/oskari/control/statistics/plugins/sotka/parser/SotkaIndicatorParser.java b/service-statistics-sotka/src/main/java/fi/nls/oskari/control/statistics/plugins/sotka/parser/SotkaIndicatorParser.java
index e31425e73..6aaa958f0 100755
--- a/service-statistics-sotka/src/main/java/fi/nls/oskari/control/statistics/plugins/sotka/parser/SotkaIndicatorParser.java
+++ b/service-statistics-sotka/src/main/java/fi/nls/oskari/control/statistics/plugins/sotka/parser/SotkaIndicatorParser.java
@@ -77,7 +77,7 @@ StatisticalIndicator createIndicator(JSONObject json, Map<String, Long> sotkaLay
             ind.setSource(toLocalizationMap(json.getJSONObject("organization").getJSONObject("title")));
 
             return setupMetadata(ind, sotkaLayersToOskariLayers);
-        } catch (JSONException e) {
+        } catch (Exception e) {
             LOG.error(e, "Could not read data from Sotka Indicator JSON.");
             return null;
         }
@@ -340,6 +340,7 @@ private StatisticalIndicator setupMetadata(StatisticalIndicator ind, Map<String,
         if (ind.getLayers().isEmpty()) {
             // we can't show this indicator since it doesn't link to any region set
             // if at this point we don't have layers -> ignore the indicator by returning null
+            LOG.debug("Indicator:", ind.getId(), "doesn't have any of regionsets linked to datasource, ignoring");
             return null;
         }