mirror/php-phpbb
1
0
Fork 0
mirror of https://github.com/phpbb/phpbb.git synced 2025-07-31 14:00:31 +02:00
Code Issues Releases Wiki Activity

Merge remote-tracking branch 'github-bantu/ticket/12937' into develop-ascraeus

Browse Source 
* github-bantu/ticket/12937:
  [ticket/12937] Do not return false. Return null for get and array() for all.
This commit is contained in:
Nils Adermann
2014-08-08 17:23:07 +02:00
parent f6da266109 c5f0c1aa8e
commit 13ccb352ce
2 changed files with 22 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files

39
phpBB/phpbb/config_php_file.php
View File

@@ -71,59 +71,44 @@ class config_php_file
/**
* Returns an associative array containing the variables defined by the config file.
*
* @return bool|array Return the content of the config file or false if the file does not exists.
* @return array Return the content of the config file or an empty array if the file does not exists.
*/
public function get_all()
{
if (!$this->load_config_file())
{
return false;
}
$this->load_config_file();
return $this->config_data;
}
/**
* Return the value of a variable defined into the config.php file and false if the variable does not exist.
* Return the value of a variable defined into the config.php file or null if the variable does not exist.
*
* @param string $variable The name of the variable
* @return mixed
* @return mixed Value of the variable or null if the variable is not defined.
*/
public function get($variable)
{
if (!$this->load_config_file())
{
return false;
}
$this->load_config_file();
return isset($this->config_data[$variable]) ? $this->config_data[$variable] : false;
return isset($this->config_data[$variable]) ? $this->config_data[$variable] : null;
}
/**
* Load the config file and store the information.
*
* @return bool True if the file was correctly loaded, false otherwise.
* @return null
*/
protected function load_config_file()
{
if (!$this->config_loaded)
if (!$this->config_loaded && file_exists($this->config_file))
{
if (file_exists($this->config_file))
{
$this->defined_vars = get_defined_vars();
$this->defined_vars = get_defined_vars();
require($this->config_file);
$this->config_data = array_diff_key(get_defined_vars(), $this->defined_vars);
require($this->config_file);
$this->config_data = array_diff_key(get_defined_vars(), $this->defined_vars);
$this->config_loaded = true;
}
else
{
return false;
}
$this->config_loaded = true;
}
return true;
}
/**