aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test/controller/parameters
diff options
context:
space:
mode:
authorSantiago Pastorino <santiago@wyeworks.com>2012-10-12 00:50:20 -0200
committerSantiago Pastorino <santiago@wyeworks.com>2012-10-12 01:01:26 -0200
commitbdd105d8b91c5d0881ab78e36a65a79fdca4a7fb (patch)
tree4a4057eccbf31f3231a5ebc593c04ea69a68fdf9 /actionpack/test/controller/parameters
parentb91a90e49606719a65fecd7c9d703f45df7b7f73 (diff)
downloadrails-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/parameters')
-rw-r--r--actionpack/test/controller/parameters/nested_parameters_test.rb6
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]