From 17486e3aecc47fbad31f189a9b68904bc18c59fb Mon Sep 17 00:00:00 2001 From: Simon Brandhof Date: Tue, 6 Jan 2015 11:31:52 +0100 Subject: SONAR-6016 End of support of Ruby source viewer extensions --- .../main/java/org/sonar/server/ui/JRubyFacade.java | 12 ----- .../app/controllers/drilldown_controller.rb | 10 +--- .../WEB-INF/app/controllers/resource_controller.rb | 62 ---------------------- .../java/org/sonar/api/web/NavigationSection.java | 2 + 4 files changed, 3 insertions(+), 83 deletions(-) diff --git a/server/sonar-server/src/main/java/org/sonar/server/ui/JRubyFacade.java b/server/sonar-server/src/main/java/org/sonar/server/ui/JRubyFacade.java index 61e8e691b94..8185c7828a1 100644 --- a/server/sonar-server/src/main/java/org/sonar/server/ui/JRubyFacade.java +++ b/server/sonar-server/src/main/java/org/sonar/server/ui/JRubyFacade.java @@ -199,18 +199,6 @@ public final class JRubyFacade { return get(Views.class).getPages(section, resourceScope, resourceQualifier, resourceLanguage, (String[]) availableMeasures); } - public List> getResourceTabs() { - return get(Views.class).getPages(NavigationSection.RESOURCE_TAB, null, null, null, null); - } - - public List> getResourceTabs(String scope, String qualifier, String language, Object[] availableMeasures) { - return get(Views.class).getPages(NavigationSection.RESOURCE_TAB, scope, qualifier, language, (String[]) availableMeasures); - } - - public List> getResourceTabsForMetric(String scope, String qualifier, String language, Object[] availableMeasures, String metric) { - return get(Views.class).getPagesForMetric(NavigationSection.RESOURCE_TAB, scope, qualifier, language, (String[]) availableMeasures, metric); - } - public ViewProxy getPage(String id) { return get(Views.class).getPage(id); } diff --git a/server/sonar-web/src/main/webapp/WEB-INF/app/controllers/drilldown_controller.rb b/server/sonar-web/src/main/webapp/WEB-INF/app/controllers/drilldown_controller.rb index 43574f0858f..4ffaac8abc3 100644 --- a/server/sonar-web/src/main/webapp/WEB-INF/app/controllers/drilldown_controller.rb +++ b/server/sonar-web/src/main/webapp/WEB-INF/app/controllers/drilldown_controller.rb @@ -172,15 +172,7 @@ class DrilldownController < ApplicationController end def display_metric_viewers?(resource, metric_key) - return true if resource.file? - java_facade.getResourceTabsForMetric(resource.scope, resource.qualifier, resource.language, resource.last_snapshot.metric_keys.to_java(:string), metric_key).each do |tab| - tab.getUserRoles().each do |role| - if has_role?(role, resource) - return true - end - end - end - false + return resource.file? end def display_violation_viewers?(snapshot) diff --git a/server/sonar-web/src/main/webapp/WEB-INF/app/controllers/resource_controller.rb b/server/sonar-web/src/main/webapp/WEB-INF/app/controllers/resource_controller.rb index 7c9fbee8507..02014815da9 100644 --- a/server/sonar-web/src/main/webapp/WEB-INF/app/controllers/resource_controller.rb +++ b/server/sonar-web/src/main/webapp/WEB-INF/app/controllers/resource_controller.rb @@ -41,66 +41,4 @@ class ResourceController < ApplicationController redirect_to url_for(:controller => 'component', :action => 'index') + '#' + anchor end - # deprecated stuff for drilldown - def view - require_parameters 'id' - @resource = Project.by_key(params[:id]) - access_denied unless has_role?(:user, @resource) - @snapshot = @resource.last_snapshot - load_extensions() if @snapshot - if @extension - render :partial => 'view' - else - not_found('Extension not found') - end - end - - # - # Call by new component viewer to display plugin extension - # - # GET /resource/extension?id=&tab=extension_key - def extension - @resource = Project.by_key(params[:id]) - not_found('Resource not found') unless @resource - @resource = @resource.permanent_resource - access_denied unless has_role?(:user, @resource) - - @snapshot = @resource.last_snapshot - load_extensions() if @snapshot - if @extension - render :partial => 'extension' - else - not_found('Extension not found') - end - end - - private - - def load_extensions - @extensions=[] - java_facade.getResourceTabs(@resource.scope, @resource.qualifier, @resource.language, @snapshot.metric_keys.to_java(:string)).each do |tab| - if tab.getUserRoles().empty? - @extensions<SONAR-5321 */ @Deprecated String RESOURCE_TAB = "resource_tab"; + String CONFIGURATION = "configuration"; /** -- cgit v1.2.3