Changeset 482 for trunk/grails-app/controllers/EntryDetailedController.groovy

Timestamp:

Apr 13, 2010, 11:02:59 AM (14 years ago)

Author:

gav

Message:

Refactor entry creation to provide correct error messages when trying to create a new entry on a complete task.

File:

• trunk/grails-app/controllers/EntryDetailedController.groovy (modified) (2 diffs)

trunk/grails-app/controllers/EntryDetailedController.groovy

@@ -93 +93 @@
 
     def create = {
-        try {
-            def taskInstance = Task.get(params.taskInstance.id)
-            def entryInstance = new Entry()
-            entryInstance.task = taskInstance
-            entryInstance.entryType = EntryType.get(params.entryType.id)
-            return ['entryInstance':entryInstance]
+        if(!params.taskInstance?.id || !params.entryType?.id) {
+            flash.message = g.message(code:"entry.create.no.params")
+            redirect(controller:"taskDetailed", action:"search")
+            return
         }
-        catch(Exception e) {
-            flash.message = "Please select a task, then 'Add Entry'."
+
+        def taskInstance = Task.read(params.taskInstance.id)
+
+        if(!taskInstance) {
+            flash.message = g.message(code:"task.notFound")
             redirect(controller:"taskDetailed", action:"search")
+            return
         }
-    }
+
+        // Check for Complete task.
+        if(taskInstance.taskStatus.id == 3) {
+            flash.errorMessage = g.message(code:"task.operationNotPermittedOnCompleteTask")
+            redirect(controller:"taskDetailed", action:"show", id: taskInstance.id)
+            return
+        }
+
+        def entryInstance = new Entry()
+        entryInstance.task = taskInstance
+        entryInstance.entryType = EntryType.read(params.entryType.id)
+        return ['entryInstance': entryInstance]
+    } // create
 
     def save = {
@@ -112 +126 @@
             flash.message = "Entry created."
             redirect(controller: "taskDetailed", action: "show", id: result.taskId)
+            return
         }
-        else {
-            if(result.entryInstance) {
-                render(view:'create',model:[entryInstance: result.entryInstance])
-            }
-            else {
-                flash.message = "Could not create entry."
-                redirect(controller: "taskDetailed", action:"search")
-            }
 
+        if(result.error.code != "default.create.failure") {
+            params.errorMessage = g.message(code: result.error.code)
         }
-    }
 
-}
+        render(view:'create',model:[entryInstance: result.entryInstance])
+    } // save
+
+} // end class