[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16389233#comment-16389233
 ] 

ASF GitHub Bot commented on ARTEMIS-1715:
-

Github user michaelandrepearce commented on the issue:

https://github.com/apache/activemq-artemis/pull/1913
  
@clebertsuconic yup, looks better, i've merged.


> Disable to remove a divert from hawtio console
> --
>
> Key: ARTEMIS-1715
> URL: https://issues.apache.org/jira/browse/ARTEMIS-1715
> Project: ActiveMQ Artemis
>  Issue Type: Bug
>Reporter: Stanislav Knot
>Assignee: Stanislav Knot
>Priority: Major
>
> Removing divert is broken (broker tries to remove diverted address, not 
> divert itself). It should be disabled to remove divert.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16389230#comment-16389230
 ] 

ASF GitHub Bot commented on ARTEMIS-1715:
-

Github user asfgit closed the pull request at:

https://github.com/apache/activemq-artemis/pull/1913


> Disable to remove a divert from hawtio console
> --
>
> Key: ARTEMIS-1715
> URL: https://issues.apache.org/jira/browse/ARTEMIS-1715
> Project: ActiveMQ Artemis
>  Issue Type: Bug
>Reporter: Stanislav Knot
>Assignee: Stanislav Knot
>Priority: Major
>
> Removing divert is broken (broker tries to remove diverted address, not 
> divert itself). It should be disabled to remove divert.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16389193#comment-16389193
 ] 

ASF GitHub Bot commented on ARTEMIS-1715:
-

Github user stanlyDoge commented on the issue:

https://github.com/apache/activemq-artemis/pull/1913
  
@clebertsuconic Done


> Disable to remove a divert from hawtio console
> --
>
> Key: ARTEMIS-1715
> URL: https://issues.apache.org/jira/browse/ARTEMIS-1715
> Project: ActiveMQ Artemis
>  Issue Type: Bug
>Reporter: Stanislav Knot
>Assignee: Stanislav Knot
>Priority: Major
>
> Removing divert is broken (broker tries to remove diverted address, not 
> divert itself). It should be disabled to remove divert.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16388897#comment-16388897
 ] 

ASF GitHub Bot commented on ARTEMIS-1715:
-

Github user clebertsuconic commented on the issue:

https://github.com/apache/activemq-artemis/pull/1913
  
@michaelandrepearce is this good to merge? (the PR check failure is 
probably not related).

@stanlyDoge if you could please rebase to have a re-run?


> Disable to remove a divert from hawtio console
> --
>
> Key: ARTEMIS-1715
> URL: https://issues.apache.org/jira/browse/ARTEMIS-1715
> Project: ActiveMQ Artemis
>  Issue Type: Bug
>Reporter: Stanislav Knot
>Assignee: Stanislav Knot
>Priority: Major
>
> Removing divert is broken (broker tries to remove diverted address, not 
> divert itself). It should be disabled to remove divert.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16387478#comment-16387478
 ] 

ASF GitHub Bot commented on ARTEMIS-1715:
-

Github user stanlyDoge commented on a diff in the pull request:

https://github.com/apache/activemq-artemis/pull/1913#discussion_r172447340
  
