Changeset 97 for trunk/src/grails-app/controllers

Timestamp:

Apr 1, 2009, 5:07:23 AM (16 years ago)

Author:

gav

Message:

Squashed the password validation bug in Person edit

Location:

trunk/src/grails-app/controllers

Files:

• AssignedPersonDetailedController.groovy (modified) (1 diff)
• PersonController.groovy (modified) (6 diffs)

trunk/src/grails-app/controllers/AssignedPersonDetailedController.groovy

@@ -1 +1 @@
 import org.codehaus.groovy.grails.plugins.springsecurity.Secured
 
-class AssignedPersonController extends BaseController {
+class AssignedPersonDetailedController extends BaseController {
     
     def index = { redirect(action:list,params:params) }

trunk/src/grails-app/controllers/PersonController.groovy

@@ -1 +1 @@
 import org.codehaus.groovy.grails.plugins.springsecurity.Secured
 
-@Secured(['ROLE_AppAdmin', 'ROLE_Manager'])
-class PersonController {
+@Secured(['ROLE_Manager'])
+class PersonController extends BaseAppAdminController {
 
         def authenticateService
@@ -54 +54 @@
                                 Authority.findAll().each { it.removeFromPersons(person) }
                 
-//                 try {
-                    person.delete(flush:true)
+                try {
+                    person.delete()
                     flash.message = "Person $params.id deleted."
-//                 }
-//                 catch(Exception e) {
-//                     flash.message = "Could not delete '$person.loginName' due to database constraints, but all authorities have been removed."
-//                     redirect action: edit, id: person.id
-//                 }
-
+                    redirect(action:list)
+                }
+                catch(org.springframework.dao.DataIntegrityViolationException e) {
+                    flash.message = "Could not delete '$person.loginName' due to database constraints, but all authorities have been removed."
+                    redirect(action:show,id:params.id)
+                }
                         }
                 }
@@ -68 +68 @@
                         flash.message = "Person not found with id $params.id"
                 }
-
-                redirect action: list
         }
 
@@ -100 +98 @@
                         person.errors.rejectValue 'version', "person.optimistic.locking.failure",
                                 "Another user has updated this Person while you were editing."
-                                render view: 'edit', model: buildPersonModel(person)
+            render view: 'edit', model: buildPersonModel(person)
                         return
                 }
@@ -106 +104 @@
                 person.properties = params
 
-        if (person.pass != "") {
-            if (!person.hasErrors()) {
+        if(params.pass == "") {
+            person.pass = "InsertNothingToClearValidation"
+        }
+        else {
+            if (person.validate()) {
                 person.password = authenticateService.encodePassword(params.pass)
             }
-        }
-        else {
-            person.pass = "NothingToClearValidation"
         }
 
@@ -118 +116 @@
             Authority.findAll().each { it.removeFromPersons(person) }
             addRoles(person)
+            flash.message = "Person '$params.id - $params.loginName' updated."
             redirect action: show, id: person.id
         }