You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by yu...@apache.org on 2012/06/26 20:37:18 UTC

svn commit: r1354157 - in /incubator/ambari/trunk: CHANGES.txt hmc/js/configureServices.js

Author: yusaku
Date: Tue Jun 26 18:37:16 2012
New Revision: 1354157

URL: http://svn.apache.org/viewvc?rev=1354157&view=rev
Log:
AMBARI-578. Custom Config page: don't allow form submission if there are client-side validation errors (Contributed by yusaku)

Modified:
    incubator/ambari/trunk/CHANGES.txt
    incubator/ambari/trunk/hmc/js/configureServices.js

Modified: incubator/ambari/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/incubator/ambari/trunk/CHANGES.txt?rev=1354157&r1=1354156&r2=1354157&view=diff
==============================================================================
--- incubator/ambari/trunk/CHANGES.txt (original)
+++ incubator/ambari/trunk/CHANGES.txt Tue Jun 26 18:37:16 2012
@@ -6,6 +6,9 @@ characters wide.
 
 Release 1.0.0 - unreleased
 
+  AMBARI-578. Custom Config page: don't allow form submission if there are
+  client-side validation errors (yusaku)
+
   AMBARI-411. The HBase puppet templates could include some more config knobs
   (Jaimin Jetly via yusaku)
 

Modified: incubator/ambari/trunk/hmc/js/configureServices.js
URL: http://svn.apache.org/viewvc/incubator/ambari/trunk/hmc/js/configureServices.js?rev=1354157&r1=1354156&r2=1354157&view=diff
==============================================================================
--- incubator/ambari/trunk/hmc/js/configureServices.js (original)
+++ incubator/ambari/trunk/hmc/js/configureServices.js Tue Jun 26 18:37:16 2012
@@ -2,6 +2,10 @@ var globalOptionsInfo = null;
            
 Y.one('#configureClusterAdvancedSubmitButtonId').on('click',function (e) {
   
+  if (this.hasClass('disabled')) {
+    return;
+  }
+  
   var opts = configureServicesUtil.generateUserOpts();
 
   e.target.set('disabled', true);