$a->config[$family][$key] = $val;
return $val;
- }
- else {
+ } else {
$a->config[$family][$key] = '!<unset>!';
}
return $default_value;
public static function set($family, $key, $value) {
global $a;
+ $stored = self::get($family, $key);
+
+ if ($stored == $value) {
+ return true;
+ }
+
$a->config[$family][$key] = $value;
// manage array value
- $dbvalue = (is_array($value) ? serialize($value):$value);
+ $dbvalue = (is_array($value) ? serialize($value) : $value);
$dbvalue = (is_bool($dbvalue) ? intval($dbvalue) : $dbvalue);
- // The "INSERT" command is very cost intense. It saves performance to do it this way.
- $ret = q("SELECT `v` FROM `config` WHERE `cat` = '%s' AND `k` = '%s' ORDER BY `id` DESC LIMIT 1",
- dbesc($family),
- dbesc($key)
- );
-
- if (!$ret) {
+ if (is_null($stored)) {
$ret = q("INSERT INTO `config` (`cat`, `k`, `v`) VALUES ('%s', '%s', '%s') ON DUPLICATE KEY UPDATE `v` = '%s'",
dbesc($family),
dbesc($key),
dbesc($dbvalue),
dbesc($dbvalue)
);
- } elseif ($ret[0]['v'] != $dbvalue) {
+ } else {
$ret = q("UPDATE `config` SET `v` = '%s' WHERE `cat` = '%s' AND `k` = '%s'",
dbesc($dbvalue),
dbesc($family),
dbesc($key)
);
}
- if ($ret)
+ if ($ret) {
return $value;
-
+ }
return $ret;
}
public static function delete($family, $key) {
global $a;
- if (x($a->config[$family],$key))
+ if (x($a->config[$family],$key)) {
unset($a->config[$family][$key]);
+ }
$ret = q("DELETE FROM `config` WHERE `cat` = '%s' AND `k` = '%s'",
dbesc($family),
dbesc($key)