Fixes and improvements to identify on vector layers (#2051)

* Fixes and improvements to identify on vector layers * Fixed lint issues
geosolutions-it · Jul 24, 2017 · 008969b · 008969b
1 parent b170c52
commit 008969b
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 4 deletions.
diff --git a/web/client/components/data/identify/viewers/row/PropertiesViewer.jsx b/web/client/components/data/identify/viewers/row/PropertiesViewer.jsx
@@ -8,6 +8,7 @@ const PropTypes = require('prop-types');
  */
 
 const React = require('react');
+const {isString} = require('lodash');
 
 const alwaysExcluded = ["exclude", "titleStyle", "listStyle", "componentStyle", "title"];
 
@@ -47,7 +48,7 @@ class PropertiesViewer extends React.Component {
             .filter(this.toExlude)
             .map((key) => {
                 return (
-                    <p key={key} style={this.props.listStyle}><b>{key}</b> {this.props[key]}</p>
+                    <p key={key} style={this.props.listStyle}><b>{key}</b> {this.renderProperty(this.props[key])}</p>
                 );
             });
     };
@@ -80,6 +81,12 @@ class PropertiesViewer extends React.Component {
         );
     };
 
+    renderProperty = (prop) => {
+        if (isString(prop)) {
+            return prop;
+        }
+        return JSON.stringify(prop);
+    };
     render() {
         return (
             <div style={this.props.componentStyle}>

diff --git a/web/client/components/map/leaflet/Feature.jsx b/web/client/components/map/leaflet/Feature.jsx
@@ -175,7 +175,10 @@ class Feature extends React.Component {
             this._layer.on('click', (event) => {
                 if (this.props.onClick) {
                     this.props.onClick({
-                        pixel: event.containerPoint,
+                        pixel: {
+                            x: event.originalEvent && event.originalEvent.x,
+                            y: event.originalEvent && event.originalEvent.y
+                        },
                         latlng: event.latlng
                     });
                 }

diff --git a/web/client/reducers/mapInfo.js b/web/client/reducers/mapInfo.js
@@ -146,7 +146,7 @@ function mapInfo(state = {}, action) {
         }
         let resolution = action.metadata && action.metadata.resolution || 1;
         let bufferedPoint = buffer(point, (action.metadata.buffer || 1) * resolution, unit);
-        const intersected = action.layer.features.filter(
+        const intersected = (action.layer.features || []).filter(
                     (feature) => {
                         try {
                             // TODO: instead of create a fixed buffer, we should check the feature style to create the proper buffer.

diff --git a/web/client/utils/mapinfo/vector.js b/web/client/utils/mapinfo/vector.js
@@ -16,7 +16,7 @@ module.exports = {
                 lng: props.point.latlng.lng
             },
             metadata: {
-                fields: Object.keys(layer.features[0].properties),
+                fields: layer.features && layer.features.length && Object.keys(layer.features[0].properties) || [],
                 title: layer.name,
                 resolution: props.map && props.map && props.map.zoom && MapUtils.getCurrentResolution(props.map.zoom, 0, 21, 96),
                 buffer: props.buffer,