Changeset 1227


Ignore:
Timestamp:
Nov 30, 2010, 5:59:51 PM (6 years ago)
Author:
work@…
Message:
  • improved in-code comment
File:
1 edited

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
Note: See TracChangeset for help on using the changeset viewer.