Knah-Tsaeb/MyShaarli
Knah-Tsaeb/MyShaarli
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Merge pull request #1063 from ArthurHoaro/hotfix/legacy-warnings

Browse source 
Fix warnings when upgrading from legacy SebSauvage version
This commit is contained in:
ArthurHoaro 2018-01-31 12:18:31 +01:00 committed by GitHub
commit 2cbf4acdde
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 18 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
application/config/ConfigPhp.php
View file

@ -83,10 +83,10 @@ class ConfigPhp implements ConfigIO
$out = array();
foreach (self::$ROOT_KEYS as $key) {
$out[$key] = $GLOBALS[$key];
$out[$key] = isset($GLOBALS[$key]) ? $GLOBALS[$key] : '';
}
$out['config'] = $GLOBALS['config'];
$out['plugins'] = !empty($GLOBALS['plugins']) ? $GLOBALS['plugins'] : array();
$out['config'] = isset($GLOBALS['config']) ? $GLOBALS['config'] : [];
$out['plugins'] = isset($GLOBALS['plugins']) ? $GLOBALS['plugins'] : [];
return $out;
}

14
tests/config/ConfigPhpTest.php
View file

@ -36,6 +36,20 @@ class ConfigPhpTest extends \PHPUnit_Framework_TestCase
$this->assertEquals(array(), $this->configIO->read('nope'));
}
/**
* Read an empty existent config file -> array with blank default values.
*/
public function testReadEmpty()
{
$dataFile = 'tests/utils/config/emptyConfigPhp.php';
$conf = $this->configIO->read($dataFile);
$this->assertEmpty($conf['login']);
$this->assertEmpty($conf['title']);
$this->assertEmpty($conf['titleLink']);
$this->assertEmpty($conf['config']);
$this->assertEmpty($conf['plugins']);
}
/**
* Write a new config file.
*/

1
tests/utils/config/emptyConfigPhp.php Normal file
View file

@ -0,0 +1 @@
<?php