From c6b0be1ace1286145cbd1e0752550fefda72d47e Mon Sep 17 00:00:00 2001
From: Fred G <fredg02@hotmail.com>
Date: Tue, 24 Feb 2015 02:04:41 +0100
Subject: [PATCH] SlaveOrLabelColumn: added links to slaves, added whitespace
 between label and description

---
 .../extracolumns/SlaveOrLabelColumn.java      | 27 ++++++++-----------
 .../SlaveOrLabelColumn/column.jelly           | 13 +++++++--
 2 files changed, 22 insertions(+), 18 deletions(-)

diff --git a/src/main/java/jenkins/plugins/extracolumns/SlaveOrLabelColumn.java b/src/main/java/jenkins/plugins/extracolumns/SlaveOrLabelColumn.java
index ea9ba4d..ccaf75a 100644
--- a/src/main/java/jenkins/plugins/extracolumns/SlaveOrLabelColumn.java
+++ b/src/main/java/jenkins/plugins/extracolumns/SlaveOrLabelColumn.java
@@ -1,7 +1,7 @@
 /*
  * The MIT License
  *
- * Copyright (c) 2013, Stephan Krull
+ * Copyright (c) 2013, Stephan Krull, Frederic Gurr
  *
  * Permission is hereby granted, free of charge, to any person obtaining a copy
  * of this software and associated documentation files (the "Software"), to deal
@@ -47,27 +47,22 @@ public class SlaveOrLabelColumn extends ListViewColumn {
     public SlaveOrLabelColumn() {
     }
 
-    public String getInfo(Job<?, ?> job) {
-
+    public Label getLabel(Job<?, ?> job) {
         if(!(job instanceof AbstractProject)){
             LOGGER.finest("Not an instance of " + AbstractProject.class.getCanonicalName() + ". Cannot get info.");
-            return "";
+            return null;
         }
         
         AbstractProject<?, ?> project = AbstractProject.class.cast(job);
-        Label projectLabel = project.getAssignedLabel();
-        if (projectLabel == null || projectLabel.isEmpty()){
-            return "N/A";
-        }
-
-        if (projectLabel.isSelfLabel()){
-            return projectLabel.getName();
+        return project.getAssignedLabel();
+    }
+    
+    public String getDescription(Label label) {
+        if (label == null) {
+            return "";
         }
-
-        String desc = projectLabel.getDescription();
-        desc = (desc == null || desc.length() < 1) ? "" : "(" + desc + ")";
-
-        return projectLabel.getName()+ desc;
+        String desc = label.getDescription();
+        return (desc == null || desc.length() < 1) ? "" : "(" + desc + ")";
     }
 
     @Extension
diff --git a/src/main/resources/jenkins/plugins/extracolumns/SlaveOrLabelColumn/column.jelly b/src/main/resources/jenkins/plugins/extracolumns/SlaveOrLabelColumn/column.jelly
index 1844a03..ebc51fa 100644
--- a/src/main/resources/jenkins/plugins/extracolumns/SlaveOrLabelColumn/column.jelly
+++ b/src/main/resources/jenkins/plugins/extracolumns/SlaveOrLabelColumn/column.jelly
@@ -1,7 +1,7 @@
 <!--
   The MIT License
 
-  Copyright (c) 2013, Stephan Krull
+  Copyright (c) 2013, Stephan Krull, Frederic Gurr
 
   Permission is hereby granted, free of charge, to any person obtaining a copy
   of this software and associated documentation files (the "Software"), to deal
@@ -24,5 +24,14 @@
 
 <?jelly escape-by-default='true'?>
 <j:jelly xmlns:j="jelly:core">
-  <td data="${it.getInfo(job)}">${it.getInfo(job)}</td>
+  <j:set var="label" value="${it.getLabel(job)}"/>
+  <j:set var="desc" value="${it.getDescription(label)}"/>
+  <td data="${label}">
+    <j:choose>
+      <j:when test="${label == null}">N/A</j:when>
+      <j:when test="${label.equals('')}">N/A</j:when>
+      <j:when test="${label.isSelfLabel()}"><a href="${rootURL}/computer/${label.getName()}">${label.getName()}</a></j:when>
+      <j:otherwise>${label.getName()} ${desc}</j:otherwise>
+    </j:choose>
+  </td>
 </j:jelly>