PConfig : Check for is_null($uid) as well as false. We actually allow $uid = 0 though it shouldn't normally happen.
This commit is contained in:
parent
bc46f70a90
commit
0c3d5e99a2
@ -17,7 +17,7 @@ class PConfig {
|
|||||||
*/
|
*/
|
||||||
|
|
||||||
static public function Load($uid) {
|
static public function Load($uid) {
|
||||||
if($uid === false)
|
if(is_null($uid) || $uid === false)
|
||||||
return false;
|
return false;
|
||||||
|
|
||||||
if(! array_key_exists($uid, \App::$config))
|
if(! array_key_exists($uid, \App::$config))
|
||||||
@ -61,7 +61,7 @@ class PConfig {
|
|||||||
|
|
||||||
static public function Get($uid,$family,$key,$instore = false) {
|
static public function Get($uid,$family,$key,$instore = false) {
|
||||||
|
|
||||||
if($uid === false)
|
if(is_null($uid) || $uid === false)
|
||||||
return false;
|
return false;
|
||||||
|
|
||||||
if(! array_key_exists($uid, \App::$config))
|
if(! array_key_exists($uid, \App::$config))
|
||||||
@ -102,7 +102,7 @@ class PConfig {
|
|||||||
// we provide a function backtrace in the logs so that we can find
|
// we provide a function backtrace in the logs so that we can find
|
||||||
// and fix the calling function.
|
// and fix the calling function.
|
||||||
|
|
||||||
if($uid === false) {
|
if(is_null($uid) || $uid === false) {
|
||||||
btlogger('UID is FALSE!', LOGGER_NORMAL, LOG_ERR);
|
btlogger('UID is FALSE!', LOGGER_NORMAL, LOG_ERR);
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
@ -172,6 +172,9 @@ class PConfig {
|
|||||||
|
|
||||||
static public function Delete($uid, $family, $key) {
|
static public function Delete($uid, $family, $key) {
|
||||||
|
|
||||||
|
if(is_null($uid) || $uid === false)
|
||||||
|
return false;
|
||||||
|
|
||||||
$ret = false;
|
$ret = false;
|
||||||
|
|
||||||
if(array_key_exists($key, \App::$config[$uid][$family]))
|
if(array_key_exists($key, \App::$config[$uid][$family]))
|
||||||
|
Reference in New Issue
Block a user