Changeset 1943


Ignore:
Timestamp:
Jun 28, 2011, 2:30:03 PM (5 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 edited

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>
Note: See TracChangeset for help on using the changeset viewer.