From f463abbe53488747725ea2e45dfc4a98ada02b8e Mon Sep 17 00:00:00 2001
From: Jesse Glick <jglick@cloudbees.com>
Date: Thu, 9 Oct 2014 15:24:00 -0400
Subject: [PATCH] [FIXED JENKINS-25074] Make sure to call
 Job.getBuildHealthReports just once per t:buildHealth render.

---
 changelog.html                                       | 3 +++
 core/src/main/resources/lib/hudson/buildHealth.jelly | 5 +++--
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/changelog.html b/changelog.html
index 95055d81c0..8d0ba443d3 100644
--- a/changelog.html
+++ b/changelog.html
@@ -55,6 +55,9 @@ Upcoming changes</a>
 <!-- Record your changes in the trunk here. -->
 <div id="trunk" style="display:none"><!--=TRUNK-BEGIN=-->
 <ul class=image>
+  <li class=bug>
+    Build health computed repeatedly for a single Weather column cell.
+    (<a href="https://issues.jenkins-ci.org/browse/JENKINS-25074">issue 25074</a>)
   <li class=rfe>
     Missing workspace page should use 404 status code.
     (<a href="https://issues.jenkins-ci.org/browse/JENKINS-10450">issue 10450</a>)
diff --git a/core/src/main/resources/lib/hudson/buildHealth.jelly b/core/src/main/resources/lib/hudson/buildHealth.jelly
index dd3e52c04b..8800175675 100644
--- a/core/src/main/resources/lib/hudson/buildHealth.jelly
+++ b/core/src/main/resources/lib/hudson/buildHealth.jelly
@@ -31,8 +31,9 @@ THE SOFTWARE.
 -->
 <?jelly escape-by-default='true'?>
 <j:jelly xmlns:j="jelly:core" xmlns:x="jelly:xml" xmlns:st="jelly:stapler" xmlns:d="jelly:define" xmlns:l="/lib/layout" xmlns:t="/lib/hudson" xmlns:f="/lib/form">
-    <j:set var="buildHealth" value="${job.buildHealth}"/>
     <j:set var="healthReports" value="${job.buildHealthReports}"/>
+    <j:new var="emptyHealthReport" className="hudson.model.HealthReport"/>
+    <j:set var="buildHealth" value="${empty(healthReports) ? emptyHealthReport : healthReports[0]}"/>
     <j:if test="${td}"><j:set var="useTdElement" value="x"/></j:if>
     <x:element name="${useTdElement!=null?'td':'div'}">
         <x:attribute name="data">${buildHealth.score}</x:attribute>
@@ -74,7 +75,7 @@ THE SOFTWARE.
                         </tr>
                     </thead>
                     <tbody>
-                        <j:forEach var="rpt" items="${job.buildHealthReports}">
+                        <j:forEach var="rpt" items="${healthReports}">
                             <tr>
                                 <td align="left">
                                     <l:icon class="${rpt.iconClassName} icon-sm" />
-- 
GitLab