Changeset 1943

Show
Ignore:
Timestamp:
28-06-11 14:30:03 (3 years ago)
Author:
robert@…
Message:

- Fixed a bug in studywizard in saving study writers (saving a study with writers gave a concurrentmodificationexception on studies.dbnp.org)
- Adjusted 'previous searches' list to link to the original search with external searches

Location:
trunk/grails-app
Files:
2 modified

Legend:

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

    r1942 r1943  
    14541454                   if( study.persons ) { 
    14551455                           // removing persons from study 
    1456                            study.persons.each { 
     1456                           // Create a clone of persons list in order to avoid 
     1457                           // concurrentModification exceptions. See http://blog.springsource.com/2010/07/02/gorm-gotchas-part-2/ 
     1458                           def persons = [] + study.persons; 
     1459                           ersons.each { 
    14571460                                   study.removeFromPersons(it) 
    14581461                                   it.delete() 
     
    15111514            
    15121515           if (type == "readers") { 
    1513                    if (study.readers) 
    1514                            study.readers.clear() 
     1516                   if (study.readers) { 
     1517                           study.readers.clear(); 
     1518                   } 
    15151519                            
    15161520                   users.each { study.addToReaders(it) } 
    15171521           } else if (type == "writers") { 
     1522                            
    15181523                   if (study.writers) { 
    1519                            study.writers.each { 
    1520                                         study.removeFromWriters(it) 
    1521                            } 
     1524                           study.writers.clear(); 
    15221525                   } 
    15231526 
  • trunk/grails-app/views/advancedQuery/list.gsp

    r1908 r1943  
    7777                                <td>${search.getNumResults()}</td> 
    7878                                <td><g:formatDate date="${search.executionDate}" format="HH:mm" /></td> 
    79                                 <td><g:link action="show" id="${search.id}"><img border="0" src="${fam.icon(name: 'application_form_magnify')}" alt="Show" /></g:link> 
    80                                 <td><g:link action="discard" id="${search.id}"><img border="0" src="${fam.icon(name: 'basket_remove')}" alt="Discard" /></g:link> 
     79                                <td><a href="${search.url.encodeAsHTML()}"><img border="0" src="${fam.icon(name: 'application_form_magnify')}" alt="Show" /></a></td> 
     80                                <td><g:link action="discard" id="${search.id}"><img border="0" src="${fam.icon(name: 'basket_remove')}" alt="Discard" /></g:link></td> 
    8181                        </tr> 
    8282                </g:each>