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

FIX: Upload_Validator failed to fetch max size from PHP ini values (fixes #6999)

Browse Source
This commit is contained in:
Loz Calver 2017-06-06 14:28:03 +01:00
parent 058ba813e6
commit 4ad2cae864
No known key found for this signature in database
GPG Key ID: F91FFC2D7E0CAEDB
2 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
filesystem/Upload.php
View File

@ -396,7 +396,7 @@ class Upload_Validator {
if (empty($this->allowedMaxFileSize)) {
// Set default max file sizes if there isn't
$fileSize = Config::inst()->get('Upload_Validator', 'default_max_file_size');
if (isset($fileSize)) {
if (!empty($fileSize)) {
$this->setAllowedMaxFileSize($fileSize);
} else {
// When no default is present, use maximum set by PHP

6
tests/filesystem/UploadTest.php
View File

@ -134,6 +134,12 @@ class UploadTest extends SapphireTest {
}
public function testGetAllowedMaxFileSize() {
// Check the max file size defaults to PHP settings
$maxPhpSize = min(File::ini2bytes(ini_get('upload_max_filesize')), File::ini2bytes(ini_get('post_max_size')));
$v = new UploadTest_Validator();
$retrievedSize = $v->getAllowedMaxFileSize('[image]');
$this->assertEquals($maxPhpSize, $retrievedSize, 'Max file size did not default to PHP value');
// Check the max file size uses the config values
$configMaxFileSizes = array(
'[image]' => '1k',