--- Diff: 
artemis-hawtio/artemis-plugin/src/main/webapp/plugin/js/artemisPlugin.js ---
@@ -96,6 +96,9 @@ var ARTEMIS = (function(ARTEMIS) {
  .when('/artemis/deleteAddress', {
templateUrl: ARTEMIS.templatePath + 'deleteAddress.html'
  })
+ .when('/artemis/deleteDivert', {
--- End diff --

Well, it shows page without such button. But I agree it would be better to 
hide this button in "toolbar". I will take a look at this.


> Disable to remove a divert from hawtio console
> --
>
> Key: ARTEMIS-1715
> URL: https://issues.apache.org/jira/browse/ARTEMIS-1715
> Project: ActiveMQ Artemis
>  Issue Type: Bug
>Reporter: Stanislav Knot
>Assignee: Stanislav Knot
>Priority: Major
>
> Removing divert is broken (broker tries to remove diverted address, not 
> divert itself). It should be disabled to remove divert.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16387474#comment-16387474
 ] 

ASF GitHub Bot commented on ARTEMIS-1715:
-

Github user michaelandrepearce commented on a diff in the pull request:

https://github.com/apache/activemq-artemis/pull/1913#discussion_r172446748
  
--- Diff: 
artemis-hawtio/artemis-plugin/src/main/webapp/plugin/js/artemisPlugin.js ---
@@ -96,6 +96,9 @@ var ARTEMIS = (function(ARTEMIS) {
  .when('/artemis/deleteAddress', {
templateUrl: ARTEMIS.templatePath + 'deleteAddress.html'
  })
+ .when('/artemis/deleteDivert', {
--- End diff --

Then the consoles should hide the button, if divert is selected.


> Disable to remove a divert from hawtio console
> --
>
> Key: ARTEMIS-1715
> URL: https://issues.apache.org/jira/browse/ARTEMIS-1715
> Project: ActiveMQ Artemis
>  Issue Type: Bug
>Reporter: Stanislav Knot
>Assignee: Stanislav Knot
>Priority: Major
>
> Removing divert is broken (broker tries to remove diverted address, not 
> divert itself). It should be disabled to remove divert.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16387471#comment-16387471
 ] 

ASF GitHub Bot commented on ARTEMIS-1715:
-

Github user stanlyDoge commented on a diff in the pull request:

https://github.com/apache/activemq-artemis/pull/1913#discussion_r172446094
  
--- Diff: 
artemis-hawtio/artemis-plugin/src/main/webapp/plugin/js/artemisPlugin.js ---
@@ -96,6 +96,9 @@ var ARTEMIS = (function(ARTEMIS) {
  .when('/artemis/deleteAddress', {
templateUrl: ARTEMIS.templatePath + 'deleteAddress.html'
  })
+ .when('/artemis/deleteDivert', {
--- End diff --

Current behaviour is this: user has divert selected and wants to remove it. 
So he click on Delete. But hawtio console does not recognize divert and tries 
to remove address at which the divert is created.


> Disable to remove a divert from hawtio console
> --
>
> Key: ARTEMIS-1715
> URL: https://issues.apache.org/jira/browse/ARTEMIS-1715
> Project: ActiveMQ Artemis
>  Issue Type: Bug
>Reporter: Stanislav Knot
>Assignee: Stanislav Knot
>Priority: Major
>
> Removing divert is broken (broker tries to remove diverted address, not 
> divert itself). It should be disabled to remove divert.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16386695#comment-16386695
 ] 

ASF GitHub Bot commented on ARTEMIS-1715:
-

Github user michaelandrepearce commented on a diff in the pull request:

https://github.com/apache/activemq-artemis/pull/1913#discussion_r172316744
  
--- Diff: 
artemis-hawtio/artemis-plugin/src/main/webapp/plugin/js/artemisPlugin.js ---
@@ -231,6 +234,17 @@ var ARTEMIS = (function(ARTEMIS) {
  }
   });
 
+  workspace.subLevelTabs.push({
--- End diff --

If the intent is not to delete a divert as per reply to previous comment, 
then this should not be added


> Disable to remove a divert from hawtio console
> --
>
> Key: ARTEMIS-1715
> URL: https://issues.apache.org/jira/browse/ARTEMIS-1715
> Project: ActiveMQ Artemis
>  Issue Type: Bug
>Reporter: Stanislav Knot
>Assignee: Stanislav Knot
>Priority: Major
>
> Removing divert is broken (broker tries to remove diverted address, not 
> divert itself). It should be disabled to remove divert.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16386697#comment-16386697
 ] 

ASF GitHub Bot commented on ARTEMIS-1715:
-

Github user michaelandrepearce commented on a diff in the pull request:

https://github.com/apache/activemq-artemis/pull/1913#discussion_r172316762
  
--- Diff: 
artemis-hawtio/artemis-plugin/src/main/webapp/plugin/js/artemisPlugin.js ---
@@ -96,6 +96,9 @@ var ARTEMIS = (function(ARTEMIS) {
  .when('/artemis/deleteAddress', {
templateUrl: ARTEMIS.templatePath + 'deleteAddress.html'
  })
+ .when('/artemis/deleteDivert', {
--- End diff --

If the intent is not to delete a divert as per reply to previous comment, 
then this should not be added


> Disable to remove a divert from hawtio console
> --
>
> Key: ARTEMIS-1715
> URL: https://issues.apache.org/jira/browse/ARTEMIS-1715
> Project: ActiveMQ Artemis
>  Issue Type: Bug
>Reporter: Stanislav Knot
>Assignee: Stanislav Knot
>Priority: Major
>
> Removing divert is broken (broker tries to remove diverted address, not 
> divert itself). It should be disabled to remove divert.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16385821#comment-16385821
 ] 

ASF GitHub Bot commented on ARTEMIS-1715:
-

Github user michaelandrepearce commented on a diff in the pull request:

https://github.com/apache/activemq-artemis/pull/1913#discussion_r172121469
  
--- Diff: 
artemis-hawtio/artemis-plugin/src/main/webapp/plugin/html/deleteDivert.html ---
@@ -0,0 +1,24 @@
+
+
--- End diff --

Eg there is code in this pr that signifies otherwise


> Disable to remove a divert from hawtio console
> --
>
> Key: ARTEMIS-1715
> URL: https://issues.apache.org/jira/browse/ARTEMIS-1715
> Project: ActiveMQ Artemis
>  Issue Type: Bug
>Reporter: Stanislav Knot
>Assignee: Stanislav Knot
>Priority: Major
>
> Removing divert is broken (broker tries to remove diverted address, not 
> divert itself). It should be disabled to remove divert.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16385819#comment-16385819
 ] 

ASF GitHub Bot commented on ARTEMIS-1715:
-

Github user michaelandrepearce commented on a diff in the pull request:

https://github.com/apache/activemq-artemis/pull/1913#discussion_r172121106
  
--- Diff: 
artemis-hawtio/artemis-plugin/src/main/webapp/plugin/html/deleteDivert.html ---
@@ -0,0 +1,24 @@
+
+
--- End diff --

If that’s the case please close this pr 


> Disable to remove a divert from hawtio console
> --
>
> Key: ARTEMIS-1715
> URL: https://issues.apache.org/jira/browse/ARTEMIS-1715
> Project: ActiveMQ Artemis
>  Issue Type: Bug
>Reporter: Stanislav Knot
>Assignee: Stanislav Knot
>Priority: Major
>
> Removing divert is broken (broker tries to remove diverted address, not 
> divert itself). It should be disabled to remove divert.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16385786#comment-16385786
 ] 

ASF GitHub Bot commented on ARTEMIS-1715:
-

Github user stanlyDoge commented on a diff in the pull request:

https://github.com/apache/activemq-artemis/pull/1913#discussion_r172115995
  
--- Diff: 
artemis-hawtio/artemis-plugin/src/main/webapp/plugin/html/deleteDivert.html ---
@@ -0,0 +1,24 @@
+
+
--- End diff --

Hi Michael. After discussion with @andytaylor we decided that divert should 
not be deletable from web console, so there is no need to have pop-up.


> Disable to remove a divert from hawtio console
> --
>
> Key: ARTEMIS-1715
> URL: https://issues.apache.org/jira/browse/ARTEMIS-1715
> Project: ActiveMQ Artemis
>  Issue Type: Bug
>Reporter: Stanislav Knot
>Assignee: Stanislav Knot
>Priority: Major
>
> Removing divert is broken (broker tries to remove diverted address, not 
> divert itself). It should be disabled to remove divert.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16385563#comment-16385563
 ] 

ASF GitHub Bot commented on ARTEMIS-1715:
-

Github user michaelandrepearce commented on a diff in the pull request:

https://github.com/apache/activemq-artemis/pull/1913#discussion_r172086640
  
--- Diff: 
artemis-hawtio/artemis-plugin/src/main/webapp/plugin/html/deleteDivert.html ---
@@ -0,0 +1,24 @@
+
+
--- End diff --

Please see other delete's note the extra are you sure kind of pop up. This 
should be the same, see 
https://github.com/stanlyDoge/activemq-artemis/blob/0809b7794e16cd8981209f30f7ec3adf6ce0a114/artemis-hawtio/artemis-plugin/src/main/webapp/plugin/html/deleteAddress.html


> Disable to remove a divert from hawtio console
> --
>
> Key: ARTEMIS-1715
> URL: https://issues.apache.org/jira/browse/ARTEMIS-1715
> Project: ActiveMQ Artemis
>  Issue Type: Bug
>Reporter: Stanislav Knot
>Assignee: Stanislav Knot
>Priority: Major
>
> Removing divert is broken (broker tries to remove diverted address, not 
> divert itself). It should be disabled to remove divert.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16385558#comment-16385558
 ] 

ASF GitHub Bot commented on ARTEMIS-1715:
-

Github user michaelandrepearce commented on a diff in the pull request:

https://github.com/apache/activemq-artemis/pull/1913#discussion_r172086296
  
--- Diff: 
artemis-hawtio/artemis-plugin/src/main/webapp/plugin/js/artemisPlugin.js ---
@@ -96,6 +96,9 @@ var ARTEMIS = (function(ARTEMIS) {
  .when('/artemis/deleteAddress', {
templateUrl: ARTEMIS.templatePath + 'deleteAddress.html'
  })
+ . when('/artemis/deleteDivert', {
--- End diff --

space between . and when


> Disable to remove a divert from hawtio console
> --
>
> Key: ARTEMIS-1715
> URL: https://issues.apache.org/jira/browse/ARTEMIS-1715
> Project: ActiveMQ Artemis
>  Issue Type: Bug
>Reporter: Stanislav Knot
>Assignee: Stanislav Knot
>Priority: Major
>
> Removing divert is broken (broker tries to remove diverted address, not 
> divert itself). It should be disabled to remove divert.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (ARTEMIS-1715) Disable to remove a divert from hawtio console

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/ARTEMIS-1715?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16382062#comment-16382062
 ] 

ASF GitHub Bot commented on ARTEMIS-1715:
-

GitHub user stanlyDoge opened a pull request:

https://github.com/apache/activemq-artemis/pull/1913

ARTEMIS-1715 Disable to remove a divert from hawtio console



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/stanlyDoge/activemq-artemis ARTEMIS-1715

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/activemq-artemis/pull/1913.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #1913


commit 0809b7794e16cd8981209f30f7ec3adf6ce0a114
Author: Stanislav Knot 
Date:   2018-03-01T14:29:22Z

ARTEMIS-1715 Disable to remove a divert from hawtio console




> Disable to remove a divert from hawtio console
> --
>
> Key: ARTEMIS-1715
> URL: https://issues.apache.org/jira/browse/ARTEMIS-1715
> Project: ActiveMQ Artemis
>  Issue Type: Bug
>Reporter: Stanislav Knot
>Assignee: Stanislav Knot
>Priority: Major
>
> Removing divert is broken (broker tries to remove diverted address, not 
> divert itself). It should be disabled to remove divert.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)