Changeset 775 for branches/features/taskProcedureRework/grails-app/controllers/TaskProcedureDetailedController.groovy

Timestamp:

Feb 3, 2011, 5:12:40 AM (13 years ago)

Author:

gav

Message:

Add optimistic locking checks and transactional save for TaskProcedure.

File:

• branches/features/taskProcedureRework/grails-app/controllers/TaskProcedureDetailedController.groovy (modified) (7 diffs)

branches/features/taskProcedureRework/grails-app/controllers/TaskProcedureDetailedController.groovy

@@ -5 +5 @@
 
     def filterService
-    def authService
     def taskProcedureService
 
@@ -69 +68 @@
             }
             catch(org.springframework.dao.DataIntegrityViolationException e) {
-                flash.message = "TaskProcedure ${params.id} could not be deleted"
+                flash.errorMessage = "TaskProcedure ${params.id} could not be deleted"
                 redirect(controller:'taskDetailed',
                                 action:'show',
@@ -77 +76 @@
         }
         else {
-            flash.message = "TaskProcedure not found with id ${params.id}"
+            flash.errorMessage = "TaskProcedure not found with id ${params.id}"
             redirect(action:list)
         }
@@ -91 +90 @@
 
         if(!taskProcedureInstance) {
-            flash.message = "TaskProcedure not found with id ${params.id}"
+            flash.errorMessage = "TaskProcedure not found with id ${params.id}"
             redirect(action:list)
         }
@@ -112 +111 @@
 
         if(result.error.code == "default.not.found") {
-            flash.message = g.message(code: result.error.code, args: result.error.args)
+            flash.errorMessage = g.message(code: result.error.code, args: result.error.args)
             redirect(action:list)
+            return
+        }
+
+        if(result.error.code == "default.optimistic.locking.failure") {
+            flash.errorMessage = g.message(code: result.error.code, args: result.error.args)
+            redirect(controller:'taskDetailed',
+                            action:'show',
+                            id:result.taskProcedureInstance.linkedTask.id,
+                            params:[showTab:"showProcedureTab"])
             return
         }
@@ -130 +138 @@
         params.linkedTask = Task.get(params.taskInstance.id)
 
+        // Task already has a taskProcedure.
+        if(params.linkedTask.taskProcedure) {
+            flash.errorMessage = g.message(code: 'default.optimistic.locking.failure')
+            redirect(controller:'taskDetailed',
+                            action:'show',
+                            id:params.linkedTask.id,
+                            params:[showTab:"showProcedureTab"])
+            return
+        }
+
+        // Task does not have a primaryAsset.
         if(!params.linkedTask?.primaryAsset) {
             flash.errorMessage = "Please set a Primary Asset first, then go to the Procedure tab."
@@ -142 +161 @@
 
     def save = {
-        def taskProcedureInstance = new TaskProcedure(params)
-        taskProcedureInstance.createdBy = authService.currentUser
-        taskProcedureInstance.lastUpdatedBy = authService.currentUser
-        def taskInstance = Task.get(params.linkedTask.id)
+        def result = taskProcedureService.save(params)
 
-        // Gaps in the html index's can be created by adding 2 items and removing the first one.
-        // This creates a gap at the missing index where LazyList will return a null.
-        def nullMaintenanceActions = taskProcedureInstance.maintenanceActions.findAll {!it}
-        if (nullMaintenanceActions) {
-            taskProcedureInstance.maintenanceActions.removeAll(nullMaintenanceActions)
+        if(!result.error) {
+            flash.message = g.message(code: "default.create.success", args: ["TaskProcedure", result.taskProcedureInstance.id])
+            redirect(controller:'taskDetailed',
+                            action:'show',
+                            id:result.taskProcedureInstance.linkedTask.id,
+                            params:[showTab:"showProcedureTab"])
+            return
         }
 
-        if(!taskProcedureInstance.hasErrors() && taskProcedureInstance.save(flush: true)) {
-             // Also sets: taskInstance.taskProcedure = taskProcedureInstance
-            taskProcedureInstance.addToTasks(taskInstance)
-            flash.message = "TaskProcedure ${taskProcedureInstance.id} created."
+        if(result.error.code == "default.optimistic.locking.failure") {
+            flash.errorMessage = g.message(code: result.error.code, args: result.error.args)
             redirect(controller:'taskDetailed',
                             action:'show',
-                            id:taskProcedureInstance.linkedTask.id,
+                            id:result.taskProcedureInstance.linkedTask.id,
                             params:[showTab:"showProcedureTab"])
+            return
         }
-        else {
-            // Populate maintenanceAction errors for display.
-            taskProcedureInstance.maintenanceActions.each { it.validate() }
-            render(view:'create',model:[taskProcedureInstance:taskProcedureInstance])
-        }
+
+        render(view:'create', model:[taskProcedureInstance: result.taskProcedureInstance])
     }