SONAR-4573 Remove the hack which leads do display the 'Project Permissions' page...

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/layouts/_menu_resource_settings.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/layouts/_menu_resource_settings.html.erb
index d6362174d1b2b8ed21e9a5cba1d8a82cbc5f483d..fde05bbc14dfab324b14e6e8c94daf5b865fb699 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/layouts/_menu_resource_settings.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/layouts/_menu_resource_settings.html.erb
@@ -19,10 +19,7 @@
       <% if (is_admin && @project.project?) %>
          <li><a href="<%= ApplicationController.root_context -%>/project/links/<%= @project.id -%>"><%= message('project_links.page') -%></a></li>
       <% end %>
-      <%
-         # NOTE: we keep "@project.view?" in the test for backward compatibility with the Views plugin
-         if (is_admin && controller.java_facade.getResourceTypeBooleanProperty(@project.qualifier, 'hasRolePolicy') || @project.view?)
-      %>
+      <% if is_admin && controller.java_facade.getResourceTypeBooleanProperty(@project.qualifier, 'hasRolePolicy') %>
       <li><a href="<%= ApplicationController.root_context -%>/project_roles/index/<%= @project.id -%>"><%= message('roles.page') -%></a></li>
       <% end %>
       <% if is_admin && controller.java_facade.getResourceTypeBooleanProperty(@project.qualifier, 'modifiable_history') %>

diff --git a/sonar-server/src/main/webapp/WEB-INF/lib/resourceable.rb b/sonar-server/src/main/webapp/WEB-INF/lib/resourceable.rb
index edb4a03277d3c390bea2d3a1fd4d6a8b0312fd6a..32513ba6695dc9cae9b10733990288ae198bdcec 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/lib/resourceable.rb
+++ b/sonar-server/src/main/webapp/WEB-INF/lib/resourceable.rb
@@ -53,10 +53,12 @@ module Resourceable
     qualifier==QUALIFIER_LIB
   end
 
+  # No more used since SonarQube 4.0 and should not be used anymore
   def view?
     qualifier==QUALIFIER_VIEW
   end
 
+  # No more used and should not be used
   def subview?
     qualifier==QUALIFIER_SUBVIEW
   end