Changeset 1565

Show
Ignore:
Timestamp:
25-02-11 14:27:48 (3 years ago)
Author:
robert@…
Message:

Fixed #260 (study wizard in safari) and fixed a bug when adding a publication with a lot of authors (>255 characters)

Location:
trunk
Files:
5 modified

Legend:

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

    r1430 r1565  
    3131            pubMedID: params.get( 'publication-pubMedID' ), 
    3232            DOI: params.get( 'publication-doi' ) 
    33         ) 
     33        ); 
     34         
     35                // Check whether the autorsList is not too long. If it is, split it 
     36                println "authors length: " + publication.authorsList.size() 
     37                if( publication.authorsList.size() > 255 ) { 
     38                        def postfix = " et al."; 
     39                        def split = publication.authorsList[ 0..255 - postfix.size()].lastIndexOf( ", " ); 
     40                        publication.authorsList = publication.authorsList[ 0..split-1] + postfix; 
     41                         
     42                        println "new authors list: " + publication.authorsList.size() + " - " + publication.authorsList 
     43                } 
    3444 
    3545        def message; 
     
    160170                    pubMedID: params.get( 'publication-pubMedID' ), 
    161171                    DOI: params.get( 'publication-doi' ) 
    162                 ).save(flush:true); 
     172                ); 
     173                         
     174                                // Check whether the autorsList is not too long. If it is, split it 
     175                                println "authors length: " + publication.authorsList.size() 
     176                                if( publication.authorsList.size() > 255 ) { 
     177                                        def postfix = " et al."; 
     178                                        def split = publication.authorsList[ 0..255 - postfix.size()].lastIndexOf( ", " ); 
     179                                        publication.authorsList = publication.authorsList[ 0..split-1] + postfix; 
     180                                         
     181                                        println "new authors list: " + publication.authorsList.size() + " - " + publication.authorsList 
     182                                } 
     183                         
     184                                publication.save(flush:true); 
    163185            } 
    164186 
  • trunk/grails-app/taglib/dbnp/studycapturing/WizardTagLib.groovy

    r1548 r1565  
    107107                        body 
    108108                ) 
    109  
    110                 attrs.description = ''; 
    111  
    112                 // render 'Add publication button' 
    113                 baseElement.call( 
    114                         '_publicationAddButton', 
    115                         attrs, 
    116                         body 
    117                 ) 
    118109        } 
    119110 
     
    183174                } 
    184175                out << '<input type="hidden" name="' + attrs.name + '_ids" value="' + ids + '" id="' + attrs.name + '_ids">'; 
     176                 
     177                out << _publicationAddButton( attrs, body ); 
    185178        } 
    186179 
     
    194187                } 
    195188         
    196                 out << '<input type="button" onClick="openPublicationDialog(\'' + attrs.name + '\' );" value="Add Publication">'; 
     189                out << '<input class="addButton" type="button" onClick="openPublicationDialog(\'' + attrs.name + '\' );" value="Add Publication">'; 
    197190        } 
    198191         
     
    353346                baseElement.call( 
    354347                        '_userList', 
    355                         attrs, 
    356                         body 
    357                 ) 
    358  
    359                 attrs.description = ''; 
    360  
    361                 // render 'Add user button' 
    362                 baseElement.call( 
    363                         '_userAddButton', 
    364348                        attrs, 
    365349                        body 
     
    429413                } 
    430414                out << '<input type="hidden" name="' + attrs.name + '_ids" value="' + ids + '" id="' + attrs.name + '_ids">'; 
     415                 
     416                out << _userAddButton( attrs, body ); 
    431417        } 
    432418 
     
    439425                } 
    440426 
    441                 out << '<input type="button" onClick="openUserDialog(\'' + attrs.name + '\' );" value="Add User">'; 
     427                out << '<input class="addButton" type="button" onClick="openUserDialog(\'' + attrs.name + '\' );" value="Add User">'; 
    442428        } 
    443429         
  • trunk/grails-app/views/simpleWizard/study.gsp

    r1553 r1565  
    4444                                        <h2>Publications</h2> 
    4545                                        <div class="content"> 
    46                                                 <af:publicationSelectElement noForm=true name="publication" value="${study?.publications}"/> 
     46                                                <af:publicationSelectElement noForm="true" name="publication" value="${study?.publications}"/> 
    4747                                        </div> 
    4848                                </div> 
  • trunk/web-app/css/simplewizard.css

    r1553 r1565  
    66.simpleWizard input, .simpleWizard select, .simpleWizard textarea { width: 220px; } 
    77.simpleWizard .tableEditor input, .simpleWizard .tableEditor  select { width: 100px; } 
     8.simpleWizard .element .input .addButton { margin-left: 0px; } 
    89 
    910div .simpleWizard .description { width: 170px; } 
  • trunk/web-app/css/templates.css

    r1286 r1565  
    9595} 
    9696 
     97.element .input .addButton { 
     98        margin-left: 255px; 
     99} 
     100 
    97101.element .input .isExample { 
    98102    color: #006DBA;