Merge pull request #5394 from MrPetovan/bug/fix-config-set-return-value
Change (P)Config::set return value to bool
This commit is contained in:
commit
2fa6cc0000
6 changed files with 4 additions and 6 deletions
|
@ -110,7 +110,7 @@ class Config extends BaseObject
|
|||
* @param string $key The configuration key to set
|
||||
* @param mixed $value The value to store
|
||||
*
|
||||
* @return mixed Stored $value or false if the database update failed
|
||||
* @return bool Operation success
|
||||
*/
|
||||
public static function set($family, $key, $value)
|
||||
{
|
||||
|
|
|
@ -53,7 +53,7 @@ interface IConfigAdapter
|
|||
* @param string $key The configuration key to set
|
||||
* @param mixed $value The value to store
|
||||
*
|
||||
* @return mixed Stored $value or false if the database update failed
|
||||
* @return bool Operation success
|
||||
*/
|
||||
public function set($cat, $k, $value);
|
||||
|
||||
|
|
|
@ -57,7 +57,7 @@ interface IPConfigAdapter
|
|||
* @param string $k The configuration key to set
|
||||
* @param string $value The value to store
|
||||
*
|
||||
* @return mixed Stored $value or false
|
||||
* @return bool Operation success
|
||||
*/
|
||||
public function set($uid, $cat, $k, $value);
|
||||
|
||||
|
|
|
@ -112,7 +112,6 @@ class JITConfigAdapter extends BaseObject implements IConfigAdapter
|
|||
|
||||
if ($result) {
|
||||
$this->in_db[$cat][$k] = true;
|
||||
return $value;
|
||||
}
|
||||
|
||||
return $result;
|
||||
|
|
|
@ -98,7 +98,6 @@ class JITPConfigAdapter extends BaseObject implements IPConfigAdapter
|
|||
|
||||
if ($result) {
|
||||
$this->in_db[$uid][$cat][$k] = true;
|
||||
return $value;
|
||||
}
|
||||
|
||||
return $result;
|
||||
|
|
|
@ -107,7 +107,7 @@ class PConfig extends BaseObject
|
|||
* @param string $key The configuration key to set
|
||||
* @param string $value The value to store
|
||||
*
|
||||
* @return mixed Stored $value or false
|
||||
* @return bool Operation success
|
||||
*/
|
||||
public static function set($uid, $family, $key, $value)
|
||||
{
|
||||
|
|
Loading…
Reference in a new issue