SONAR-4568 On a boolean rule parameter, this is not possible to set back the paramete...

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/controllers/rules_configuration_controller.rb b/sonar-server/src/main/webapp/WEB-INF/app/controllers/rules_configuration_controller.rb
index 1450440a5b2b7c09ffa6e7a3a3807e1fb70879e1..b0b9dab865963ca6fec97b478126be2d56046912 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/controllers/rules_configuration_controller.rb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/controllers/rules_configuration_controller.rb
@@ -307,7 +307,7 @@ class RulesConfigurationController < ApplicationController
   def update_param
     verify_post_request
     access_denied unless has_role?(:profileadmin)
-    require_parameters :profile_id, :param_id, :active_rule_id, :value
+    require_parameters :profile_id, :param_id, :active_rule_id
     profile = Profile.find(params[:profile_id].to_i)
     rule_param = RulesParameter.find(params[:param_id].to_i)
     active_rule = ActiveRule.find(params[:active_rule_id].to_i)
@@ -327,7 +327,8 @@ class RulesConfigurationController < ApplicationController
       active_param = nil
       java_facade.ruleParamChanged(profile.id, active_rule.id, rule_param.name, old_value, nil, current_user.name)
     end
-                  # let's reload the active rule
+
+    # let's reload the active rule
     active_rule = ActiveRule.find(active_rule.id)
     render :partial => 'rule', :locals => {:profile => profile, :rule => active_rule.rule, :active_rule => active_rule}
   end

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/_rule_param.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/_rule_param.html.erb
index 34afac5ad91bf9f1c85832ae5235022232dfa1ee..76037b191c3f87b3beda8ba0d1009718874e5ed1 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/_rule_param.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/_rule_param.html.erb
@@ -8,7 +8,7 @@
 <td class="form-key-cell"><%= parameter.name -%></td>
 
 <td class="form-val-cell">
-    <form  onsubmit="$j.ajax({  url: '<%= ApplicationController.root_context -%>/rules_configuration/update_param/<%=active_param_id-%>?active_rule_id=<%=active_rule_id-%>&param_id=<%=parameter.id-%>&profile_id=<%=profile.id-%>',
+    <form onsubmit="$j.ajax({  url: '<%= ApplicationController.root_context -%>/rules_configuration/update_param/<%=active_param_id-%>?active_rule_id=<%=active_rule_id-%>&param_id=<%=parameter.id-%>&profile_id=<%=profile.id-%>',
                                 type:'post',beforeSend: function(request){$j('#param_loading_<%=parameter.id-%>').show();$j('#error_<%=parameter.id-%>').hide();},
                                 data: $j(this).serialize(),
                                 success: function(htmlSuccess){$j('#rule_<%=rule.id-%>').replaceWith(htmlSuccess);},
@@ -23,7 +23,7 @@
   <span id="text_<%= parameter.id -%>"><%= param_value_input(parameter, param_value, :disabled => read_only) -%></span>
 
   <% unless read_only %>
-    <%= submit_tag message('update_verb') %>
+    <%= submit_tag(message('update_verb'), :id => 'submit_' + parameter.id.to_s) %>
     <img src="<%= ApplicationController.root_context -%>/images/loading.gif" style="display:none;" id="param_loading_<%= parameter.id -%>">
     <% if active_parameter and active_parameter.errors.size>0 %>
       <span class="error"><%= active_parameter.errors.on 'value' %></span>