Changeset 1227 for trunk/grails-app/controllers

Timestamp:

Nov 30, 2010, 5:59:51 PM (10 years ago)

Author:

work@…

Message:

• improved in-code comment

File:

• trunk/grails-app/controllers/dbnp/studycapturing/WizardController.groovy (modified) (1 diff)

trunk/grails-app/controllers/dbnp/studycapturing/WizardController.groovy

@@ -922 +922 @@
                         }
 
-                        // Grails tends to dynamically load objects. This is a nice
-                        // feature, but can cause problems (see ticket #223) as objects
-                        // get instantiated multiple times which causes the POST variable
-                        // to have a different 'identity' as the actual instances. The
-                        // workaround is to 'touch' them here so they get instantiated:
+                        // Grails tends to lazily initialize objects. While in theory this
+                        // is a nice feature, it does not work well with our complex setup
+                        // using the Identity class to match POST variables with object
+                        // instances. This lazy initialization caused two issues:
+                        // #147 and #223, and both are now resolved by forcing objects to
+                        // be instantiated / initialized when a study is loaded from the
+                        // database
                         study.hasMany.each { name, type ->
                                 // dynamically instantiate all identity classes