Changeset 2167


Ignore:
Timestamp:
Feb 6, 2012, 1:13:11 PM (5 years ago)
Author:
business@…
Message:

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

File:
1 edited

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