diff options
author | Matthew Somerville <matthew@mysociety.org> | 2019-10-18 17:12:39 +0100 |
---|---|---|
committer | Matthew Somerville <matthew@mysociety.org> | 2019-10-18 17:13:09 +0100 |
commit | c54dac7884b74cf71f2b105e55432b43053eba5e (patch) | |
tree | 357cd00514f59684bcebca8879907b18c28f3a3c | |
parent | a23084bfca7fc7ed4d91d0c6d79689c0d6979de4 (diff) |
Do not clear asset attributes on category change.
-rw-r--r-- | web/cobrands/fixmystreet/assets.js | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/web/cobrands/fixmystreet/assets.js b/web/cobrands/fixmystreet/assets.js index cb466a47e..0bd6d411b 100644 --- a/web/cobrands/fixmystreet/assets.js +++ b/web/cobrands/fixmystreet/assets.js @@ -107,7 +107,9 @@ OpenLayers.Layer.VectorAsset = OpenLayers.Class(OpenLayers.Layer.Vector, { // It's possible an asset has been selected before a category (e.g. if // assets are showing for a whole category group. So on category change, - // make sure we check if any attribute fields need setting/clearing. + // make sure we check if any attribute fields need setting. We don't + // clear if not, because that might clear e.g. attributes set by a layer + // using `usrn`. changeCategory: function() { if (!fixmystreet.map) { return; @@ -115,8 +117,6 @@ OpenLayers.Layer.VectorAsset = OpenLayers.Class(OpenLayers.Layer.Vector, { var feature = fixmystreet.assets.selectedFeature(); if (feature) { this.setAttributeFields(feature); - } else { - this.clearAttributeFields(); } }, |