]> source.dussan.org Git - sonarqube.git/commitdiff
SONAR-4923 Remove remaining usage of rule_notes and active_rule_notes
authorJulien Lancelot <julien.lancelot@sonarsource.com>
Wed, 22 Jan 2014 12:56:31 +0000 (13:56 +0100)
committerJulien Lancelot <julien.lancelot@sonarsource.com>
Wed, 22 Jan 2014 13:01:00 +0000 (14:01 +0100)
sonar-server/src/main/webapp/WEB-INF/app/controllers/issue_controller.rb
sonar-server/src/main/webapp/WEB-INF/app/models/profile.rb
sonar-server/src/main/webapp/WEB-INF/app/models/rule.rb
sonar-server/src/main/webapp/WEB-INF/app/views/issue/_rule.html.erb

index afd842cb875f1d9758df9a6dfd88112e09ee57b3..58df6c0e39fd731cae017f793dd1897b714143f1 100644 (file)
@@ -186,7 +186,7 @@ class IssueController < ApplicationController
     verify_ajax_request
     require_parameters :id
     rule_key = params[:id].split(':')
-    @rule = Rule.first(:conditions => ['plugin_name=? and plugin_rule_key=?', rule_key[0], rule_key[1]], :include => :rule_note)
+    @rule = Rule.first(:conditions => ['plugin_name=? and plugin_rule_key=?', rule_key[0], rule_key[1]])
     render :partial => 'issue/rule'
   end
 
index 217be41cd8c739f8fc1cdfe456966fa8cc77bbe9..d9dfd4410fbda95383585f8ba03d0073c43a56e1 100644 (file)
@@ -22,7 +22,7 @@ class Profile < ActiveRecord::Base
 
   has_many :alerts, :dependent => :delete_all
   has_many :active_rules, :class_name => 'ActiveRule', :foreign_key => 'profile_id', :dependent => :destroy, :include => ['rule']
-  has_many :active_rules_with_params, :class_name => 'ActiveRule', :foreign_key => 'profile_id', :include => ['active_rule_parameters', 'active_rule_note']
+  has_many :active_rules_with_params, :class_name => 'ActiveRule', :foreign_key => 'profile_id', :include => ['active_rule_parameters']
   has_many :changes, :class_name => 'ActiveRuleChange', :dependent => :destroy
 
   validates_uniqueness_of :name, :scope => :language, :case_sensitive => false, :message => Api::Utils.message('quality_profiles.already_exists')
index 4fb806c1fe43a2c088708afdcd03558b9fa5e0b4..444d6d61e54d6dc462d960daf80b865fc9cb37f9 100644 (file)
@@ -272,7 +272,7 @@ class Rule < ActiveRecord::Base
     conditions << "id IN (:ids)"
     values[:ids] = Array(Internal.rules.findIds(params))
 
-    includes=(options[:include_parameters_and_notes] ? [:rules_parameters, :rule_note] : nil)
+    includes=(options[:include_parameters_and_notes] ? [:rules_parameters] : nil)
     rules = Rule.all(:include => includes, :conditions => [conditions.join(" AND "), values])
     rules = Rule.sort_by(rules, options[:sort_by])
     filter(rules, options)
index a0165ac138feb914e64ff6571db1b7dece93627b..f93e91898f2955308b34aecafc76f6764d42a16b 100644 (file)
@@ -6,9 +6,9 @@
   <% end %>
 </div>
 
-<% if @rule.note %>
+<% if @rule.note_data %>
   <div class="marginbottom10">
-    <%= @rule.note.html_text -%>
+    <%= Api::Utils.markdown_to_html(@rule.note_data) -%>
   </div>
 <% end %>
 
@@ -16,4 +16,4 @@
   <%= h @rule.plugin_name -%>
   &nbsp;<%= image_tag 'sep12.png' -%>&nbsp;
   <a href="#" onclick="return openIssueRulePopup(this)"><%= h @rule.plugin_rule_key -%></a>
-</div>
\ No newline at end of file
+</div>