]> git.mxchange.org Git - friendica.git/blobdiff - src/Database/DBA.php
Remove deprecated code
[friendica.git] / src / Database / DBA.php
index 1cf10ba4addd8472edf9a95359ec20acd7258ff0..f4d4bb11dd1907b644873899cf6daf621906491c 100644 (file)
@@ -56,7 +56,7 @@ class DBA
        }
 
        /**
-        * @brief Returns the MySQL server version string
+        * Returns the MySQL server version string
         *
         * This function discriminate between the deprecated mysql API and the current
         * object-oriented mysqli API. Example of returned string: 5.5.46-0+deb8u1
@@ -69,7 +69,7 @@ class DBA
        }
 
        /**
-        * @brief Returns the selected database name
+        * Returns the selected database name
         *
         * @return string
         * @throws \Exception
@@ -79,19 +79,30 @@ class DBA
                return DI::dba()->databaseName();
        }
 
+       /**
+        * Escape all SQL unsafe data
+        *
+        * @param string $str
+        * @return string escaped string
+        */
        public static function escape($str)
        {
                return DI::dba()->escape($str);
        }
 
+       /**
+        * Checks if the database is connected
+        *
+        * @return boolean is the database connected?
+        */
        public static function connected()
        {
                return DI::dba()->connected();
        }
 
        /**
-        * @brief Replaces ANY_VALUE() function by MIN() function,
-        *  if the database server does not support ANY_VALUE().
+        * Replaces ANY_VALUE() function by MIN() function,
+        * if the database server does not support ANY_VALUE().
         *
         * Considerations for Standard SQL, or MySQL with ONLY_FULL_GROUP_BY (default since 5.7.5).
         * ANY_VALUE() is available from MySQL 5.7.5 https://dev.mysql.com/doc/refman/5.7/en/miscellaneous-functions.html
@@ -106,7 +117,7 @@ class DBA
        }
 
        /**
-        * @brief beautifies the query - useful for "SHOW PROCESSLIST"
+        * beautifies the query - useful for "SHOW PROCESSLIST"
         *
         * This is safe when we bind the parameters later.
         * The parameter values aren't part of the SQL.
@@ -127,7 +138,7 @@ class DBA
        }
 
        /**
-        * @brief Convert parameter array to an universal form
+        * Convert parameter array to an universal form
         * @param array $args Parameter array
         * @return array universalized parameter array
         */
@@ -144,8 +155,8 @@ class DBA
        }
 
        /**
-        * @brief Executes a prepared statement that returns data
-        * @usage Example: $r = p("SELECT * FROM `item` WHERE `guid` = ?", $guid);
+        * Executes a prepared statement that returns data
+        * 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
@@ -162,7 +173,7 @@ class DBA
        }
 
        /**
-        * @brief Executes a prepared statement like UPDATE or INSERT that doesn't return data
+        * Executes a prepared statement like UPDATE or INSERT that doesn't return data
         *
         * Please use DBA::delete, DBA::insert, DBA::update, ... instead
         *
@@ -178,7 +189,7 @@ class DBA
        }
 
        /**
-        * @brief Check if data exists
+        * Check if data exists
         *
         * @param string|array $table     Table name or array [schema => table]
         * @param array        $condition array of fields for condition
@@ -196,7 +207,6 @@ class DBA
         *
         * Please use DBA::selectFirst or DBA::exists whenever this is possible.
         *
-        * @brief Fetches the first row
         * @param string $sql SQL statement
         * @return array first row of query
         * @throws \Exception
@@ -209,7 +219,7 @@ class DBA
        }
 
        /**
-        * @brief Returns the number of affected rows of the last statement
+        * Returns the number of affected rows of the last statement
         *
         * @return int Number of rows
         */
@@ -219,7 +229,7 @@ class DBA
        }
 
        /**
-        * @brief Returns the number of columns of a statement
+        * Returns the number of columns of a statement
         *
         * @param object Statement object
         * @return int Number of columns
@@ -229,7 +239,7 @@ class DBA
                return DI::dba()->columnCount($stmt);
        }
        /**
-        * @brief Returns the number of rows of a statement
+        * Returns the number of rows of a statement
         *
         * @param PDOStatement|mysqli_result|mysqli_stmt Statement object
         * @return int Number of rows
@@ -240,7 +250,7 @@ class DBA
        }
 
        /**
-        * @brief Fetch a single row
+        * Fetch a single row
         *
         * @param mixed $stmt statement object
         * @return array current row
@@ -251,7 +261,7 @@ class DBA
        }
 
        /**
-        * @brief Insert a row into a table
+        * Insert a row into a table
         *
         * @param string|array $table               Table name or array [schema => table]
         * @param array        $param               parameter array
@@ -266,7 +276,7 @@ class DBA
        }
 
        /**
-        * @brief Fetch the id of the last insert command
+        * Fetch the id of the last insert command
         *
         * @return integer Last inserted id
         */
