Changeset 819 for trunk/grails-app/domain

Timestamp:

Aug 17, 2010, 3:37:15 PM (10 years ago)

Author:

duh

Message:

• removed test code from wizard

Location:

trunk

Files:

• . (modified) (1 prop)
• grails-app/domain/dbnp/studycapturing/Study.groovy (modified) (4 diffs)

trunk

@@ -8 +8 @@
 [:]
 .settings
+.skip-studies

@@ -8 +8 @@
 [:]
 .settings
+.skip-studies

trunk/grails-app/domain/dbnp/studycapturing/Study.groovy

@@ -249 +249 @@
                 // This should remove the event group itself too, because of the cascading belongsTo relation
                 this.removeFromSamplingEvents(samplingEvent)
+
                 // But apparently it needs an explicit delete() too
                 // (Which can be verified by outcommenting this line, then SampleTests.testDeleteViaParentSamplingEvent fails
@@ -262 +263 @@
          */
         String deleteEventGroup(EventGroup eventGroup) {
-                // TODO THIS DOESNT WORK!!!!
-                println "-"
-                println "-"
-                println "-"
-                println "-"
-                println "-"
-                println "REMOVING AN EVENTGROUP DOES NOT DELETE SAMPLES"
-                println "-"
-                println "-"
-                println "-"
-                println "-"
-                println "-"
-
-
-
                 String msg = "EventGroup ${eventGroup} was deleted"
 
@@ -318 +304 @@
                                         msg += ", sample '${it.name}' was deleted"
                                         this.removeFromSamples( it )
+
                                         // Also here, contrary to documentation, an extra delete() is needed
+                                        // otherwise date is not properly deleted!
                                         it.delete()
                                 }
@@ -344 +332 @@
                 println ".remove eventGroup '${eventGroup.name}' from study '${this.title}'"
                 this.removeFromEventGroups(eventGroup)
+
                 // Also here, contrary to documentation, an extra delete() is needed
+                // otherwise cascaded deletes are not properly performed
                 eventGroup.delete()
 

@@ -8 +8 @@
 [:]
 .settings
+.skip-studies