# HG changeset patch
# User Sebastien Jodogne <s.jodogne@gmail.com>
# Date 1788770790 -7200
# Node ID 53b0cc4006ea76b64f36bcc0c0ce00081572ade3
# Parent  632cb930ac87e879cb335f5d20f2417925871a05
allow to select imported features

diff -r 632cb930ac87 -r 53b0cc4006ea ViewerPlugin/WebApplication/viewer.js
--- a/ViewerPlugin/WebApplication/viewer.js	Mon Sep 07 10:39:49 2026 +0200
+++ b/ViewerPlugin/WebApplication/viewer.js	Mon Sep 07 10:46:30 2026 +0200
@@ -438,7 +438,10 @@
     },
 
     UpdateAnnotationProperty: function(prop) {
-      if (this.selectedFeature && prop.featureProp) {
+      if (this.selectedFeature &&
+          prop.featureProp &&
+          this.drawSource !== null &&
+          this.drawSource.hasFeature(this.selectedFeature)) {
         this.selectedFeature.set(prop.featureProp, prop.value);
         this.SaveUserFeatures();
       }
@@ -570,7 +573,9 @@
 
     DeleteSelectedAnnotation: function() {
       var selected = this.selectAnnotation.getFeatures();
-      if (selected.getLength() > 0) {
+      if (selected.getLength() > 0 &&
+          this.drawSource !== null &&
+          this.drawSource.hasFeature(selected.item(0))) {
         this.modalDeleteAnnotation.show();
       }
     },
@@ -900,6 +905,17 @@
         return layer;
       }
 
+      function IsFeatureVisible(feature) {
+        var layerId = feature.get('layer-id');
+        var userLayer = GetLayerById(layerId);
+        if (userLayer !== null) {
+          return userLayer.visible;
+        }
+
+        var importedLayer = GetImportedLayerById(layerId);
+        return importedLayer !== null && importedLayer.visible;
+      }
+
       // Single vector source holding all features from all layers
       this.drawSource = new ol.source.Vector();
 
@@ -975,9 +991,9 @@
       // The condition restricts user-click selection to the dedicated select tool only,
       // preventing spurious selection events when starting a draw near an existing feature.
       this.selectAnnotation = new ol.interaction.Select({
-        layers: [ this.drawLayer ],
+        layers: [ this.drawLayer, this.drawImportedLayer ],
         filter: function(feature) {
-          return GetLayerOfFeature(feature).visible;
+          return IsFeatureVisible(feature);
         },
         condition: function(e) {
           return ol.events.condition.singleClick(e) && app.activeDrawTool === 'select';
@@ -1053,7 +1069,12 @@
             that.AddReadOnlyProperty('Area', FormatArea(geometry.getArea(), that.map.getView().getProjection()));
           }
 
-          that.AddEditableProperty('Label', feature.get('label') || '', 'label');
+          if (that.drawSource !== null &&
+              that.drawSource.hasFeature(feature)) {
+            that.AddEditableProperty('Label', feature.get('label') || '', 'label');
+          } else {
+            that.AddReadOnlyProperty('Label', feature.get('label') || '');
+          }
 
           /*
           // TODO