Changeset 1732


Ignore:
Timestamp:
Apr 8, 2011, 2:13:21 PM (6 years ago)
Author:
work@…
Message:
  • resolved an issue with identity (ticket #367)
Location:
trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/application.properties

    r1726 r1732  
    1313plugins.db-util=0.4
    1414plugins.famfamfam=1.0.1
    15 plugins.gdt=0.0.31
     15plugins.gdt=0.0.32
    1616plugins.gdtimporter=0.2.4
    1717plugins.grom=0.2.2
  • trunk/grails-app/controllers/dbnp/studycapturing/StudyWizardController.groovy

    r1722 r1732  
    149149                                                // load study
    150150                                                if (this.loadStudy(flow, flash, [studyid:flow.jump.id],authenticationService.getLoggedInUser())) {
     151                                                        // iterate through eventgroups and get their identifiers. Somehow
     152                                                        // we really need to do this here, otherwise (only!) the eventGroup's
     153                                                        // identifiers keep iterating in the events page, messing up the
     154                                                        // GET request, and deleting all samples
     155                                                        //      use case:       1) load study
     156                                                        //                              2) click samples tab
     157                                                        //                              3) click events tab (eventGroup identifiers increases for the SAME
     158                                                        //                                 eventGroup while you would expect the identifier for a eventGroup
     159                                                        //                                 to remain stable)
     160                                                        //                              4) click next or sample tab --> get parameters are wrong so all samples
     161                                                        //                                 are deleted
     162                                                        //                              5) click on the events tab --> the eventGroup identifiers are now stable?!
     163                                                        // The only thing that makes EventGroup different from the other entities is that
     164                                                        // EventGroup extends Identity directly, while the other entities extend TemplateEntity
     165                                                        // which in turn extends Identity.
     166                                                        //
     167                                                        // The following line fixed the issue, but it is extremely weird we even have to do this...
     168                                                        flow.study.eventGroups.each { it.getIdentifier() }
     169
    151170                                                        toStudyPage()
    152171                                                } else {
Note: See TracChangeset for help on using the changeset viewer.