Changeset 1736


Ignore:
Timestamp:
Apr 8, 2011, 4:03:23 PM (7 years ago)
Author:
work@…
Message:
  • resolves issue #384, studyPerson, publications and users are not deleted properly
Location:
trunk/grails-app
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/grails-app/controllers/dbnp/studycapturing/StudyWizardController.groovy

    r1733 r1736  
    13161316
    13171317                   // First remove the publication that are not present in the array
    1318                    if( study.publications )
    1319                            study.publications.removeAll { publication -> !publicationIDs.find { id -> id == publication.id } }
     1318                   if( study.publications ) {
     1319                           study.publications.findAll { publication -> !publicationIDs.find { id -> id == publication.id } }.each {
     1320                                   study.removeFromPublications(it)
     1321                           }
     1322                   }
    13201323
    13211324                   // Add those publications not yet present in the database
     
    13331336           } else {
    13341337                   log.info('.no publications selected.')
    1335                    if( study.publications )
    1336                            study.publications.clear()
     1338                   if( study.publications ) {
     1339                           study.publications.each {
     1340                                   study.removeFromPublications(it)
     1341                           }
     1342                   }
    13371343           }
    13381344   }
     
    13631369                   // First remove the contacts that are not present in the array
    13641370                   if( study.persons ) {
    1365                            study.persons.removeAll {
     1371                           study.persons.findAll {
    13661372                                   studyperson -> !contactIDs.find { ids -> (ids.person == studyperson.person.id) && (ids.role == studyperson.role.id) }
     1373                           }.each {
     1374                                   study.removeFromPersons(it)
     1375                                   it.delete()
    13671376                           }
    13681377                   }
     
    13941403           } else {
    13951404                   log.info('.no persons selected.')
    1396                    if( study.persons )
    1397                            study.persons.clear()
     1405                   if( study.persons ) {
     1406                           // removing persons from study
     1407                           study.persons.each {
     1408                                   study.removeFromPersons(it)
     1409                                   it.delete()
     1410                           }
     1411                   }
    13981412           }
    13991413   }
     
    14531467                   users.each { study.addToReaders(it) }
    14541468           } else if (type == "writers") {
    1455                    if (study.writers)
    1456                            study.writers.clear()
     1469                   if (study.writers) {
     1470                           study.writers.each {
     1471                                        study.removeFromWriters(it)
     1472                           }
     1473                   }
    14571474
    14581475                   users.each { study.addToWriters(it) }
  • trunk/grails-app/domain/dbnp/studycapturing/Person.groovy

    r1457 r1736  
    4646    }
    4747
    48 
     48        def String toString() {
     49                return "${lastName}, ${firstName}"
     50        }
    4951}
  • trunk/grails-app/domain/dbnp/studycapturing/PersonAffiliation.groovy

    r1457 r1736  
    1212 */
    1313class PersonAffiliation extends Identity {
    14 
    1514        String institute
    1615        String department
Note: See TracChangeset for help on using the changeset viewer.