Changeset 454 for trunk/grails-app/domain

Timestamp:

May 21, 2010, 10:04:49 AM (10 years ago)

Author:

roberth

Message:

Improved study list to be able to handle one or more studies. Also improved performance issues with Subject and Sample classes. I have set the domainFields property to a static property, filled it in the class description and attached an ontology to it in the bootstrap (because it can't be done in the class description). It might not be the prettiest solution, it does improve the performance of the study detail page and study comparison with 90%.

Location:

trunk/grails-app/domain/dbnp/studycapturing

Files:

• Sample.groovy (modified) (2 diffs)
• Subject.groovy (modified) (2 diffs)

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

@@ -19 +19 @@
          * @return List
          */
-        List<TemplateField> giveDomainFields() {
+        List<TemplateField> giveDomainFields() { return Sample.domainFields }
+        static List<TemplateField> domainFields =
                 [
                         new TemplateField(
@@ -27 +28 @@
                         new TemplateField(
                                 name: 'material',
-                                type: TemplateFieldType.ONTOLOGYTERM,
-                                ontologies: [Ontology.findByNcboId(1005)])
+                                type: TemplateFieldType.ONTOLOGYTERM )
                 ]
-        }
 
         static constraints = {

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

@@ -21 +21 @@
          * @return List
          */
-        List<TemplateField> giveDomainFields() {
+        List<TemplateField> giveDomainFields() { return Subject.domainFields }
+        static List<TemplateField> domainFields =
                 [
                         new TemplateField(
@@ -29 +30 @@
                         new TemplateField(
                                 name: 'species',
-                                type: TemplateFieldType.ONTOLOGYTERM,
-                                ontologies: [Ontology.findByNcboId(1132)])
+                                type: TemplateFieldType.ONTOLOGYTERM)
                 ]
-        }
 }