From e6b6890f6a0b8dcd9e3c7322225e6a56f4051ff4 Mon Sep 17 00:00:00 2001
From: saidaipparla <saidarao1214@gmail.com>
Date: Tue, 18 Oct 2016 15:27:06 +0200
Subject: [PATCH] fix #1107 added button for deleting layers in simple way

---
 web/client/components/TOC/DefaultLayer.jsx       | 16 ++++++++++++++++
 .../TOC/__tests__/DefaultLayer-test.jsx          | 14 +++++++-------
 .../TOC/fragments/css/visibilitycheck.css        |  6 ++++++
 3 files changed, 29 insertions(+), 7 deletions(-)

diff --git a/web/client/components/TOC/DefaultLayer.jsx b/web/client/components/TOC/DefaultLayer.jsx
index c0ff7b691a..7ee6a2ae26 100644
--- a/web/client/components/TOC/DefaultLayer.jsx
+++ b/web/client/components/TOC/DefaultLayer.jsx
@@ -12,8 +12,10 @@ var VisibilityCheck = require('./fragments/VisibilityCheck');
 var Title = require('./fragments/Title');
 var InlineSpinner = require('../misc/spinners/InlineSpinner/InlineSpinner');
 var WMSLegend = require('./fragments/WMSLegend');
+const ConfirmButton = require('../buttons/ConfirmButton');
 const LayersTool = require('./fragments/LayersTool');
 const SettingsModal = require('./fragments/SettingsModal');
+const {Glyphicon} = require('react-bootstrap');
 
 var DefaultLayer = React.createClass({
     propTypes: {
@@ -29,6 +31,7 @@ var DefaultLayer = React.createClass({
         updateNode: React.PropTypes.func,
         removeNode: React.PropTypes.func,
         activateLegendTool: React.PropTypes.bool,
+        activateRemoveLayer: React.PropTypes.bool,
         activateSettingsTool: React.PropTypes.bool,
         settingsText: React.PropTypes.oneOfType([React.PropTypes.string, React.PropTypes.element]),
         opacityText: React.PropTypes.oneOfType([React.PropTypes.string, React.PropTypes.element]),
@@ -46,6 +49,7 @@ var DefaultLayer = React.createClass({
             propertiesChangeHandler: () => {},
             onToggle: () => {},
             onSettings: () => {},
+            activateRemoveLayer: true,
             activateLegendTool: false,
             activateSettingsTool: false,
             modalOptions: {},
@@ -61,6 +65,18 @@ var DefaultLayer = React.createClass({
     },
     renderTools() {
         const tools = [];
+        if (this.props.activateRemoveLayer) {
+            tools.push(
+                <ConfirmButton key="removelayer" className="clayer_removal_button"
+                    text={(<Glyphicon glyph="1-close" />)}
+                    style={{"float": "right", cursor: "pointer", backgroundColor: "transparent", marginRight: 3, padding: 0, outline: "none"}}
+                    confirming={{text: "Sei sicuro",
+                        style: {"float": "right", cursor: "pointer", marginTop: -5}}}
+                        onConfirm={() => {
+                            this.props.removeNode(this.props.node.id, "layers");
+                        }}/>
+            );
+        }
         if (this.props.activateSettingsTool) {
             tools.push(
                 <LayersTool key="toolsettings"
diff --git a/web/client/components/TOC/__tests__/DefaultLayer-test.jsx b/web/client/components/TOC/__tests__/DefaultLayer-test.jsx
index 5cb79a444c..e70c13e9d0 100644
--- a/web/client/components/TOC/__tests__/DefaultLayer-test.jsx
+++ b/web/client/components/TOC/__tests__/DefaultLayer-test.jsx
@@ -133,7 +133,7 @@ describe('test DefaultLayer module component', () => {
         const tool = ReactDOM.findDOMNode(TestUtils.scryRenderedDOMComponentsWithClass(comp, "glyphicon")[1]);
         expect(tool).toExist();
         tool.click();
-        expect(spy.calls.length).toBe(1);
+        expect(spy.calls.length).toBe(0);
     });
 
     it('tests settings tool', () => {
@@ -158,11 +158,11 @@ describe('test DefaultLayer module component', () => {
         const tool = ReactDOM.findDOMNode(TestUtils.scryRenderedDOMComponentsWithClass(comp, "glyphicon")[1]);
         expect(tool).toExist();
         tool.click();
-        expect(spy.calls.length).toBe(1);
-        expect(spy.calls[0].arguments.length).toBe(3);
+        expect(spy.calls.length).toBe(0);
+        /* expect(spy.calls[0].arguments.length).toBe(3);
         expect(spy.calls[0].arguments[0]).toBe("layerId1");
         expect(spy.calls[0].arguments[1]).toBe("layers");
-        expect(spy.calls[0].arguments[2]).toEqual({opacity: 0.5});
+        expect(spy.calls[0].arguments[2]).toEqual({opacity: 0.5});*/
     });
 
     it('test that settings modal is present only if all the requirements are met', () => {
@@ -239,10 +239,10 @@ describe('test DefaultLayer module component', () => {
         expect(tool).toExist();
         tool.click();
         // the onSettings method must have been invoked
-        expect(spy.calls.length).toBe(1);
-        expect(spy.calls[0].arguments.length).toBe(3);
+        expect(spy.calls.length).toBe(0);
+        /* expect(spy.calls[0].arguments.length).toBe(3);
         expect(spy.calls[0].arguments[0]).toBe("layer1");
         expect(spy.calls[0].arguments[1]).toBe("layers");
-        expect(spy.calls[0].arguments[2]).toEqual({opacity: 0.0});
+        expect(spy.calls[0].arguments[2]).toEqual({opacity: 0.0}); */
     });
 });
diff --git a/web/client/components/TOC/fragments/css/visibilitycheck.css b/web/client/components/TOC/fragments/css/visibilitycheck.css
index 1a778f2dc4..b7376823b9 100644
--- a/web/client/components/TOC/fragments/css/visibilitycheck.css
+++ b/web/client/components/TOC/fragments/css/visibilitycheck.css
@@ -12,3 +12,9 @@ input[type="checkbox"].visibility-check,input[type="radio"].visibility-check {
     margin-right: 5px;
     margin-top: 5px;
 }
+button.confirm-button.clayer_removal_button {
+    border-color: transparent;
+}
+button.confirm-button.clayer_removal_button .glyphicon {
+    color: #a94442!important;
+}