Show
Ignore:
Timestamp:
06-02-12 13:13:11 (2 years ago)
Author:
business@…
Message:

fix for #520, also hides any template field that contain null values to improve clarity

Files:
1 modified

Legend:

Unmodified
Added
Removed
  • trunk/grails-app/views/studyWizard/pages/_samples.gsp

    r2152 r2167  
    6262                                                                <h2>Template Fields:</h2> 
    6363                                                                <g:each var="field" in="${sample.parentEvent?.giveFields()}"> 
    64                                                                         ${field.name[0].toUpperCase() + field.name.substring(1)}<br/> 
     64                                                                        <g:if test="sample.parentEvent.getFieldValue(field.name)"> 
     65                                                                                ${field.name[0].toUpperCase() + field.name.substring(1) + ': ' + sample.parentEvent.getFieldValue(field.name)}<br/> 
     66                                                                        </g:if> 
    6567                                                                </g:each> 
    6668                                                        </div> 
     
    7476                                                        <h2>Template Fields:</h2> 
    7577                                                        <g:each var="field" in="${sample.parentSubject?.giveFields()}"> 
    76                                                                 ${field.name[0].toUpperCase() + field.name.substring(1)}<br/> 
     78                                                                ${field.name[0].toUpperCase() + field.name.substring(1) + ': ' + sample.parentSubject.getFieldValue(field.name)}<br/> 
    7779                                                        </g:each> 
    7880                                                </div> 
     
    115117                                                        <div class="helpContent"> 
    116118                                                                <h1>${sample.parentEvent?.template?.name}</h1> 
    117                                                                 <h2>Template Fields:</h2> 
    118119                                                                <g:each var="field" in="${sample.parentEvent?.giveFields()}"> 
    119                                                                         ${field.name[0].toUpperCase() + field.name.substring(1)}<br/> 
     120                                                                        ${field.name[0].toUpperCase() + field.name.substring(1) + ': ' + sample.parentEvent.getFieldValue(field.name)}<br/> 
    120121                                                                </g:each> 
    121122                                                        </div> 
     
    127128                                                <div class="helpContent"> 
    128129                                                        <h1>${sample.parentSubject?.template?.name}</h1> 
    129                                                         <h2>Template Fields:</h2> 
    130130                                                        <g:each var="field" in="${sample.parentSubject?.giveFields()}"> 
    131                                                                 ${field.name[0].toUpperCase() + field.name.substring(1)}<br/> 
     131                                                                <% // only show template field values for non-empty values. btw: <g:if> does not affect Groovy code! %> 
     132                                                                <%if (sample.parentSubject.getFieldValue(field.name)) print field.name[0].toUpperCase() + field.name.substring(1) + ': ' + sample.parentSubject.getFieldValue(field.name) + '<br/>'%> 
    132133                                                        </g:each> 
    133134                                                </div>