tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-10-22 14:05:37 +02:00
Code Issues Projects Releases Wiki Activity

Merge pull request #6952 from sminnee/fix-1666

Browse Source 
FIX: Better error checking for non-writable temp paths
This commit is contained in:
Damian Mooyman 2017-05-23 13:00:10 +12:00 committed by GitHub
commit cd0cac69ce
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/Core/TempPath.php
View File

@ -60,8 +60,6 @@ function getTempParentFolder($base = null)
$base = BASE_PATH;
}
$worked = true;
// first, try finding a silverstripe-cache dir built off the base path
$tempPath = $base . DIRECTORY_SEPARATOR . 'silverstripe-cache';
if (@file_exists($tempPath)) {
@ -77,7 +75,7 @@ function getTempParentFolder($base = null)
str_replace(array(' ', '/', ':', '\\'), '-', $base);
if (!@file_exists($tempPath)) {
$oldUMask = umask(0);
$worked = @mkdir($tempPath, 0777);
@mkdir($tempPath, 0777);
umask($oldUMask);
// if the folder already exists, correct perms
@ -87,15 +85,18 @@ function getTempParentFolder($base = null)
}
}
$worked = @file_exists($tempPath) && @is_writable($tempPath);
// failing to use the system path, attempt to create a local silverstripe-cache dir
if (!$worked) {
$worked = true;
$tempPath = $base . DIRECTORY_SEPARATOR . 'silverstripe-cache';
if (!@file_exists($tempPath)) {
$oldUMask = umask(0);
$worked = @mkdir($tempPath, 0777);
@mkdir($tempPath, 0777);
umask($oldUMask);
}
$worked = @file_exists($tempPath) && @is_writable($tempPath);
}
if (!$worked) {