Do not set attribute fields on hidden layers.

This did not really matter before because you were either hiding an
asset layer (and thus its fields), or selecting a new layer/asset (and
so the last thing would be setting the attributes for the new asset. But
it was unnecessary, and now we're updating an inspector form and the
fields may apply to more than one layer.

1 files changed, 3 insertions, 0 deletions

diff --git a/web/cobrands/fixmystreet/assets.js b/web/cobrands/fixmystreet/assets.js
index ac4cb891c..367e2d33c 100644
--- a/web/cobrands/fixmystreet/assets.js
+++ b/web/cobrands/fixmystreet/assets.js
@@ -120,6 +120,9 @@ OpenLayers.Layer.VectorAsset = OpenLayers.Class(OpenLayers.Layer.Vector, {
         if (!fixmystreet.map) {
             return;
         }
+        if (!this.getVisibility()) {
+          return;
+        }
         var feature = fixmystreet.assets.selectedFeature();
         if (feature) {
             this.setAttributeFields(feature);