Fixes #17757 - Check perm before redirecting to manifest page

Katello · Mar 8, 2017 · cad5e14 · cad5e14
1 parent b941f26
commit cad5e14
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/..._katello/app/assets/javascripts/bastion_katello/subscriptions/subscriptions.controller.js b/..._katello/app/assets/javascripts/bastion_katello/subscriptions/subscriptions.controller.js
@@ -64,7 +64,7 @@ angular.module('Bastion.subscriptions').controller('SubscriptionsController',
 
         $scope.$on('$stateChangeSuccess', function () {
             $scope.subscriptions.$promise.then(function () {
-                if ($scope.subscriptions.results.length < 1) {
+                if ($scope.subscriptions.results.length < 1 && $scope.permitted('import_manifest')) {
                     $scope.transitionTo('subscriptions-manifest.import');
                 }
             });

diff --git a/...ion_katello/app/assets/javascripts/bastion_katello/subscriptions/views/subscriptions.html b/...ion_katello/app/assets/javascripts/bastion_katello/subscriptions/views/subscriptions.html
@@ -18,7 +18,7 @@ <h2 translate>Subscriptions</h2>
   </div>
 
   <span data-block="no-rows-message" translate>
-    You currently don't have any Subscriptions, you can add Subscriptions by importing a manifest using the button on the right labeled 'Manage Manifest'.
+    You currently don't have any Subscriptions.
   </span>
 
    <span data-block="no-search-results-message" translate>