Changeset 1227

Show
Ignore:
Timestamp:
30-11-10 17:59:51 (3 years ago)
Author:
work@…
Message:

- improved in-code comment

Files:
1 modified

Legend:

Unmodified
Added
Removed
  • trunk/grails-app/controllers/dbnp/studycapturing/WizardController.groovy

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