Ignore:
Timestamp:
Jan 12, 2011, 2:42:06 PM (6 years ago)
Author:
business@…
Message:

added extra check in sample validation to avoid NPEs in case of broken Study objects

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/grails-app/domain/dbnp/studycapturing/Sample.groovy

    r1314 r1373  
    8282                        def error = false
    8383
     84                        // check whether obj.parent.samples is not null at this stage to avoid null pointer exception
    8485                        if (obj.parent) {
    8586
    86                                 // check if there is exactly one sample with this name in the study (this one)
    87                                 if (obj.parent.samples.findAll{ it.name == obj.name}.size() > 1) {
    88                                         error = true
    89                                         errors.rejectValue(
    90                                                 'name',
    91                                                 'sample.UniqueNameViolation',
    92                                                 [obj.name, obj.parent] as Object[],
    93                                                 'Sample name {0} appears multiple times in study {1}'
    94                                                 )
     87                                if (obj.parent.samples) {
     88
     89                                        // check if there is exactly one sample with this name in the study (this one)
     90                                        if (obj.parent.samples.findAll{ it.name == obj.name}.size() > 1) {
     91                                                error = true
     92                                                errors.rejectValue(
     93                                                        'name',
     94                                                        'sample.UniqueNameViolation',
     95                                                        [obj.name, obj.parent] as Object[],
     96                                                        'Sample name {0} appears multiple times in study {1}'
     97                                                        )
     98                                        }
    9599                                }
    96100                        }
Note: See TracChangeset for help on using the changeset viewer.