You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by kw...@apache.org on 2015/12/01 13:04:08 UTC
svn commit: r1717401 - in
/qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management:
addAuthenticationProvider.js preferencesprovider/PreferencesProviderForm.js
Author: kwall
Date: Tue Dec 1 12:04:08 2015
New Revision: 1717401
URL: http://svn.apache.org/viewvc?rev=1717401&view=rev
Log:
QPID-6922: [Web Management UI] Fix defect causing error during edit of authentication provider
Modified:
qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management/addAuthenticationProvider.js
qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management/preferencesprovider/PreferencesProviderForm.js
Modified: qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management/addAuthenticationProvider.js
URL: http://svn.apache.org/viewvc/qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management/addAuthenticationProvider.js?rev=1717401&r1=1717400&r2=1717401&view=diff
==============================================================================
--- qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management/addAuthenticationProvider.js (original)
+++ qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management/addAuthenticationProvider.js Tue Dec 1 12:04:08 2015
@@ -61,7 +61,9 @@ define(["dojo/dom",
var that = this;
this.authenticationProviderName = registry.byId("addAuthenticationProvider.name");
this.authenticationProviderName.set("regExpGen", util.nameOrContextVarRegexp);
- this.authenticationProviderName.on("change", function(newValue){that.preferencesProviderForm.preferencesProviderNameWidget.set("value",newValue);});
+ this.authenticationProviderName.on("change", function(newValue){
+ that.preferencesProviderForm.setPreferencesProviderName(newValue);
+ });
this.dialog = registry.byId("addAuthenticationProvider");
this.addButton = registry.byId("addAuthenticationProvider.addButton");
@@ -110,7 +112,7 @@ define(["dojo/dom",
else
{
that.preferencesProviderForm.reset();
- that.preferencesProviderForm.preferencesProviderNameWidget.set("value", actualData.name);
+ that.preferencesProviderForm.setPreferencesProviderName(actualData.name);
}
that.authenticationProviderName.set("value", actualData.name);
that._show();
Modified: qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management/preferencesprovider/PreferencesProviderForm.js
URL: http://svn.apache.org/viewvc/qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management/preferencesprovider/PreferencesProviderForm.js?rev=1717401&r1=1717400&r2=1717401&view=diff
==============================================================================
--- qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management/preferencesprovider/PreferencesProviderForm.js (original)
+++ qpid/java/trunk/broker-plugins/management-http/src/main/java/resources/js/qpid/management/preferencesprovider/PreferencesProviderForm.js Tue Dec 1 12:04:08 2015
@@ -95,6 +95,13 @@ function (util, declare, array, domConst
{
return this.preferencesProviderNameWidget.get("value");
},
+ setPreferencesProviderName: function(name)
+ {
+ if (!(this.data && this.data.name))
+ {
+ this.preferencesProviderNameWidget.set("value", name);
+ }
+ },
setMetadata: function(metadata)
{
this.metadata = metadata;
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org