X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=src%2FDatabase%2FDBA.php;h=bf3004ead4761eebdc2d46fc026c8240597a47cf;hb=5d464c353f2630e2b37a4b4d6afc4565dcd100e6;hp=70b43e122502ece25fa2631bcefc85664eefe0eb;hpb=cdd8e4fd9dd7f72706cce3f4c4adeba7114456aa;p=friendica.git diff --git a/src/Database/DBA.php b/src/Database/DBA.php index 70b43e1225..bf3004ead4 100644 --- a/src/Database/DBA.php +++ b/src/Database/DBA.php @@ -6,13 +6,18 @@ namespace Friendica\Database; // Please use App->getConfigVariable() instead. //use Friendica\Core\Config; +use Friendica\Core\Logger; use Friendica\Core\System; use Friendica\Util\DateTimeFormat; use mysqli; +use mysqli_result; +use mysqli_stmt; use PDO; use PDOException; use PDOStatement; +require_once 'include/dba.php'; + /** * @class MySQL database class * @@ -20,6 +25,15 @@ use PDOStatement; */ class DBA { + /** + * Lowest possible date value + */ + const NULL_DATE = '0001-01-01'; + /** + * Lowest possible datetime value + */ + const NULL_DATETIME = '0001-01-01 00:00:00'; + public static $connected = false; private static $server_info = ''; @@ -87,6 +101,7 @@ class DBA self::$connection->setAttribute(PDO::ATTR_EMULATE_PREPARES, false); self::$connected = true; } catch (PDOException $e) { + /// @TODO At least log exception, don't ignore it! } } @@ -212,7 +227,7 @@ class DBA } $r = self::p("EXPLAIN ".$query); - if (!DBM::is_result($r)) { + if (!self::isResult($r)) { return; } @@ -247,11 +262,16 @@ class DBA } public static function escape($str) { - switch (self::$driver) { - case 'pdo': - return substr(@self::$connection->quote($str, PDO::PARAM_STR), 1, -1); - case 'mysqli': - return @self::$connection->real_escape_string($str); + if (self::$connected) { + switch (self::$driver) { + case 'pdo': + return substr(@self::$connection->quote($str, PDO::PARAM_STR), 1, -1); + + case 'mysqli': + return @self::$connection->real_escape_string($str); + } + } else { + return str_replace("'", "\\'", $str); } } @@ -265,7 +285,7 @@ class DBA switch (self::$driver) { case 'pdo': $r = self::p("SELECT 1"); - if (DBM::is_result($r)) { + if (self::isResult($r)) { $row = self::toArray($r); $connected = ($row[0]['1'] == '1'); } @@ -364,7 +384,7 @@ class DBA * @usage Example: $r = p("SELECT * FROM `item` WHERE `guid` = ?", $guid); * * Please only use it with complicated queries. - * For all regular queries please use dba::select or dba::exists + * For all regular queries please use DBA::select or DBA::exists * * @param string $sql SQL statement * @return bool|object statement object or result object @@ -393,7 +413,7 @@ class DBA if ((substr_count($sql, '?') != count($args)) && (count($args) > 0)) { // Question: Should we continue or stop the query here? - logger('Parameter mismatch. Query "'.$sql.'" - Parameters '.print_r($args, true), LOGGER_DEBUG); + Logger::log('Parameter mismatch. Query "'.$sql.'" - Parameters '.print_r($args, true), Logger::DEBUG); } $sql = self::cleanQuery($sql); @@ -533,7 +553,7 @@ class DBA $error = self::$error; $errorno = self::$errorno; - logger('DB Error '.self::$errorno.': '.self::$error."\n". + Logger::log('DB Error '.self::$errorno.': '.self::$error."\n". System::callstack(8)."\n".self::replaceParameters($sql, $args)); // On a lost connection we try to reconnect - but only once. @@ -541,14 +561,14 @@ class DBA if (self::$in_retrial || !self::reconnect()) { // It doesn't make sense to continue when the database connection was lost if (self::$in_retrial) { - logger('Giving up retrial because of database error '.$errorno.': '.$error); + Logger::log('Giving up retrial because of database error '.$errorno.': '.$error); } else { - logger("Couldn't reconnect after database error ".$errorno.': '.$error); + Logger::log("Couldn't reconnect after database error ".$errorno.': '.$error); } exit(1); } else { // We try it again - logger('Reconnected after database error '.$errorno.': '.$error); + Logger::log('Reconnected after database error '.$errorno.': '.$error); self::$in_retrial = true; $ret = self::p($sql, $args); self::$in_retrial = false; @@ -560,7 +580,7 @@ class DBA self::$errorno = $errorno; } - $a->save_timestamp($stamp1, 'database'); + $a->saveTimestamp($stamp1, 'database'); if ($a->getConfigValue('system', 'db_log')) { $stamp2 = microtime(true); @@ -582,7 +602,7 @@ class DBA /** * @brief Executes a prepared statement like UPDATE or INSERT that doesn't return data * - * Please use dba::delete, dba::insert, dba::update, ... instead + * Please use DBA::delete, DBA::insert, DBA::update, ... instead * * @param string $sql SQL statement * @return boolean Was the query successfull? False is returned only if an error occurred @@ -617,13 +637,13 @@ class DBA $error = self::$error; $errorno = self::$errorno; - logger('DB Error '.self::$errorno.': '.self::$error."\n". + Logger::log('DB Error '.self::$errorno.': '.self::$error."\n". System::callstack(8)."\n".self::replaceParameters($sql, $params)); // On a lost connection we simply quit. // A reconnect like in self::p could be dangerous with modifications if ($errorno == 2006) { - logger('Giving up because of database error '.$errorno.': '.$error); + Logger::log('Giving up because of database error '.$errorno.': '.$error); exit(1); } @@ -631,7 +651,7 @@ class DBA self::$errorno = $errorno; } - $a->save_timestamp($stamp, "database_write"); + $a->saveTimestamp($stamp, "database_write"); return $retval; } @@ -677,7 +697,7 @@ class DBA /** * Fetches the first row * - * Please use dba::selectFirst or dba::exists whenever this is possible. + * Please use DBA::selectFirst or DBA::exists whenever this is possible. * * @brief Fetches the first row * @param string $sql SQL statement @@ -799,7 +819,7 @@ class DBA } } - $a->save_timestamp($stamp1, 'database'); + $a->saveTimestamp($stamp1, 'database'); return $columns; } @@ -811,12 +831,12 @@ class DBA * @param array $param parameter array * @param bool $on_duplicate_update Do an update on a duplicate entry * - * @return boolean was the insert successfull? + * @return boolean was the insert successful? */ public static function insert($table, $param, $on_duplicate_update = false) { if (empty($table) || empty($param)) { - logger('Table and fields have to be set'); + Logger::log('Table and fields have to be set'); return false; } @@ -924,13 +944,11 @@ class DBA switch (self::$driver) { case 'pdo': - if (self::$connection->inTransaction()) { - break; - } - if (!self::$connection->beginTransaction()) { + if (!self::$connection->inTransaction() && !self::$connection->beginTransaction()) { return false; } break; + case 'mysqli': if (!self::$connection->begin_transaction()) { return false; @@ -949,10 +967,13 @@ class DBA if (!self::$connection->inTransaction()) { return true; } + return self::$connection->commit(); + case 'mysqli': return self::$connection->commit(); } + return true; } @@ -985,6 +1006,7 @@ class DBA } $ret = self::$connection->rollBack(); break; + case 'mysqli': $ret = self::$connection->rollback(); break; @@ -1030,7 +1052,7 @@ class DBA public static function delete($table, array $conditions, array $options = [], $in_process = false, array &$callstack = []) { if (empty($table) || empty($conditions)) { - logger('Table and conditions have to be set'); + Logger::log('Table and conditions have to be set'); return false; } @@ -1050,7 +1072,12 @@ class DBA $commands[$key] = ['table' => $table, 'conditions' => $conditions]; - $cascade = defaults($options, 'cascade', true); + // Don't use "defaults" here, since it would set "false" to "true" + if (isset($options['cascade'])) { + $cascade = $options['cascade']; + } else { + $cascade = true; + } // To speed up the whole process we cache the table relations if ($cascade && count(self::$relation) == 0) { @@ -1116,7 +1143,7 @@ class DBA if ((count($command['conditions']) > 1) || is_int($first_key)) { $sql = "DELETE FROM `" . $command['table'] . "`" . $condition_string; - logger(self::replaceParameters($sql, $conditions), LOGGER_DATA); + Logger::log(self::replaceParameters($sql, $conditions), Logger::DATA); if (!self::e($sql, $conditions)) { if ($do_transaction) { @@ -1146,7 +1173,7 @@ class DBA $sql = "DELETE FROM `" . $table . "` WHERE `" . $field . "` IN (" . substr(str_repeat("?, ", count($field_values)), 0, -2) . ");"; - logger(self::replaceParameters($sql, $field_values), LOGGER_DATA); + Logger::log(self::replaceParameters($sql, $field_values), Logger::DATA); if (!self::e($sql, $field_values)) { if ($do_transaction) { @@ -1197,7 +1224,7 @@ class DBA public static function update($table, $fields, $condition, $old_fields = []) { if (empty($table) || empty($fields) || empty($condition)) { - logger('Table, fields and condition have to be set'); + Logger::log('Table, fields and condition have to be set'); return false; } @@ -1290,7 +1317,7 @@ class DBA * * $params = array("order" => array("id", "received" => true), "limit" => 10); * - * $data = dba::select($table, $fields, $condition, $params); + * $data = DBA::select($table, $fields, $condition, $params); */ public static function select($table, array $fields = [], array $condition = [], array $params = []) { @@ -1332,7 +1359,7 @@ class DBA * or: * $condition = ["`uid` = ? AND `network` IN (?, ?)", 1, 'dfrn', 'dspr']; * - * $count = dba::count($table, $condition); + * $count = DBA::count($table, $condition); */ public static function count($table, array $condition = []) { @@ -1386,7 +1413,7 @@ class DBA /* Workaround for MySQL Bug #64791. * Never mix data types inside any IN() condition. * In case of mixed types, cast all as string. - * Logic needs to be consistent with dba::p() data types. + * Logic needs to be consistent with DBA::p() data types. */ $is_int = false; $is_alpha = false; @@ -1446,7 +1473,7 @@ class DBA $limit_string = ''; if (isset($params['limit']) && is_int($params['limit'])) { - $limit_string = " LIMIT " . $params['limit']; + $limit_string = " LIMIT " . intval($params['limit']); } if (isset($params['limit']) && is_array($params['limit'])) { @@ -1517,7 +1544,7 @@ class DBA case 'mysqli': // MySQLi offers both a mysqli_stmt and a mysqli_result class. // We should be careful not to assume the object type of $stmt - // because dba::p() has been able to return both types. + // because DBA::p() has been able to return both types. if ($stmt instanceof mysqli_stmt) { $stmt->free_result(); $ret = $stmt->close(); @@ -1530,8 +1557,105 @@ class DBA break; } - $a->save_timestamp($stamp1, 'database'); + $a->saveTimestamp($stamp1, 'database'); return $ret; } + + /** + * @brief Return a list of database processes + * + * @return array + * 'list' => List of processes, separated in their different states + * 'amount' => Number of concurrent database processes + */ + public static function processlist() + { + $ret = self::p("SHOW PROCESSLIST"); + $data = self::toArray($ret); + + $s = []; + + $processes = 0; + $states = []; + foreach ($data as $process) { + $state = trim($process["State"]); + + // Filter out all non blocking processes + if (!in_array($state, ["", "init", "statistics", "updating"])) { + ++$states[$state]; + ++$processes; + } + } + + $statelist = ""; + foreach ($states as $state => $usage) { + if ($statelist != "") { + $statelist .= ", "; + } + $statelist .= $state.": ".$usage; + } + return(["list" => $statelist, "amount" => $processes]); + } + + /** + * Checks if $array is a filled array with at least one entry. + * + * @param mixed $array A filled array with at least one entry + * + * @return boolean Whether $array is a filled array or an object with rows + */ + public static function isResult($array) + { + // It could be a return value from an update statement + if (is_bool($array)) { + return $array; + } + + if (is_object($array)) { + return self::numRows($array) > 0; + } + + return (is_array($array) && (count($array) > 0)); + } + + /** + * @brief Callback function for "esc_array" + * + * @param mixed $value Array value + * @param string $key Array key + * @param boolean $add_quotation add quotation marks for string values + * @return void + */ + private static function escapeArrayCallback(&$value, $key, $add_quotation) + { + if (!$add_quotation) { + if (is_bool($value)) { + $value = ($value ? '1' : '0'); + } else { + $value = self::escape($value); + } + return; + } + + if (is_bool($value)) { + $value = ($value ? 'true' : 'false'); + } elseif (is_float($value) || is_integer($value)) { + $value = (string) $value; + } else { + $value = "'" . self::escape($value) . "'"; + } + } + + /** + * @brief Escapes a whole array + * + * @param mixed $arr Array with values to be escaped + * @param boolean $add_quotation add quotation marks for string values + * @return void + */ + public static function escapeArray(&$arr, $add_quotation = false) + { + array_walk($arr, 'self::escapeArrayCallback', $add_quotation); + } }