AMBARI-20432. ambari-web shows redundant configuration for "Number of nodes used by Hive's LLAP". (jaimin)
Project: http://git-wip-us.apache.org/repos/asf/ambari/repo Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/785d7dac Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/785d7dac Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/785d7dac Branch: refs/heads/branch-feature-AMBARI-12556 Commit: 785d7dac9f4cf685eeba0ee4447da9f50ebbc0d1 Parents: 72b3084 Author: Jaimin Jetly <jai...@hortonworks.com> Authored: Mon Mar 13 17:52:44 2017 -0700 Committer: Jaimin Jetly <jai...@hortonworks.com> Committed: Mon Mar 13 17:53:04 2017 -0700 ---------------------------------------------------------------------- .../mixins/common/configs/widgets/unique/num_llap_nodes.js | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/ambari/blob/785d7dac/ambari-web/app/mixins/common/configs/widgets/unique/num_llap_nodes.js ---------------------------------------------------------------------- diff --git a/ambari-web/app/mixins/common/configs/widgets/unique/num_llap_nodes.js b/ambari-web/app/mixins/common/configs/widgets/unique/num_llap_nodes.js index 781a40a..e3e7efe 100644 --- a/ambari-web/app/mixins/common/configs/widgets/unique/num_llap_nodes.js +++ b/ambari-web/app/mixins/common/configs/widgets/unique/num_llap_nodes.js @@ -35,6 +35,10 @@ App.NumLlapNodesWidgetMixin = Em.Mixin.create({ * @type {boolean} */ doNotShowWidget: function () { + var self = this; + Em.run.next(function () { + self.handleReadOnlyAttribute(); + }); if (this.get('readOnly')) { return false; } @@ -42,12 +46,12 @@ App.NumLlapNodesWidgetMixin = Em.Mixin.create({ }.property('isPropertyUndefined', 'config.showAsTextBox', 'readOnly'), handleReadOnlyAttribute: function () { - var readOnly = this.get('readOnly'); + var readOnly = this.get('readOnly') || false; this.set('disabled', readOnly); this.set('supportSwitchToTextBox', !readOnly); var action = readOnly ? 'disable' : 'enable'; this.toggleSlider(action); - }.observes('readOnly'), + }, toggleWidgetView: function() { this._super();
