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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to