Author: rgodfrey
Date: Thu Jun 21 12:46:09 2012
New Revision: 1352521
URL: http://svn.apache.org/viewvc?rev=1352521&view=rev
Log:
QPID-3999 : prevent double call to event handlers
Modified:
qpid/branches/java-config-and-management/qpid/java/broker-plugins/management/src/main/java/resources/js/qpid/management/Exchange.js
qpid/branches/java-config-and-management/qpid/java/broker-plugins/management/src/main/java/resources/js/qpid/management/VirtualHost.js
qpid/branches/java-config-and-management/qpid/java/broker-plugins/management/src/main/java/resources/js/qpid/management/moveCopyMessages.js
Modified:
qpid/branches/java-config-and-management/qpid/java/broker-plugins/management/src/main/java/resources/js/qpid/management/Exchange.js
URL:
http://svn.apache.org/viewvc/qpid/branches/java-config-and-management/qpid/java/broker-plugins/management/src/main/java/resources/js/qpid/management/Exchange.js?rev=1352521&r1=1352520&r2=1352521&view=diff
==============================================================================
---
qpid/branches/java-config-and-management/qpid/java/broker-plugins/management/src/main/java/resources/js/qpid/management/Exchange.js
(original)
+++
qpid/branches/java-config-and-management/qpid/java/broker-plugins/management/src/main/java/resources/js/qpid/management/Exchange.js
Thu Jun 21 12:46:09 2012
@@ -22,6 +22,7 @@ define(["dojo/_base/xhr",
"dojo/parser",
"dojo/query",
"dojo/_base/connect",
+ "dijit/registry",
"qpid/common/properties",
"qpid/common/updater",
"qpid/common/util",
@@ -29,7 +30,7 @@ define(["dojo/_base/xhr",
"qpid/common/UpdatableStore",
"qpid/management/addBinding",
"dojo/domReady!"],
- function (xhr, parser, query, connect, properties, updater, util,
formatter, UpdatableStore, addBinding) {
+ function (xhr, parser, query, connect, registry, properties, updater,
util, formatter, UpdatableStore, addBinding) {
function Exchange(name, parent, controller) {
this.name = name;
@@ -75,7 +76,7 @@ define(["dojo/_base/xhr",
var addBindingButton = query(".addBindingButton",
contentPane.containerNode)[0];
- connect.connect(addBindingButton, "onclick",
+ connect.connect(registry.byNode(addBindingButton),
"onClick",
function(evt){
addBinding.show({ virtualhost:
that.getVirtualHostName(),
exchange:
that.getExchangeName()});
Modified:
qpid/branches/java-config-and-management/qpid/java/broker-plugins/management/src/main/java/resources/js/qpid/management/VirtualHost.js
URL:
http://svn.apache.org/viewvc/qpid/branches/java-config-and-management/qpid/java/broker-plugins/management/src/main/java/resources/js/qpid/management/VirtualHost.js?rev=1352521&r1=1352520&r2=1352521&view=diff
==============================================================================
---
qpid/branches/java-config-and-management/qpid/java/broker-plugins/management/src/main/java/resources/js/qpid/management/VirtualHost.js
(original)
+++
qpid/branches/java-config-and-management/qpid/java/broker-plugins/management/src/main/java/resources/js/qpid/management/VirtualHost.js
Thu Jun 21 12:46:09 2012
@@ -22,6 +22,7 @@ define(["dojo/_base/xhr",
"dojo/parser",
"dojo/query",
"dojo/_base/connect",
+ "dijit/registry",
"qpid/common/properties",
"qpid/common/updater",
"qpid/common/util",
@@ -30,7 +31,7 @@ define(["dojo/_base/xhr",
"qpid/management/addQueue",
"qpid/management/addExchange",
"dojo/domReady!"],
- function (xhr, parser, query, connect, properties, updater, util,
formatter, UpdatableStore, addQueue, addExchange) {
+ function (xhr, parser, query, connect, registry, properties, updater,
util, formatter, UpdatableStore, addQueue, addExchange) {
function VirtualHost(name, parent, controller) {
this.name = name;
@@ -63,10 +64,10 @@ define(["dojo/_base/xhr",
that.vhostUpdater.update();
var addQueueButton = query(".addQueueButton",
contentPane.containerNode)[0];
- connect.connect(addQueueButton, "onclick",
function(evt){ addQueue.show(that.name) });
+ connect.connect(registry.byNode(addQueueButton),
"onClick", function(evt){ addQueue.show(that.name) });
var addExchangeButton =
query(".addExchangeButton", contentPane.containerNode)[0];
- connect.connect(addExchangeButton, "onclick",
function(evt){ addExchange.show(that.name) });
+
connect.connect(registry.byNode(addExchangeButton), "onClick", function(evt){
addExchange.show(that.name) });
}});
};
Modified:
qpid/branches/java-config-and-management/qpid/java/broker-plugins/management/src/main/java/resources/js/qpid/management/moveCopyMessages.js
URL:
http://svn.apache.org/viewvc/qpid/branches/java-config-and-management/qpid/java/broker-plugins/management/src/main/java/resources/js/qpid/management/moveCopyMessages.js?rev=1352521&r1=1352520&r2=1352521&view=diff
==============================================================================
---
qpid/branches/java-config-and-management/qpid/java/broker-plugins/management/src/main/java/resources/js/qpid/management/moveCopyMessages.js
(original)
+++
qpid/branches/java-config-and-management/qpid/java/broker-plugins/management/src/main/java/resources/js/qpid/management/moveCopyMessages.js
Thu Jun 21 12:46:09 2012
@@ -47,7 +47,7 @@ define(["dojo/_base/xhr",
var cancelButton = query(".moveMessageCancel")[0];
- connect.connect(cancelButton, "onclick",
+ connect.connect(registry.byNode(cancelButton),
"onClick",
function(evt){
event.stop(evt);
registry.byId("moveMessages").hide();
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