@@ -276,7 +286,7 @@ class DBA
        }
 
        /**
-        * @brief Locks a table for exclusive write access
+        * Locks a table for exclusive write access
         *
         * This function can be extended in the future to accept a table array as well.
         *
@@ -291,7 +301,7 @@ class DBA
        }
 
        /**
-        * @brief Unlocks all locked tables
+        * Unlocks all locked tables
         *
         * @return boolean was the unlock successful?
         * @throws \Exception
@@ -302,7 +312,7 @@ class DBA
        }
 
        /**
-        * @brief Starts a transaction
+        * Starts a transaction
         *
         * @return boolean Was the command executed successfully?
         */
@@ -312,7 +322,7 @@ class DBA
        }
 
        /**
-        * @brief Does a commit
+        * Does a commit
         *
         * @return boolean Was the command executed successfully?
         */
@@ -322,7 +332,7 @@ class DBA
        }
 
        /**
-        * @brief Does a rollback
+        * Does a rollback
         *
         * @return boolean Was the command executed successfully?
         */
@@ -332,7 +342,7 @@ class DBA
        }
 
        /**
-        * @brief Delete a row from a table
+        * Delete a row from a table
         *
         * @param string|array $table      Table name
         * @param array        $conditions Field condition(s)
@@ -349,9 +359,9 @@ class DBA
        }
 
        /**
-        * @brief Updates rows
+        * Updates rows in the database.
         *
-        * Updates rows in the database. When $old_fields is set to an array,
+        * When $old_fields is set to an array,
         * the system will only do an update if the fields in that array changed.
         *
         * Attention:
@@ -385,7 +395,6 @@ class DBA
        /**
         * Retrieve a single record from a table and returns it in an associative array
         *
-        * @brief Retrieve a single record from a table
         * @param string|array $table     Table name or array [schema => table]
         * @param array        $fields
         * @param array        $condition
@@ -400,7 +409,7 @@ class DBA
        }
 
        /**
-        * @brief Select rows from a table and fills an array with the data
+        * Select rows from a table and fills an array with the data
         *
         * @param string|array $table     Table name or array [schema => table]
         * @param array        $fields    Array of selected fields, empty for all
@@ -417,7 +426,7 @@ class DBA
        }
 
        /**
-        * @brief Select rows from a table
+        * Select rows from a table
         *
         * @param string|array $table     Table name or array [schema => table]
         * @param array        $fields    Array of selected fields, empty for all
@@ -445,7 +454,7 @@ class DBA
        }
 
        /**
-        * @brief Counts the rows from a table satisfying the provided condition
+        * Counts the rows from a table satisfying the provided condition
         *
         * @param string|array $table     Table name or array [schema => table]
         * @param array        $condition array of fields for condition
@@ -510,7 +519,7 @@ class DBA
        }
 
        /**
-        * @brief Returns the SQL condition string built from the provided condition array
+        * Returns the SQL condition string built from the provided condition array
         *
         * This function operates with two modes.
         * - Supplied with a filed/value associative array, it builds simple strict
@@ -529,69 +538,98 @@ class DBA
         */
        public static function buildCondition(array &$condition = [])
        {
+               $condition = self::collapseCondition($condition);
+               
                $condition_string = '';
                if (count($condition) > 0) {
-                       reset($condition);
-                       $first_key = key($condition);
-                       if (is_int($first_key)) {
-                               $condition_string = " WHERE (" . array_shift($condition) . ")";
-                       } else {
-                               $new_values = [];
-                               $condition_string = "";
-                               foreach ($condition as $field => $value) {
-                                       if ($condition_string != "") {
-                                               $condition_string .= " AND ";
-                                       }
-                                       if (is_array($value)) {
-                                               if (count($value)) {
-                                                       /* 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.
-                                                        */
-                                                       $is_int = false;
-                                                       $is_alpha = false;
-                                                       foreach ($value as $single_value) {
-                                                               if (is_int($single_value)) {
-                                                                       $is_int = true;
-                                                               } else {
-                                                                       $is_alpha = true;
-                                                               }
-                                                       }
+                       $condition_string = " WHERE (" . array_shift($condition) . ")";
+               }
 
-                                                       if ($is_int && $is_alpha) {
-                                                               foreach ($value as &$ref) {
-                                                                       if (is_int($ref)) {
-                                                                               $ref = (string)$ref;
-                                                                       }
-                                                               }
-                                                               unset($ref); //Prevent accidental re-use.
-                                                       }
+               return $condition_string;
+       }
+
+       /**
+        * Collapse an associative array condition into a SQL string + parameters condition array.
+        *
+        * ['uid' => 1, 'network' => ['dspr', 'apub']]
+        *
+        * gets transformed into
+        *
+        * ["`uid` = ? AND `network` IN (?, ?)", 1, 'dspr', 'apub']
+        *
+        * @param array $condition
+        * @return array
+        */
+       public static function collapseCondition(array $condition)
+       {
+               // Ensures an always true condition is returned
+               if (count($condition) < 1) {
+                       return ['1'];
+               }
+
+               reset($condition);
+               $first_key = key($condition);
+
+               if (is_int($first_key)) {
+                       // Already collapsed
+                       return $condition;
+               }
+
+               $values = [];
+               $condition_string = "";
+               foreach ($condition as $field => $value) {
+                       if ($condition_string != "") {
+                               $condition_string .= " AND ";
+                       }
 
-                                                       $new_values = array_merge($new_values, array_values($value));
-                                                       $placeholders = substr(str_repeat("?, ", count($value)), 0, -2);
-                                                       $condition_string .= self::quoteIdentifier($field) . " IN (" . $placeholders . ")";
+                       if (is_array($value)) {
+                               if (count($value)) {
+                                       /* 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.
+                                        */
+                                       $is_int = false;
+                                       $is_alpha = false;
+                                       foreach ($value as $single_value) {
+                                               if (is_int($single_value)) {
+                                                       $is_int = true;
                                                } else {
-                                                       // Empty value array isn't supported by IN and is logically equivalent to no match
-                                                       $condition_string .= "FALSE";
+                                                       $is_alpha = true;
                                                }
-                                       } elseif (is_null($value)) {
-                                               $condition_string .= self::quoteIdentifier($field) . " IS NULL";
-                                       } else {
-                                               $new_values[$field] = $value;
-                                               $condition_string .= self::quoteIdentifier($field) . " = ?";
                                        }
+
+                                       if ($is_int && $is_alpha) {
+                                               foreach ($value as &$ref) {
+                                                       if (is_int($ref)) {
+                                                               $ref = (string)$ref;
+                                                       }
+                                               }
+                                               unset($ref); //Prevent accidental re-use.
+                                       }
+
+                                       $values = array_merge($values, array_values($value));
+                                       $placeholders = substr(str_repeat("?, ", count($value)), 0, -2);
+                                       $condition_string .= self::quoteIdentifier($field) . " IN (" . $placeholders . ")";
+                               } else {
+                                       // Empty value array isn't supported by IN and is logically equivalent to no match
+                                       $condition_string .= "FALSE";
                                }
-                               $condition_string = " WHERE (" . $condition_string . ")";
-                               $condition = $new_values;
+                       } elseif (is_null($value)) {
+                               $condition_string .= self::quoteIdentifier($field) . " IS NULL";
+                       } else {
+                               $values[$field] = $value;
+                               $condition_string .= self::quoteIdentifier($field) . " = ?";
                        }
                }
 
-               return $condition_string;
+               $condition = array_merge([$condition_string], array_values($values));
+
+               return $condition;
        }
 
        /**
-        * @brief Returns the SQL parameter string built from the provided parameter array
+        * Returns the SQL parameter string built from the provided parameter array
         *
         * @param array $params
         * @return string
@@ -631,7 +669,7 @@ class DBA
        }
 
        /**
-        * @brief Fills an array with data from a query
+        * Fills an array with data from a query
         *
         * @param object $stmt statement object
         * @param bool   $do_close
@@ -643,7 +681,7 @@ class DBA
        }
 
        /**
-        * @brief Returns the error number of the last query
+        * Returns the error number of the last query
         *
         * @return string Error number (0 if no error)
         */
@@ -653,7 +691,7 @@ class DBA
        }
 
        /**
-        * @brief Returns the error message of the last query
+        * Returns the error message of the last query
         *
         * @return string Error message ('' if no error)
         */
@@ -663,7 +701,7 @@ class DBA
        }
 
        /**
-        * @brief Closes the current statement
+        * Closes the current statement
         *
         * @param object $stmt statement object
         * @return boolean was the close successful?
@@ -674,7 +712,7 @@ class DBA
        }
 
        /**
-        * @brief Return a list of database processes
+        * Return a list of database processes
         *
         * @return array
         *      'list' => List of processes, separated in their different states
@@ -699,7 +737,7 @@ class DBA
        }
 
        /**
-        * @brief Escapes a whole array
+        * Escapes a whole array
         *
         * @param mixed   $arr           Array with values to be escaped
         * @param boolean $add_quotation add quotation marks for string values