AMBARI-4668. Request/task names shown for Maintenance Mode toggling don't make sense. (jaimin)

ambari-web/app/controllers/main/host/details.js

@@ -974,8 +974,9 @@ App.MainHostDetailsController = Em.Controller.extend({
   onOffPassiveModeForHost: function(context) {
     var state = context.active ? 'PASSIVE' : 'ACTIVE';
     var self = this;
+    var message = context.label + ' for host';
     App.showConfirmationPopup(function() {
-          self.hostPassiveModeRequest(state, context.label)
+          self.hostPassiveModeRequest(state, message)
         },
         Em.I18n.t('hosts.passiveMode.popup').format(context.active ? 'On' : 'Off',this.get('content.hostName'))
     );
@@ -1226,22 +1227,24 @@ App.MainHostDetailsController = Em.Controller.extend({
   turnOnOffPassiveConfirmation: function(event){
     var self = this;
     var component = event.context;
-    var state, onOff;
+    var state, onOff, message;
     if (component.get("passiveState") == "PASSIVE") {
       onOff = "Off";
       state = "ACTIVE";
+      message =  Em.I18n.t('passiveState.turnOffFor').format(component.get('displayName'));
     } else {
       onOff = "On";
       state = "PASSIVE";
+      message = Em.I18n.t('passiveState.turnOnFor').format(component.get('displayName'));
     }
     App.showConfirmationPopup(function() {
-          self.turnOnOffPassive(state, component)
+          self.turnOnOffPassive(state, component, message)
         },
         Em.I18n.t('hosts.passiveMode.popup').format(onOff,component.get('displayName'))
     );
   },
 
-  turnOnOffPassive: function(state,component) {
+  turnOnOffPassive: function(state,component,message) {
     var hostName = this.get('content.hostName');
     App.ajax.send({
       name: 'host_component.passive',
@@ -1251,7 +1254,7 @@ App.MainHostDetailsController = Em.Controller.extend({
         hostName: hostName,
         componentName: component.get('componentName'),
         passive_state: state,
-        requestInfo: component.get('componentName') + " " + state
+        requestInfo: message
       },
       success: 'updateComponentPassiveState'
     });

ambari-web/app/messages.js

@@ -190,8 +190,8 @@ Em.I18n.translations = {
   'common.hostLog.popup.errorLog.value': 'errors-{0}.txt',
   'passiveState.turnOn':'Turn On Maintenance Mode',
   'passiveState.turnOff':'Turn Off Maintenance Mode',
-  'passiveState.turnOnFor':'Turn On Maintenance Mode for',
-  'passiveState.turnOffFor':'Turn Off Maintenance Mode for',
+  'passiveState.turnOnFor':'Turn On Maintenance Mode for {0}',
+  'passiveState.turnOffFor':'Turn Off Maintenance Mode for {0}',
 
   'requestInfo.installComponents':'Install Components',
   'requestInfo.installServices':'Install Services',

ambari-web/app/views/main/host/hosts_table_menu_view.js

@@ -93,7 +93,7 @@ App.HostTableMenuView = Em.View.extend({
         operationData: Em.Object.create({
           state: 'PASSIVE',
           action: 'PASSIVE_STATE',
-          message: Em.I18n.t('passiveState.turnOnFor'),
+          message: Em.I18n.t('passiveState.turnOnFor').format(App.format.components[componentNameForOtherActions]),
           componentName: componentNameForOtherActions
         })
       }),
@@ -102,7 +102,7 @@ App.HostTableMenuView = Em.View.extend({
         operationData: Em.Object.create({
           state: 'ACTIVE',
           action: 'PASSIVE_STATE',
-          message: Em.I18n.t('passiveState.turnOffFor'),
+          message: Em.I18n.t('passiveState.turnOffFor').format(App.format.components[componentNameForOtherActions]),
           componentName: componentNameForOtherActions
         })
       })
@@ -152,7 +152,7 @@ App.HostTableMenuView = Em.View.extend({
         operationData: Em.Object.create({
           state: 'PASSIVE',
           action: 'PASSIVE_STATE',
-          message: Em.I18n.t('passiveState.turnOn')
+          message: Em.I18n.t('passiveState.turnOnFor').format('hosts')
         })
       }),
       Em.Object.create({
@@ -160,7 +160,7 @@ App.HostTableMenuView = Em.View.extend({
         operationData: Em.Object.create({
           state: 'ACTIVE',
           action: 'PASSIVE_STATE',
-          message: Em.I18n.t('passiveState.turnOff')
+          message: Em.I18n.t('passiveState.turnOffFor').format('hosts')
         })
       })
     ]);

ambari-web/app/views/main/service/item.js

@@ -55,8 +55,13 @@ App.MainServiceItemView = Em.View.extend({
       default:
         options.push({action: 'runSmokeTest', cssClass: 'icon-thumbs-up-alt', 'label': Em.I18n.t('services.service.actions.run.smoke'), disabled:disabled});
     }
-    var passiveLabel = service.get('passiveState') === "ACTIVE" ? Em.I18n.t('passiveState.turnOn') : Em.I18n.t('passiveState.turnOff');
-    options.push({action:'turnOnOffPassive', cssClass: 'icon-medkit', context:passiveLabel, 'label':passiveLabel , disabled: false});
+    var requestLabel = service.get('passiveState') === "ACTIVE" ?
+      Em.I18n.t('passiveState.turnOnFor').format(App.Service.DisplayNames[serviceName]) :
+      Em.I18n.t('passiveState.turnOffFor').format(App.Service.DisplayNames[serviceName]);
+    var passiveLabel = service.get('passiveState') === "ACTIVE" ?
+      Em.I18n.t('passiveState.turnOn') :
+      Em.I18n.t('passiveState.turnOff');
+    options.push({action:'turnOnOffPassive', cssClass: 'icon-medkit', context:requestLabel, 'label':passiveLabel , disabled: false});
     return options;
   }.property('controller.content', 'controller.isStopDisabled'),
   isMaintenanceActive: function() {