Changeset 152 for trunk/grails-app/domain

Timestamp:

Jan 29, 2010, 7:38:01 PM (14 years ago)

Author:

jahn

Message:

Currently Sample extends Event. The Sample class might better be merged into the Event class.

Location:

trunk/grails-app/domain/dbnp/studycapturing

Files:

• Sample.groovy (modified) (1 diff)
• SamplingEvent.groovy (modified) (1 diff)

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

@@ -12 +12 @@
     // or should this be inferred via the parent SamplingEvent?
 
-    // TODO: should Sample also carry a reference to its parent subject,
-    // or should this be inferred via the parent SamplingEvent?
-
-    String name // should be unique with respect to the parent study (which can be inferred
+    String name      // should be unique with respect to the parent study (which can be inferred)
     Term material
+    // don't we need a member, that describes the quantity of the sample?
 
     static constraints = {
     }
+
 }

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

@@ -3 +3 @@
 /**
  * The SamplingEvent class describes a sampling event, an event that also results in one or more samples.
+ *
+ * NOTE: according to documentation, super classes and subclasses share the same table.
+ *       thus, there is actually no reason not to merge the sampling with the Event super class.
+ *       this is especially true, since the subclass adds only one has-many relation ship.
  */
+
 class SamplingEvent extends Event {