Manage errors on featureType load

geosolutions-it · Feb 14, 2017 · 46e9285 · 46e9285
1 parent 74456ea
commit 46e9285
Show file tree

Hide file tree

Showing 7 changed files with 42 additions and 11 deletions.
diff --git a/web/client/components/data/query/QueryBuilder.jsx b/web/client/components/data/query/QueryBuilder.jsx
@@ -21,6 +21,8 @@ const QueryBuilder = React.createClass({
         featureTypeConfigUrl: React.PropTypes.string,
         useMapProjection: React.PropTypes.bool,
         attributes: React.PropTypes.array,
+        featureTypeError: React.PropTypes.string,
+        featureTypeErrorText: React.PropTypes.node,
         groupLevels: React.PropTypes.number,
         filterFields: React.PropTypes.array,
         groupFields: React.PropTypes.array,
@@ -56,6 +58,7 @@ const QueryBuilder = React.createClass({
             groupFields: [],
             filterFields: [],
             attributes: [],
+            featureTypeError: "",
             spatialField: {},
             removeButtonIcon: "glyphicon glyphicon-minus",
             addButtonIcon: "glyphicon glyphicon-plus",
@@ -98,20 +101,23 @@ const QueryBuilder = React.createClass({
             }
         };
     },
+    componentDidMount() {
+        if (this.props.featureTypeConfigUrl && this.props.attributes.length < 1) {
+            this.props.attributeFilterActions.onLoadFeatureTypeConfig(
+                this.props.featureTypeConfigUrl, this.props.params);
+        }
+    },
     componentWillReceiveProps(props) {
         let url = props.featureTypeConfigUrl;
         let params = props.params !== this.props.params ? props.params : this.props.params;
         if (url !== this.props.featureTypeConfigUrl) {
             this.props.attributeFilterActions.onLoadFeatureTypeConfig(url, params);
         }
     },
-    componentDidMount() {
-        if (this.props.featureTypeConfigUrl && this.props.attributes.length < 1) {
-            this.props.attributeFilterActions.onLoadFeatureTypeConfig(
-                this.props.featureTypeConfigUrl, this.props.params);
-        }
-    },
     render() {
+        if (this.props.featureTypeError !== "") {
+            return (<div style={{margin: "0 auto", "text-align": "center"}}>{this.props.featureTypeErrorText}</div>);
+        }
         return this.props.attributes.length > 0 ? (
             <div id="queryFormPanel">
                 <QueryToolbar

diff --git a/web/client/components/data/query/__tests__/QueryBuilder-test.jsx b/web/client/components/data/query/__tests__/QueryBuilder-test.jsx
@@ -93,4 +93,22 @@ describe('QueryBuilder', () => {
         let childNodes = queryBuilderDOMNode.actual.childNodes;
         expect(childNodes.length).toBe(2);
     });
+
+    it('creates the QueryBuilder component in error state', () => {
+
+        let attributeFilterActions = {
+            onLoadFeatureTypeConfig: () => {}
+        };
+        let spy = expect.spyOn(attributeFilterActions, 'onLoadFeatureTypeConfig');
+
+        const querybuilder = ReactDOM.render(<QueryBuilder
+            featureTypeError={"true"}
+            featureTypeErrorText={"bla bla"}
+            attributeFilterActions={attributeFilterActions}
+            featureTypeConfigUrl={"randomurl"} />,
+        document.getElementById("container"));
+
+        expect(querybuilder).toExist();
+        expect(spy.calls.length).toEqual(1);
+    });
 });
diff --git a/web/client/plugins/TOC.jsx b/web/client/plugins/TOC.jsx
@@ -69,6 +69,7 @@ const SmartQueryForm = connect((state) => {
         groupFields: state.queryform.groupFields,
         filterFields: state.queryform.filterFields,
         attributes: state.query && state.query.typeName && state.query.featureTypes && state.query.featureTypes[state.query.typeName] && state.query.featureTypes[state.query.typeName].attributes,
+        featureTypeError: state.query && state.query.typeName && state.query.featureTypes && state.query.featureTypes[state.query.typeName] && state.query.featureTypes[state.query.typeName].error,
         spatialField: state.queryform.spatialField,
         showDetailsPanel: state.queryform.showDetailsPanel,
         toolbarEnabled: state.queryform.toolbarEnabled,
@@ -237,7 +238,9 @@ const LayerTree = React.createClass({
     renderQueryPanel() {
         return (<div>
             <Button id="query-close-button" bsStyle="primary" key="menu-button" className="square-button" onClick={this.props.onToggleQuery.bind(this, null, null)}><Glyphicon glyph="arrow-left"/></Button>
-            <SmartQueryForm/>
+            <SmartQueryForm
+                featureTypeErrorText={<Message msgId="layerProperties.featureTypeError"/>}
+                />
         </div>);
     },
     render() {

diff --git a/web/client/translations/data.de-DE b/web/client/translations/data.de-DE
@@ -35,7 +35,8 @@
             "delete": "Löschen",
             "deleteLayer":"Ebene löschen",
             "deleteLayerMessage": "Möchtest du wirklich diese Ebene löschen?",
-            "confirmDelete": "Bist du sicher?"
+            "confirmDelete": "Bist du sicher?",
+            "featureTypeError": "Layerattribute können nicht geladen werden"
         },
         "background": "Hintergrund",
         "language": "Sprache",

diff --git a/web/client/translations/data.en-US b/web/client/translations/data.en-US
@@ -35,7 +35,8 @@
             "delete": "Delete",
             "deleteLayer":"Delete Layer",
             "deleteLayerMessage": "Do you really want to delete this Layer?",
-            "confirmDelete": "Are you sure?"
+            "confirmDelete": "Are you sure?",
+            "featureTypeError": "Cannot load layer attributes"
         },
         "background": "Background",
         "language": "Language",

diff --git a/web/client/translations/data.fr-FR b/web/client/translations/data.fr-FR
@@ -35,7 +35,8 @@
             "delete": "Effacer",
             "deleteLayer":"Supprimer le calque",
             "deleteLayerMessage": "Voulez-vous vraiment supprimer cette couche?",
-            "confirmDelete": "Êtes-vous sûr?"
+            "confirmDelete": "Êtes-vous sûr?",
+            "featureTypeError": "Impossible de charger les attributs de calque"
         },
         "enable": "Activer",
         "background": "Fond de plan",

diff --git a/web/client/translations/data.it-IT b/web/client/translations/data.it-IT
@@ -35,7 +35,8 @@
             "delete": "Elimina",
             "deleteLayer":"Elimina Livello",
             "deleteLayerMessage": "Sei sicuro di voler eliminare questo livello?",
-            "confirmDelete": "Sei sicuro?"
+            "confirmDelete": "Sei sicuro?",
+            "featureTypeError": "Impossibile caricare gli attributi del layer"
         },
         "background": "Sfondo",
         "language": "Lingua",