From 58a6fdc5ed20797aff3f362416ae24db161a6d6a Mon Sep 17 00:00:00 2001
From: Eelco Dolstra <eelco.dolstra@logicblox.com>
Date: Wed, 28 Aug 2013 15:17:08 +0200
Subject: [PATCH] Remove the job status pages

They're mostly redundant since there is a faster "jobs" tab on
the jobset pages now.  The only thing the latter lacks is the
ability to see status change times, but those are quite expensive
to compute, and are visible on build pages if you really need them.
---
 src/lib/Hydra/Base/Controller/ListBuilds.pm | 11 +++++------
 src/root/topbar.tt                          |  7 -------
 2 files changed, 5 insertions(+), 13 deletions(-)

diff --git a/src/lib/Hydra/Base/Controller/ListBuilds.pm b/src/lib/Hydra/Base/Controller/ListBuilds.pm
index f3c69263..cbdc9be0 100644
--- a/src/lib/Hydra/Base/Controller/ListBuilds.pm
+++ b/src/lib/Hydra/Base/Controller/ListBuilds.pm
@@ -23,12 +23,11 @@ sub getJobStatus {
 }
 
 
-sub jobstatus : Chained('get_builds') PathPart Args(0) {
-    my ($self, $c) = @_;
-    $c->stash->{template} = 'jobstatus.tt';
-    $c->stash->{latestBuilds} = [getJobStatus($self, $c)->all];
-}
-
+#sub jobstatus : Chained('get_builds') PathPart Args(0) {
+#    my ($self, $c) = @_;
+#    $c->stash->{template} = 'jobstatus.tt';
+#    $c->stash->{latestBuilds} = [getJobStatus($self, $c)->all];
+#}
 
 
 # A convenient way to see all the errors - i.e. things demanding
diff --git a/src/root/topbar.tt b/src/root/topbar.tt
index 8b92e9ba..013eae28 100644
--- a/src/root/topbar.tt
+++ b/src/root/topbar.tt
@@ -39,7 +39,6 @@
       <li class="divider"></li>
       [% INCLUDE menuItem uri = c.uri_for(c.controller('Project').action_for('project'), [project.name]) title = "Overview" %]
       [% INCLUDE menuItem uri = c.uri_for(c.controller('Project').action_for('all'), [project.name]) title = "Latest builds" %]
-      [% INCLUDE menuItem uri = c.uri_for(c.controller('Project').action_for('jobstatus'), [project.name]) title = "Job status" %]
       [% INCLUDE menuItem uri = c.uri_for(c.controller('Project').action_for('errors'), [project.name]) title = "Errors" %]
       <li class="divider"></li>
       [% INCLUDE menuItem uri = c.uri_for('/project' project.name 'channel' 'latest') title = "Channel" %]
@@ -64,9 +63,6 @@
       [% INCLUDE menuItem
         uri = c.uri_for(c.controller('Jobset').action_for('all'), [project.name, jobset.name])
         title = "Latest builds" %]
-      [% INCLUDE menuItem
-        uri = c.uri_for(c.controller('Jobset').action_for('jobstatus'), [project.name, jobset.name])
-        title = "Job status" %]
       [% INCLUDE menuItem
         uri = c.uri_for(c.controller('Jobset').action_for('errors'), [project.name, jobset.name])
         title = "Errors" %]
@@ -111,9 +107,6 @@
       [% INCLUDE menuItem
         uri = c.uri_for(c.controller('Job').action_for('all'), [project.name, jobset.name, job.name])
         title = "Latest builds" %]
-      [% INCLUDE menuItem
-        uri = c.uri_for(c.controller('Job').action_for('jobstatus'), [project.name, jobset.name, job.name])
-        title = "Job status" %]
       [% INCLUDE menuItem
         uri = c.uri_for(c.controller('Job').action_for('errors'), [project.name, jobset.name, job.name])
         title = "Errors" %]