You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by on...@apache.org on 2015/05/15 13:57:58 UTC
ambari git commit: AMBARI-11165. Configs: Recommendations API call
should not be made if config being changed is empty (onechiporenko)
Repository: ambari
Updated Branches:
refs/heads/trunk dfc77fb21 -> a52976e2b
AMBARI-11165. Configs: Recommendations API call should not be made if config being changed is empty (onechiporenko)
Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/a52976e2
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/a52976e2
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/a52976e2
Branch: refs/heads/trunk
Commit: a52976e2ba62f8ca0a8d664f9994d1e98d418c0b
Parents: dfc77fb
Author: Oleg Nechiporenko <on...@apache.org>
Authored: Fri May 15 14:54:20 2015 +0300
Committer: Oleg Nechiporenko <on...@apache.org>
Committed: Fri May 15 14:54:20 2015 +0300
----------------------------------------------------------------------
ambari-web/app/views/common/controls_view.js | 1 +
1 file changed, 1 insertion(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ambari/blob/a52976e2/ambari-web/app/views/common/controls_view.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/views/common/controls_view.js b/ambari-web/app/views/common/controls_view.js
index 3bafde1..6dd82b7 100644
--- a/ambari-web/app/views/common/controls_view.js
+++ b/ambari-web/app/views/common/controls_view.js
@@ -87,6 +87,7 @@ App.SupportsDependentConfigs = Ember.Mixin.create({
* @returns {$.Deferred}
*/
sendRequestRorDependentConfigs: function(config) {
+ if (!config.get('isValid')) return $.Deferred().resolve().promise();
if (App.get('isClusterSupportsEnhancedConfigs') && ['mainServiceInfoConfigsController','wizardStep7Controller'].contains(this.get('controller.name'))) {
var name = config.get('name');
var controller = this.get('controller');