Updating default application templates to use update instead of update_attributes
This commit is contained in:
parent
03ac174f2d
commit
0433d79724
|
@ -59,8 +59,8 @@ module Rails
|
|||
end
|
||||
|
||||
# PATCH/PUT update
|
||||
def update_attributes(params=nil)
|
||||
"#{name}.update_attributes(#{params})"
|
||||
def update(params=nil)
|
||||
"#{name}.update(#{params})"
|
||||
end
|
||||
|
||||
# POST create
|
||||
|
|
|
@ -71,7 +71,7 @@ class <%= controller_class_name %>Controller < ApplicationController
|
|||
# PATCH/PUT <%= route_url %>/1.json
|
||||
def update
|
||||
respond_to do |format|
|
||||
if @<%= orm_instance.update_attributes("#{singular_table_name}_params") %>
|
||||
if @<%= orm_instance.update("#{singular_table_name}_params") %>
|
||||
<%- if options[:html] -%>
|
||||
format.html { redirect_to @<%= singular_table_name %>, notice: <%= "'#{human_name} was successfully updated.'" %> }
|
||||
<%- end -%>
|
||||
|
|
|
@ -35,7 +35,7 @@ class ScaffoldControllerGeneratorTest < Rails::Generators::TestCase
|
|||
end
|
||||
|
||||
assert_instance_method :update, content do |m|
|
||||
assert_match(/@user\.update_attributes\(user_params\)/, m)
|
||||
assert_match(/@user\.update\(user_params\)/, m)
|
||||
assert_match(/@user\.errors/, m)
|
||||
end
|
||||
|
||||
|
|
|
@ -45,7 +45,7 @@ class ScaffoldGeneratorTest < Rails::Generators::TestCase
|
|||
end
|
||||
|
||||
assert_instance_method :update, content do |m|
|
||||
assert_match(/@product_line\.update_attributes\(product_line_params\)/, m)
|
||||
assert_match(/@product_line\.update\(product_line_params\)/, m)
|
||||
assert_match(/@product_line\.errors/, m)
|
||||
end
|
||||
|
||||
|
@ -162,7 +162,7 @@ class ScaffoldGeneratorTest < Rails::Generators::TestCase
|
|||
end
|
||||
|
||||
assert_instance_method :update, content do |m|
|
||||
assert_match(/@admin_role\.update_attributes\(admin_role_params\)/, m)
|
||||
assert_match(/@admin_role\.update\(admin_role_params\)/, m)
|
||||
assert_match(/@admin_role\.errors/, m)
|
||||
end
|
||||
|
||||
|
|
Loading…
Reference in New Issue