@geogirafe/lib-geoportal 1.2.0-dev.2657953927 → 1.2.0-dev.2666667702

package/components/advancedfilter/component.d.ts

@@ -12,6 +12,7 @@ declare class AdvancedFilterComponent extends GirafeHTMLElement implements IGira
     panelTitle: string;
     panelTogglePath: string;
     loading: boolean;
+    private readonly debouncedUpdateFilterItems;
     layers: LayerWms[];
     layerFilterElements: LayerFilterElement[];
     columnAliasHelper: ColumnAliasHelper;
@@ -40,7 +41,12 @@ declare class AdvancedFilterComponent extends GirafeHTMLElement implements IGira
      */
     private addFilterElementFromLayer;
     onFilterRemoved(filterElement: LayerFilterElement): void;
+    /**
+     * Removes a filter element from the UI and optionally from the layer.
+     * Note that the filter should only be removed from the layer if this was triggered by a user action in this component.
+     */
     private removeFilterElement;
+    private onFilterChangedInLayerTree;
     onLayerChanged(filterElement: LayerFilterElement, e: Event): Promise<void>;
     onConditionRemoved(filterElement: LayerFilterElement, condition: FilterConditionElement): void;
     onConditionAdded(filterElement: LayerFilterElement): void;

package/components/advancedfilter/component.js

@@ -11,6 +11,7 @@ import LayerWms from '../../models/layers/layerwms.js';
 import ColumnAliasHelper from '../../tools/utils/aliases.js';
 import LayerFilterElement from './layerfilterelement.js';
 import GroupLayer from '../../models/layers/grouplayer.js';
+import { debounce } from '../../tools/utils/debounce.js';
 import { UsedInTemplateOnly } from '../../decorators.js';
 import FilterConditionElement from './filterconditionelement.js';
 import LayerCsvExporter from './layercsvexporter.js';
