From df60053e9cace2c51f52bd513ac00ccc3882281b Mon Sep 17 00:00:00 2001
From: Asher Baker <asherkin@limetech.org>
Date: Sat, 19 Jan 2013 10:10:56 -0800
Subject: [PATCH] Don't scream about local storage not being writable if a path
 hasn't been configured

Summary: D4497

Test Plan: Reloaded setup check, saw no issues.

Reviewers: epriestley

CC: aran, Korvin

Differential Revision: https://secure.phabricator.com/D4537
---
 .../config/check/PhabricatorSetupCheckStorage.php           | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/applications/config/check/PhabricatorSetupCheckStorage.php b/src/applications/config/check/PhabricatorSetupCheckStorage.php
index 70fe5bf386..dd2d009f3d 100644
--- a/src/applications/config/check/PhabricatorSetupCheckStorage.php
+++ b/src/applications/config/check/PhabricatorSetupCheckStorage.php
@@ -3,8 +3,10 @@
 final class PhabricatorSetupCheckStorage extends PhabricatorSetupCheck {
 
   protected function executeChecks() {
-    $local_key = 'storage.local-disk.path';
-    $local_path = PhabricatorEnv::getEnvConfig($local_key);
+    $local_path = PhabricatorEnv::getEnvConfig('storage.local-disk.path');
+    if (!$local_path) {
+      return;
+    }
 
     if (!Filesystem::pathExists($local_path) ||
         !is_readable($local_path) ||