More warnings fix (#1565)

geosolutions-it · Mar 15, 2017 · d8207c4 · d8207c4
1 parent 2394a46
commit d8207c4
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 12 deletions.
diff --git a/docs/developer-guide/map-plugin.md b/docs/developer-guide/map-plugin.md
@@ -60,7 +60,7 @@ const TestSupport = React.createClass({
 
 module.exports = TestSupport;
 ```
- * include the tool(s) in the requires section of plugins.js amd give it a **name**:
+ * include the tool(s) in the requires section of plugins.js and give it a **name**:
 
 ```js
 module.exports = {
@@ -97,3 +97,5 @@ module.exports = {
   }
 }
 ```
+
+NOTE: When using the "impl" configuration you are responsible for the correct configuration of such tool, **remember to add any other property it may require in the configuration**.
diff --git a/package.json b/package.json
@@ -130,7 +130,7 @@
     "react-responsive": "1.1.3",
     "react-selectize": "2.0.3",
     "react-share": "1.8.0",
-    "react-sidebar": "2.1.2",
+    "react-sidebar": "2.3.0",
     "react-sortable-items": "https://github.com/geosolutions-it/react-sortable-items/tarball/react15",
     "react-spinkit": "2.1.1",
     "react-swipeable-views": "0.11.1",

diff --git a/web/client/components/catalog/Catalog.jsx b/web/client/components/catalog/Catalog.jsx
@@ -135,11 +135,11 @@ const Catalog = React.createClass({
           maxButtons={5}
           activePage={page + 1}
           onSelect={this.handlePage} />
-        <div className="push-right">
-            <Message msgId="catalog.pageInfo" msgParams={{start, end: start + returned - 1, total}} />
-            {this.renderLoading()}
-        </div>
-  </div>);
+            <div className="push-right">
+                <Message msgId="catalog.pageInfo" msgParams={{start, end: start + returned - 1, total}} />
+                {this.renderLoading()}
+            </div>
+          </div>);
     },
     renderRecords() {
         return (<div>
@@ -166,16 +166,15 @@ const Catalog = React.createClass({
         }
     },
     renderButtons() {
-        // TODO check this part in ms2, customize for webmapper
         const buttons = [];
         if (this.props.includeSearchButton) {
             buttons.push(<Button bsStyle="primary" style={this.props.buttonStyle} onClick={this.search}
-                        className={this.props.buttonClassName}>
+                        className={this.props.buttonClassName} key="catalog_search_button">
                         {this.renderLoading()} <Message msgId="catalog.search"/>
                     </Button>);
         }
         if (this.props.includeResetButton) {
-            buttons.push(<Button style={this.props.buttonStyle} onClick={this.reset}>
+            buttons.push(<Button style={this.props.buttonStyle} onClick={this.reset} key="catalog_reset_button">
                         <Message msgId="catalog.reset"/>
                     </Button>);
         }
@@ -188,7 +187,7 @@ const Catalog = React.createClass({
         return null;
     },
     renderFormats() {
-        return this.props.formats.map((format) => <option value={format.name}>{format.label}</option>);
+        return this.props.formats.map((format) => <option value={format.name} key={format.name}>{format.label}</option>);
     },
     render() {
         const textSearch = (<FormGroup><FormControl

diff --git a/web/client/components/data/identify/DefaultViewer.jsx b/web/client/components/data/identify/DefaultViewer.jsx
@@ -135,7 +135,6 @@ const DefaultViewer = React.createClass({
                 <Container {...this.props.containerProps}
                     onChangeIndex={(index) => {this.setState({index}); }}
                     ref="container"
-                    defaultActiveKey={0}
                     index={this.state.index || 0}
                     key={"swiper"}
                     className="swipeable-view"