SONAR-2048: Move inheritance settings to separate tab

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/controllers/profiles_controller.rb b/sonar-server/src/main/webapp/WEB-INF/app/controllers/profiles_controller.rb
index dbc44d5410750d8267c95a0133ea02bcfae2983c..d0bea4321890c960097fd10d313cdfa63398f56c 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/controllers/profiles_controller.rb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/controllers/profiles_controller.rb
@@ -20,7 +20,8 @@
 class ProfilesController < ApplicationController
   SECTION=Navigation::SECTION_CONFIGURATION
 
-  verify :method => :post, :only => ['create', 'delete', 'copy', 'set_as_default', 'restore', 'set_projects', 'rename'], :redirect_to => { :action => 'index' }
+  # GETs should be safe (see http://www.w3.org/2001/tag/doc/whenToUseGet.html)
+  verify :method => :post, :only => ['create', 'delete', 'copy', 'set_as_default', 'restore', 'set_projects', 'rename', 'change_parent'], :redirect_to => { :action => 'index' }
 
   # the backup action is allow to non-admin users : see http://jira.codehaus.org/browse/SONAR-2039
   before_filter :admin_required, :except => [ 'index', 'show', 'projects', 'permalinks', 'export', 'backup', 'inheritance' ]
@@ -195,6 +196,23 @@ class ProfilesController < ApplicationController
   end
 
 
+  #
+  #
+  # POST /profiles/change_parent?id=<profile id>&parent_name=<parent profile name>
+  #
+  #
+  def change_parent
+    id = params[:id].to_i
+    parent_name = params[:parent_name]
+    if parent_name.blank?
+      java_facade.changeParentProfile(id, nil)
+    else
+      java_facade.changeParentProfile(id, parent_name)
+    end
+    redirect_to :action => 'inheritance', :id => id
+  end
+
+
   #
   #
   # GET /profiles/permalinks?id=<profile id>

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 0dd68d535ffc2220fbfb5dab2e78e98b9a58aba5..163b5e29de18add44604eda2d44cc45b4d3ad4f2 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
@@ -29,7 +29,7 @@ class RulesConfigurationController < ApplicationController
   RULE_PRIORITIES = Sonar::RulePriority.as_options.reverse\r
 \r
   # GETs should be safe (see http://www.w3.org/2001/tag/doc/whenToUseGet.html)\r
-  verify :method => :post, :only => ['activate_rule', 'update_param', 'bulk_edit', 'create', 'update', 'delete', 'change_parent', 'revert_rule'], :redirect_to => { :action => 'index' }\r
+  verify :method => :post, :only => ['activate_rule', 'update_param', 'bulk_edit', 'create', 'update', 'delete', 'revert_rule'], :redirect_to => { :action => 'index' }\r
 \r
   before_filter :admin_required, :except => [ 'index', 'export' ]\r
 \r
@@ -73,23 +73,6 @@ class RulesConfigurationController < ApplicationController
     end\r
 \r
   end\r
-\r
-
-  #
-  #
-  # POST /rules_configuration/change_parent?id=<profile id>&parent_name=<parent profile name>
-  #
-  #
-  def change_parent
-    id = params[:id].to_i
-    parent_name = params[:parent_name]
-    if parent_name.blank?
-      java_facade.changeParentProfile(id, nil)
-    else
-      java_facade.changeParentProfile(id, parent_name)
-    end
-    redirect_to :action => 'index', :id => params[:id]
-  end
 
 
   #

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/profiles/inheritance.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/profiles/inheritance.html.erb
index 4bf37458890b86ac6a9845f28f2e984e1cfe1dfa..dcd2c770ce36e13fe417cfe15a744dadc4469bb4 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/profiles/inheritance.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/profiles/inheritance.html.erb
@@ -5,7 +5,7 @@
 
 <% if !@profile.provided? %>
   <% form_tag({:action => 'change_parent'}, {:method => 'post'}) do %>
-    <%= hidden_field_tag "id", @id %>
+    <%= hidden_field_tag "id", @profile.id %>
     Parent profile: <%= select_tag "parent_name", options_for_select(@select_parent, @profile.parent_name), :disabled => !is_admin? %>
     <%= submit_tag "Change", :id => 'submit_parent', :disabled => !is_admin? %>
   <% end %>