Update Transactions and fix errors

Reduce quoting level, and fix errors introduced by prior commits
This commit is contained in:
steve-lad 2021-02-08 17:41:19 +01:00 committed by Diederik de Groot
parent 4a3f21dc5d
commit 5892afc653
No known key found for this signature in database
GPG key ID: AFA728250A1BECD6

View file

@ -112,19 +112,19 @@ class dbinterface
case 'HWextension': case 'HWextension':
$raw_settings = $this->getDb_model_info($get = "extension", $format_list = "model"); $raw_settings = $this->getDb_model_info($get = "extension", $format_list = "model");
break; break;
case 'get_colums_sccpdevice': case 'get_columns_sccpdevice':
$sql = "DESCRIBE sccpdevice"; $sql = 'DESCRIBE sccpdevice';
$stmt = $db->prepare($sql); $stmt = $db->prepare($sql);
break; break;
case 'get_colums_sccpuser': case 'get_columns_sccpuser':
$sql = "DESCRIBE sccpuser"; $sql = 'DESCRIBE sccpuser';
$stmts = $db->prepare($sql); $stmts = $db->prepare($sql);
break; break;
case 'get_sccpdevice_byid': case 'get_sccpdevice_byid':
$sql = 'SELECT t1.*, types.dns, types.buttons, types.loadimage, types.nametemplate as nametemplate, ' $sql = 'SELECT t1.*, types.dns, types.buttons, types.loadimage, types.nametemplate as nametemplate,
. 'addon.buttons as addon_buttons FROM sccpdevice AS t1 ' addon.buttons as addon_buttons FROM sccpdevice AS t1
. 'LEFT JOIN sccpdevmodel as types ON t1.type=types.model ' LEFT JOIN sccpdevmodel as types ON t1.type=types.model
. 'LEFT JOIN sccpdevmodel as addon ON t1.addon=addon.model WHERE name="' . $data['id'] . ''; LEFT JOIN sccpdevmodel as addon ON t1.addon=addon.model WHERE name =\'' . $data['id'] . '\'';
$stmt = $db->prepare($sql); $stmt = $db->prepare($sql);
break; break;
case "get_sccpuser": case "get_sccpuser":
@ -245,6 +245,13 @@ class dbinterface
function write($table_name = "", $save_value = array(), $mode = 'update', $key_fld = "", $hwid = "") function write($table_name = "", $save_value = array(), $mode = 'update', $key_fld = "", $hwid = "")
{ {
//dbug('entering write for table', $table_name);
if ($table_name === 'sccpdevmodel'){
dbug('entering write with save_value', $save_value);
dbug('entering write with mode', $mode);
dbug('entering write with key_fld', $key_fld);
dbug('entering write with hwid', $hwid);
}
// mode clear - Empty table before update // mode clear - Empty table before update
// mode update - update / replace record // mode update - update / replace record
global $db; global $db;
@ -269,25 +276,22 @@ class dbinterface
if ($mode == 'clear') { if ($mode == 'clear') {
// $sql = 'truncate `sccpsettings`'; // $sql = 'truncate `sccpsettings`';
$db->prepare('TRUNCATE sccpsettings')->execute(); $db->prepare('TRUNCATE sccpsettings')->execute();
$stmt = $db->prepare('INSERT INTO sccpsettings (`keyword`, `data`, `seq`, `type`) VALUES (?,?,?,?)'); $stmt = $db->prepare('INSERT INTO sccpsettings (keyword, data, seq, type) VALUES (?,?,?,?)');
$result = $db->executeMultiple($stmt, $save_value); $result = $db->executeMultiple($stmt, $save_value);
} else { } else {
if (!empty($delete_value)) { if (!empty($delete_value)) {
$stmt = $db->prepare('DELETE FROM sccpsettings WHERE `keyword`=?'); $stmt = $db->prepare('DELETE FROM sccpsettings WHERE keyword = ?');
$result = $db->executeMultiple($stmt, $delete_value); $result = $db->executeMultiple($stmt, $delete_value);
} }
if (!empty($save_value)) { if (!empty($save_value)) {
$stmt = $db->prepare('REPLACE INTO sccpsettings (`keyword`, `data`, `seq`, `type`) VALUES (?,?,?,?)'); $stmt = $db->prepare('REPLACE INTO sccpsettings (keyword, data, seq, type) VALUES (?,?,?,?)');
$result = $db->executeMultiple($stmt, $save_value); $result = $db->executeMultiple($stmt, $save_value);
} }
} }
break; break;
case 'sccpdevmodel': case 'sccpdevmodel': // Fall through to next intentionally
break; case 'sccpdevice': // Fall through to next intentionally
case 'sccpdevice':
break;
case 'sccpuser': case 'sccpuser':
$sql_db = $table_name;
$sql_key = ""; $sql_key = "";
$sql_var = ""; $sql_var = "";
foreach ($save_value as $key_v => $data) { foreach ($save_value as $key_v => $data) {
@ -304,40 +308,39 @@ class dbinterface
} }
} }
if (!empty($sql_var)) { if (!empty($sql_var)) {
if ($mode == 'delete') { switch ($mode) {
$req = 'DELETE FROM sccpuser WHERE ' . $sql_key . ''; case 'delete':
} else { $req = 'DELETE FROM '. $table_name . ' WHERE ' . $sql_key;
if ($mode == 'update') { break;
$req = 'UPDATE sccpuser SET ' . $sql_var . 'WHERE ' . $sql_key . ''; case 'update':
} else { $req = 'UPDATE ' . $table_name . ' SET ' . $sql_var . ' WHERE ' . $sql_key;
$req = 'REPLACE INTO sccpuser SET ' . $sql_var . ''; break;
} default:
$req = 'REPLACE INTO ' . $table_name . ' SET ' . $sql_var;
} }
} }
$result = $db->prepare($req)->execute(); $result = $db->prepare($req)->execute();
break; break;
case 'sccpbuttons': case 'sccpbuttons':
if (($mode == 'clear') || ($mode == 'delete')) { switch ($mode) {
$sql = 'DELETE FROM sccpbuttonconfig WHERE ref=' . $hwid . ''; case 'clear': // no break here as clear is same as delete
$result = $db->prepare($sql)->execute(); case 'delete':
} $sql = 'DELETE FROM sccpbuttonconfig WHERE ref=' . $hwid . '';
if ($mode == 'delete') { $result = $db->prepare($sql)->execute();
break; break;
} case 'replace':
if (empty($save_value)) { if (!empty($save_value)) {
break; $sql = 'UPDATE sccpbuttonconfig SET name =? WHERE ref = ? AND reftype =? AND instance =? AND buttontype =?';
} $stmt = $db->prepare($sql);
if ($mode == 'replace') { $result= $db->executeMultiple($stmt, $save_value);
$sql = 'UPDATE sccpbuttonconfig SET `name`=? WHERE `ref`= ? AND `reftype`=? AND `instance`=? AND `buttontype`=?'; }
// $sql = 'INSERT INTO `sccpbuttonconfig` (`ref`, `reftype`,`instance`, `buttontype`, `name`, `options`) VALUES (?,?,?,?,?,?);'; break;
// die(print_r($save_value,1)); default:
$stmt = $db->prepare($sql); if (!empty($save_value)) {
$result= $db->executeMultiple($stmt, $save_value); $sql = 'INSERT INTO sccpbuttonconfig (ref, reftype, instance, buttontype, name, options) VALUES (?,?,?,?,?,?)';
} else { $stmt = $db->prepare($sql);
$sql = 'INSERT INTO sccpbuttonconfig (`ref`, `reftype`,`instance`, `buttontype`, `name`, `options`) VALUES (?,?,?,?,?,?)'; $result = $db->executeMultiple($stmt, $save_value);
// die(print_r($save_value,1)); }
$stmt = $db->prepare($sql);
$result = $db->executeMultiple($stmt, $save_value);
} }
} }
return $result; return $result;