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 2013/03/23 05:29:58 UTC
svn commit: r1460091 - in /incubator/ambari/trunk: CHANGES.txt
ambari-web/app/messages.js ambari-web/app/models/service_config.js
ambari-web/app/templates/common/configs/service_config.hbs
Author: yusaku
Date: Sat Mar 23 04:29:58 2013
New Revision: 1460091
URL: http://svn.apache.org/r1460091
Log:
AMBARI-1645. Undo should not be allowed on component hosts. (yusaku)
Modified:
incubator/ambari/trunk/CHANGES.txt
incubator/ambari/trunk/ambari-web/app/messages.js
incubator/ambari/trunk/ambari-web/app/models/service_config.js
incubator/ambari/trunk/ambari-web/app/templates/common/configs/service_config.hbs
Modified: incubator/ambari/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/incubator/ambari/trunk/CHANGES.txt?rev=1460091&r1=1460090&r2=1460091&view=diff
==============================================================================
--- incubator/ambari/trunk/CHANGES.txt (original)
+++ incubator/ambari/trunk/CHANGES.txt Sat Mar 23 04:29:58 2013
@@ -509,6 +509,10 @@ Trunk (unreleased changes):
BUG FIXES
+ AMBARI-1645. Undo should not be allowed on component hosts. (yusaku)
+
+ AMBARI-1644. Service summary page flickers. (yusaku)
+
AMBARI-1689. 500 Exception creating service component during install. (Sumit
Mohanty via swagle)
Modified: incubator/ambari/trunk/ambari-web/app/messages.js
URL: http://svn.apache.org/viewvc/incubator/ambari/trunk/ambari-web/app/messages.js?rev=1460091&r1=1460090&r2=1460091&view=diff
==============================================================================
--- incubator/ambari/trunk/ambari-web/app/messages.js (original)
+++ incubator/ambari/trunk/ambari-web/app/messages.js Sat Mar 23 04:29:58 2013
@@ -112,6 +112,8 @@ Em.I18n.translations = {
'common.upgrade': 'Upgrade',
'common.security':'Security',
'common.cluster':'Cluster',
+ 'common.exception':'Exception',
+ 'common.undo':'Undo',
'hostPopup.noHostsToShow':'No hosts to show',
'hostPopup.noTasksToShow':'No tasks to show',
@@ -313,6 +315,7 @@ Em.I18n.translations = {
'installer.step7.header':'Customize Services',
'installer.step7.body':'We have come up with recommended configurations for the services you selected. Customize them as you see fit.',
'installer.step7.attentionNeeded':'<b>Attention:</b> Some configurations need your attention before you can proceed.',
+ 'installer.step7.config.addProperty':'Add Property',
'installer.step7.ConfigErrMsg.header':'Custom configuration error: ',
'installer.step7.ConfigErrMsg.message':'Error in custom configuration. Some properties entered in the box are already exposed on this page',
Modified: incubator/ambari/trunk/ambari-web/app/models/service_config.js
URL: http://svn.apache.org/viewvc/incubator/ambari/trunk/ambari-web/app/models/service_config.js?rev=1460091&r1=1460090&r2=1460091&view=diff
==============================================================================
--- incubator/ambari/trunk/ambari-web/app/models/service_config.js (original)
+++ incubator/ambari/trunk/ambari-web/app/models/service_config.js Sat Mar 23 04:29:58 2013
@@ -177,6 +177,22 @@ App.ServiceConfigProperty = Ember.Object
var isEditable = this.get('isEditable');
return isEditable && dValue != null && value !== dValue;
}.property('value', 'defaultValue', 'isEditable'),
+
+ /**
+ * Don't show "Undo" for hosts on Installer Step7
+ */
+ cantBeUndone: function() {
+ var types = ["masterHost", "slaveHosts", "masterHosts", "slaveHost"];
+ var displayType = this.get('displayType');
+ var result = false;
+ types.forEach(function(type) {
+ if (type === displayType) {
+ result = true;
+ return;
+ }
+ });
+ return result;
+ }.property('displayType'),
initialValue: function () {
var masterComponentHostsInDB = App.db.getMasterComponentHosts();
Modified: incubator/ambari/trunk/ambari-web/app/templates/common/configs/service_config.hbs
URL: http://svn.apache.org/viewvc/incubator/ambari/trunk/ambari-web/app/templates/common/configs/service_config.hbs?rev=1460091&r1=1460090&r2=1460091&view=diff
==============================================================================
--- incubator/ambari/trunk/ambari-web/app/templates/common/configs/service_config.hbs (original)
+++ incubator/ambari/trunk/ambari-web/app/templates/common/configs/service_config.hbs Sat Mar 23 04:29:58 2013
@@ -39,13 +39,16 @@
<div {{bindAttr class="errorMessage:error: :control-group"}}>
{{view viewClass serviceConfigBinding="this" categoryConfigsBinding="view.categoryConfigs" }}
{{#if isOverridable}}
- <a class="action" {{action "createOverrideProperty" this target="view" }} ><i class="icon-plus-sign"></i>Exception</a>
+ <a class="action" {{action "createOverrideProperty" this target="view" }} ><i class="icon-plus-sign"></i>{{t common.exception}}</a>
{{/if}}
- {{#if isNotDefaultValue}}
- <a class="action" {{action "doRestoreDefaultValue" this target="view" }} ><i class="icon-undo"></i>Undo</a>
+ {{#if cantBeUndone}}
+ {{else}}
+ {{#if isNotDefaultValue}}
+ <a class="action" {{action "doRestoreDefaultValue" this target="view" }} ><i class="icon-undo"></i>{{t common.undo}}</a>
+ {{/if}}
{{/if}}
{{#if isRemovable}}
- <a class="action" {{action "removeProperty" this target="view" }} ><i class="icon-minus-sign"></i>Remove</a>
+ <a class="action" {{action "removeProperty" this target="view" }} ><i class="icon-minus-sign"></i>{{t common.remove}}</a>
{{/if}}
<span class="help-inline">{{errorMessage}}</span>
</div>
@@ -59,7 +62,7 @@
{{! For Advanced, Advanced Core Site, Advanced HDFS Site sections, show the 'Add Property' link.}}
{{#if category.canAddProperty }}
<div>
- <a href="#" {{action "showAddPropertyWindow" this target="view" }} >Add Property...</a>
+ <a href="#" {{action "showAddPropertyWindow" this target="view" }} >{{t installer.step7.config.addProperty}}...</a>
</div>
{{/if}}
</form>