Changeset 819


Ignore:
Timestamp:
Aug 17, 2010, 3:37:15 PM (8 years ago)
Author:
duh
Message:
  • removed test code from wizard
Location:
trunk
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk

    • Property svn:ignore
      •  

        old new  
        88[:]
        99.settings
         10.skip-studies
  • trunk/grails-app/controllers/dbnp/studycapturing/WizardController.groovy

    r805 r819  
    318318                                if (!flow.event || (flow.event instanceof Event && type == "sample") || (flow.event instanceof SamplingEvent && type == "event")) {
    319319                                        // create new instance
    320                                         flow.event = (type == "event") ? new Event(template: template, parent: flow.study) : new SamplingEvent(template: template, parent: flow.study)
     320                                        flow.event = (type == "event") ? new Event(template: template) : new SamplingEvent(template: template)
    321321                                } else {
    322322                                        flow.event.template = template
     
    351351                                flash.errors = [:]
    352352
     353                                // add event to study
     354                                if (flow.event instanceof SamplingEvent) {
     355                                        flow.study.addToSamplingEvents( flow.event )
     356                                } else {
     357                                        flow.study.addToEvents( flow.event )
     358                                }
     359
    353360                                // validate event
    354361                                if (flow.event.validate()) {
    355                                         // validates properly
    356                                         if (flow.event instanceof SamplingEvent) {
    357                                                 flow.study.addToSamplingEvents( flow.event )
    358                                         } else {
    359                                                 flow.study.addToEvents( flow.event )
    360                                         }
    361 
    362362                                        // remove event from the flowscope
    363363                                        flow.remove('event')
     
    366366                                } else {
    367367                                        // event does not validate
     368                                        // remove from study
     369                                        if (flow.event instanceof SamplingEvent) {
     370                                                flow.study.removeFromSamplingEvents( flow.event )
     371                                        } else {
     372                                                flow.study.removeFromEvents( flow.event )
     373                                        }
     374
     375                                        // append errors
    368376                                        this.appendErrors(flow.event, flash.errors)
    369377                                        error()
     
    406414                                flow.study.addToEventGroups(
    407415                                        new EventGroup(
    408                                                 name    : groupName,
    409                                                 parent  : flow.study
     416                                                name    : groupName
    410417                                        )
    411418                                )
     
    527534                                                                flow.study.addToSamples(
    528535                                                                        new Sample(
    529                                                                                 parent: flow.study,
    530536                                                                                parentSubject: subject,
    531537                                                                                parentEvent: samplingEvent,
     
    631637                                        // save study
    632638                                        println ".saving study"
    633                                         if (!flow.study.save()) {
     639                                        if (!flow.study.save(flush:true)) {
    634640                                                this.appendErrors(flow.study, flash.errors)
    635641                                                throw new Exception('error saving study')
     
    945951                                        name            : subjectName,
    946952                                        species         : species,
    947                                         template        : template,
    948                                         parent          : flow.study
     953                                        template        : template
    949954                                )
     955
     956                                // add it to the study
     957                                flow.study.addToSubjects( subject )
    950958
    951959                                // validate subject
    952960                                if (subject.validate()) {
    953                                         // add it to the study
    954                                         flow.study.addToSubjects( subject )
    955961                                        println ".added subject "+subject
    956962                                } else {
    957963                                        // whoops?
     964                                        flow.study.removeFromSubjects( subject )
     965
     966                                        // append errors
    958967                                        this.appendErrors(subject, flash.errors)
    959968                                        errors = true
  • trunk/grails-app/domain/dbnp/studycapturing/Study.groovy

    r817 r819  
    249249                // This should remove the event group itself too, because of the cascading belongsTo relation
    250250                this.removeFromSamplingEvents(samplingEvent)
     251
    251252                // But apparently it needs an explicit delete() too
    252253                // (Which can be verified by outcommenting this line, then SampleTests.testDeleteViaParentSamplingEvent fails
     
    262263         */
    263264        String deleteEventGroup(EventGroup eventGroup) {
    264                 // TODO THIS DOESNT WORK!!!!
    265                 println "-"
    266                 println "-"
    267                 println "-"
    268                 println "-"
    269                 println "-"
    270                 println "REMOVING AN EVENTGROUP DOES NOT DELETE SAMPLES"
    271                 println "-"
    272                 println "-"
    273                 println "-"
    274                 println "-"
    275                 println "-"
    276 
    277 
    278 
    279265                String msg = "EventGroup ${eventGroup} was deleted"
    280266
     
    318304                                        msg += ", sample '${it.name}' was deleted"
    319305                                        this.removeFromSamples( it )
     306
    320307                                        // Also here, contrary to documentation, an extra delete() is needed
     308                                        // otherwise date is not properly deleted!
    321309                                        it.delete()
    322310                                }
     
    344332                println ".remove eventGroup '${eventGroup.name}' from study '${this.title}'"
    345333                this.removeFromEventGroups(eventGroup)
     334
    346335                // Also here, contrary to documentation, an extra delete() is needed
     336                // otherwise cascaded deletes are not properly performed
    347337                eventGroup.delete()
    348338
Note: See TracChangeset for help on using the changeset viewer.