WINDUP-754 Group the identical hint labels with number - e.g. Oracle …

…proprietary type reference 5×
windup · Nov 19, 2015 · 7e11ed2 · 7e11ed2
1 parent 051444a
commit 7e11ed2
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/reporting/impl/src/main/resources/reports/resources/css/windup.css b/reporting/impl/src/main/resources/reports/resources/css/windup.css
@@ -99,6 +99,7 @@ h3.panel-title span.s {
 .panel .panel-body .info h4:first-child { margin-top: 0; }
 .panel .panel-body .info .rule-link.tag { font-variant-position: super; font-weight: normal; position: relative; top: -0.3ex; }
 
+.java-application .notifications li.warning small { color: #888; font-style: italic; }
 
 
 dl:last-child {

diff --git a/rules-java/api/src/main/resources/reports/templates/java_application.ftl b/rules-java/api/src/main/resources/reports/templates/java_application.ftl
@@ -64,11 +64,18 @@
             <h4 class="warns">Warnings: ${warnings} items</h4>
             <#if warnings != 0 >
                 <ul class='notifications'>
+                    <#assign map = {}>
                     <#list sourceReportModel.sourceFileModel.classificationModels.iterator() as classification>
-                        <li class='warning clsf'>${classification.classification?html}</li>
+                        <#assign count = (map[classification.classification]!0) + 1>
+                        <#assign map += {classification.classification : count}>
                     </#list>
                     <#list sourceReportModel.sourceFileModel.inlineHints.iterator() as hintLine>
-                        <li class='warning hint'>${hintLine.title?html}</li>
+                        <#assign count = (map[hintLine.title]!0) + 1>
+                        <#assign map += {hintLine.title : count}>
+                    </#list>
+                    <#list map?keys as key>
+                        <#assign count = map[key]>
+                        <li class="warning"> ${key?html} <small>${count}&#215;</small></li>
                     </#list>
                 </ul>
             </#if>
@@ -269,7 +276,7 @@
         <script src="resources/js/jquery-1.10.1.min.js"></script>
         <script src="resources/libraries/flot/jquery.flot.min.js"></script>
         <script src="resources/libraries/flot/jquery.flot.pie.min.js"></script>
-         <script src="resources/js/windup-overview.js"/>
+        <script src="resources/js/windup-overview.js"/>
         <script src="resources/js/bootstrap.min.js"></script>
 
         <@render_pie project=reportModel.projectModel recursive=true elementID="application_pie" includeTags=reportModel.includeTags excludeTags=reportModel.excludeTags />