diff options
author | Santiago Pastorino <santiago@wyeworks.com> | 2012-10-12 00:50:20 -0200 |
---|---|---|
committer | Santiago Pastorino <santiago@wyeworks.com> | 2012-10-12 01:01:26 -0200 |
commit | bdd105d8b91c5d0881ab78e36a65a79fdca4a7fb (patch) | |
tree | 4a4057eccbf31f3231a5ebc593c04ea69a68fdf9 /actionpack/test/controller | |
parent | b91a90e49606719a65fecd7c9d703f45df7b7f73 (diff) | |
download | rails-bdd105d8b91c5d0881ab78e36a65a79fdca4a7fb.tar.gz rails-bdd105d8b91c5d0881ab78e36a65a79fdca4a7fb.tar.bz2 rails-bdd105d8b91c5d0881ab78e36a65a79fdca4a7fb.zip |
When executing permit with just a key that points to a hash, DO NOT allow all the hash
params.require(:person).permit(:projects_attributes) was returning
=> {"projects_attributes"=>{"0"=>{"name"=>"Project 1"}}}
When should return
=> {}
You should be doing ...
params.require(:person).permit(projects_attributes: :name)
to get just the projects attributes you want to allow
Diffstat (limited to 'actionpack/test/controller')
-rw-r--r-- | actionpack/test/controller/parameters/nested_parameters_test.rb | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/actionpack/test/controller/parameters/nested_parameters_test.rb b/actionpack/test/controller/parameters/nested_parameters_test.rb index 41f5b6e127..d287e79cba 100644 --- a/actionpack/test/controller/parameters/nested_parameters_test.rb +++ b/actionpack/test/controller/parameters/nested_parameters_test.rb @@ -15,18 +15,22 @@ class NestedParametersTest < ActiveSupport::TestCase details: { pages: 200, genre: "Tragedy" + }, + id: { + isbn: 'x' } }, magazine: "Mjallo!" }) - permitted = params.permit book: [ :title, { authors: [ :name ] }, { details: :pages } ] + permitted = params.permit book: [ :title, { authors: [ :name ] }, { details: :pages }, :id ] assert permitted.permitted? assert_equal "Romeo and Juliet", permitted[:book][:title] assert_equal "William Shakespeare", permitted[:book][:authors][0][:name] assert_equal "Christopher Marlowe", permitted[:book][:authors][1][:name] assert_equal 200, permitted[:book][:details][:pages] + assert_nil permitted[:book][:id] assert_nil permitted[:book][:details][:genre] assert_nil permitted[:book][:authors][0][:born] assert_nil permitted[:magazine] |