incubator-ambari-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From yus...@apache.org
Subject svn commit: r1363463 - in /incubator/ambari/trunk: CHANGES.txt hmc/js/configureServicesUtils.js
Date Thu, 19 Jul 2012 18:30:23 GMT
Author: yusaku
Date: Thu Jul 19 18:30:22 2012
New Revision: 1363463

URL: http://svn.apache.org/viewvc?rev=1363463&view=rev
Log:
AMBARI-654. Enter the value of the service properties that are not meant to be displayed on
UI in ServiceConfig table (Contributed by Jaimin Jetly)

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

Modified: incubator/ambari/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/incubator/ambari/trunk/CHANGES.txt?rev=1363463&r1=1363462&r2=1363463&view=diff
==============================================================================
--- incubator/ambari/trunk/CHANGES.txt (original)
+++ incubator/ambari/trunk/CHANGES.txt Thu Jul 19 18:30:22 2012
@@ -81,6 +81,9 @@ Trunk (unreleased changes)
 
   BUG FIXES
 
+    AMBARI-654. Enter the value of the service properties that are not meant to
+    be displayed on UI in ServiceConfig table (Jaimin Jetly via yusaku) 
+
     AMBARI-602. Fix install docs for 64-bit only support and how to pre-setup
     cluster for ambari-agent for testing (hitesh)
 

Modified: incubator/ambari/trunk/hmc/js/configureServicesUtils.js
URL: http://svn.apache.org/viewvc/incubator/ambari/trunk/hmc/js/configureServicesUtils.js?rev=1363463&r1=1363462&r2=1363463&view=diff
==============================================================================
--- incubator/ambari/trunk/hmc/js/configureServicesUtils.js (original)
+++ incubator/ambari/trunk/hmc/js/configureServicesUtils.js Thu Jul 19 18:30:22 2012
@@ -27,7 +27,7 @@
     function convertDisplayType(displayType) {
       switch (displayType) {
         case "NODISPLAY":
-          return "NODISPLAY";
+          return "hidden";
         case "TEXT":
           return "text";
         case "PASSWORD":
@@ -58,7 +58,11 @@
 
       var checked = (type === 'checkbox' && option.value === 'true') ? 'checked'
: '';
 
-      var retString = '<div class="formElement">' +
+      var retString = '<div class="formElement"';
+      if (type === "hidden") {
+        retString += 'style = "display:none"';
+      }
+      retString += '>' +
         '<label for="' + service + '">' + option.displayName + '</label>' +
         '<input class="' + unitClass + '" type="' + type + '" id="' + property + '" name="'
+ service + '" value="' + option.value + '" ' + checked;
       if (readOnlyFlag) {
@@ -107,9 +111,6 @@
             // service has configs, so needs render
             var type = convertDisplayType(service.properties[property].type);
             // Y.log("TYPE: " + type + "Property: " + property);
-            if (type === "NODISPLAY") {
-                continue;
-            }
             serviceNeedsRender = true;
             var unit = service.properties[property].unit;
             var displayAttributes = null;



Mime
View raw message