[JENKINS-24436] Use rootUrl rather that jobBaseUrl when linking to jo…

…b results.
jenkinsci · Oct 25, 2014 · 1a89911 · jglick · Oct 25, 2014 · uhafner
1 parent 7fd072e
commit 1a89911
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/main/java/hudson/plugins/cobertura/CoverageColumn.java b/src/main/java/hudson/plugins/cobertura/CoverageColumn.java
@@ -49,7 +49,7 @@ public String getUrl(final Job<?, ?> project) {
             return null;
         }
         else {
-            return action.getUrlName();
+            return project.getUrl() + action.getUrlName();
         }
     }
 

diff --git a/src/main/resources/hudson/plugins/cobertura/CoverageColumn/column.jelly b/src/main/resources/hudson/plugins/cobertura/CoverageColumn/column.jelly
@@ -5,7 +5,7 @@
       ${it.getCoverage(job)}
     </j:if>
     <j:if test="${it.hasUrl(job)}">
-      <a href="${jobBaseUrl}${job.shortUrl}${it.getUrl(job)}">${it.getCoverage(job)}</a>
+      <a href="${rootURL}/${it.getUrl(job)}">${it.getCoverage(job)}</a>
     </j:if>
   </td>
 </j:jelly>
diff --git a/src/main/resources/hudson/plugins/cobertura/Messages.properties b/src/main/resources/hudson/plugins/cobertura/Messages.properties
@@ -9,7 +9,7 @@ MavenCoberturaPublisher.displayName=Publish Cobertura Coverage Report
 
 CoverageColumn.columnName=Cobertura coverage
 CoverageColumn.columnHeader=Coverage %
-CoverageColumn.empty=n/a
+CoverageColumn.empty=-
 CoverageColumn.line={0}%
 CoverageColumn.branch={0}%
 CoverageColumn.both=L:{0}% - B:{1}%