Merge pull request #187 from Geoportail-Luxembourg/GITLAB-28-fix_add_…

…layer allow to add a layer from another theme
Geoportail-Luxembourg · Jan 30, 2025 · 9463e14 · 9463e14
2 parents 3bfa7cb + fe034d9
commit 9463e14
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/src/components/catalog/catalog-tree.vue b/src/components/catalog/catalog-tree.vue
@@ -61,7 +61,7 @@ function toggleParent(node: LayerTreeNodeModel, is3d: boolean) {
 }
 
 function toggleLayer(node: LayerTreeNodeModel, is3d: boolean) {
-  layers.toggleLayer(+node.id, !node.checked, is3d)
+  layers.toggleLayer(+node.id, !node.checked, is3d, true)
 }
 </script>
 

diff --git a/src/composables/layers/layers.composable.ts b/src/composables/layers/layers.composable.ts
@@ -149,15 +149,21 @@ export default function useLayers() {
     }
   }
 
-  function toggleLayer(id: LayerId, show = true, is3d: boolean) {
+  function toggleLayer(
+    id: LayerId,
+    show = true,
+    is3d: boolean,
+    fromCurrentTheme: boolean
+  ) {
     const themeStore = useThemeStore()
     const mapStore = useMapStore()
-
     // the cast from ThemeNodeModel | undefined to Layer might not be correct.
     // in the themes fixture only WMS layers correspond to the Layer definition,
     // whereas WMTS layers have "layer" property
     const store = is3d ? themeStore.layerTrees_3d : themeStore.theme
-    const layer = <Layer>themes.findById(id, store)
+    const layer = <Layer>(
+      themes.findById(id, fromCurrentTheme ? store : undefined)
+    )
 
     if (layer) {
       const linkedLayers = layer.metadata?.linked_layers || []