Changeset 459

Timestamp:

May 25, 2010, 12:05:59 PM (13 years ago)

Author:

vinlud

Message:

Check empty string ontology selector

Location:

trunk/grails-app

Files:

• controllers/dbnp/studycapturing/TermEditorController.groovy (modified) (3 diffs)
• domain/dbnp/data/Term.groovy (modified) (1 diff)

trunk/grails-app/controllers/dbnp/studycapturing/TermEditorController.groovy

@@ -66 +66 @@
                                 println params
                                 def ontology = Ontology.findByNcboVersionedId( params.get('term-ontology_id') as int )
+                def strTerm = params.get('term')
 
                                 // do we have an ontology?
@@ -71 +72 @@
                                         // TODO: if ontology is missing, create it
                     // pending possible addition to OntoCAT BioportalOntologyService API of search by versioned Ontology Id
+                    println "Ontology is empty"
                                 }
 
                                 // instantiate term with parameters
                                 def term = new Term(
-                                        name: params.get('term'),
+                                        name: strTerm,
                                         ontology: ontology,
                                         accession: params.get('term-concept_id')
@@ -92 +94 @@
                                                 println it
                                         }
-                                        flash.errors = term.errors
                                         error()
                     flash.message = "Term addition failed"

trunk/grails-app/domain/dbnp/data/Term.groovy

@@ -24 +24 @@
                 accession(unique: 'ontology')   // Accession should be unique within an ontology
                 name(unique: 'ontology')        // Preferred name should be unique within an ontology
+        name(size: 1..255)              // Name should be a non-empty string
         }