archive/phabricator
1
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

Don't scream about local storage not being writable if a path hasn't been configured

Browse Source 
Summary: D4497

Test Plan: Reloaded setup check, saw no issues.

Reviewers: epriestley

CC: aran, Korvin

Differential Revision: https://secure.phabricator.com/D4537
This commit is contained in:
Asher Baker
2013-01-19 10:10:56 -08:00
committed by epriestley
parent 42ee1d0ed6
commit df60053e9c
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/applications/config/check/PhabricatorSetupCheckStorage.php
View File

@@ -3,8 +3,10 @@
final class PhabricatorSetupCheckStorage extends PhabricatorSetupCheck { final class PhabricatorSetupCheckStorage extends PhabricatorSetupCheck {
protected function executeChecks() { protected function executeChecks() {
$local_key = 'storage.local-disk.path'; $local_path = PhabricatorEnv::getEnvConfig('storage.local-disk.path');
$local_path = PhabricatorEnv::getEnvConfig($local_key); if (!$local_path) {
return;
}
if (!Filesystem::pathExists($local_path) || if (!Filesystem::pathExists($local_path) ||
!is_readable($local_path) || !is_readable($local_path) ||