Index: /trunk/grails-app/controllers/AssignedGroupDetailedController.groovy
===================================================================
--- /trunk/grails-app/controllers/AssignedGroupDetailedController.groovy	(revision 246)
+++ /trunk/grails-app/controllers/AssignedGroupDetailedController.groovy	(revision 247)
@@ -3,5 +3,10 @@
 class AssignedGroupDetailedController extends BaseController {
 
-    def index = { redirect(action:list,params:params) }
+    def assignedGroupService
+
+    def index = {
+        flash.message = g.message(code: "assignedGroup.task.not.found")
+        redirect(controller:"taskDetailed", action:"search")
+    }
 
     // the delete, save and update actions only accept POST requests
@@ -9,99 +14,104 @@
 
     def list = {
-        params.max = Math.min( params.max ? params.max.toInteger() : 10,  100)
-        [ assignedGroupInstanceList: AssignedGroup.list( params ), assignedGroupInstanceTotal: AssignedGroup.count() ]
+            def result = assignedGroupService.list(params)
+            if(!result.error) {
+                return [ assignedGroupInstanceList: result.assignedGroupInstanceList,
+                                assignedGroupInstanceTotal: result.assignedGroupInstanceTotal ]
+            }
+
+            flash.message = g.message(code: result.error.code, args: result.error.args)
+            redirect( url: resource(dir:'') )
     }
 
     def show = {
-        def assignedGroupInstance = AssignedGroup.get( params.id )
+        def result = assignedGroupService.show(params)
 
-        if(!assignedGroupInstance) {
-            flash.message = "AssignedGroup not found with id ${params.id}"
-            redirect(action:list)
-        }
-        else { return [ assignedGroupInstance : assignedGroupInstance ] }
+        if(!result.error)
+            return [ assignedGroupInstance: result.assignedGroupInstance ]
+
+        flash.message = g.message(code: result.error.code, args: result.error.args)
+
+        if(params.task?.id)
+            redirect(controller:"taskDetailed", action:"show", id: params.task.id)
+
+        redirect(controller:"taskDetailed", action:"search")
     }
 
     def delete = {
-        def assignedGroupInstance = AssignedGroup.get( params.id )
-        if(assignedGroupInstance) {
-            try {
-                def taskId = assignedGroupInstance.task.id
-                assignedGroupInstance.delete()
-                flash.message = "AssignedGroup ${params.id} deleted"
-                redirect(controller:"taskDetailed", action:"show", id: taskId)
-            }
-            catch(org.springframework.dao.DataIntegrityViolationException e) {
-                flash.message = "AssignedGroup ${params.id} could not be deleted"
-                redirect(action:show,id:params.id)
-            }
+        def result = assignedGroupService.delete(params)
+
+        if(!result.error) {
+            flash.message = g.message(code: "default.delete.success", args: ["AssignedGroup", params.id])
+            redirect(controller:"taskDetailed", action:"show", id: params.task?.id)
+            return
         }
-        else {
-            flash.message = "AssignedGroup not found with id ${params.id}"
-            redirect(action:list)
+
+        flash.message = g.message(code: result.error.code, args: result.error.args)
+
+        if(result.error.code == "default.not.found") {
+            redirect(controller:"taskDetailed", action:"show", id: params.task?.id)
+            return
         }
+
+        redirect(action:show, id: params.id)
     }
 
     def edit = {
-        def assignedGroupInstance = AssignedGroup.get( params.id )
+        def result = assignedGroupService.edit(params)
 
-        if(!assignedGroupInstance) {
-            flash.message = "AssignedGroup not found with id ${params.id}"
-            redirect(action:list)
-        }
-        else {
-            return [ assignedGroupInstance : assignedGroupInstance ]
-        }
+        if(!result.error)
+            return [ assignedGroupInstance : result.assignedGroupInstance ]
+
+        flash.message = g.message(code: result.error.code, args: result.error.args)
+
+        redirect(controller:"taskDetailed", action:"show", id: params.task?.id)
+
     }
 
     def update = {
-        def assignedGroupInstance = AssignedGroup.get( params.id )
-        if(assignedGroupInstance) {
-            if(params.version) {
-                def version = params.version.toLong()
-                if(assignedGroupInstance.version > version) {
+        def result = assignedGroupService.update(params)
 
-                    assignedGroupInstance.errors.rejectValue("version", "assignedGroup.optimistic.locking.failure", "Another user has updated this AssignedGroup while you were editing.")
-                    render(view:'edit',model:[assignedGroupInstance:assignedGroupInstance])
-                    return
-                }
-            }
-            assignedGroupInstance.properties = params
-            if(!assignedGroupInstance.hasErrors() && assignedGroupInstance.save(flush: true)) {
-                flash.message = "AssignedGroup ${params.id} updated"
-                redirect(action:show,id:assignedGroupInstance.id)
-            }
-            else {
-                render(view:'edit',model:[assignedGroupInstance:assignedGroupInstance])
-            }
+        if(!result.error) {
+            flash.message = g.message(code: "default.update.success", args: ["AssignedGroup", params.id])
+            redirect(action:show, id: params.id)
+            return
         }
-        else {
-            flash.message = "AssignedGroup not found with id ${params.id}"
-            redirect(action:edit,id:params.id)
+
+        if(result.error.code == "default.not.found") {
+            flash.message = g.message(code: result.error.code, args: result.error.args)
+            redirect(controller:"taskDetailed", action:"show", id: params.task?.id)
+            return
         }
+
+        render(view:'edit', model:[assignedGroupInstance: result.assignedGroupInstance.attach()])
     }
 
     def create = {
-        if(!params.task?.id) {
-            flash.message = "Please select a task and then 'Add Assigned Group'"
+        def result = assignedGroupService.create(params)
+
+        if(!result.error)
+            return [assignedGroupInstance: result.assignedGroupInstance]
+
+        if(result.error.code == "assignedGroup.task.not.found") {
+            flash.message = g.message(code: result.error.code, args: result.error.args)
             redirect(controller: "taskDetailed", action: "search")
+            return
         }
-        else {
-            def assignedGroupInstance = new AssignedGroup()
-            assignedGroupInstance.properties = params
-            return ['assignedGroupInstance':assignedGroupInstance]
-        }
+
+        flash.message = g.message(code: result.error.code, args: result.error.args)
+        redirect(controller:"taskDetailed", action:"show", id: params.task?.id)
     }
 
     def save = {
-        def assignedGroupInstance = new AssignedGroup(params)
+        def result = assignedGroupService.save(params)
 
-        if(!assignedGroupInstance.hasErrors() && assignedGroupInstance.save(flush: true)) {
-            flash.message = "AssignedGroup ${assignedGroupInstance.id} created"
-            redirect(controller:"taskDetailed", action:"show", id: params.task.id)
+        if(!result.error) {
+            flash.message = g.message(code: "default.create.success", args: ["AssignedGroup", result.assignedGroupInstance.id])
+            redirect(controller:"taskDetailed", action:"show", id: result.assignedGroupInstance.task.id)
+            return
         }
-        else {
-            render(view:'create',model:[assignedGroupInstance:assignedGroupInstance])
-        }
+
+        render(view:'edit', model:[assignedGroupInstance: result.assignedGroupInstance])
     }
+
 }
Index: /trunk/grails-app/i18n/messages.properties
===================================================================
--- /trunk/grails-app/i18n/messages.properties	(revision 246)
+++ /trunk/grails-app/i18n/messages.properties	(revision 247)
@@ -94,4 +94,16 @@
 inventoryMovement.inventoryItem.notFound=Inventory Item not found.
 
+assignedGroup.task.not.found=Please select a task and then ''Add Assigned Group''.
+
+default.list.failure=Could not generate list for class {0}.
+default.not.found={0} {1} not found, it may have been deleted.
+default.delete.success={0} {1} deleted.
+default.delete.failure={0} {1} could not be deleted.
+default.update.success={0} {1} updated.
+default.update.failure={0} {1} could not be updated.
+default.create.success={0} {1} created.
+default.create.failure={0} {1} could not be created.
+default.optimistic.locking.failure=Another user has updated this item while you were editing, please check the updated values.
+
 default.doesnt.match.message=Property [{0}] of class [{1}] with value [{2}] does not match the required pattern [{3}]
 default.invalid.url.message=Property [{0}] of class [{1}] with value [{2}] is not a valid URL
@@ -113,5 +125,4 @@
 default.paginate.prev=Previous
 default.paginate.next=Next
-default.optimistic.locking.failure=Another user has updated this item while you were editing, please check the updated values.
 
 # Rich UI plugin - Calendar
Index: /trunk/grails-app/services/AssignedGroupService.groovy
===================================================================
--- /trunk/grails-app/services/AssignedGroupService.groovy	(revision 247)
+++ /trunk/grails-app/services/AssignedGroupService.groovy	(revision 247)
@@ -0,0 +1,145 @@
+class AssignedGroupService {
+    boolean transactional = false
+
+    def list(params) {
+        def result = [:]
+        def fail = { Map m ->
+            result.error = [ code: m.code, args: ["AssignedGroup"] ]
+            return result
+        }
+
+        params.max = Math.min( params.max ? params.max.toInteger() : 10,  100)
+        result.assignedGroupInstanceList = AssignedGroup.list(params)
+        result.assignedGroupInstanceTotal = AssignedGroup.count()
+
+        if(!result.assignedGroupInstanceList || !result.assignedGroupInstanceTotal)
+            return fail(code:"default.list.failure")
+
+        // Success.
+        return result
+    }
+
+    def show(params) {
+        def result = [:]
+        def fail = { Map m ->
+            result.error = [ code: m.code, args: ["AssignedGroup", params.id] ]
+            return result
+        }
+
+        result.assignedGroupInstance = AssignedGroup.get(params.id)
+
+        if(!result.assignedGroupInstance)
+            return fail(code:"default.not.found")
+
+        // Success.
+        return result
+    }
+
+    def delete(params) {
+        def result = [:]
+        def fail = { Map m ->
+            result.error = [ code: m.code, args: ["AssignedGroup", params.id] ]
+            return result
+        }
+
+        result.assignedGroupInstance = AssignedGroup.get(params.id)
+
+        if(!result.assignedGroupInstance)
+            return fail(code:"default.not.found")
+
+        try {
+            result.assignedGroupInstance.delete(flush:true)
+            return result //Success.
+        }
+        catch(org.springframework.dao.DataIntegrityViolationException e) {
+            return fail(code:"default.delete.failure")
+        }
+
+    }
+
+    def edit(params) {
+        def result = [:]
+        def fail = { Map m ->
+            result.error = [ code: m.code, args: ["AssignedGroup", params.id] ]
+            return result
+        }
+
+        result.assignedGroupInstance = AssignedGroup.get(params.id)
+
+        if(!result.assignedGroupInstance)
+            return fail(code:"default.not.found")
+
+        // Success.
+        return result
+    }
+
+    def update(params) {
+        AssignedGroup.withTransaction { status ->
+            def result = [:]
+
+            def fail = { Map m ->
+                status.setRollbackOnly()
+                if(result.assignedGroupInstance && m.field)
+                    result.assignedGroupInstance.errors.rejectValue(m.field, m.code)
+                result.error = [ code: m.code, args: ["AssignedGroup", params.id] ]
+                return result
+            }
+
+            result.assignedGroupInstance = AssignedGroup.get(params.id)
+
+            if(!result.assignedGroupInstance)
+                return fail(code:"default.not.found")
+
+            // Optimistic locking check.
+            if(params.version) {
+                if(result.assignedGroupInstance.version > params.version.toLong())
+                    return fail(field:"version", code:"default.optimistic.locking.failure")
+            }
+
+            result.assignedGroupInstance.properties = params
+
+            if(result.assignedGroupInstance.hasErrors() || !result.assignedGroupInstance.save())
+                return fail(code:"default.update.failure")
+
+            // Success.
+            return result
+
+        } //end withTransaction
+    }  // end update()
+
+    def create(params) {
+        def result = [:]
+        def fail = { Map m ->
+            result.error = [ code: m.code, args: ["AssignedGroup", params.id] ]
+            return result
+        }
+
+        if(!params.task?.id)
+            fail(code:"assignedGroup.task.not.found")
+
+        result.assignedGroupInstance = new AssignedGroup()
+        result.assignedGroupInstance.properties = params
+
+        // success
+        return result
+    }
+
+    def save(params) {
+        def result = [:]
+        def fail = { Map m ->
+            if(result.assignedGroupInstance && m.field)
+                result.assignedGroupInstance.errors.rejectValue(m.field, m.code)
+            result.error = [ code: m.code, args: ["AssignedGroup", params.id] ]
+            return result
+        }
+
+        result.assignedGroupInstance = new AssignedGroup(params)
+
+        if(result.assignedGroupInstance.hasErrors() || !result.assignedGroupInstance.save(flush: true))
+            return fail(code:"default.create.failure")
+
+        // success
+        return result
+    }
+
+} // end class
Index: /trunk/grails-app/views/assignedGroupDetailed/create.gsp
===================================================================
--- /trunk/grails-app/views/assignedGroupDetailed/create.gsp	(revision 246)
+++ /trunk/grails-app/views/assignedGroupDetailed/create.gsp	(revision 247)
@@ -21,5 +21,5 @@
             </g:hasErrors>
             <g:form action="save" method="post" >
-                <input type="hidden"  name="task.id" value="${assignedGroupInstance?.task?.id}" />
+                <g:hiddenField name="task.id" value="${assignedGroupInstance?.task?.id}" />
                 <div class="dialog">
                     <table>
Index: /trunk/grails-app/views/assignedGroupDetailed/edit.gsp
===================================================================
--- /trunk/grails-app/views/assignedGroupDetailed/edit.gsp	(revision 246)
+++ /trunk/grails-app/views/assignedGroupDetailed/edit.gsp	(revision 247)
@@ -21,6 +21,7 @@
             </g:hasErrors>
             <g:form method="post" >
-                <input type="hidden" name="id" value="${assignedGroupInstance?.id}" />
-                <input type="hidden" name="version" value="${assignedGroupInstance?.version}" />
+                <g:hiddenField name="id" value="${assignedGroupInstance?.id}" />
+                <g:hiddenField name="task.id" value="${assignedGroupInstance?.task?.id}" />
+                <g:hiddenField name="version" value="${assignedGroupInstance?.version}" />
                 <div class="dialog">
                     <table>
Index: /trunk/grails-app/views/assignedGroupDetailed/show.gsp
===================================================================
--- /trunk/grails-app/views/assignedGroupDetailed/show.gsp	(revision 246)
+++ /trunk/grails-app/views/assignedGroupDetailed/show.gsp	(revision 247)
@@ -51,5 +51,6 @@
             <div class="buttons">
                 <g:form>
-                    <input type="hidden" name="id" value="${assignedGroupInstance?.id}" />
+                    <g:hiddenField name="id" value="${assignedGroupInstance?.id}" />
+                    <g:hiddenField name="task.id" value="${assignedGroupInstance?.task?.id}" />
                     <span class="button"><g:actionSubmit class="edit" value="Edit" /></span>
                     <span class="button"><g:actionSubmit class="delete" onclick="return confirm('Are you sure?');" value="Delete" /></span>
Index: /trunk/grails-app/views/taskDetailed/edit.gsp
===================================================================
--- /trunk/grails-app/views/taskDetailed/edit.gsp	(revision 246)
+++ /trunk/grails-app/views/taskDetailed/edit.gsp	(revision 247)
@@ -5,5 +5,5 @@
         <meta http-equiv="Content-Type" content="text/html; charset=UTF-8"/>
         <meta name="layout" content="main" />
-        <title>Edit Task #${taskInstance.id}</title>
+        <title>Edit Task #${taskInstance?.id}</title>
         <nav:resources override="true"/>
         <resource:dateChooser />
@@ -36,5 +36,5 @@
                             </tr>
 
-                            <g:if test="${taskInstance.parentTask}">
+                            <g:if test="${taskInstance?.parentTask}">
                                 <tr class="prop">
                                     <td valign="top" class="name">Parent Task:</td>
@@ -166,5 +166,5 @@
                                 <ul>
                                 <g:each var="a" in="${taskInstance?.assignedGroups?}">
-                                    <li><g:link controller="assignedGroupDetailed" action="edit" id="${a.id}">${a?.encodeAsHTML()}</g:link></li>
+                                    <li><g:link controller="assignedGroupDetailed" action="edit" id="${a.id}" params="['task.id':taskInstance?.id]">${a?.encodeAsHTML()}</g:link></li>
                                 </g:each>
                                 </ul>
@@ -181,5 +181,5 @@
                                 <ul>
                                 <g:each var="a" in="${taskInstance?.assignedPersons?}">
-                                    <li><g:link controller="assignedPersonDetailed" action="edit" id="${a.id}">${a?.encodeAsHTML()}</g:link></li>
+                                    <li><g:link controller="assignedPersonDetailed" action="edit" id="${a.id}" params="['task.id':taskInstance?.id]">${a?.encodeAsHTML()}</g:link></li>
                                 </g:each>
                                 </ul>
Index: /trunk/grails-app/views/taskDetailed/show.gsp
===================================================================
--- /trunk/grails-app/views/taskDetailed/show.gsp	(revision 246)
+++ /trunk/grails-app/views/taskDetailed/show.gsp	(revision 247)
@@ -189,8 +189,8 @@
                                             <ul>
                                             <g:each var="a" in="${taskInstance.assignedGroups}">
-                                                <li><g:link controller="assignedGroupDetailed" action="edit" id="${a.id}">${a?.encodeAsHTML()}</g:link></li>
+                                                <li><g:link controller="assignedGroupDetailed" action="edit" id="${a.id}" params="['task.id':taskInstance.id]">${a?.encodeAsHTML()}</g:link></li>
                                             </g:each>
                                             </ul>
-                                            <g:link controller="assignedGroupDetailed" params="['task.id':taskInstance?.id]" action="create">Add Assigned Group</g:link>
+                                            <g:link controller="assignedGroupDetailed" action="create" params="['task.id':taskInstance.id]">Add Assigned Group</g:link>
                                         </td>
 
@@ -462,5 +462,5 @@
                                 <div class="buttons">
                                     <span class="button">
-                                        <g:actionSubmit value="Create" action="create"  class="add"/>
+                                        <g:actionSubmit value="Add" action="create"  class="add"/>
                                     </span>
                                 </div>
@@ -664,5 +664,5 @@
                                     </g:else>
                                     <span class="button"><g:actionSubmit action="listSubTasks" class="table" value="List" /></span>
-                                    <span class="button"><g:actionSubmit action="createSubTask" class="add" value="Create" /></span>
+                                    <span class="button"><g:actionSubmit action="createSubTask" class="save" value="Create" /></span>
                                 </g:form>
                             </div>
@@ -678,5 +678,5 @@
                                 <g:form>
                                     <g:hiddenField name="id" value="${taskInstance?.id}" />
-                                    <span class="button"><g:actionSubmit action="createSubTask" class="add" value="Create" /></span>
+                                    <span class="button"><g:actionSubmit action="createSubTask" class="save" value="Create" /></span>
                                 </g:form>
                             </div>