Changeset 960

Show
Ignore:
Timestamp:
21-10-10 12:24:40 (3 years ago)
Author:
business@…
Message:

changed parentEvent in Sample to nullable. Please make sure that any code that depends on the assertion that Sample always has a parentEvent is changed accordingly! In this commit done for getSamples in RestController?.

Location:
trunk/grails-app
Files:
2 modified

Legend:

Unmodified
Added
Removed
  • trunk/grails-app/controllers/RestController.groovy

    r959 r960  
    156156                                        def item = [  
    157157                                                'sampleToken' : sample.name, 
    158                                                 'material'        : sample.material.name, 
    159                                                 'subject'         : sample.parentSubject.name, 
    160                                                 'event'           : sample.parentEvent.template.name, 
    161                                                 'startTime'       : sample.parentEvent.getStartTimeString() 
     158                                                'material'        : sample.material?.name, 
     159                                                'subject'         : sample.parentSubject?.name, 
     160                                                'event'           : sample.parentEvent?.template?.name, 
     161                                                'startTime'       : sample.parentEvent?.getStartTimeString() 
    162162                                        ] 
    163163                                        items.push item  
  • trunk/grails-app/domain/dbnp/studycapturing/Sample.groovy

    r959 r960  
    5454                // The parent subject is optional, e.g. in a biobank of samples the subject could be unknown or non-existing. 
    5555                parentSubject(nullable:true) 
    56  
     56                // The same holds for parentEvent 
     57                parentEvent(nullable:true) 
     58                 
    5759                // The material domain field is optional 
    5860                material(nullable: true)