[BUGFIX] Unused TDParams in ColumnsContentObject()
[Packages/TYPO3.CMS.git] / typo3 / sysext / core / Classes / Database / DatabaseConnection.php
index f23c109..23fc313 100644 (file)
@@ -339,7 +339,7 @@ class DatabaseConnection {
                $mmWhere .= ($local_table and $foreign_table) ? ' AND ' : '';
                $tables = ($local_table ? $local_table . ',' : '') . $mm_table;
                if ($foreign_table) {
-                       $mmWhere .= ($foreign_table_as ? $foreign_table_as : $foreign_table) . '.uid=' . $mm_table . '.uid_foreign';
+                       $mmWhere .= ($foreign_table_as ?: $foreign_table) . '.uid=' . $mm_table . '.uid_foreign';
                        $tables .= ',' . $foreign_table . ($foreign_table_as ? ' AS ' . $foreign_table_as : '');
                }
                return $this->exec_SELECTquery($select, $tables, $mmWhere . ' ' . $whereClause, $groupBy, $orderBy, $limit);
@@ -434,7 +434,7 @@ class DatabaseConnection {
                $resultSet = $this->exec_SELECTquery('COUNT(' . $field . ')', $table, $where);
                if ($resultSet !== FALSE) {
                        list($count) = $this->sql_fetch_row($resultSet);
-                       $count = intval($count);
+                       $count = (int)$count;
                        $this->sql_free_result($resultSet);
                }
                return $count;
@@ -555,7 +555,7 @@ class DatabaseConnection {
                                }
                        }
                        // Build query
-                       $query = 'UPDATE ' . $table . ' SET ' . implode(',', $fields) . (strlen($where) > 0 ? ' WHERE ' . $where : '');
+                       $query = 'UPDATE ' . $table . ' SET ' . implode(',', $fields) . ((string)$where !== '' ? ' WHERE ' . $where : '');
                        if ($this->debugOutput || $this->store_lastBuiltQuery) {
                                $this->debug_lastBuiltQuery = $query;
                        }
@@ -580,7 +580,7 @@ class DatabaseConnection {
                                $hookObject->DELETEquery_preProcessAction($table, $where, $this);
                        }
                        // Table and fieldnames should be "SQL-injection-safe" when supplied to this function
-                       $query = 'DELETE FROM ' . $table . (strlen($where) > 0 ? ' WHERE ' . $where : '');
+                       $query = 'DELETE FROM ' . $table . ((string)$where !== '' ? ' WHERE ' . $where : '');
                        if ($this->debugOutput || $this->store_lastBuiltQuery) {
                                $this->debug_lastBuiltQuery = $query;
                        }
@@ -608,13 +608,13 @@ class DatabaseConnection {
                }
                // Table and fieldnames should be "SQL-injection-safe" when supplied to this function
                // Build basic query
-               $query = 'SELECT ' . $select_fields . ' FROM ' . $from_table . (strlen($where_clause) > 0 ? ' WHERE ' . $where_clause : '');
+               $query = 'SELECT ' . $select_fields . ' FROM ' . $from_table . ((string)$where_clause !== '' ? ' WHERE ' . $where_clause : '');
                // Group by
-               $query .= strlen($groupBy) > 0 ? ' GROUP BY ' . $groupBy : '';
+               $query .= (string)$groupBy !== '' ? ' GROUP BY ' . $groupBy : '';
                // Order by
-               $query .= strlen($orderBy) > 0 ? ' ORDER BY ' . $orderBy : '';
+               $query .= (string)$orderBy !== '' ? ' ORDER BY ' . $orderBy : '';
                // Group by
-               $query .= strlen($limit) > 0 ? ' LIMIT ' . $limit : '';
+               $query .= (string)$limit !== '' ? ' LIMIT ' . $limit : '';
                // Return query
                if ($this->debugOutput || $this->store_lastBuiltQuery) {
                        $this->debug_lastBuiltQuery = $query;
@@ -634,7 +634,7 @@ class DatabaseConnection {
        public function SELECTsubquery($select_fields, $from_table, $where_clause) {
                // Table and fieldnames should be "SQL-injection-safe" when supplied to this function
                // Build basic query:
-               $query = 'SELECT ' . $select_fields . ' FROM ' . $from_table . (strlen($where_clause) > 0 ? ' WHERE ' . $where_clause : '');
+               $query = 'SELECT ' . $select_fields . ' FROM ' . $from_table . ((string)$where_clause !== '' ? ' WHERE ' . $where_clause : '');
                // Return query
                if ($this->debugOutput || $this->store_lastBuiltQuery) {
                        $this->debug_lastBuiltQuery = $query;
@@ -863,12 +863,12 @@ class DatabaseConnection {
         * Useful when you want to make sure an array contains only integers before imploding them in a select-list.
         *
         * @param array $arr Array with values
-        * @return array The input array with all values passed through intval()
+        * @return array The input array with all values cast to (int)
         * @see cleanIntList()
         */
        public function cleanIntArray($arr) {
                foreach ($arr as $k => $v) {
-                       $arr[$k] = intval($arr[$k]);
+                       $arr[$k] = (int)$arr[$k];
                }
                return $arr;
        }
@@ -878,7 +878,7 @@ class DatabaseConnection {
         * Useful when you want to make sure a commalist of supposed integers really contain only integers; You want to know that when you don't trust content that could go into an SQL statement.
         *
         * @param string $list List of comma-separated values which should be integers
-        * @return string The input list but with every value passed through intval()
+        * @return string The input list but with every value cast to (int)
         * @see cleanIntArray()
         */
        public function cleanIntList($list) {
@@ -1786,7 +1786,7 @@ class DatabaseConnection {
         */
        public function debug($func, $query = '') {
                $error = $this->sql_error();
-               if ($error || (int) $this->debugOutput === 2) {
+               if ($error || (int)$this->debugOutput === 2) {
                        \TYPO3\CMS\Core\Utility\DebugUtility::debug(
                                array(
                                        'caller' => 'TYPO3\\CMS\\Core\\Database\\DatabaseConnection::' . $func,
@@ -1866,7 +1866,7 @@ class DatabaseConnection {
                ) {
                        // Raw HTML output
                        $explainMode = 1;
-               } elseif ((int) $this->explainOutput == 3 && is_object($GLOBALS['TT'])) {
+               } elseif ((int)$this->explainOutput == 3 && is_object($GLOBALS['TT'])) {
                        // Embed the output into the TS admin panel
                        $explainMode = 2;
                } else {