From b91dafab417ef90e132421df1f223afa764ace79 Mon Sep 17 00:00:00 2001
From: Dan Cryer <dan@block8.co.uk>
Date: Wed, 3 Dec 2014 13:15:33 +0000
Subject: [PATCH] Fixing commit links

---
 PHPCI/Controller/HomeController.php | 8 +++++++-
 PHPCI/View/BuildsTable.phtml        | 2 +-
 PHPCI/View/Home/index.phtml         | 2 +-
 3 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/PHPCI/Controller/HomeController.php b/PHPCI/Controller/HomeController.php
index f84bebe0..0bb176dc 100644
--- a/PHPCI/Controller/HomeController.php
+++ b/PHPCI/Controller/HomeController.php
@@ -46,7 +46,13 @@ class HomeController extends \PHPCI\Controller
 
         $projects = $this->projectStore->getWhere(array(), 50, 0, array(), array('title' => 'ASC'));
 
-        $this->view->builds   = $this->buildStore->getLatestBuilds(null, 10);
+        $builds = $this->buildStore->getLatestBuilds(null, 10);
+
+        foreach ($builds as &$build) {
+            $build = BuildFactory::getBuild($build);
+        }
+
+        $this->view->builds   = $builds;
         $this->view->projects = $projects['items'];
         $this->view->summary  = $this->getSummaryHtml($projects);
 
diff --git a/PHPCI/View/BuildsTable.phtml b/PHPCI/View/BuildsTable.phtml
index 6da8a268..d45a8730 100644
--- a/PHPCI/View/BuildsTable.phtml
+++ b/PHPCI/View/BuildsTable.phtml
@@ -51,7 +51,7 @@ switch($build->getStatus())
         <?php
             if ($build->getCommitId() !== 'Manual') {
                 print sprintf(
-                    '<a href="%s">%s (%s)</a>',
+                    '<a href="%s" target="_blank">%s (%s)</a>',
                     $build->getCommitLink(),
                     substr($build->getCommitId(), 0, 7),
                     $build->getCommitterEmail()
diff --git a/PHPCI/View/Home/index.phtml b/PHPCI/View/Home/index.phtml
index 183aa366..db351c2e 100644
--- a/PHPCI/View/Home/index.phtml
+++ b/PHPCI/View/Home/index.phtml
@@ -74,7 +74,7 @@
                                     <?php
                                     if ($build->getCommitId() !== 'Manual') {
                                         print sprintf(
-                                            '<a href="%s">%s (%s)</a>',
+                                            '<a href="%s" target="_blank">%s (%s)</a>',
                                             $build->getCommitLink(),
                                             substr($build->getCommitId(), 0, 7),
                                             $build->getCommitterEmail()