Index: trunk/grails-app/controllers/PersonController.groovy
===================================================================
--- trunk/grails-app/controllers/PersonController.groovy	(revision 149)
+++ trunk/grails-app/controllers/PersonController.groovy	(revision 150)
@@ -7,10 +7,10 @@
     def filterService
 
-	// the delete, save and update actions only accept POST requests
-	static Map allowedMethods = [delete: 'POST', save: 'POST', update: 'POST']
+    // the delete, save and update actions only accept POST requests
+    static Map allowedMethods = [delete: 'POST', save: 'POST', update: 'POST']
 
-	def index = {
-		redirect action: list, params: params
-	}
+    def index = {
+        redirect action: list, params: params
+    }
 
     def list = {
@@ -27,5 +27,5 @@
     }
 
-	def show = {
+    def show = {
 
         // In the case of an actionSubmit button, rewrite action name from 'index'.
@@ -33,37 +33,37 @@
         { params.action='show' }
 
-		def person = Person.get(params.id)
-		if (!person) {
-			flash.message = "Person not found with id $params.id"
-			redirect action: list
-			return
-		}
-		List roleNames = []
-		for (role in person.authorities) {
-			roleNames << role.authority
-		}
-		roleNames.sort { n1, n2 ->
-			n1 <=> n2
-		}
-		[person: person, roleNames: roleNames]
-	}
+        def person = Person.get(params.id)
+        if (!person) {
+            flash.message = "Person not found with id $params.id"
+            redirect action: list
+            return
+        }
+        List roleNames = []
+        for (role in person.authorities) {
+            roleNames << role.authority
+        }
+        roleNames.sort { n1, n2 ->
+            n1 <=> n2
+        }
+        [person: person, roleNames: roleNames]
+    }
 
-	/**
-	 * Person delete action. Before removing an existing person,
-	 * they should be removed from those authorities which they are involved.
-	 */
-	def delete = {
+    /**
+    * Person delete action. Before removing an existing person,
+    * they should be removed from those authorities which they are involved.
+    */
+    def delete = {
 
-		def person = Person.get(params.id)
-		if (person) {
-			def authPrincipal = authenticateService.principal()
-			// Avoid self-delete.
-			if (!(authPrincipal instanceof String) && authPrincipal.username == person.loginName) {
-				flash.message = "You cannot delete yourself, please login as another manager and try again."
+        def person = Person.get(params.id)
+        if (person) {
+            def authPrincipal = authenticateService.principal()
+            // Avoid self-delete.
+            if (!(authPrincipal instanceof String) && authPrincipal.username == person.loginName) {
+                flash.message = "You cannot delete yourself, please login as another manager and try again."
                 redirect(action:show,id:params.id)
-			}
-			else {
-				//first, delete this person from Persons_Authorities table.
-				Authority.findAll().each { it.removeFromPersons(person) }
+            }
+            else {
+                //first, delete this person from Persons_Authorities table.
+                Authority.findAll().each { it.removeFromPersons(person) }
                 person.isActive = false
                 person.save(flush: true)
@@ -78,12 +78,12 @@
                     redirect(action:show,id:params.id)
                 }
-			}
-		}
-		else {
-			flash.message = "Person not found with id $params.id"
-		}
-	}
+            }
+        }
+        else {
+            flash.message = "Person not found with id $params.id"
+        }
+    }
 
-	def edit = {
+    def edit = {
 
         // In the case of an actionSubmit button, rewrite action name from 'index'.
@@ -91,35 +91,35 @@
         { params.action='edit' }
 
-		def person = Person.get(params.id)
-		if (!person) {
-			flash.message = "Person not found with id $params.id"
-			redirect action: list
-			return
-		}
-        flash.message = "To allow login at least the 'ROLE_AppUser' authority must be given."
-		return buildPersonModel(person)
-	}
+        def person = Person.get(params.id)
+        if (!person) {
+            flash.message = "Person not found with id $params.id"
+            redirect action: list
+            return
+        }
+        params.message = "To allow login at least the 'ROLE_AppUser' authority must be given."
+        return buildPersonModel(person)
+    }
 
