PR: MPDASHBOARD-32

maven.dashboard.report.showempty property not honored.


git-svn-id: https://svn.apache.org/repos/asf/maven/maven-1/plugins/trunk@367861 13f79535-47bb-0310-9956-ffa450edef68

dashboard/src/plugin-resources/templates/dashboard.jsl

@@ -85,7 +85,7 @@
     <j:set var="shallDisplay" value="true"/>
     <j:if test="${context.getVariable('maven.dashboard.report.showempty') == 'false'}">
       <x:set var="notEmptyElems" select="aggregator[not(text() = '-')]"/>
-      <j:if test="${notEmptyElems.isEmpty() == 'true'}">
+      <j:if test="${empty(notEmptyElems)}">
         <j:set var="shallDisplay" value="false"/>
       </j:if>
     </j:if>

dashboard/xdocs/changes.xml

@@ -25,6 +25,7 @@
   </properties>
   <body>
     <release version="1.9-SNAPSHOT" date="in svn">
+      <action dev="ltheussl" type="fix" issue="MPDASHBOARD-32" due-to="Wim Deblauwe"><code>maven.dashboard.report.showempty</code> property not honored.</action>
       <action dev="aheritier" type="fix" issue="MPDASHBOARD-34" due-to="Philippe Kernevez">Cobertura aggregator don't support offline mode. From now the Cobertura aggregator works only with MAVEN 1.1.</action>
       <action dev="aheritier" type="update" issue="MPDASHBOARD-33" due-to="carlos@apache.org">Update the default checkstyle report location to make it compatible with Checkstyle plugin 3.0+ (MPCHECKSTYLE-40).</action>
       <action dev="carlos" type="add" issue="MPDASHBOARD-31">Added Cobertura aggregator.</action>