* Avoid argc side-effects

* Fix undefined array keys in JITConfigAdapter::set()

* Avoid argv side effects in ApiTest
This commit is contained in:
Pierre Rudloff 2018-07-18 22:30:55 +02:00 committed by Hypolite Petovan
parent b265bf2ec3
commit 6e1ed0b0d3
2 changed files with 19 additions and 0 deletions

View file

@ -96,6 +96,13 @@ class JITConfigAdapter extends BaseObject implements IConfigAdapter
$stored = $this->get($cat, $k, null, true);
if (!isset($this->in_db[$cat])) {
$this->in_db[$cat] = [];
}
if (!isset($this->in_db[$cat][$k])) {
$this->in_db[$cat] = false;
}
if (($stored === $dbvalue) && $this->in_db[$cat][$k]) {
return true;
}

View file

@ -68,6 +68,18 @@ class ApiTest extends DatabaseTest
Config::set('system', 'theme', 'system_theme');
}
/**
* Cleanup variables used by tests.
*/
protected function tearDown()
{
parent::tearDown();
$app = get_app();
$app->argc = 1;
$app->argv = ['home'];
}
/**
* Assert that an user array contains expected keys.
* @param array $user User array