Merge pull request #1763 from olivergondza/JENKINS-29286

[FIXED JENKINS-29286] Use byte size to sort disk space monitor columns

core/src/main/java/hudson/node_monitors/DiskSpaceMonitorDescriptor.java

@@ -37,6 +37,8 @@ import java.math.BigDecimal;
 import java.text.ParseException;
 import java.util.Locale;
 
+import org.kohsuke.accmod.Restricted;
+import org.kohsuke.accmod.restrictions.DoNotUse;
 import org.kohsuke.stapler.export.ExportedBean;
 import org.kohsuke.stapler.export.Exported;
 
@@ -81,6 +83,14 @@ public abstract class DiskSpaceMonitorDescriptor extends AbstractAsyncNodeMonito
             return path;
         }
 
+        // Needed for jelly that does not seem to be able to access properties
+        // named 'size' as it confuses it with built-in size method and fails
+        // to parse the expression expecting '()'.
+        @Restricted(DoNotUse.class)
+        public long getFreeSize() {
+            return size;
+        }
+
         /**
          * Gets GB left.
          */

core/src/main/resources/hudson/node_monitors/DiskSpaceMonitor/column.jelly

@@ -29,7 +29,7 @@ THE SOFTWARE.
       <td align="right" data="-1">N/A</td>
     </j:when>
     <j:otherwise>
-      <td align="right" data="${data}"><j:out value="${data.toHtml()}"/></td>
+      <td align="right" data="${data.freeSize}"><j:out value="${data.toHtml()}"/></td>
     </j:otherwise>
   </j:choose>
-</j:jelly>
+</j:jelly>
\ No newline at end of file

core/src/main/resources/hudson/node_monitors/SwapSpaceMonitor/column.jelly

@@ -32,4 +32,4 @@ THE SOFTWARE.
       <td align="right" data="${from.toMB(data)}"><j:out value="${from.toHtml(data)}"/></td>
     </j:otherwise>
   </j:choose>
-</j:jelly>
+</j:jelly>
\ No newline at end of file

core/src/main/resources/hudson/node_monitors/TemporarySpaceMonitor/column.jelly

@@ -29,7 +29,7 @@ THE SOFTWARE.
       <td align="right" data="-1">N/A</td>
     </j:when>
     <j:otherwise>
-      <td align="right" data="${data}"><j:out value="${data.toHtml()}"/></td>
+      <td align="right" data="${data.freeSize}"><j:out value="${data.toHtml()}"/></td>
     </j:otherwise>
   </j:choose>
-</j:jelly>
+</j:jelly>
\ No newline at end of file