Ignore:
Timestamp:
Oct 9, 2009, 10:11:43 PM (15 years ago)
Author:
gav
Message:

Add params.message to Person create and edit views.
Code format to 4 spaces, no tabs.

Location:
trunk/grails-app/controllers
Files:
3 edited

Legend:

Unmodified
Added
Removed

  • trunk/grails-app/controllers/AppCoreController.groovy

    r149 r150  
    5555            def personInstance = Person.get(authenticateService.userDomain().id)
    5656            return [ personInstance : personInstance ]       
    57         } 
     57        }
    5858
    5959        if (request.method == 'POST') {

  • trunk/grails-app/controllers/AuthorityController.groovy

    r116 r150  
    33class AuthorityController extends BaseAppAdminController {
    44
    5         // the delete, save and update actions only accept POST requests
    6         static Map allowedMethods = [delete: 'POST', save: 'POST', update: 'POST']
     5    // the delete, save and update actions only accept POST requests
     6    static Map allowedMethods = [delete: 'POST', save: 'POST', update: 'POST']
    77
    8         def authenticateService
     8    def authenticateService
    99
    10         def index = {
    11                 redirect action: list, params: params
    12         }
     10    def index = {
     11        redirect action: list, params: params
     12    }
    1313
    14         /**
    15         * Display the list authority page.
    16         */
    17         def list = {
    18                 if (!params.max) {
    19                         params.max = 10
    20                 }
    21                 [authorityList: Authority.list(params)]
    22         }
     14    /**
     15    * Display the list authority page.
     16    */
     17    def list = {
     18        if (!params.max) {
     19            params.max = 10
     20        }
     21        [authorityList: Authority.list(params)]
     22    }
    2323
    24         /**
    25         * Display the show authority page.
    26         */
    27         def show = {
    28                 def authority = Authority.get(params.id)
    29                 if (!authority) {
    30                         flash.message = "Authority not found with id $params.id"
    31                         redirect action: list
    32                         return
    33                 }
     24    /**
     25    * Display the show authority page.
     26    */
     27    def show = {
     28        def authority = Authority.get(params.id)
     29        if (!authority) {
     30            flash.message = "Authority not found with id $params.id"
     31            redirect action: list
     32            return
     33        }
    3434
    35                 [authority: authority]
    36         }
     35        [authority: authority]
     36    }
    3737
    38         /**
    39         * Delete an authority.
    40         */
    41         def delete = {
    42                 def authority = Authority.get(params.id)
    43                 if (!authority) {
    44                         flash.message = "Authority not found with id $params.id"
    45                         redirect action: list
    46                         return
    47                 }
     38    /**
     39    * Delete an authority.
     40    */
     41    def delete = {
     42        def authority = Authority.get(params.id)
     43        if (!authority) {
     44            flash.message = "Authority not found with id $params.id"
     45            redirect action: list
     46            return
     47        }
    4848
    49                 authenticateService.deleteRole(authority)
     49        authenticateService.deleteRole(authority)
    5050
    51                 flash.message = "Authority $params.id deleted."
    52                 redirect action: list
    53         }
     51        flash.message = "Authority $params.id deleted."
     52        redirect action: list
     53    }
    5454
    55         /**
    56         * Display the edit authority page.
    57         */
    58         def edit = {
    59                 def authority = Authority.get(params.id)
    60                 if (!authority) {
    61                         flash.message = "Authority not found with id $params.id"
    62                         redirect action: list
    63                         return
    64                 }
     55    /**
     56    * Display the edit authority page.
     57    */
     58    def edit = {
     59        def authority = Authority.get(params.id)
     60        if (!authority) {
     61            flash.message = "Authority not found with id $params.id"
     62            redirect action: list
     63            return
     64        }
    6565
    66                 [authority: authority]
    67         }
     66        [authority: authority]
     67    }
    6868
    69         /**
    70         * Authority update action.
    71         */
    72         def update = {
     69    /**
     70    * Authority update action.
     71    */
     72    def update = {
    7373
    74                 def authority = Authority.get(params.id)
    75                 if (!authority) {
    76                         flash.message = "Authority not found with id $params.id"
    77                         redirect action: edit, id: params.id
    78                         return
    79                 }
     74        def authority = Authority.get(params.id)
     75        if (!authority) {
     76            flash.message = "Authority not found with id $params.id"
     77            redirect action: edit, id: params.id
     78            return
     79        }
    8080
    81                 long version = params.version.toLong()
    82                 if (authority.version > version) {
    83                         authority.errors.rejectValue 'version', 'authority.optimistic.locking.failure',
    84                                 'Another user has updated this Authority while you were editing.'
    85                         render view: 'edit', model: [authority: authority]
    86                         return
    87                 }
     81        long version = params.version.toLong()
     82        if (authority.version > version) {
     83            authority.errors.rejectValue 'version', 'authority.optimistic.locking.failure',
     84                'Another user has updated this Authority while you were editing.'
     85            render view: 'edit', model: [authority: authority]
     86            return
     87        }
    8888
    89                 if (authenticateService.updateRole(authority, params)) {
    90                         authenticateService.clearCachedRequestmaps()
    91                         redirect action: show, id: authority.id
    92                 }
    93                 else {
    94                         render view: 'edit', model: [authority: authority]
    95                 }
    96         }
     89        if (authenticateService.updateRole(authority, params)) {
     90            authenticateService.clearCachedRequestmaps()
     91            redirect action: show, id: authority.id
     92        }
     93        else {
     94            render view: 'edit', model: [authority: authority]
     95        }
     96    }
    9797
    98         /**
    99         * Display the create new authority page.
    100         */
    101         def create = {
    102                 [authority: new Authority()]
    103         }
     98    /**
     99    * Display the create new authority page.
     100    */
     101    def create = {
     102        [authority: new Authority()]
     103    }
    104104
    105         /**
    106         * Save a new authority.
    107         */
    108         def save = {
     105    /**
     106    * Save a new authority.
     107    */
     108    def save = {
    109109
    110                 def authority = new Authority()
    111                 authority.properties = params
    112                 if (authority.save()) {
    113                         redirect action: show, id: authority.id
    114                 }
    115                 else {
    116                         render view: 'create', model: [authority: authority]
    117                 }
    118         }
     110        def authority = new Authority()
     111        authority.properties = params
     112        if (authority.save()) {
     113            redirect action: show, id: authority.id
     114        }
     115        else {
     116            render view: 'create', model: [authority: authority]
     117        }
     118    }
    119119}

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

    r149 r150  
    77    def filterService
    88
    9         // the delete, save and update actions only accept POST requests
    10         static Map allowedMethods = [delete: 'POST', save: 'POST', update: 'POST']
     9    // the delete, save and update actions only accept POST requests
     10    static Map allowedMethods = [delete: 'POST', save: 'POST', update: 'POST']
    1111
    12         def index = {
    13                 redirect action: list, params: params
    14         }
     12    def index = {
     13        redirect action: list, params: params
     14    }
    1515
    1616    def list = {
    … …  
    2727    }
    2828
    29         def show = {
     29    def show = {
    3030
    3131        // In the case of an actionSubmit button, rewrite action name from 'index'.
    … …  
    3333        { params.action='show' }
    3434
    35                 def person = Person.get(params.id)
    36                 if (!person) {
    37                         flash.message = "Person not found with id $params.id"
    38                         redirect action: list
    39                         return
    40                 }
    41                 List roleNames = []
    42                 for (role in person.authorities) {
    43                         roleNames << role.authority
    44                 }
    45                 roleNames.sort { n1, n2 ->
    46                         n1 <=> n2
    47                 }
    48                 [person: person, roleNames: roleNames]
    49         }
     35        def person = Person.get(params.id)
     36        if (!person) {
     37            flash.message = "Person not found with id $params.id"
     38            redirect action: list
     39            return
     40        }
     41        List roleNames = []
     42        for (role in person.authorities) {
     43            roleNames << role.authority
     44        }
     45        roleNames.sort { n1, n2 ->
     46            n1 <=> n2
     47        }
     48        [person: person, roleNames: roleNames]
     49    }
    5050
    51         /**
    52         * Person delete action. Before removing an existing person,
    53         * they should be removed from those authorities which they are involved.
    54         */
    55         def delete = {
     51    /**
     52    * Person delete action. Before removing an existing person,
     53    * they should be removed from those authorities which they are involved.
     54    */
     55    def delete = {
    5656
    57                 def person = Person.get(params.id)
    58                 if (person) {
    59                         def authPrincipal = authenticateService.principal()
    60                         // Avoid self-delete.
    61                         if (!(authPrincipal instanceof String) && authPrincipal.username == person.loginName) {
    62                                 flash.message = "You cannot delete yourself, please login as another manager and try again."
     57        def person = Person.get(params.id)
     58        if (person) {
     59            def authPrincipal = authenticateService.principal()
     60            // Avoid self-delete.
     61            if (!(authPrincipal instanceof String) && authPrincipal.username == person.loginName) {
     62                flash.message = "You cannot delete yourself, please login as another manager and try again."
    6363                redirect(action:show,id:params.id)
    64                         }
    65                         else {
    66                                 //first, delete this person from Persons_Authorities table.
    67                                 Authority.findAll().each { it.removeFromPersons(person) }
     64            }
     65            else {
     66                //first, delete this person from Persons_Authorities table.
     67                Authority.findAll().each { it.removeFromPersons(person) }
    6868                person.isActive = false
    6969                person.save(flush: true)
    … …  
    7878                    redirect(action:show,id:params.id)
    7979                }
    80                         }
    81                 }
    82                 else {
    83                         flash.message = "Person not found with id $params.id"
    84                 }
    85         }
     80            }
     81        }
     82        else {
     83            flash.message = "Person not found with id $params.id"
     84        }
     85    }
    8686
    87         def edit = {
     87    def edit = {
    8888
    8989        // In the case of an actionSubmit button, rewrite action name from 'index'.
    … …  
    9191        { params.action='edit' }
    9292
    93                 def person = Person.get(params.id)
    94                 if (!person) {
    95                         flash.message = "Person not found with id $params.id"
    96                         redirect action: list
    97                         return
    98                 }
    99         flash.message = "To allow login at least the 'ROLE_AppUser' authority must be given."
    100                 return buildPersonModel(person)
    101         }
     93        def person = Person.get(params.id)
     94        if (!person) {
     95            flash.message = "Person not found with id $params.id"
     96            redirect action: list
     97            return
     98        }
     99        params.message = "To allow login at least the 'ROLE_AppUser' authority must be given."
     100        return buildPersonModel(person)
     101    }
    102102
    103         /**
    104         * Person update action.
    105         */
    106         def update = {
     103    /**
     104        * Person update action.
     105        */
     106    def update = {
    107107
    108                 def person = Person.get(params.id)
    109                 if (!person) {
    110                         flash.message = "Person not found with id $params.id"
    111                         redirect action: edit, id: params.id
    112                         return
    113                 }
     108        def person = Person.get(params.id)
     109        if (!person) {
     110            flash.message = "Person not found with id $params.id"
     111            redirect action: edit, id: params.id
     112            return
     113        }
    114114
    115                 long version = params.version.toLong()
    116                 if (person.version > version) {
    117                         person.errors.rejectValue 'version', "person.optimistic.locking.failure",
    118                                 "Another user has updated this Person while you were editing."
     115        long version = params.version.toLong()
     116        if (person.version > version) {
     117            person.errors.rejectValue 'version', "person.optimistic.locking.failure",
     118                "Another user has updated this Person while you were editing."
    119119            render view: 'edit', model: buildPersonModel(person)
    120                         return
    121                 }
     120            return
     121        }
    122122
    123                 person.properties = params
     123        person.properties = params
    124124
    125125        if(params.pass == "") {
    … …  
    142142        }
    143143
    144         }
     144    }
    145145
    146         def create = {
    147         flash.message = "To allow login at least the 'ROLE_AppUser' authority must be given."
    148                 [person: new Person(params), authorityList: Authority.list()]
    149         }
     146    def create = {
     147        params.message = "To allow login at least the 'ROLE_AppUser' authority must be given."
     148        [person: new Person(params), authorityList: Authority.list()]
     149    }
    150150
    151         /**
    152         * Person save action.
    153         */
    154         def save = {
     151    /**
     152        * Person save action.
     153        */
     154    def save = {
    155155
    156                 def person = new Person()
    157                 person.properties = params
    158                 person.password = authenticateService.encodePassword(params.pass)
    159                 if (person.save()) {
    160                         addRoles(person)
    161                         redirect action: show, id: person.id
    162                 }
    163                 else {
    164                         render view: 'create', model: [authorityList: Authority.list(), person: person]
    165                 }
    166         }
     156        def person = new Person()
     157        person.properties = params
     158        person.password = authenticateService.encodePassword(params.pass)
     159        if (person.save()) {
     160            addRoles(person)
     161            redirect action: show, id: person.id
     162        }
     163        else {
     164            render view: 'create', model: [authorityList: Authority.list(), person: person]
     165        }
     166    }
    167167
    168         private void addRoles(person) {
    169                 for (String key in params.keySet()) {
    170                         if (key.contains('ROLE') && 'on' == params.get(key)) {
    171                                 Authority.findByAuthority(key).addToPersons(person)
    172                         }
    173                 }
    174         }
     168    private void addRoles(person) {
     169        for (String key in params.keySet()) {
     170            if (key.contains('ROLE') && 'on' == params.get(key)) {
     171                Authority.findByAuthority(key).addToPersons(person)
     172            }
     173        }
     174    }
    175175
    176         private Map buildPersonModel(person) {
     176    private Map buildPersonModel(person) {
    177177
    178                 List roles = Authority.list()
    179                 roles.sort { r1, r2 ->
    180                         r1.authority <=> r2.authority
    181                 }
    182                 Set userRoleNames = []
    183                 for (role in person.authorities) {
    184                         userRoleNames << role.authority
    185                 }
    186                 LinkedHashMap<Authority, Boolean> roleMap = [:]
    187                 for (role in roles) {
    188                         roleMap[(role)] = userRoleNames.contains(role.authority)
    189                 }
     178        List roles = Authority.list()
     179        roles.sort { r1, r2 ->
     180            r1.authority <=> r2.authority
     181        }
     182        Set userRoleNames = []
     183        for (role in person.authorities) {
     184            userRoleNames << role.authority
     185        }
     186        LinkedHashMap<Authority, Boolean> roleMap = [:]
     187        for (role in roles) {
     188            roleMap[(role)] = userRoleNames.contains(role.authority)
     189        }
    190190
    191                 return [person: person, roleMap: roleMap]
    192         }
     191        return [person: person, roleMap: roleMap]
     192    }
    193193}
Note: See TracChangeset for help on using the changeset viewer.