[FIXED JENKINS-16935] Give the default no-name package a non-empty name.

The empty name breaks the display and hyperlinks.
jenkinsci · Feb 22, 2013 · b798592 · b798592
1 parent 0a13b5c
commit b798592
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/src/main/java/hudson/plugins/jacoco/report/AbstractReport.java b/src/main/java/hudson/plugins/jacoco/report/AbstractReport.java
@@ -33,7 +33,7 @@ public void setName(String name) {
     }
 
     public String getDisplayName() {
-        return name;
+        return getName();
     }
 
     /**

diff --git a/src/main/java/hudson/plugins/jacoco/report/PackageReport.java b/src/main/java/hudson/plugins/jacoco/report/PackageReport.java
@@ -15,6 +15,15 @@
  */
 public final class PackageReport extends AggregatedReport<CoverageReport,PackageReport,ClassReport> {
 
+    /**
+     * Give the default no-name package a non-empty name.
+     */
+    @Override
+    public String getName() {
+        String n = super.getName();
+        return n.length() == 0 ? "(default)" : n;
+    }
+
     @Override
     public void setName(String name) {
         super.setName(name.replaceAll("/", "."));