From e3fd094a09b0b7268a0b5d94f6fa34b3de664835 Mon Sep 17 00:00:00 2001
From: epriestley <git@epriestley.com>
Date: Mon, 23 Jun 2014 09:40:15 -0700
Subject: [PATCH] Minor, fix `empty()` call for older versions of PHP

Summary: While it's fine on 5.5.8, older versions of PHP are not happy about
this.
---
 .../config/controller/PhabricatorConfigWelcomeController.php   | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/applications/config/controller/PhabricatorConfigWelcomeController.php b/src/applications/config/controller/PhabricatorConfigWelcomeController.php
index 6eb9fffe08..9f0b343ba2 100644
--- a/src/applications/config/controller/PhabricatorConfigWelcomeController.php
+++ b/src/applications/config/controller/PhabricatorConfigWelcomeController.php
@@ -133,7 +133,8 @@ final class PhabricatorConfigWelcomeController
       $content);
 
     $settings_href = PhabricatorEnv::getURI('/settings/');
-    $have_settings = !empty($viewer->loadPreferences()->getPreferences());
+    $prefs = $viewer->loadPreferences()->getPreferences();
+    $have_settings = !empty($prefs);
     if ($have_settings) {
       $content = pht(
         "You've adjusted at least one setting on your account.\n\n".