AMBARI-11609. User with Read-Only access can't visit service configs page (onechiporenko)

ambari-web/app/controllers/main/service/info/configs.js

@@ -135,7 +135,7 @@ App.MainServiceInfoConfigsController = Em.Controller.extend(App.ServerValidatorM
    * @type {boolean}
    */
   canEdit: function () {
-    return this.get('isCurrentSelected') && !this.get('isCompareMode');
+    return this.get('isCurrentSelected') && !this.get('isCompareMode') && App.isAccessible('MANAGER');
   }.property('isCurrentSelected', 'isCompareMode'),
 
   serviceConfigs: function () {
@@ -851,15 +851,26 @@ App.MainServiceInfoConfigsController = Em.Controller.extend(App.ServerValidatorM
     if (!App.Service.find().someProperty('serviceName', 'RANGER')) {
       App.config.removeRangerConfigs(this.get('stepConfigs'));
     }
-    this.getRecommendationsForDependencies(null, true, function() {
-      self.setProperties({
-        dataIsLoaded: true,
-        versionLoaded: true,
-        isInit: false
-      });
-      Em.run.next(function(){
-         self.set('hash', self.getHash())
-      });
+    if (App.isAccessible('MANAGER')) {
+      this.getRecommendationsForDependencies(null, true, self._onLoadComplete.bind(self));
+    }
+    else {
+      self._onLoadComplete();
+    }
+  },
+
+  /**
+   * @method _getRecommendationsForDependenciesCallback
+   */
+  _onLoadComplete: function () {
+    var self = this;
+    this.setProperties({
+      dataIsLoaded: true,
+      versionLoaded: true,
+      isInit: false
+    });
+    Em.run.next(function(){
+      self.set('hash', self.getHash());
     });
   },
 

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

@@ -18,13 +18,17 @@
 var App = require('app');
 
 App.MainHostServiceConfigsView = Em.View.extend({
+
   templateName: require('templates/main/host/configs_service'),
+
   didInsertElement: function () {
-    var controller = this.get('controller');
-    controller.loadStep();
+    this.get('controller').loadStep();
   },
+
   isConfigsEditable: false,
+
   content: function () {
     return App.router.get('mainHostDetailsController.content');
   }.property('App.router.mainHostDetailsController.content')
+
 });

ambari-web/test/controllers/main/service/info/config_test.js

@@ -1253,4 +1253,30 @@ describe("App.MainServiceInfoConfigsController", function () {
 
   });
 
+  describe('#_onLoadComplete', function () {
+
+    beforeEach(function () {
+      sinon.stub(Em.run, 'next', Em.K);
+      mainServiceInfoConfigsController.setProperties({
+        dataIsLoaded: false,
+        versionLoaded: false,
+        isInit: true
+      });
+    });
+
+    afterEach(function () {
+      Em.run.next.restore();
+    });
+
+    it('should update flags', function () {
+
+      mainServiceInfoConfigsController._onLoadComplete();
+      expect(mainServiceInfoConfigsController.get('dataIsLoaded')).to.be.true;
+      expect(mainServiceInfoConfigsController.get('versionLoaded')).to.be.true;
+      expect(mainServiceInfoConfigsController.get('isInit')).to.be.false;
+
+    });
+
+  });
+
 });