fix problem with invalid nested parameters and uncomment the function…

…al test that exposed the problem
grails · Apr 3, 2009 · 0132b66 · 0132b66
1 parent 8a4e13e
commit 0132b66
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/...functional-tests/apps/commandObjects/test/functional/CommandObjectsFunctionalTests.groovy b/...functional-tests/apps/commandObjects/test/functional/CommandObjectsFunctionalTests.groovy
@@ -5,7 +5,6 @@ class CommandObjectsFunctionalTests extends functionaltestplugin.FunctionalTestC
          assertContentContains 'name: fred, age:45'
     }
 
-/* TODO - uncomment
 	void testCommandObjectsAdditionalParams() {
         get('/test/testCommand?name=fred&age=45&x=23&y=343')
         assertStatus 200
@@ -14,9 +13,7 @@ class CommandObjectsFunctionalTests extends functionaltestplugin.FunctionalTestC
         get('/test/testCommand?name=fred&age=45&name.x=23&name.y=343')
         assertStatus 200
         assertContentContains 'name: fred, age:45'		
-
 	}
-*/
 
 	void testCommandObjectsWithNestedObject() {
         get('/test/testCommandWithNestedValues?name=fred&age=45&x=23&y=343&nested.value=test')

diff --git a/grails/src/web/org/codehaus/groovy/grails/web/binding/GrailsDataBinder.java b/grails/src/web/org/codehaus/groovy/grails/web/binding/GrailsDataBinder.java
@@ -116,6 +116,7 @@ public GrailsDataBinder(Object target, String objectName) {
         }
         setDisallowedFields(disallowed);
         setAllowedFields(ALL_OTHER_FIELDS_ALLOWED_BY_DEFAULT);
+        setIgnoreInvalidFields(true);
 
     }