add method to recursively fetch parent layers in ArcGIS Mapservice gr…

…oups

lib/Models/Catalog/Esri/ArcGisMapServerCatalogGroup.ts

@@ -49,6 +49,18 @@ export class MapServerStratum extends LoadableStratum(
     ) as this;
   }
 
+  findParentLayers(layerId: number): number[] {
+    const parentLayerIds: number[] = [];
+    const layer = this.layers.find((l) => l.id === layerId);
+    if (layer !== undefined) {
+      parentLayerIds.push(layer.id);
+      if (layer.parentLayerId !== -1) {
+        parentLayerIds.push(...this.findParentLayers(layer.parentLayerId));
+      }
+    }
+    return parentLayerIds;
+  }
+
   @computed get name() {
     if (
       this._mapServer.documentInfo &&
@@ -190,11 +202,12 @@ export class MapServerStratum extends LoadableStratum(
     }
     const id = this._catalogGroup.uniqueId;
     //if parent layer is not -1 then this is sublayer so we define its ID like that
-    const layerId =
+    let layerId =
       id +
       "/" +
       (layer.parentLayerId !== -1 ? layer.parentLayerId + "/" : "") +
       layer.id;
+
     let model: ArcGisMapServerCatalogItem | ArcGisMapServerCatalogGroup;
 
     // Treat layer as a group if it has type "Group Layer" - or has subLayers
@@ -220,7 +233,14 @@ export class MapServerStratum extends LoadableStratum(
         ArcGisMapServerCatalogItem,
         layerId
       );
+
       if (existingModel === undefined) {
+        const parentLayers = this.findParentLayers(layer.id);
+
+        if (parentLayers.length > 0) {
+          layerId = id + "/" + parentLayers.reverse().join("/");
+        }
+
         model = new ArcGisMapServerCatalogItem(
           layerId,
           this._catalogGroup.terria

lib/ReactViews/DataCatalog/DataCatalogGroup.jsx

@@ -101,6 +101,7 @@ class DataCatalogGroup extends React.Component {
   render() {
     const group = this.props.group;
     const { t } = this.props;
+
     return (
       <CatalogGroup
         text={this.getNameOrPrettyUrl()}