@@ -31,6 +32,7 @@ ${this.htmlUnsafe(this.feedbackTemplateHtml ?? '')}`;
     panelTitle = 'advanced-filter-panel';
     panelTogglePath = 'interface.advancedFilterPanelVisible';
     loading = false;
+    debouncedUpdateFilterItems = debounce(this.updateFilterItemsByLayerState.bind(this), 200);
     layers = [];
     layerFilterElements = [];
     columnAliasHelper;
@@ -48,15 +50,13 @@ ${this.htmlUnsafe(this.feedbackTemplateHtml ?? '')}`;
         super.girafeTranslate();
     }
     registerEvents() {
-        this.subscribe(/layers\.layersList\..*/, () => {
-            if (this.isPanelVisible) {
-                void this.updateFilterItemsByLayerState();
-            }
-        });
+        // Listen for filter changes on the layer list, triggered by changes in the simple filter component
+        this.subscribe(/layers\.layersList\..*\.filter/, (oldFilter, newFilter, layer) => this.onFilterChangedInLayerTree(oldFilter, newFilter, layer));
+        // Listen for layer visibility changes
         this.subscribe(/layers\.layersList\..*\.activeState/, (_oldValue, _newValue, layer) => {
             if (layer instanceof LayerWms && this.isPanelVisible && !this.state.loading) {
-                this.updateFilterableLayerList();
-                this.render();
+                // Debounce the filter element updates to avoid premature action / rendering while the layer tree is still changing
+                void this.debouncedUpdateFilterItems();
             }
         });
         this.subscribe('loading', (_, loading) => {
@@ -88,18 +88,17 @@ ${this.htmlUnsafe(this.feedbackTemplateHtml ?? '')}`;
         //  Update which layers are active and queryable
         this.updateFilterableLayerList();
         // Remove filter elements that are no longer needed
-        for (const filterElement of this.layerFilterElements) {
+        const filtersToBeRemoved = this.layerFilterElements.filter((filterElement) => {
             const layerInTree = this.layers.find((l) => l.id === filterElement.layer?.id);
-            if (
+            return (
             // Filter element does not have a layer yet
             filterElement.layer === undefined ||
                 // Layer isn't in layer tree anymore
                 (filterElement.layer && !layerInTree) ||
                 // Filter in layer was removed by simple filter component
-                (filterElement.layer && layerInTree.filter === undefined && filterElement.hasAppliedConditions())) {
-                this.removeFilterElement(filterElement);
-            }
-        }
+                (filterElement.layer && layerInTree.filter === undefined && filterElement.hasAppliedConditions()));
+        });
+        filtersToBeRemoved.forEach((filterElement) => this.removeFilterElement(filterElement, false));
         // Add filter elements for layers that are new in the layer tree and do not have a filter element yet
         for (const layer of this.layers) {
             if (layer.hasFilter && !this.layerFilterElements.some((f) => f.layer?.id === layer.id)) {
@@ -148,20 +147,38 @@ ${this.htmlUnsafe(this.feedbackTemplateHtml ?? '')}`;
         this.layerFilterElements.push(filterElement);
     }
     onFilterRemoved(filterElement) {
-        this.removeFilterElement(filterElement);
+        this.removeFilterElement(filterElement, true);
         this.render();
     }
-    removeFilterElement(filterElement) {
+    /**
+     * Removes a filter element from the UI and optionally from the layer.
+     * Note that the filter should only be removed from the layer if this was triggered by a user action in this component.
+     */
+    removeFilterElement(filterElement, removeFilterFromLayer = true) {
         const idx = this.layerFilterElements.indexOf(filterElement);
         if (idx !== -1) {
             filterElement.removeConditions();
             this.layerFilterElements.splice(idx, 1);
-            if (filterElement.layer?.filter) {
+            if (removeFilterFromLayer && filterElement.layer?.filter) {
                 // Note: Changing layer.filter will trigger the subscription that re-renders the component
                 filterElement.layer.filter = undefined;
             }
         }
     }
+    onFilterChangedInLayerTree(oldFilter, newFilter, layer) {
+        if (oldFilter === undefined || newFilter === undefined) {
+            // Cleanly add or remove filter elements in the UI
+            void this.updateFilterItemsByLayerState();
+            return;
+        }
+        // Refresh the displayed filter element with the new filter
+        for (const filterElement of this.layerFilterElements) {
+            if (filterElement.layer?.id === layer.id) {
+                filterElement.setFromLayer(layer).then(() => this.render());
+                break;
+            }
+        }
+    }
     async onLayerChanged(filterElement, e) {
         if (!filterElement.conditions.length) {
             // Display an empty condition while layer metadata is being loaded
@@ -176,7 +193,7 @@ ${this.htmlUnsafe(this.feedbackTemplateHtml ?? '')}`;
             this.layerFilterElements.push(newFilter);
             newFilter?.setLayer(layer).then((_) => this.render());
             // Remove filter from previous layer and from UI
-            this.removeFilterElement(filterElement);
+            this.removeFilterElement(filterElement, true);
         }
         else {
             // Simply switch out layers

package/package.json

@@ -5,7 +5,7 @@
     "name": "GeoGirafe PSC",
     "url": "https://doc.geogirafe.org"
   },
-  "version": "1.2.0-dev.2657953927",
+  "version": "1.2.0-dev.2666667702",
   "type": "module",
   "engines": {
     "node": ">=20.19.0"

package/templates/public/about.json

@@ -1 +1 @@
-{"version":"1.2.0-dev.2657953927", "build":"2657953927", "date":"07/07/2026"}
+{"version":"1.2.0-dev.2666667702", "build":"2666667702", "date":"10/07/2026"}

package/tools/configuration/girafeconfig.js

@@ -4,7 +4,7 @@ export var HighlightMode;
     HighlightMode["HighlightAll"] = "highlight-all";
     HighlightMode["HighlightPerLayer"] = "highlight-per-layer";
 })(HighlightMode || (HighlightMode = {}));
-import { defaultNewsSanitizeConfig, defaultPermalinkSanitizeConfig, defaultQuerySanitizeConfig } from './sanitizeconfig.defaults';
+import { defaultNewsSanitizeConfig, defaultPermalinkSanitizeConfig, defaultQuerySanitizeConfig } from './sanitizeconfig-defaults.js';
 class GirafeConfig {
     general;
     languages;

package/tools/main.d.ts

@@ -11,7 +11,7 @@ export { default as PluginManager } from './auth/pluginmanager.js';
 export { default as ConfigManager } from './configuration/configmanager.js';
 export type { HighlightMode } from './configuration/girafeconfig.js';
 export { default as GirafeConfig } from './configuration/girafeconfig.js';
-export { defaultNewsSanitizeConfig, defaultPermalinkSanitizeConfig, defaultQuerySanitizeConfig } from './configuration/sanitizeconfig.defaults';
+export { defaultNewsSanitizeConfig, defaultPermalinkSanitizeConfig, defaultQuerySanitizeConfig } from './configuration/sanitizeconfig-defaults.js';
 export { default as GirafeContext } from './context/context.js';
 export type { default as IGirafeContext } from './context/icontext.js';
 export { default as CesiumDrawing } from './drawing/cesiumDrawing.js';
@@ -111,8 +111,8 @@ export type { GgUserInteractionListener } from './state/userInteractionManager.j
 export { default as UserInteractionManager } from './state/userInteractionManager.js';
 export type { GgUserInteractionEvent } from './state/userinteractionevent.js';
 export { gGEventDependencies, isPrimaryPointerAction, isAlternateMouseClick, isMouseWheelClick } from './state/userinteractionevent.js';
-export type { SwState } from './sw/service-worker.tools';
-export { swLog, SwHelper, IndexedDbHelper, CacheHelper } from './sw/service-worker.tools';
+export type { SwState } from './sw/service-worker-tools.js';
+export { swLog, SwHelper, IndexedDbHelper, CacheHelper } from './sw/service-worker-tools.js';
 export { default as CustomThemesManager } from './themes/customthemesmanager.js';
 export { default as ThemeFavoritesManager } from './themes/themefavoritesmanager.js';
 export { DEFAULT_OPACITY, OPACITY_FOR_DEFAULT_BASEMAP } from './themes/themes-config.js';

package/tools/main.js

@@ -10,7 +10,7 @@ export { default as OpenIdConnectManager } from './auth/openidconnectmanager.js'
 export { default as PluginManager } from './auth/pluginmanager.js';
 export { default as ConfigManager } from './configuration/configmanager.js';
 export { default as GirafeConfig } from './configuration/girafeconfig.js';
-export { defaultNewsSanitizeConfig, defaultPermalinkSanitizeConfig, defaultQuerySanitizeConfig } from './configuration/sanitizeconfig.defaults';
+export { defaultNewsSanitizeConfig, defaultPermalinkSanitizeConfig, defaultQuerySanitizeConfig } from './configuration/sanitizeconfig-defaults.js';
 export { default as GirafeContext } from './context/context.js';
 export { default as CesiumDrawing } from './drawing/cesiumDrawing.js';
 export { default as DrawingFeature } from './drawing/drawingFeature.js';
@@ -81,7 +81,7 @@ export { default as StateToggleManager } from './state/stateToggleManager.js';
 export { default as StateManager } from './state/statemanager.js';
 export { default as UserInteractionManager } from './state/userInteractionManager.js';
 export { gGEventDependencies, isPrimaryPointerAction, isAlternateMouseClick, isMouseWheelClick } from './state/userinteractionevent.js';
-export { swLog, SwHelper, IndexedDbHelper, CacheHelper } from './sw/service-worker.tools';
+export { swLog, SwHelper, IndexedDbHelper, CacheHelper } from './sw/service-worker-tools.js';
 export { default as CustomThemesManager } from './themes/customthemesmanager.js';
 export { default as ThemeFavoritesManager } from './themes/themefavoritesmanager.js';
 export { DEFAULT_OPACITY, OPACITY_FOR_DEFAULT_BASEMAP } from './themes/themes-config.js';

package/tools/offline/offlinemanager.js

@@ -1,6 +1,6 @@
 // SPDX-License-Identifier: Apache-2.0
 import GirafeSingleton from '../../base/GirafeSingleton.js';
-import { IndexedDbHelper } from '../sw/service-worker.tools';
+import { IndexedDbHelper } from '../sw/service-worker-tools.js';
 import { Feature } from 'ol';
 import VectorLayer from 'ol/layer/Vector.js';
 import VectorSource from 'ol/source/Vector.js';

package/tools/sw/service-worker.js

@@ -1,7 +1,7 @@
 // SPDX-License-Identifier: Apache-2.0
 // NOTE REG: Here the extension ".js" MUST be present in the import,
 // otherwise the service-worker won't be able to find the transpiled file
-import { SwHelper, IndexedDbHelper, swLog, CacheHelper } from './service-worker.tools.js';
+import { SwHelper, IndexedDbHelper, swLog, CacheHelper } from './service-worker-tools.js';
 /**
  * The service worker state needs to be persisted
  * because when not using the app, the service worker can be paused by the browser