]> source.dussan.org Git - sonarqube.git/commitdiff
SONAR-7236 When clicking on TechDebt Ratio on New Code, we should drilldown on that...
authorStas Vilchik <vilchiks@gmail.com>
Wed, 20 Jan 2016 11:15:51 +0000 (12:15 +0100)
committerStas Vilchik <vilchiks@gmail.com>
Wed, 20 Jan 2016 11:16:08 +0000 (12:16 +0100)
server/sonar-server/src/main/resources/org/sonar/server/dashboard/widget/debt_overview.erb
server/sonar-web/src/main/js/components/shared/drilldown-link.js
server/sonar-web/src/main/webapp/WEB-INF/app/helpers/application_helper.rb

index 0ea3e8204c8033fd6408d94631b9e5cdcad7ad32..5fd5a81f58cfe852c65d1336a9075771acd00286 100644 (file)
@@ -34,7 +34,7 @@
                 <span class="nowrap">
                   <%= format_measure(new_sqale_debt_ratio,
                                      :period => dashboard_configuration.period_index,
-                                     :url => url_for_drilldown('new_technical_debt', :period => dashboard_configuration.period_index)) %>
+                                     :url => url_for_drilldown('new_sqale_debt_ratio', :period => dashboard_configuration.period_index)) %>
                 </span>
               </div>
             <% end %>
index bb5dd28a7dec9bec5353fb504042fd7bfc13fff3..ea50a0390be0000f32a88b49b42ce4a28fceb4d2 100644 (file)
@@ -68,9 +68,7 @@ const HIGHLIGHT_MAPPING = {
 
   'new_overall_coverage': 'new_overall_uncovered_lines',
   'new_overall_line_coverage': 'new_overall_uncovered_lines',
-  'new_overall_branch_coverage': 'new_overall_uncovered_conditions',
-
-  'new_sqale_debt_ratio': 'new_technical_debt'
+  'new_overall_branch_coverage': 'new_overall_uncovered_conditions'
 };
 
 
index c19dd8e990f5a8298480c19b1950ec9c51f37495..982956acd762c91b4d2d4185aaed2ba29876ab09 100644 (file)
@@ -235,11 +235,7 @@ module ApplicationHelper
           url += 'resolved=false'
       end
     else
-      if metric_key == 'new_sqale_debt_ratio'
-        url = url_for(options.merge({:controller => 'drilldown', :action => 'measures', :metric => 'new_technical_debt', :highlight => 'new_sqale_debt_ratio', :id => options[:resource]||@resource.id}))
-      else
-        url = url_for(options.merge({:controller => 'drilldown', :action => 'measures', :metric => metric_key, :id => options[:resource]||@resource.id}))
-      end
+      url = url_for(options.merge({:controller => 'drilldown', :action => 'measures', :metric => metric_key, :id => options[:resource]||@resource.id}))
     end
 
     url