-	/**
-	 * Person update action.
-	 */
-	def update = {
+    /**
+        * Person update action.
+        */
+    def update = {
 
-		def person = Person.get(params.id)
-		if (!person) {
-			flash.message = "Person not found with id $params.id"
-			redirect action: edit, id: params.id
-			return
-		}
+        def person = Person.get(params.id)
+        if (!person) {
+            flash.message = "Person not found with id $params.id"
+            redirect action: edit, id: params.id
+            return
+        }
 
-		long version = params.version.toLong()
-		if (person.version > version) {
-			person.errors.rejectValue 'version', "person.optimistic.locking.failure",
-				"Another user has updated this Person while you were editing."
+        long version = params.version.toLong()
+        if (person.version > version) {
+            person.errors.rejectValue 'version', "person.optimistic.locking.failure",
+                "Another user has updated this Person while you were editing."
             render view: 'edit', model: buildPersonModel(person)
-			return
-		}
+            return
+        }
 
-		person.properties = params
+        person.properties = params
 
         if(params.pass == "") {
@@ -142,52 +142,52 @@
         }
 
-	}
+    }
 
-	def create = {
-        flash.message = "To allow login at least the 'ROLE_AppUser' authority must be given."
-		[person: new Person(params), authorityList: Authority.list()]
-	}
+    def create = {
+        params.message = "To allow login at least the 'ROLE_AppUser' authority must be given."
+        [person: new Person(params), authorityList: Authority.list()]
+    }
 
-	/**
-	 * Person save action.
-	 */
-	def save = {
+    /**
+        * Person save action.
+        */
+    def save = {
 
-		def person = new Person()
-		person.properties = params
-		person.password = authenticateService.encodePassword(params.pass)
-		if (person.save()) {
-			addRoles(person)
-			redirect action: show, id: person.id
-		}
-		else {
-			render view: 'create', model: [authorityList: Authority.list(), person: person]
-		}
-	}
+        def person = new Person()
+        person.properties = params
+        person.password = authenticateService.encodePassword(params.pass)
+        if (person.save()) {
+            addRoles(person)
+            redirect action: show, id: person.id
+        }
+        else {
+            render view: 'create', model: [authorityList: Authority.list(), person: person]
+        }
+    }
 
-	private void addRoles(person) {
-		for (String key in params.keySet()) {
-			if (key.contains('ROLE') && 'on' == params.get(key)) {
-				Authority.findByAuthority(key).addToPersons(person)
-			}
-		}
-	}
+    private void addRoles(person) {
+        for (String key in params.keySet()) {
+            if (key.contains('ROLE') && 'on' == params.get(key)) {
+                Authority.findByAuthority(key).addToPersons(person)
+            }
+        }
+    }
 
-	private Map buildPersonModel(person) {
+    private Map buildPersonModel(person) {
 
-		List roles = Authority.list()
-		roles.sort { r1, r2 ->
-			r1.authority <=> r2.authority
-		}
-		Set userRoleNames = []
-		for (role in person.authorities) {
-			userRoleNames << role.authority
-		}
-		LinkedHashMap<Authority, Boolean> roleMap = [:]
-		for (role in roles) {
-			roleMap[(role)] = userRoleNames.contains(role.authority)
-		}
+        List roles = Authority.list()
+        roles.sort { r1, r2 ->
+            r1.authority <=> r2.authority
+        }
+        Set userRoleNames = []
+        for (role in person.authorities) {
+            userRoleNames << role.authority
+        }
+        LinkedHashMap<Authority, Boolean> roleMap = [:]
+        for (role in roles) {
+            roleMap[(role)] = userRoleNames.contains(role.authority)
+        }
 
-		return [person: person, roleMap: roleMap]
-	}
+        return [person: person, roleMap: roleMap]
+    }
 }
