}
}
- public function connect()
+ /**
+ * Tries to connect to database
+ *
+ * @return bool Success
+ */
+ public function connect(): bool
{
if (!is_null($this->connection) && $this->connected()) {
return $this->connected;
*
* @return string with either "pdo" or "mysqli"
*/
- public function getDriver()
+ public function getDriver(): string
{
return $this->driver;
}
* This function discriminate between the deprecated mysql API and the current
* object-oriented mysqli API. Example of returned string: 5.5.46-0+deb8u1
*
- * @return string
+ * @return string Database server information
*/
- public function serverInfo()
+ public function serverInfo(): string
{
if ($this->server_info == '') {
switch ($this->driver) {
/**
* Returns the selected database name
*
- * @return string
+ * @return string Database name
* @throws \Exception
*/
- public function databaseName()
+ public function databaseName(): string
{
$ret = $this->p("SELECT DATABASE() AS `db`");
$data = $this->toArray($ret);
* Analyze a database query and log this if some conditions are met.
*
* @param string $query The database query that will be analyzed
- *
+ * @return void
* @throws \Exception
*/
- private function logIndex($query)
+ private function logIndex(string $query)
{
if (!$this->configCache->get('system', 'db_log_index')) {
* Removes every not allowlisted character from the identifier string
*
* @param string $identifier
- *
* @return string sanitized identifier
* @throws \Exception
*/
- private function sanitizeIdentifier($identifier)
+ private function sanitizeIdentifier(string $identifier): string
{
return preg_replace('/[^A-Za-z0-9_\-]+/', '', $identifier);
}
}
}
- public function isConnected()
+ /**
+ * Returns connected flag
+ *
+ * @return bool Whether connection to database was success
+ */
+ public function isConnected(): bool
{
return $this->connected;
}
+ /**
+ * Checks connection status
+ *
+ * @return bool Whether connection to database was success
+ */
public function connected()
{
$connected = false;
*
* @return string The input SQL string modified if necessary.
*/
- public function anyValueFallback($sql)
+ public function anyValueFallback(string $sql): string
{
$server_info = $this->serverInfo();
if (version_compare($server_info, '5.7.5', '<') ||
*
* @return string The replaced SQL query
*/
- private function replaceParameters($sql, $args)
+ private function replaceParameters(string $sql, array $args): string
{
$offset = 0;
foreach ($args as $param => $value) {
* @return bool|object statement object or result object
* @throws \Exception
*/
- public function p($sql)
+ public function p(string $sql)
{
$this->profiler->startRecording('database');
* @return boolean Was the query successfull? False is returned only if an error occurred
* @throws \Exception
*/
- public function e($sql)
+ public function e(string $sql): bool
{
+ $retval = false;
$this->profiler->startRecording('database_write');
* @return boolean Are there rows for that condition?
* @throws \Exception
*/
- public function exists($table, $condition)
+ public function exists($table, array $condition): bool
{
if (empty($table)) {
return false;
*
* @param string $sql SQL statement
*
- * @return array first row of query
+ * @return array|bool first row of query or false on failure
* @throws \Exception
*/
- public function fetchFirst($sql)
+ public function fetchFirst(string $sql)
{
$params = DBA::getParam(func_get_args());
*
* @return int Number of rows
*/
- public function affectedRows()
+ public function affectedRows(): int
{
return $this->affected_rows;
}
*
* @return int Number of columns
*/
- public function columnCount($stmt)
+ public function columnCount($stmt): int
{
if (!is_object($stmt)) {
return 0;
*
* @return int Number of rows
*/
- public function numRows($stmt)
+ public function numRows($stmt): int
{
if (!is_object($stmt)) {
return 0;
*
* @param bool|PDOStatement|mysqli_stmt $stmt statement object
*
- * @return array|false current row
+ * @return array|bool Current row or false on failure
*/
public function fetch($stmt)
{
* @return boolean was the insert successful?
* @throws \Exception
*/
- public function insert($table, array $param, int $duplicate_mode = self::INSERT_DEFAULT)
+ public function insert($table, array $param, int $duplicate_mode = self::INSERT_DEFAULT): bool
{
if (empty($table) || empty($param)) {
$this->logger->info('Table and fields have to be set');
* @return boolean was the insert successful?
* @throws \Exception
*/
- public function replace($table, array $param)
+ public function replace($table, array $param): bool
{
if (empty($table) || empty($param)) {
$this->logger->info('Table and fields have to be set');
*
* @return integer Last inserted id
*/
- public function lastInsertId()
+ public function lastInsertId(): int
{
switch ($this->driver) {
case self::PDO:
* @return boolean was the lock successful?
* @throws \Exception
*/
- public function lock($table)
+ public function lock($table): bool
{
// See here: https://dev.mysql.com/doc/refman/5.7/en/lock-tables-and-transactions.html
if ($this->driver == self::PDO) {
* @return boolean was the unlock successful?
* @throws \Exception
*/
- public function unlock()
+ public function unlock(): bool
{
// See here: https://dev.mysql.com/doc/refman/5.7/en/lock-tables-and-transactions.html
$this->performCommit();
return true;
}
- protected function performCommit()
+ /**
+ * Performs the commit
+ *
+ * @return boolean Was the command executed successfully?
+ */
+ protected function performCommit(): bool
{
switch ($this->driver) {
case self::PDO:
*
* @return boolean Was the command executed successfully?
*/
- public function commit()
+ public function commit(): bool
{
if (!$this->performCommit()) {
return false;
*
* @return boolean Was the command executed successfully?
*/
- public function rollback()
+ public function rollback(): bool
{
$ret = false;
$ret = $this->connection->rollback();
break;
}
+
$this->in_transaction = false;
return $ret;
}
* @return boolean was the delete successful?
* @throws \Exception
*/
- public function delete($table, array $conditions)
+ public function delete($table, array $conditions): bool
{
if (empty($table) || empty($conditions)) {
$this->logger->info('Table and conditions have to be set');
*
* @return boolean was the update successfull?
* @throws \Exception
+ * @todo Implement "bool $update_on_duplicate" to avoid mixed type for $old_fields
*/
- public function update($table, $fields, $condition, $old_fields = [], $params = [])
+ public function update($table, array $fields, array $condition, $old_fields = [], array $params = [])
{
if (empty($table) || empty($fields) || empty($condition)) {
$this->logger->info('Table, fields and condition have to be set');
* @throws \Exception
* @see $this->select
*/
- public function selectFirst($table, array $fields = [], array $condition = [], $params = [])
+ public function selectFirst($table, array $fields = [], array $condition = [], array $params = [])
{
$params['limit'] = 1;
$result = $this->select($table, $fields, $condition, $params);
*
* @param array $fields
* @param array $options
- * @return array
+ * @return array Escaped fields
*/
- private function escapeFields(array $fields, array $options)
+ private function escapeFields(array $fields, array $options): array
{
// In the case of a "GROUP BY" we have to add all the ORDER fields to the fieldlist.
// This needs to done to apply the "ANY_VALUE(...)" treatment from below to them.
*/
public function count($table, array $condition = [], array $params = [])
{
+ // @TODO Can we dump this to have ": int" as returned type-hint?
if (empty($table)) {
return false;
}
* @param array $fields
* @return array casted fields
*/
- public function castFields(string $table, array $fields) {
+ public function castFields(string $table, array $fields): array
+ {
// When there is no data, we don't need to do something
if (empty($fields)) {
return $fields;
*
* @return string Error message ('' if no error)
*/
- public function errorMessage()
+ public function errorMessage(): string
{
return $this->error;
}
* @param string $name
* @return string content
*/
- public function getVariable(string $name)
+ public function getVariable(string $name): string
{
$result = $this->fetchFirst("SHOW GLOBAL VARIABLES WHERE `Variable_name` = ?", $name);
return $result['Value'] ?? null;
*
* @return boolean Whether $array is a filled array or an object with rows
*/
- public function isResult($array)
+ public function isResult($array): bool
{
// It could be a return value from an update statement
if (is_bool($array)) {
* @param mixed $value Array value
* @param string $key Array key
* @param boolean $add_quotation add quotation marks for string values
- *
* @return void
*/
- private function escapeArrayCallback(&$value, $key, $add_quotation)
+ private function escapeArrayCallback(&$value, string $key, bool $add_quotation)
{
if (!$add_quotation) {
if (is_bool($value)) {
*
* @param mixed $arr Array with values to be escaped
* @param boolean $add_quotation add quotation marks for string values
- *
* @return void
*/
- public function escapeArray(&$arr, $add_quotation = false)
+ public function escapeArray(&$arr, bool $add_quotation = false)
{
array_walk($arr, [$this, 'escapeArrayCallback'], $add_quotation);
}
/**
* Replaces a string in the provided fields of the provided table
*
- * @param string $table_name
+ * @param string $table_name Table name
* @param array $fields List of field names in the provided table
* @param string $search
* @param string $replace
+ * @return void
* @throws \Exception
*/
public function replaceInTableFields(string $table_name, array $fields, string $search, string $replace)
projects / friendica.git / blobdiff