From 665d78e7d6b447e1ee39418b43e7968e496181b7 Mon Sep 17 00:00:00 2001
From: Sam Georges <sam@daftspunk.com>
Date: Sat, 1 Nov 2014 11:38:41 +1100
Subject: [PATCH] If there is no build number, don't display the build
 scoreboard item

---
 modules/system/controllers/Updates.php       |  8 ++++----
 modules/system/controllers/updates/index.htm | 16 +++++++++-------
 2 files changed, 13 insertions(+), 11 deletions(-)

diff --git a/modules/system/controllers/Updates.php b/modules/system/controllers/Updates.php
index 6d392390b..853d76145 100644
--- a/modules/system/controllers/Updates.php
+++ b/modules/system/controllers/Updates.php
@@ -48,10 +48,10 @@ class Updates extends Controller
      */
     public function index()
     {
-        $this->vars['core_build'] = Parameters::get('system::core.build', '???');
-        $this->vars['project_id'] = Parameters::get('system::project.id');
-        $this->vars['project_name'] = Parameters::get('system::project.name');
-        $this->vars['project_owner'] = Parameters::get('system::project.owner');
+        $this->vars['coreBuild'] = Parameters::get('system::core.build');
+        $this->vars['projectId'] = Parameters::get('system::project.id');
+        $this->vars['projectName'] = Parameters::get('system::project.name');
+        $this->vars['projectOwner'] = Parameters::get('system::project.owner');
         return $this->asExtension('ListController')->index();
     }
 
diff --git a/modules/system/controllers/updates/index.htm b/modules/system/controllers/updates/index.htm
index 84dd39100..33c932846 100644
--- a/modules/system/controllers/updates/index.htm
+++ b/modules/system/controllers/updates/index.htm
@@ -2,10 +2,10 @@
     <div data-control="toolbar">
         <div class="scoreboard-item title-value">
             <h4><?= e(trans('system::lang.project.name')) ?></h4>
-            <?php if ($project_id): ?>
-                <p class="oc-icon-chain"><?= $project_name ?></p>
+            <?php if ($projectId): ?>
+                <p class="oc-icon-chain"><?= $projectName ?></p>
                 <p class="description">
-                    <?= e(trans('system::lang.project.owner_label')) ?>: <?= $project_owner ?>
+                    <?= e(trans('system::lang.project.owner_label')) ?>: <?= $projectOwner ?>
                     (<a
                         href="javascript:;"
                         data-request-confirm="<?= e(trans('system::lang.project.detach_confirm')) ?>"
@@ -24,10 +24,12 @@
                 </p>
             <?php endif ?>
         </div>
-        <div class="scoreboard-item title-value">
-            <h4><?= e(trans('system::lang.updates.core_build')) ?></h4>
-            <p><?= $core_build ?></p>
-        </div>
+        <?php if ($coreBuild): ?>
+            <div class="scoreboard-item title-value">
+                <h4><?= e(trans('system::lang.updates.core_build')) ?></h4>
+                <p><?= $coreBuild ?></p>
+            </div>
+        <?php endif ?>
     </div>
 </div>