From 3ff2934400c9d9c38c2c52c46851f0c29d06788d Mon Sep 17 00:00:00 2001
From: Rob Vermaas <rob.vermaas@gmail.com>
Date: Tue, 23 Feb 2010 13:08:58 +0000
Subject: [PATCH]  * hdyra: show last succeeded build and changes since on
 build status summary if build has failed

---
 src/root/build.tt  | 24 +++++++++++++++---------
 src/root/common.tt |  2 +-
 2 files changed, 16 insertions(+), 10 deletions(-)

diff --git a/src/root/build.tt b/src/root/build.tt
index a497362e..0007e62d 100644
--- a/src/root/build.tt
+++ b/src/root/build.tt
@@ -176,14 +176,23 @@
 		[% END %]
 
         [% IF !build.finished %]
-                    [% INCLUDE renderBuildSteps type="Running" %]
+            [% INCLUDE renderBuildSteps type="Running" %]
 		[% END %]
 
 		[% IF build.finished %]
-			[% IF build.buildsteps && (build.resultInfo.buildstatus == 2 || build.resultInfo.buildstatus == 5)%]
+			[% IF build.buildsteps && build.resultInfo.buildstatus != 0 %]
 				[% INCLUDE renderBuildSteps type="Failed" %]
 			[% END %]
 
+            [% IF build.resultInfo.buildstatus != 0 && prevSuccessfulBuild %]
+                <h2>Changes</h2>
+                <p>
+                    The previous successful build of this job was [% INCLUDE renderFullBuildLink build=prevSuccessfulBuild %]. The 
+                    following changes were made since:
+                </p>
+                [% INCLUDE renderInputDiff build2=build , build1=prevSuccessfulBuild %]
+            [% END %]
+
 			[% IF build.resultInfo.errormsg && build.resultInfo.buildstatus != 5 %]
 		
 				<h2 id="nix-error">Nix error output</h2>
@@ -384,16 +393,13 @@
 		  </tbody>
 		</table>
 
+        <p/>
+
         [% IF prevBuild %]
-            <h2>Changes since previous build : [% INCLUDE renderFullBuildLink build=prevBuild %]</h2>
+            <h3>Changes since previous build : [% INCLUDE renderFullBuildLink build=prevBuild %]</h3>
             [% INCLUDE renderInputDiff build2=build , build1=prevBuild %]
         [% END %]
-
-        [% IF prevBuild && prevSuccessfulBuild.id != prevBuild.id %]
-            <h2>Changes since previous successful build : [% INCLUDE renderFullBuildLink build=prevSuccessfulBuild %]</h2>
-            [% INCLUDE renderInputDiff build2=build , build1=prevSuccessfulBuild %]
-        [% END %]
-        
+       
 	</div>
 
 
diff --git a/src/root/common.tt b/src/root/common.tt
index a91ecb96..00833890 100644
--- a/src/root/common.tt
+++ b/src/root/common.tt
@@ -198,7 +198,7 @@
 [% END -%]
 
 [% BLOCK renderFullBuildLink; %] 
-	Job [% INCLUDE renderFullJobNameOfBuild build=build %] <a href="[% c.uri_for('/build' build.id) %]">build [% build.id %]
+	[% INCLUDE renderFullJobNameOfBuild build=build %] <a href="[% c.uri_for('/build' build.id) %]">build [% build.id %]</a>
 [% END %]
 
 [% BLOCK renderBuildStatusIcon; %]