Dont change params hash. Use dup instead

Dmitriy Zaporozhets
1 parent ce1b79af
Showing 1 changed file with 3 additions and 2 deletions Show diff stats
app/controllers/admin/groups_controller.rb
@@ -32,13 +32,14 @@ class Admin::GroupsController &lt; AdminController
   end
  
   def update
-    owner_id = params[:group].delete(:owner_id)
+    group_params = params[:group].dup
+    owner_id =group_params.delete(:owner_id)
  
     if owner_id
       @group.owner = User.find(owner_id)
     end
  
-    if @group.update_attributes(params[:group])
+    if @group.update_attributes(group_params)
       redirect_to [:admin, @group], notice: 'Group was successfully updated.'
     else
       render action: "edit"
...	...	@@ -32,13 +32,14 @@ class Admin::GroupsController < AdminController
32	32	end
33	33
34	34	def update
35		- owner_id = params[:group].delete(:owner_id)
	35	+ group_params = params[:group].dup
	36	+ owner_id =group_params.delete(:owner_id)
36	37
37	38	if owner_id
38	39	@group.owner = User.find(owner_id)
39	40	end
40	41
41		- if @group.update_attributes(params[:group])
	42	+ if @group.update_attributes(group_params)
42	43	redirect_to [:admin, @group], notice: 'Group was successfully updated.'
43	44	else
44	45	render action: "edit"
...	...