diff options
author | Simon Brandhof <simon.brandhof@gmail.com> | 2011-09-08 12:26:33 +0200 |
---|---|---|
committer | Simon Brandhof <simon.brandhof@gmail.com> | 2011-09-08 16:01:49 +0200 |
commit | 4b65687f7bc58677f6b3845312c3845e0c938600 (patch) | |
tree | a2c53b1c23050f1ca94c35aff2c26f57830e0cac /sonar-server | |
parent | 50632107856202e4955f5dc62fd946293c99dbcb (diff) | |
download | sonarqube-4b65687f7bc58677f6b3845312c3845e0c938600.tar.gz sonarqube-4b65687f7bc58677f6b3845312c3845e0c938600.zip |
SONAR-2781 The annotation @UserRole should be optional on web page extensions
Diffstat (limited to 'sonar-server')
-rw-r--r-- | sonar-server/src/main/webapp/WEB-INF/app/controllers/resource_controller.rb | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/sonar-server/src/main/webapp/WEB-INF/app/controllers/resource_controller.rb b/sonar-server/src/main/webapp/WEB-INF/app/controllers/resource_controller.rb index 8cdc48be703..2ca2eadfa51 100644 --- a/sonar-server/src/main/webapp/WEB-INF/app/controllers/resource_controller.rb +++ b/sonar-server/src/main/webapp/WEB-INF/app/controllers/resource_controller.rb @@ -55,10 +55,14 @@ class ResourceController < ApplicationController def load_extensions @extensions=[] java_facade.getResourceTabs(@resource.scope, @resource.qualifier, @resource.language).each do |tab| - tab.getUserRoles().each do |role| - if has_role?(role, @resource) - @extensions<<tab - break + if tab.getUserRoles().empty? + @extensions<<tab + else + tab.getUserRoles().each do |role| + if has_role?(role, @resource) + @extensions<<tab + break + end end end end @@ -69,7 +73,6 @@ class ResourceController < ApplicationController elsif !params[:metric].blank? metric=Metric.by_key(params[:metric]) @extension=@extensions.find{|extension| extension.getDefaultTabForMetrics().include?(metric.key)} - end @extension=@extensions.find{|extension| extension.isDefaultTab()} if @extension==nil end |