[FIXED JENKINS-9559] Added patch provided by Matt: ince the the value…

… starts with a # the default sorter tried to
jenkinsci · May 2, 2011 · c91b9cc · c91b9cc
1 parent e2946f3
commit c91b9cc
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/main/java/hudson/plugins/view/dashboard/builds/LatestBuilds.java b/src/main/java/hudson/plugins/view/dashboard/builds/LatestBuilds.java
@@ -58,6 +58,10 @@ public List<Run> getFinishedBuilds() {
 		return recentBuilds;
 	}
 
+  public String getBuildColumnSortData(Run<?, ?> build) {
+    return String.valueOf(build.getNumber());
+  }
+
 	@Extension
     public static class DescriptorImpl extends Descriptor<DashboardPortlet> {
 

diff --git a/src/main/resources/hudson/plugins/view/dashboard/builds/LatestBuilds/latestbuilds.jelly b/src/main/resources/hudson/plugins/view/dashboard/builds/LatestBuilds/latestbuilds.jelly
@@ -41,7 +41,7 @@ THE SOFTWARE.
               <!--t:jobLink job="${build.parent}"/-->
               <dp:jobLink job="${build.parent}"/>
             </td>
-            <td>
+            <td data="${it.getBuildColumnSortData(build)}">
               <!-- This tag does not generate a build link relative to view for jobs included using regexp -->
               <!--t:buildLink job="${build.parent}" jobName="${build.parent.displayName}" number="${build.number}" /-->
               <a href="${h.getRelativeLinkTo(build.parent)}/${build.number}" tooltip="${build.description}">
@@ -55,4 +55,4 @@ THE SOFTWARE.
 				</j:forEach>
 			</table>
       <!--t:buildListTable builds="${builds}" jobBaseUrl="" /-->
-</j:jelly>
+</j:jelly>