Added two comments about lines modified in #1304 to fix #1291 that do…

…n't do much.
hajkmap · Aug 25, 2023 · 5e4be0b · 5e4be0b
1 parent 30aa033
commit 5e4be0b
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/new-client/src/plugins/LayerSwitcher/components/LayerGroupItem.js b/new-client/src/plugins/LayerSwitcher/components/LayerGroupItem.js
@@ -432,6 +432,7 @@ class LayerGroupItem extends Component {
     }
   };
 
+  // FIXME: The second parameter, `subLayer` is never used.
   setVisible = (la, subLayer) => {
     let l,
       subLayersToShow = null;
@@ -587,6 +588,9 @@ class LayerGroupItem extends Component {
 
     if (!visible && visibleSubLayers.length > 0) {
       layerVisibility = true;
+      // FIXME: `subLayer` below doesn't do anything since
+      // setVisible only makes use of its first parameter
+      // (see its implementation).
       this.setVisible(this.props.layer, subLayer);
     }