@masterportal/masterportalapi 2.62.0 → 2.63.0

package/CHANGELOG.md

@@ -2,6 +2,31 @@
  All important changes in this project are stored in this file.
 
  The [Semantic Versioning](https://semver.org/spec/v2.0.0.html) is used.
+ 
+## 2.63.0 - 2026-07-17
+
+### Changed
+- The following packages have been updated:
+  - dependencies:
+    - core-js: 3.33.1 to 3.49.0
+    - dayjs: 1.11.10 to 1.11.21
+    - proj4: 2.10.0 to 2.20.9
+  - devDependencies:
+    - @parcel/config-default: 2.15.4 to 2.16.4
+    - @parcel/transformer-sass: 2.15.4 to 2.16.4
+    - canvas: 3.2.0 to 3.2.3
+    - jest: 30.1.3 to 30.4.2
+    - jest-environment-jsdom: 30.1.2 to 30.4.1
+    - jsdoc: 4.0.2 to 4.0.5
+    - parcel: 2.15.4 to 2.16.4
+    - sass: 1.69.4 to 1.101.0
+    - url: 0.11.3 to 0.11.4
+    - @cesium/engine: 23.0.0 to 26.0.0
+
+### Fixed
+- Issue \#1623: fix label for configuration of style properties as array of objects.
+
+---
 
 ## 2.62.0 - 2026-06-11
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@masterportal/masterportalapi",
-  "version": "2.62.0",
+  "version": "2.63.0",
   "description": "Basic functions of the Masterportal as api",
   "homepage": "https://bitbucket.org/geowerkstatt-hamburg/masterportalapi#readme",
   "bugs": {
@@ -33,42 +33,42 @@
     "prepare": "husky install"
   },
   "dependencies": {
-    "core-js": "^3.33.1",
-    "dayjs": "^1.11.10",
+    "core-js": "^3.49.0",
+    "dayjs": "^1.11.21",
     "ol": "10.9.0",
     "ol-mapbox-style": "13.3.0",
     "olcs": "^2.23.0",
-    "proj4": "^2.10.0"
+    "proj4": "^2.20.9"
   },
   "devDependencies": {
-    "@parcel/config-default": "^2.15.4",
-    "@parcel/transformer-sass": "^2.15.4",
+    "@parcel/config-default": "^2.16.4",
+    "@parcel/transformer-sass": "^2.16.4",
     "@swc/jest": "^0.2.39",
     "assert": "^2.1.0",
     "browserify-zlib": "^0.2.0",
     "buffer": "^6.0.3",
-    "canvas": "3.2.0",
+    "canvas": "3.2.3",
     "eslint": "8.57.1",
     "events": "^3.3.0",
     "https-browserify": "^1.0.0",
     "husky": "8.0.3",
-    "jest": "30.1.3",
+    "jest": "30.4.2",
     "jest-canvas-mock": "2.5.2",
-    "jest-environment-jsdom": "30.1.2",
+    "jest-environment-jsdom": "30.4.1",
     "jest-fetch-mock": "^3.0.3",
-    "jsdoc": "4.0.2",
-    "parcel": "^2.15.4",
+    "jsdoc": "4.0.5",
+    "parcel": "^2.16.4",
     "process": "^0.11.10",
-    "sass": "1.69.4",
+    "sass": "1.101.0",
     "stream-browserify": "^3.0.0",
     "stream-http": "^3.2.0",
     "timers-browserify": "^2.0.12",
-    "url": "^0.11.3",
+    "url": "^0.11.4",
     "use-resize-observer": "^9.1.0",
     "util": "^0.12.5"
   },
   "peerDependencies": {
-    "@cesium/engine": "^23.0.0"
+    "@cesium/engine": "^26.0.0"
   },
   "engines": {
     "node": "^22.19.0 || ^24.11.1",

package/publiccode.yml

@@ -81,8 +81,8 @@ maintenance:
 name: Masterportal
 platforms:
   - web
-releaseDate: '2026-06-11'
+releaseDate: '2026-07-17'
 roadmap: 'https://bitbucket.org/geowerkstatt-hamburg/masterportalapi/src/master/'
 softwareType: api
-softwareVersion: 2.62.0
+softwareVersion: 2.63.0
 url: 'https://bitbucket.org/geowerkstatt-hamburg/masterportalapi.git'

package/src/vectorStyle/createStyle.js

@@ -9,6 +9,35 @@ import {getRuleForIndex, getRulesForFeature} from "./lib/getRuleForIndex";
 
 const legendsOfAllStyles = [];
 let legendInformationLength = 0;
+const warnedLayerIdsWithDefaultStyleFallback = new Set();
+
+/**
+ * Warns once per layer when no styling rule matches and default styles are used.
+ * @param {object} [warningContext={}] context for the warning
+ * @param {string} [warningContext.layerId] id of the affected layer
+ * @returns {void}
+ */
+function warnAboutDefaultStyleFallback (warningContext = {}) {
+    const {layerId} = warningContext;
+
+    if (typeof layerId === "string" && layerId.length > 0) {
+        if (warnedLayerIdsWithDefaultStyleFallback.has(layerId)) {
+            return;
+        }
+        warnedLayerIdsWithDefaultStyleFallback.add(layerId);
+        console.warn(`No valid styling rule found for layer "${layerId}". Falling back to defaults.`);
+        return;
+    }
+    console.warn("No valid styling rule found. Falling back to defaults");
+}
+
+/**
+ * Clears cached default-style fallback warnings.
+ * @returns {void}
+ */
+function clearDefaultStyleFallbackWarnings () {
+    warnedLayerIdsWithDefaultStyleFallback.clear();
+}
 
 /**
  * Returns true if feature contains some kind of MultiGeometry
@@ -152,9 +181,10 @@ function getMultiGeometryStyle (geometryType, feature, rules, isClustered, wfsIm
  * @param   {object[]}  rules       styling rules to check. Array can be empty.
  * @param   {Boolean} isClustered Flag to show if feature is clustered.
  * @param   {String} wfsImgPathFromConfig path to wfsImg from Config
+ * @param   {object} [warningContext={}] context for warning deduplication
  * @returns {ol/style/Style}    style is always returned
  */
-function getGeometryStyle (feature, rules, isClustered, wfsImgPathFromConfig) {
+function getGeometryStyle (feature, rules, isClustered, wfsImgPathFromConfig, warningContext = {}) {
     const geometryType = feature ? feature.getGeometry().getType() : "cesium";
 
     // For simple geometries the first styling rule is used.
@@ -168,7 +198,7 @@ function getGeometryStyle (feature, rules, isClustered, wfsImgPathFromConfig) {
     }
 
     // fall back to default styles as configured in geomType specific styles, if no rule is matched
-    console.warn("No valid styling rule found. Falling back to defaults");
+    warnAboutDefaultStyleFallback(warningContext);
     return isMultiGeometry(geometryType)
         ? getMultiGeometryStyle(geometryType, feature, undefined, isClustered, wfsImgPathFromConfig)
         : getSimpleGeometryStyle(geometryType, feature, undefined, isClustered, wfsImgPathFromConfig);
@@ -224,14 +254,15 @@ function captureLegendFromFeature (styleId, legendInformation) {
      * @param   {ol/feature}  feature     the feature to style
      * @param   {Boolean} isClustered is feature clustered
      * @param   {String} wfsImgPathFromConfig path to wfsImg from Config
-     * @returns {ol/style/Style} style used in layer model
+    * @param   {object} [warningContext={}] context for warning deduplication
+    * @returns {ol/style/Style} style used in layer model
      */
-function createStyle (styleObject, feature, isClustered, wfsImgPathFromConfig) {
+function createStyle (styleObject, feature, isClustered, wfsImgPathFromConfig, warningContext = {}) {
     const rules = getRulesForFeature(styleObject, feature),
         // Takes first rule in array for labeling so that is giving precedence to the order in the style.json
         style = Array.isArray(rules) && rules.length > 0 ? rules[0].style : null,
         hasLabelField = style?.labelField,
-        geometryStyle = getGeometryStyle(feature, rules, isClustered, wfsImgPathFromConfig),
+        geometryStyle = getGeometryStyle(feature, rules, isClustered, wfsImgPathFromConfig, warningContext),
         legendInformation = Array.isArray(geometryStyle) ? geometryStyle[0].legendInfos : geometryStyle.legendInfos;
     let styleObjectGeometry = null;
 
@@ -327,5 +358,6 @@ export default {
     getGeometryStyle,
     getLabelStyle,
     createStyle,
+    clearDefaultStyleFallbackWarnings,
     returnLegendByStyleId
 };

package/src/vectorStyle/styles/style.js

@@ -106,9 +106,12 @@ class StyleClass {
         else if (rule?.conditions) {
             let label = "";
 
-            if (rule.conditions?.properties) {
+            if (rule.conditions?.properties && !Array.isArray(rule.conditions.properties)) {
                 label = Object.values(rule.conditions.properties).join(", ");
             }
+            if (Array.isArray(rule.conditions?.properties)) {
+                label = rule.conditions.properties.map(prop => prop.value).join(", ");
+            }
 
             if (rule.conditions?.sequence
                 && Array.isArray(rule.conditions.sequence)
@@ -117,7 +120,6 @@ class StyleClass {
                 && rule.conditions.sequence[1] >= rule.conditions.sequence[0]) {
                 label = label + " (" + rule.conditions.sequence.join("-") + ")";
             }
-
             return label;
         }
 

package/test/vectorStyle/styles/style.test.js

@@ -0,0 +1,70 @@
+import StyleClass from "../../../src/vectorStyle/styles/style";
+
+describe("StyleClass", () => {
+    describe("createLegendLabel", () => {
+        it("returns legendValue with highest precedence", () => {
+            const style = new StyleClass(),
+                label = style.createLegendLabel({
+                    conditions: {
+                        properties: {
+                            category: "Rule Label"
+                        }
+                    }
+                },
+                {
+                    attributes: {
+                        legendValue: 123
+                    }
+                });
+
+            expect(label).toBe("123");
+        });
+
+        it("creates label from properties configured as array of objects", () => {
+            const style = new StyleClass(),
+                label = style.createLegendLabel({
+                    conditions: {
+                        properties: [
+                            {
+                                key: "category",
+                                value: "A"
+                            },
+                            {
+                                key: "category",
+                                value: "B"
+                            }
+                        ]
+                    }
+                },
+                {
+                    attributes: {}
+                });
+
+            expect(label).toBe("A, B");
+        });
+
+        it("appends valid sequence to created label", () => {
+            const style = new StyleClass(),
+                label = style.createLegendLabel({
+                    conditions: {
+                        properties: {
+                            category: "A"
+                        },
+                        sequence: [10, 20]
+                    }
+                },
+                {
+                    attributes: {}
+                });
+
+            expect(label).toBe("A (10-20)");
+        });
+
+        it("returns null if no legendValue and no rule conditions exist", () => {
+            const style = new StyleClass(),
+                label = style.createLegendLabel(undefined, {attributes: {}});
+
+            expect(label).toBeNull();
+        });
+    });
+});