AMBARI-1961. Select Services: clicking on "all" selects HUE even when HUE support is toggled off. (yusaku)

git-svn-id: https://svn.apache.org/repos/asf/incubator/ambari/trunk@1469155 13f79535-47bb-0310-9956-ffa450edef68

CHANGES.txt

@@ -749,6 +749,9 @@ Trunk (unreleased changes):
 
  BUG FIXES
 
+ AMBARI-1961. Select Services: clicking on "all" selects HUE even when HUE
+ support is toggled off. (yusaku)
+
  AMBARI-1960. "Back" button can be pressed while host registration is taking
  process, even though the button seems disabled. (yusaku)
 

ambari-web/app/controllers/wizard.js

@@ -468,6 +468,7 @@ App.WizardController = Em.Controller.extend({
           displayName: displayOrderConfig[i].displayName,
           isDisabled: i === 0,
           isSelected: displayOrderConfig[i].isSelected,
+          canBeSelected: displayOrderConfig[i].canBeSelected,
           isInstalled: false,
           isHidden: displayOrderConfig[i].isHidden,
           description: entry.comment,

ambari-web/app/controllers/wizard/step4_controller.js

@@ -31,7 +31,7 @@ App.WizardStep4Controller = Em.ArrayController.extend({
    * Check whether all properties are selected
    */
   isAll: function () {
-    return this.everyProperty('isSelected', true);
+    return this.filterProperty('canBeSelected', true).everyProperty('isSelected', true);
   }.property('@each.isSelected'),
 
   /**
@@ -63,7 +63,7 @@ App.WizardStep4Controller = Em.ArrayController.extend({
    * Onclick handler for <code>select all</code> link
    */
   selectAll: function () {
-    this.setEach('isSelected', true);
+    this.filterProperty('canBeSelected', true).setEach('isSelected', true);
   },
 
   /**

ambari-web/app/data/services.js

@@ -24,6 +24,7 @@ module.exports = [
     displayName: 'HDFS',
     isDisabled: true,
     isSelected: true,
+    canBeSelected: true,
     description: Em.I18n.t('services.hdfs.description')
   },
   {
@@ -31,6 +32,7 @@ module.exports = [
     displayName: 'MapReduce',
     isDisabled: false,
     isSelected: true,
+    canBeSelected: true,
     description: Em.I18n.t('services.mapreduce.description')
   },
   {
@@ -38,6 +40,7 @@ module.exports = [
     displayName: 'Nagios',
     isDisabled: false,
     isSelected: true,
+    canBeSelected: true,
     description: Em.I18n.t('services.nagios.description')
   },
   {
@@ -45,6 +48,7 @@ module.exports = [
     displayName: 'Ganglia',
     isDisabled: false,
     isSelected: true,
+    canBeSelected: true,
     description: Em.I18n.t('services.ganglia.description')
   },
   {
@@ -52,6 +56,7 @@ module.exports = [
     displayName: 'Hive + HCat + ZooKeeper',
     isDisabled: false,
     isSelected: true,
+    canBeSelected: true,
     description: Em.I18n.t('services.hive.description')
   },
   {
@@ -59,6 +64,7 @@ module.exports = [
     displayName: 'HCatalog',
     isDisabled: false,
     isSelected: true,
+    canBeSelected: true,
     isHidden: true
   },
   {
@@ -66,6 +72,7 @@ module.exports = [
     displayName: 'WebHCat',
     isDisabled: false,
     isSelected: true,
+    canBeSelected: true,
     isHidden: true
   },
   {
@@ -73,6 +80,7 @@ module.exports = [
     displayName: 'HBase + ZooKeeper',
     isDisabled: false,
     isSelected: true,
+    canBeSelected: true,
     description: Em.I18n.t('services.hbase.description')
   },
   {
@@ -80,6 +88,7 @@ module.exports = [
     displayName: 'Pig',
     isDisabled: false,
     isSelected: true,
+    canBeSelected: true,
     description: Em.I18n.t('services.pig.description')
   },
   {
@@ -87,6 +96,7 @@ module.exports = [
     displayName: 'Sqoop',
     isDisabled: false,
     isSelected: true,
+    canBeSelected: true,
     description: Em.I18n.t('services.sqoop.description')
   },
   {
@@ -94,6 +104,7 @@ module.exports = [
     displayName: 'Oozie',
     isDisabled: false,
     isSelected: true,
+    canBeSelected: true,
     description: Em.I18n.t('services.oozie.description')
   },
   {
@@ -101,6 +112,7 @@ module.exports = [
 	  displayName: 'ZooKeeper',
     isDisabled: false,
     isSelected: true,
+    canBeSelected: true,
     isHidden: true
   },
   {
@@ -108,6 +120,7 @@ module.exports = [
     displayName: 'Hue',
     isDisabled: false,
     isSelected: App.supports.hue,
+    canBeSelected: App.supports.hue,
     isHidden: !App.supports.hue
   }
 ]