Revert "reorder command object DI"

jeffscottbrown · jeffscottbrown · commit c400552bdf65 · 2012-04-13T13:01:52.000-05:00
This reverts commit 4d825ce.
diff --git a/src/java/org/codehaus/groovy/grails/web/plugins/support/WebMetaUtils.groovy b/src/java/org/codehaus/groovy/grails/web/plugins/support/WebMetaUtils.groovy
@@ -113,10 +113,6 @@ class WebMetaUtils {
                         }
                         
                         bind.invoke(commandObject, "bindData", [commandObject, commandParams] as Object[])
-                        if(newCommandObject) {
-                            ctx.autowireCapableBeanFactory?.autowireBeanProperties(
-                                commandObject, AutowireCapableBeanFactory.AUTOWIRE_BY_NAME, false)
-                        }
                         def errors = commandObject.errors ?: new BindException(commandObject, paramType.name)
                         def constrainedProperties = commandObject.constraints?.values()
                         for (constrainedProperty in constrainedProperties) {
@@ -126,6 +122,10 @@ class WebMetaUtils {
                         }
                         commandObject.errors = errors
                         
+                        if(newCommandObject) {
+                            ctx.autowireCapableBeanFactory?.autowireBeanProperties(
+                                commandObject, AutowireCapableBeanFactory.AUTOWIRE_BY_NAME, false)
+                        }
                     }
                     catch (Exception e) {
                         throw new ControllerExecutionException("Error occurred creating command object.", e)

Original file line number	Diff line number	Diff line change
`@@ -113,10 +113,6 @@ class WebMetaUtils {`
`113`	`113`	`}`
`114`	`114`
`115`	`115`	`bind.invoke(commandObject, "bindData", [commandObject, commandParams] as Object[])`
`116`		`- if(newCommandObject) {`
`117`		`- ctx.autowireCapableBeanFactory?.autowireBeanProperties(`
`118`		`- commandObject, AutowireCapableBeanFactory.AUTOWIRE_BY_NAME, false)`
`119`		`- }`
`120`	`116`	`def errors = commandObject.errors ?: new BindException(commandObject, paramType.name)`
`121`	`117`	`def constrainedProperties = commandObject.constraints?.values()`
`122`	`118`	`for (constrainedProperty in constrainedProperties) {`
`@@ -126,6 +122,10 @@ class WebMetaUtils {`
`126`	`122`	`}`
`127`	`123`	`commandObject.errors = errors`
`128`	`124`
	`125`	`+ if(newCommandObject) {`
	`126`	`+ ctx.autowireCapableBeanFactory?.autowireBeanProperties(`
	`127`	`+ commandObject, AutowireCapableBeanFactory.AUTOWIRE_BY_NAME, false)`
	`128`	`+ }`
`129`	`129`	`}`
`130`	`130`	`catch (Exception e) {`
`131`	`131`	`throw new ControllerExecutionException("Error occurred creating command object.", e)`