[TASK] Remove superfluous parenthesis.
authorArvid Jakobsson <arvid.jakobsson@gmail.com>
Sun, 9 Sep 2012 19:59:22 +0000 (21:59 +0200)
committerChristian Kuhn <lolli@schwarzbu.ch>
Sun, 9 Sep 2012 21:46:05 +0000 (23:46 +0200)
Change ("a" . "b") . "c" into "a" . "b" . "c" using the lex-pass tool.

Change-Id: I7d10826a66a32af1fb3dbd462b028a5d12cfba89
Resolves: #40777
Related: #40095
Releases: 6.0
Reviewed-on: http://review.typo3.org/14482
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
14 files changed:
typo3/sysext/dbal/Classes/Autoloader.php
typo3/sysext/dbal/Classes/Controller/ModuleController.php
typo3/sysext/dbal/Classes/Database/DatabaseConnection.php
typo3/sysext/dbal/Classes/Database/SqlParser.php
typo3/sysext/dbal/Classes/ExtensionManager/MessageDisplay.php
typo3/sysext/dbal/Classes/Hooks/ExtensionManagerHooks.php
typo3/sysext/dbal/Classes/Hooks/InstallToolHooks.php
typo3/sysext/dbal/Classes/RecordList/RecordList.php
typo3/sysext/dbal/Tests/BaseTestCase.php
typo3/sysext/dbal/Tests/DatabaseMssqlTest.php
typo3/sysext/dbal/Tests/DatabaseOracleTest.php
typo3/sysext/dbal/Tests/DatabasePostgresqlTest.php
typo3/sysext/dbal/Tests/FakeDatabaseConnection.php
typo3/sysext/dbal/Tests/SqlParserGeneralTest.php

index 68d210a..43074d2 100644 (file)
@@ -74,7 +74,7 @@ class Autoloader {
         * @return boolean
         */
        protected function isDbalSupported() {
-               return (extension_loaded('odbc') || extension_loaded('pdo')) || extension_loaded('oci8');
+               return extension_loaded('odbc') || extension_loaded('pdo') || extension_loaded('oci8');
        }
 
        /**
index db3fef6..222f03d 100644 (file)
@@ -93,8 +93,8 @@ class ModuleController extends \TYPO3\CMS\Backend\Module\BaseScriptClass {
         */
        protected function printSqlCheck() {
                $input = \TYPO3\CMS\Core\Utility\GeneralUtility::_GP('tx_dbal');
-               $out = ((((((((((((((((((((((((((((((((('
-                       <form name="sql_check" action="' . $this->thisScript) . '" method="post" enctype="') . $GLOBALS['TYPO3_CONF_VARS']['SYS']['form_enctype']) . '">
+               $out = '
+                       <form name="sql_check" action="' . $this->thisScript . '" method="post" enctype="' . $GLOBALS['TYPO3_CONF_VARS']['SYS']['form_enctype'] . '">
                        <script type="text/javascript">
 /*<![CDATA[*/
 function updateQryForm(s) {
@@ -157,26 +157,26 @@ function updateQryForm(s) {
            <tr class="tableheader bgColor5"><th colspan="2">Easy SQL check</th></tr>
            <tr><td colspan="2">
            <select name="tx_dbal[QUERY]"size="1" onchange="updateQryForm(this.options[this.selectedIndex].value)">
-            <option value="SELECT" ') . ($input['QUERY'] === 'SELECT' ? 'selected="selected"' : '')) . '>SELECT</option>
-            <option value="INSERT" ') . ($input['QUERY'] === 'INSERT' ? 'selected="selected"' : '')) . '>INSERT</option>
-            <option value="UPDATE" ') . ($input['QUERY'] === 'UPDATE' ? 'selected="selected"' : '')) . '>UPDATE</option>
-            <option value="DELETE" ') . ($input['QUERY'] === 'DELETE' ? 'selected="selected"' : '')) . '>DELETE</option>
+            <option value="SELECT" ' . ($input['QUERY'] === 'SELECT' ? 'selected="selected"' : '') . '>SELECT</option>
+            <option value="INSERT" ' . ($input['QUERY'] === 'INSERT' ? 'selected="selected"' : '') . '>INSERT</option>
+            <option value="UPDATE" ' . ($input['QUERY'] === 'UPDATE' ? 'selected="selected"' : '') . '>UPDATE</option>
+            <option value="DELETE" ' . ($input['QUERY'] === 'DELETE' ? 'selected="selected"' : '') . '>DELETE</option>
            </select>
            </td></tr>
-           <tr id="tx-dbal-qryupdate" style="display:none;"><td></td><td><input name="tx_dbal[UPDATE]" value="') . $input['UPDATE']) . '" type="text" size="30" maxsize="100" /></td></tr>
-           <tr id="tx-dbal-qryfields"><td></td><td><input name="tx_dbal[FIELDS]" value="') . $input['FIELDS']) . '" type="text" size="30" maxsize="100" /></td></tr>
-           <tr id="tx-dbal-qryinsertvalues" style="display:none;"><td></td><td><textarea name="tx_dbal[INSERTVALUES]" cols="30" rows="4">') . $input['INSERTVALUES']) . '</textarea></td></tr>
-           <tr id="tx-dbal-qryupdatevalues" style="display:none;"><th>SET</th><td><textarea name="tx_dbal[UPDATEVALUES]" cols="30" rows="4">') . $input['UPDATEVALUES']) . '</textarea></td></tr>
-           <tr id="tx-dbal-qryfrom"><th>FROM</th><td><input name="tx_dbal[FROM]" value="') . $input['FROM']) . '" type="text" size="30" maxsize="100" /></td></tr>
-           <tr id="tx-dbal-qryinto" style="display:none;"><th>INTO</th><td><input name="tx_dbal[INTO]" value="') . $input['INTO']) . '" type="text" size="30" maxsize="100" /></td></tr>
-           <tr id="tx-dbal-qrywhere"><th>WHERE</th><td><input name="tx_dbal[WHERE]" value="') . $input['WHERE']) . '" type="text" size="30" maxsize="100" /></td></tr>
-           <tr id="tx-dbal-qrygroup"><th>GROUP BY</th><td><input name="tx_dbal[GROUP]" value="') . $input['GROUP']) . '" type="text" size="30" maxsize="100" /></td></tr>
-           <tr id="tx-dbal-qryorder"><th>ORDER BY</th><td><input name="tx_dbal[ORDER]" value="') . $input['ORDER']) . '" type="text" size="30" maxsize="100" /></td></tr>
-           <tr id="tx-dbal-qrylimit"><th>LIMIT</th><td><input name="tx_dbal[LIMIT]" value="') . $input['LIMIT']) . '" type="text" size="30" maxsize="100" /></td></tr>
+           <tr id="tx-dbal-qryupdate" style="display:none;"><td></td><td><input name="tx_dbal[UPDATE]" value="' . $input['UPDATE'] . '" type="text" size="30" maxsize="100" /></td></tr>
+           <tr id="tx-dbal-qryfields"><td></td><td><input name="tx_dbal[FIELDS]" value="' . $input['FIELDS'] . '" type="text" size="30" maxsize="100" /></td></tr>
+           <tr id="tx-dbal-qryinsertvalues" style="display:none;"><td></td><td><textarea name="tx_dbal[INSERTVALUES]" cols="30" rows="4">' . $input['INSERTVALUES'] . '</textarea></td></tr>
+           <tr id="tx-dbal-qryupdatevalues" style="display:none;"><th>SET</th><td><textarea name="tx_dbal[UPDATEVALUES]" cols="30" rows="4">' . $input['UPDATEVALUES'] . '</textarea></td></tr>
+           <tr id="tx-dbal-qryfrom"><th>FROM</th><td><input name="tx_dbal[FROM]" value="' . $input['FROM'] . '" type="text" size="30" maxsize="100" /></td></tr>
+           <tr id="tx-dbal-qryinto" style="display:none;"><th>INTO</th><td><input name="tx_dbal[INTO]" value="' . $input['INTO'] . '" type="text" size="30" maxsize="100" /></td></tr>
+           <tr id="tx-dbal-qrywhere"><th>WHERE</th><td><input name="tx_dbal[WHERE]" value="' . $input['WHERE'] . '" type="text" size="30" maxsize="100" /></td></tr>
+           <tr id="tx-dbal-qrygroup"><th>GROUP BY</th><td><input name="tx_dbal[GROUP]" value="' . $input['GROUP'] . '" type="text" size="30" maxsize="100" /></td></tr>
+           <tr id="tx-dbal-qryorder"><th>ORDER BY</th><td><input name="tx_dbal[ORDER]" value="' . $input['ORDER'] . '" type="text" size="30" maxsize="100" /></td></tr>
+           <tr id="tx-dbal-qrylimit"><th>LIMIT</th><td><input name="tx_dbal[LIMIT]" value="' . $input['LIMIT'] . '" type="text" size="30" maxsize="100" /></td></tr>
                        <tr><td></td><td style="text-align:right;"><input type="submit" value="CHECK" /></td></tr>
                        <script type="text/javascript">
 /*<![CDATA[*/
-updateQryForm(\'') . $input['QUERY']) . '\');
+updateQryForm(\'' . $input['QUERY'] . '\');
 /*]]>*/
                                </script>
                        ';
@@ -195,10 +195,10 @@ updateQryForm(\'') . $input['QUERY']) . '\');
                        $qry = $GLOBALS['TYPO3_DB']->DELETEquery($input['FROM'], $input['WHERE']);
                        break;
                }
-               $out .= ('<pre>' . htmlspecialchars($qry)) . '</pre></td></tr>';
-               $out .= ('
+               $out .= '<pre>' . htmlspecialchars($qry) . '</pre></td></tr>';
+               $out .= '
                        <tr class="tableheader bgColor5"><th colspan="2">RAW SQL check</th></tr>
-                       <tr><td colspan="2" style="text-align:right;"><textarea name="tx_dbal[RAWSQL]" cols="60" rows="5">' . $input['RAWSQL']) . '</textarea><br /><input type="submit" value="CHECK" /></td></tr>';
+                       <tr><td colspan="2" style="text-align:right;"><textarea name="tx_dbal[RAWSQL]" cols="60" rows="5">' . $input['RAWSQL'] . '</textarea><br /><input type="submit" value="CHECK" /></td></tr>';
                if (!empty($input['RAWSQL'])) {
                        $out .= '<tr class="bgColor4">';
                        $parseResult = $GLOBALS['TYPO3_DB']->SQLparser->parseSQL($input['RAWSQL']);
@@ -208,8 +208,8 @@ updateQryForm(\'') . $input['QUERY']) . '\');
                                if (!is_array($testResult)) {
                                        $out .= '<td colspan="2">' . $newQuery;
                                } else {
-                                       $out .= (('<td colspan="2">' . htmlspecialchars($testResult[0])) . '</td></tr>
-                                       <tr><th>Error:</th><td style="border:2px solid #f00;">Input query did not match the parsed and recompiled query exactly (not observing whitespace):<br />') . htmlspecialchars($testResult[1]);
+                                       $out .= '<td colspan="2">' . htmlspecialchars($testResult[0]) . '</td></tr>
+                                       <tr><th>Error:</th><td style="border:2px solid #f00;">Input query did not match the parsed and recompiled query exactly (not observing whitespace):<br />' . htmlspecialchars($testResult[1]);
                                }
                        } else {
                                $out .= '<th>Result:</th><td style="border:2px solid #f00;">' . $parseResult;
@@ -259,8 +259,8 @@ updateQryForm(\'') . $input['QUERY']) . '\');
                ksort($GLOBALS['TYPO3_DB']->cache_autoIncFields);
                foreach ($GLOBALS['TYPO3_DB']->cache_autoIncFields as $table => $field) {
                        $out .= '<tr>';
-                       $out .= ('<td>' . $table) . '</td>';
-                       $out .= ('<td>' . $field) . '</td>';
+                       $out .= '<td>' . $table . '</td>';
+                       $out .= '<td>' . $field . '</td>';
                        $out .= '</tr>';
                }
                $out .= '</tbody></table>';
@@ -270,8 +270,8 @@ updateQryForm(\'') . $input['QUERY']) . '\');
                ksort($GLOBALS['TYPO3_DB']->cache_primaryKeys);
                foreach ($GLOBALS['TYPO3_DB']->cache_primaryKeys as $table => $field) {
                        $out .= '<tr>';
-                       $out .= ('<td>' . $table) . '</td>';
-                       $out .= ('<td>' . $field) . '</td>';
+                       $out .= '<td>' . $table . '</td>';
+                       $out .= '<td>' . $field . '</td>';
                        $out .= '</tr>';
                }
                $out .= '</tbody></table>';
@@ -280,14 +280,14 @@ updateQryForm(\'') . $input['QUERY']) . '\');
                $out .= '<table border="1" cellspacing="0"><tbody><tr><th colspan="5">Table</th></tr><tr><th>Field</th><th>Type</th><th><a href="#metatypes">Metatype</a></th><th>NOT NULL</th><th>Default</th></th></tr>';
                ksort($GLOBALS['TYPO3_DB']->cache_fieldType);
                foreach ($GLOBALS['TYPO3_DB']->cache_fieldType as $table => $fields) {
-                       $out .= ('<th colspan="5">' . $table) . '</th>';
+                       $out .= '<th colspan="5">' . $table . '</th>';
                        foreach ($fields as $field => $data) {
                                $out .= '<tr>';
-                               $out .= ('<td>' . $field) . '</td>';
-                               $out .= ('<td>' . $data['type']) . '</td>';
-                               $out .= ('<td>' . $data['metaType']) . '</td>';
-                               $out .= ('<td>' . ($data['notnull'] ? 'NOT NULL' : '')) . '</td>';
-                               $out .= ('<td>' . $data['default']) . '</td>';
+                               $out .= '<td>' . $field . '</td>';
+                               $out .= '<td>' . $data['type'] . '</td>';
+                               $out .= '<td>' . $data['metaType'] . '</td>';
+                               $out .= '<td>' . ($data['notnull'] ? 'NOT NULL' : '') . '</td>';
+                               $out .= '<td>' . $data['default'] . '</td>';
                                $out .= '</tr>';
                        }
                }
@@ -314,7 +314,7 @@ updateQryForm(\'') . $input['QUERY']) . '\');
   I8: 8-byte integer
   F:  Floating point number
   N:  Numeric or decimal number</pre>';
-               $menu = ('<a href="' . $this->thisScript) . '&amp;cmd=clear">CLEAR DATA</a><hr />';
+               $menu = '<a href="' . $this->thisScript . '&amp;cmd=clear">CLEAR DATA</a><hr />';
                $menu .= '<a href="#autoincrement">auto_increment</a> | <a href="#primarykeys">Primary keys</a> | <a href="#fieldtypes">Field types</a> | <a href="#metatypes">Metatype explanation</a><hr />';
                return $menu . $out;
        }
@@ -360,12 +360,12 @@ updateQryForm(\'') . $input['QUERY']) . '\');
                                        }
                                }
                                // Create output row
-                               $tRows[] = ((((((('
+                               $tRows[] = '
                                                <tr>
-                                                       <td>' . htmlspecialchars($row['exec_time'])) . '</td>
-                                                       <td>') . htmlspecialchars($row['table_join'])) . '</td>
-                                                       <td>') . htmlspecialchars($row['script'])) . '</td>
-                                                       <td>') . htmlspecialchars($row['query'])) . '</td>
+                                                       <td>' . htmlspecialchars($row['exec_time']) . '</td>
+                                                       <td>' . htmlspecialchars($row['table_join']) . '</td>
+                                                       <td>' . htmlspecialchars($row['script']) . '</td>
+                                                       <td>' . htmlspecialchars($row['query']) . '</td>
                                                </tr>';
                        }
                        // Printing direct joins:
@@ -382,8 +382,8 @@ updateQryForm(\'') . $input['QUERY']) . '\');
                        }
                        $outStr .= '<h4>Total dependencies:</h4>' . \TYPO3\CMS\Core\Utility\DebugUtility::viewArray($tableIndex);
                        // Printing data rows:
-                       $outStr .= ('
-                                       <table border="1" cellspacing="0">' . implode('', $tRows)) . '
+                       $outStr .= '
+                                       <table border="1" cellspacing="0">' . implode('', $tRows) . '
                                        </table>';
                        break;
                case 'errors':
@@ -399,17 +399,17 @@ updateQryForm(\'') . $input['QUERY']) . '\');
                                        </tr>';
                        while ($row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res)) {
                                // Create output row
-                               $tRows[] = ((((((('
+                               $tRows[] = '
                                                <tr>
-                                                       <td>' . htmlspecialchars($row['exec_time'])) . '</td>
-                                                       <td>') . \TYPO3\CMS\Core\Utility\DebugUtility::viewArray(unserialize($row['serdata']))) . '</td>
-                                                       <td>') . htmlspecialchars($row['script'])) . '</td>
-                                                       <td>') . htmlspecialchars($row['query'])) . '</td>
+                                                       <td>' . htmlspecialchars($row['exec_time']) . '</td>
+                                                       <td>' . \TYPO3\CMS\Core\Utility\DebugUtility::viewArray(unserialize($row['serdata'])) . '</td>
+                                                       <td>' . htmlspecialchars($row['script']) . '</td>
+                                                       <td>' . htmlspecialchars($row['query']) . '</td>
                                                </tr>';
                        }
                        // Printing data rows:
-                       $outStr .= ('
-                                       <table border="1" cellspacing="0">' . implode('', $tRows)) . '
+                       $outStr .= '
+                                       <table border="1" cellspacing="0">' . implode('', $tRows) . '
                                        </table>';
                        break;
                case 'parsing':
@@ -417,14 +417,14 @@ updateQryForm(\'') . $input['QUERY']) . '\');
                        $tRows = array();
                        while ($row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res)) {
                                // Create output row
-                               $tRows[] = ('
+                               $tRows[] = '
                                                <tr>
-                                                       <td>' . htmlspecialchars($row['query'])) . '</td>
+                                                       <td>' . htmlspecialchars($row['query']) . '</td>
                                                </tr>';
                        }
                        // Printing data rows:
-                       $outStr .= ('
-                                       <table border="1" cellspacing="0">' . implode('', $tRows)) . '
+                       $outStr .= '
+                                       <table border="1" cellspacing="0">' . implode('', $tRows) . '
                                        </table>';
                        break;
                case 'where':
@@ -438,16 +438,16 @@ updateQryForm(\'') . $input['QUERY']) . '\');
                                                <td>WHERE clause</td>
                                        </tr>';
                        while ($row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res)) {
-                               $tRows[] = ((((((('
+                               $tRows[] = '
                                                <tr>
-                                                       <td>' . \TYPO3\CMS\Backend\Utility\BackendUtility::datetime($row['tstamp'])) . '</td>
-                                                       <td>') . htmlspecialchars($row['script'])) . '</td>
-                                                       <td>') . htmlspecialchars($row['tablename'])) . '</td>
-                                                               <td>') . str_replace(array('\'\'', '""', 'IS NULL', 'IS NOT NULL'), array('<span style="background-color:#ff0000;color:#ffffff;padding:2px;font-weight:bold;">\'\'</span>', '<span style="background-color:#ff0000;color:#ffffff;padding:2px;font-weight:bold;">""</span>', '<span style="background-color:#00ff00;color:#ffffff;padding:2px;font-weight:bold;">IS NULL</span>', '<span style="background-color:#00ff00;color:#ffffff;padding:2px;font-weight:bold;">IS NOT NULL</span>'), htmlspecialchars($row['whereclause']))) . '</td>
+                                                       <td>' . \TYPO3\CMS\Backend\Utility\BackendUtility::datetime($row['tstamp']) . '</td>
+                                                       <td>' . htmlspecialchars($row['script']) . '</td>
+                                                       <td>' . htmlspecialchars($row['tablename']) . '</td>
+                                                               <td>' . str_replace(array('\'\'', '""', 'IS NULL', 'IS NOT NULL'), array('<span style="background-color:#ff0000;color:#ffffff;padding:2px;font-weight:bold;">\'\'</span>', '<span style="background-color:#ff0000;color:#ffffff;padding:2px;font-weight:bold;">""</span>', '<span style="background-color:#00ff00;color:#ffffff;padding:2px;font-weight:bold;">IS NULL</span>', '<span style="background-color:#00ff00;color:#ffffff;padding:2px;font-weight:bold;">IS NOT NULL</span>'), htmlspecialchars($row['whereclause'])) . '</td>
                                                </tr>';
                        }
-                       $outStr = ('
-                                       <table border="1" cellspacing="0">' . implode('', $tRows)) . '
+                       $outStr = '
+                                       <table border="1" cellspacing="0">' . implode('', $tRows) . '
                                        </table>';
                        break;
                default:
@@ -465,13 +465,13 @@ updateQryForm(\'') . $input['QUERY']) . '\');
                                                        <td>Query</td>
                                                </tr>';
                                while ($row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res)) {
-                                       $tRows[] = ((((((((('
+                                       $tRows[] = '
                                                        <tr>
-                                                               <td>' . htmlspecialchars($row['exec_time'])) . '</td>
-                                                               <td>') . ($row['errorFlag'] ? 1 : 0)) . '</td>
-                                                               <td>') . htmlspecialchars($row['table_join'])) . '</td>
-                                                               <td>') . \TYPO3\CMS\Core\Utility\DebugUtility::viewArray(unserialize($row['serdata']))) . '</td>
-                                                               <td>') . str_replace(array('\'\'', '""', 'IS NULL', 'IS NOT NULL'), array('<span style="background-color:#ff0000;color:#ffffff;padding:2px;font-weight:bold;">\'\'</span>', '<span style="background-color:#ff0000;color:#ffffff;padding:2px;font-weight:bold;">""</span>', '<span style="background-color:#00ff00;color:#ffffff;padding:2px;font-weight:bold;">IS NULL</span>', '<span style="background-color:#00ff00;color:#ffffff;padding:2px;font-weight:bold;">IS NOT NULL</span>'), htmlspecialchars($row['query']))) . '</td>
+                                                               <td>' . htmlspecialchars($row['exec_time']) . '</td>
+                                                               <td>' . ($row['errorFlag'] ? 1 : 0) . '</td>
+                                                               <td>' . htmlspecialchars($row['table_join']) . '</td>
+                                                               <td>' . \TYPO3\CMS\Core\Utility\DebugUtility::viewArray(unserialize($row['serdata'])) . '</td>
+                                                               <td>' . str_replace(array('\'\'', '""', 'IS NULL', 'IS NOT NULL'), array('<span style="background-color:#ff0000;color:#ffffff;padding:2px;font-weight:bold;">\'\'</span>', '<span style="background-color:#ff0000;color:#ffffff;padding:2px;font-weight:bold;">""</span>', '<span style="background-color:#00ff00;color:#ffffff;padding:2px;font-weight:bold;">IS NULL</span>', '<span style="background-color:#00ff00;color:#ffffff;padding:2px;font-weight:bold;">IS NOT NULL</span>'), htmlspecialchars($row['query'])) . '</td>
                                                        </tr>';
                                }
                        } else {
@@ -486,30 +486,30 @@ updateQryForm(\'') . $input['QUERY']) . '\');
                                                        <td>Script</td>
                                                </tr>';
                                while ($row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res)) {
-                                       $tRows[] = ((((((((((((('
+                                       $tRows[] = '
                                                        <tr>
-                                                               <td>' . \TYPO3\CMS\Backend\Utility\BackendUtility::datetime($row['tstamp'])) . '</td>
-                                                               <td>') . htmlspecialchars($row['qrycount'])) . '</td>
-                                                               <td>') . ($row['error'] ? '<strong style="color:#f00">ERR</strong>' : '')) . '</td>
-                                                               <td>') . htmlspecialchars($row['calc_sum'])) . '</td>
-                                                               <td><a href="') . $this->thisScript) . '&amp;specTime=') . intval($row['tstamp'])) . '">') . htmlspecialchars($row['script'])) . '</a></td>
+                                                               <td>' . \TYPO3\CMS\Backend\Utility\BackendUtility::datetime($row['tstamp']) . '</td>
+                                                               <td>' . htmlspecialchars($row['qrycount']) . '</td>
+                                                               <td>' . ($row['error'] ? '<strong style="color:#f00">ERR</strong>' : '') . '</td>
+                                                               <td>' . htmlspecialchars($row['calc_sum']) . '</td>
+                                                               <td><a href="' . $this->thisScript . '&amp;specTime=' . intval($row['tstamp']) . '">' . htmlspecialchars($row['script']) . '</a></td>
                                                        </tr>';
                                }
                        }
-                       $outStr = ('
-                                       <table border="1" cellspacing="0">' . implode('', $tRows)) . '
+                       $outStr = '
+                                       <table border="1" cellspacing="0">' . implode('', $tRows) . '
                                        </table>';
                        break;
                }
-               $menu = ((((((((((((('
-                                       <a href="' . $this->thisScript) . '&amp;cmd=flush">FLUSH LOG</a> -
-                                       <a href="') . $this->thisScript) . '&amp;cmd=joins">JOINS</a> -
-                                       <a href="') . $this->thisScript) . '&amp;cmd=errors">ERRORS</a> -
-                                       <a href="') . $this->thisScript) . '&amp;cmd=parsing">PARSING</a> -
-                                       <a href="') . $this->thisScript) . '">LOG</a> -
-                                       <a href="') . $this->thisScript) . '&amp;cmd=where">WHERE</a> -
+               $menu = '
+                                       <a href="' . $this->thisScript . '&amp;cmd=flush">FLUSH LOG</a> -
+                                       <a href="' . $this->thisScript . '&amp;cmd=joins">JOINS</a> -
+                                       <a href="' . $this->thisScript . '&amp;cmd=errors">ERRORS</a> -
+                                       <a href="' . $this->thisScript . '&amp;cmd=parsing">PARSING</a> -
+                                       <a href="' . $this->thisScript . '">LOG</a> -
+                                       <a href="' . $this->thisScript . '&amp;cmd=where">WHERE</a> -
 
-                                       <a href="') . htmlspecialchars(\TYPO3\CMS\Core\Utility\GeneralUtility::linkThisScript())) . '" target="tx_debuglog">[New window]</a>
+                                       <a href="' . htmlspecialchars(\TYPO3\CMS\Core\Utility\GeneralUtility::linkThisScript()) . '" target="tx_debuglog">[New window]</a>
                                        <hr />
                ';
                return $menu . $outStr;
index 63055a9..3147f89 100644 (file)
@@ -361,7 +361,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                        if (isset($fdef['featureIndex']['DEFAULT'])) {
                                                $default = $fdef['featureIndex']['DEFAULT']['value'][0];
                                                if (isset($fdef['featureIndex']['DEFAULT']['value'][1])) {
-                                                       $default = ($fdef['featureIndex']['DEFAULT']['value'][1] . $default) . $fdef['featureIndex']['DEFAULT']['value'][1];
+                                                       $default = $fdef['featureIndex']['DEFAULT']['value'][1] . $default . $fdef['featureIndex']['DEFAULT']['value'][1];
                                                }
                                                $this->cache_fieldType[$table][$field]['default'] = $default;
                                        }
@@ -456,9 +456,9 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                        } else {
                                $sqlResult = mysql_query($this->lastQuery[0], $this->handlerInstance[$this->lastHandlerKey]['link']);
                                $new_id = $this->sql_insert_id();
-                               $where = ($this->cache_autoIncFields[$table] . '=') . $new_id;
+                               $where = $this->cache_autoIncFields[$table] . '=' . $new_id;
                                foreach ($this->lastQuery[1] as $field => $content) {
-                                       mysql_query((((((('UPDATE ' . $this->quoteFromTables($table)) . ' SET ') . $this->quoteFromTables($field)) . '=') . $this->fullQuoteStr($content, $table)) . ' WHERE ') . $this->quoteWhereClause($where), $this->handlerInstance[$this->lastHandlerKey]['link']);
+                                       mysql_query('UPDATE ' . $this->quoteFromTables($table) . ' SET ' . $this->quoteFromTables($field) . '=' . $this->fullQuoteStr($content, $table) . ' WHERE ' . $this->quoteWhereClause($where), $this->handlerInstance[$this->lastHandlerKey]['link']);
                                }
                        }
                        break;
@@ -472,7 +472,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                                $this->handlerInstance[$this->lastHandlerKey]->last_insert_id = $new_id;
                                        }
                                } else {
-                                       $new_id = $this->handlerInstance[$this->lastHandlerKey]->GenID(($table . '_') . $this->cache_autoIncFields[$table], $this->handlerInstance[$this->lastHandlerKey]->sequenceStart);
+                                       $new_id = $this->handlerInstance[$this->lastHandlerKey]->GenID($table . '_' . $this->cache_autoIncFields[$table], $this->handlerInstance[$this->lastHandlerKey]->sequenceStart);
                                        $fields_values[$this->cache_autoIncFields[$table]] = $new_id;
                                        if ($table != 'tx_dbal_debuglog') {
                                                $this->handlerInstance[$this->lastHandlerKey]->last_insert_id = $new_id;
@@ -493,13 +493,13 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                                        continue;
                                                }
                                                if (isset($this->cache_autoIncFields[$table]) && isset($new_id)) {
-                                                       $this->handlerInstance[$this->lastHandlerKey]->UpdateBlob($this->quoteFromTables($table), $field, $content, $this->quoteWhereClause(($this->cache_autoIncFields[$table] . '=') . $new_id));
+                                                       $this->handlerInstance[$this->lastHandlerKey]->UpdateBlob($this->quoteFromTables($table), $field, $content, $this->quoteWhereClause($this->cache_autoIncFields[$table] . '=' . $new_id));
                                                } elseif (isset($this->cache_primaryKeys[$table])) {
                                                        $where = '';
                                                        $pks = explode(',', $this->cache_primaryKeys[$table]);
                                                        foreach ($pks as $pk) {
                                                                if (isset($fields_values[$pk])) {
-                                                                       $where .= (($pk . '=') . $this->fullQuoteStr($fields_values[$pk], $table)) . ' AND ';
+                                                                       $where .= $pk . '=' . $this->fullQuoteStr($fields_values[$pk], $table) . ' AND ';
                                                                }
                                                        }
                                                        $where = $this->quoteWhereClause($where . '1=1');
@@ -517,13 +517,13 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                                        continue;
                                                }
                                                if (isset($this->cache_autoIncFields[$table]) && isset($new_id)) {
-                                                       $this->handlerInstance[$this->lastHandlerKey]->UpdateClob($this->quoteFromTables($table), $field, $content, $this->quoteWhereClause(($this->cache_autoIncFields[$table] . '=') . $new_id));
+                                                       $this->handlerInstance[$this->lastHandlerKey]->UpdateClob($this->quoteFromTables($table), $field, $content, $this->quoteWhereClause($this->cache_autoIncFields[$table] . '=' . $new_id));
                                                } elseif (isset($this->cache_primaryKeys[$table])) {
                                                        $where = '';
                                                        $pks = explode(',', $this->cache_primaryKeys[$table]);
                                                        foreach ($pks as $pk) {
                                                                if (isset($fields_values[$pk])) {
-                                                                       $where .= (($pk . '=') . $this->fullQuoteStr($fields_values[$pk], $table)) . ' AND ';
+                                                                       $where .= $pk . '=' . $this->fullQuoteStr($fields_values[$pk], $table) . ' AND ';
                                                                }
                                                        }
                                                        $where = $this->quoteWhereClause($where . '1=1');
@@ -624,7 +624,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                        } else {
                                $sqlResult = mysql_query($this->lastQuery[0], $this->handlerInstance[$this->lastHandlerKey]['link']);
                                foreach ($this->lastQuery[1] as $field => $content) {
-                                       mysql_query((((((('UPDATE ' . $this->quoteFromTables($table)) . ' SET ') . $this->quoteFromTables($field)) . '=') . $this->fullQuoteStr($content, $table)) . ' WHERE ') . $this->quoteWhereClause($where), $this->handlerInstance[$this->lastHandlerKey]['link']);
+                                       mysql_query('UPDATE ' . $this->quoteFromTables($table) . ' SET ' . $this->quoteFromTables($field) . '=' . $this->fullQuoteStr($content, $table) . ' WHERE ' . $this->quoteWhereClause($where), $this->handlerInstance[$this->lastHandlerKey]['link']);
                                }
                        }
                        break;
@@ -981,13 +981,13 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                        }
                        if (count($blobfields) || count($clobfields)) {
                                if (count($nArr)) {
-                                       $query[0] = ((((('INSERT INTO ' . $this->quoteFromTables($table)) . '
+                                       $query[0] = 'INSERT INTO ' . $this->quoteFromTables($table) . '
                                        (
-                                               ') . implode(',
-                                               ', array_keys($nArr))) . '
+                                               ' . implode(',
+                                               ', array_keys($nArr)) . '
                                        ) VALUES (
-                                               ') . implode(',
-                                               ', $nArr)) . '
+                                               ' . implode(',
+                                               ', $nArr) . '
                                        )';
                                }
                                if (count($blobfields)) {
@@ -1000,13 +1000,13 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                        $this->debug_lastBuiltQuery = $query[0];
                                }
                        } else {
-                               $query = ((((('INSERT INTO ' . $this->quoteFromTables($table)) . '
+                               $query = 'INSERT INTO ' . $this->quoteFromTables($table) . '
                                (
-                                       ') . implode(',
-                                       ', array_keys($nArr))) . '
+                                       ' . implode(',
+                                       ', array_keys($nArr)) . '
                                ) VALUES (
-                                       ') . implode(',
-                                       ', $nArr)) . '
+                                       ' . implode(',
+                                       ', $nArr) . '
                                )';
                                if ($this->debugOutput || $this->store_lastBuiltQuery) {
                                        $this->debug_lastBuiltQuery = $query;
@@ -1090,16 +1090,16 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                                } elseif ($mt[0] == 'F') {
                                                        $v = (double) $v;
                                                }
-                                               $nArr[] = ($this->quoteFieldNames($k) . '=') . (!in_array($k, $no_quote_fields) ? $this->fullQuoteStr($v, $table) : $v);
+                                               $nArr[] = $this->quoteFieldNames($k) . '=' . (!in_array($k, $no_quote_fields) ? $this->fullQuoteStr($v, $table) : $v);
                                        }
                                }
                        }
                        if (count($blobfields) || count($clobfields)) {
                                if (count($nArr)) {
-                                       $query[0] = ((('UPDATE ' . $this->quoteFromTables($table)) . '
+                                       $query[0] = 'UPDATE ' . $this->quoteFromTables($table) . '
                                                SET
-                                                       ') . implode(',
-                                                       ', $nArr)) . (strlen($where) > 0 ? '
+                                                       ' . implode(',
+                                                       ', $nArr) . (strlen($where) > 0 ? '
                                                WHERE
                                                        ' . $this->quoteWhereClause($where) : '');
                                }
@@ -1113,10 +1113,10 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                        $this->debug_lastBuiltQuery = $query[0];
                                }
                        } else {
-                               $query = ((('UPDATE ' . $this->quoteFromTables($table)) . '
+                               $query = 'UPDATE ' . $this->quoteFromTables($table) . '
                                        SET
-                                               ') . implode(',
-                                               ', $nArr)) . (strlen($where) > 0 ? '
+                                               ' . implode(',
+                                               ', $nArr) . (strlen($where) > 0 ? '
                                        WHERE
                                                ' . $this->quoteWhereClause($where) : '');
                                if ($this->debugOutput || $this->store_lastBuiltQuery) {
@@ -1144,7 +1144,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                        }
                        $table = $this->quoteFromTables($table);
                        $where = $this->quoteWhereClause($where);
-                       $query = ('DELETE FROM ' . $table) . (strlen($where) > 0 ? ' WHERE ' . $where : '');
+                       $query = 'DELETE FROM ' . $table . (strlen($where) > 0 ? ' WHERE ' . $where : '');
                        if ($this->debugOutput || $this->store_lastBuiltQuery) {
                                $this->debug_lastBuiltQuery = $query;
                        }
@@ -1177,7 +1177,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                // If there are two parameters, do mapping differently than otherwise:
                                $numrows = $splitLimit[1];
                                $offset = $splitLimit[0];
-                               $limit = ($numrows . ' OFFSET ') . $offset;
+                               $limit = $numrows . ' OFFSET ' . $offset;
                        }
                }
                $select_fields = $this->quoteFieldNames($select_fields);
@@ -1389,7 +1389,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                // If there are two parameters, do mapping differently than otherwise:
                                $numrows = $splitLimit[1];
                                $offset = $splitLimit[0];
-                               $limit = ($numrows . ' OFFSET ') . $offset;
+                               $limit = $numrows . ' OFFSET ' . $offset;
                        }
                }
                $queryComponents['LIMIT'] = $limit;
@@ -1415,14 +1415,14 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
         * @return array Precompiled SQL statement
         */
        protected function precompileSELECTquery(array $components) {
-               $parameterWrap = ('__' . dechex(time())) . '__';
+               $parameterWrap = '__' . dechex(time()) . '__';
                foreach ($components['parameters'] as $key => $params) {
                        if ($key === '?') {
                                foreach ($params as $index => $param) {
-                                       $components['parameters'][$key][$index][0] = ($parameterWrap . $param[0]) . $parameterWrap;
+                                       $components['parameters'][$key][$index][0] = $parameterWrap . $param[0] . $parameterWrap;
                                }
                        } else {
-                               $components['parameters'][$key][0] = ($parameterWrap . $params[0]) . $parameterWrap;
+                               $components['parameters'][$key][0] = $parameterWrap . $params[0] . $parameterWrap;
                        }
                }
                $select_fields = $this->SQLparser->compileFieldList($components['SELECT']);
@@ -1489,7 +1489,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                        // If there are two parameters, do mapping differently than otherwise:
                                        $numrows = $splitLimit[1];
                                        $offset = $splitLimit[0];
-                                       $limit = ($numrows . ' OFFSET ') . $offset;
+                                       $limit = $numrows . ' OFFSET ' . $offset;
                                }
                        }
                        if ($limit != '') {
@@ -1589,7 +1589,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
         */
        protected function _quoteFieldNames(array $select_fields) {
                foreach ($select_fields as $k => $v) {
-                       if (($select_fields[$k]['field'] != '' && $select_fields[$k]['field'] != '*') && !is_numeric($select_fields[$k]['field'])) {
+                       if ($select_fields[$k]['field'] != '' && $select_fields[$k]['field'] != '*' && !is_numeric($select_fields[$k]['field'])) {
                                $select_fields[$k]['field'] = $this->quoteName($select_fields[$k]['field']);
                        }
                        if ($select_fields[$k]['table'] != '' && !is_numeric($select_fields[$k]['table'])) {
@@ -1750,7 +1750,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                                        $where_clause[$k]['subquery'] = $this->quoteSELECTsubquery($v['subquery']);
                                                }
                                        } else {
-                                               if (((!isset($where_clause[$k]['value'][1]) || $where_clause[$k]['value'][1] == '') && is_string($where_clause[$k]['value'][0])) && strstr($where_clause[$k]['value'][0], '.')) {
+                                               if ((!isset($where_clause[$k]['value'][1]) || $where_clause[$k]['value'][1] == '') && is_string($where_clause[$k]['value'][0]) && strstr($where_clause[$k]['value'][0], '.')) {
                                                        $where_clause[$k]['value'][0] = $this->quoteFieldNames($where_clause[$k]['value'][0]);
                                                }
                                        }
@@ -1824,7 +1824,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
         */
        protected function _quoteOrderBy(array $orderBy) {
                foreach ($orderBy as $k => $v) {
-                       if (($orderBy[$k]['table'] === '' && $v['field'] !== '') && ctype_digit($v['field'])) {
+                       if ($orderBy[$k]['table'] === '' && $v['field'] !== '' && ctype_digit($v['field'])) {
                                continue;
                        }
                        $orderBy[$k]['field'] = $this->quoteName($orderBy[$k]['field']);
@@ -1849,7 +1849,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
         * @see quoteStr()
         */
        public function fullQuoteStr($str, $table) {
-               return ('\'' . $this->quoteStr($str, $table)) . '\'';
+               return '\'' . $this->quoteStr($str, $table) . '\'';
        }
 
        /**
@@ -1904,7 +1904,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                        return $this->handlerInstance[$handlerKey]->DataDictionary->NameQuote($name);
                } else {
                        $quote = $useBackticks ? '`' : $this->handlerInstance[$handlerKey]->nameQuote;
-                       return ($quote . $name) . $quote;
+                       return $quote . $name . $quote;
                }
        }
 
@@ -2532,7 +2532,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                // Getting real list of tables:
                switch ($this->handlerCfg['_DEFAULT']['type']) {
                case 'native':
-                       $tables_result = mysql_query(('SHOW TABLE STATUS FROM `' . TYPO3_db) . '`', $this->handlerInstance['_DEFAULT']['link']);
+                       $tables_result = mysql_query('SHOW TABLE STATUS FROM `' . TYPO3_db . '`', $this->handlerInstance['_DEFAULT']['link']);
                        if (!$this->sql_error()) {
                                while ($theTable = $this->sql_fetch_assoc($tables_result)) {
                                        $whichTables[$theTable['Name']] = $theTable;
@@ -2619,7 +2619,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                        settype($fieldRow, 'array');
                                        $fieldRow['Field'] = $fieldRow['name'];
                                        $ntype = $this->MySQLActualType($this->MetaType($fieldRow['type'], $tableName));
-                                       $ntype .= $fieldRow['max_length'] != -1 ? ($ntype == 'INT' ? '(11)' : ('(' . $fieldRow['max_length']) . ')') : '';
+                                       $ntype .= $fieldRow['max_length'] != -1 ? ($ntype == 'INT' ? '(11)' : '(' . $fieldRow['max_length'] . ')') : '';
                                        $fieldRow['Type'] = strtolower($ntype);
                                        $fieldRow['Null'] = '';
                                        $fieldRow['Key'] = '';
@@ -2782,7 +2782,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                throw new \InvalidArgumentException('Creating a database with DBAL is not supported. Did you really read the manual?', 1310027716);
                                break;
                        default:
-                               throw new \InvalidArgumentException(((('ERROR: Invalid Query type (' . $parsedQuery['type']) . ') for ->admin_query() function!: "') . htmlspecialchars($query)) . '"', 1310027740);
+                               throw new \InvalidArgumentException('ERROR: Invalid Query type (' . $parsedQuery['type'] . ') for ->admin_query() function!: "' . htmlspecialchars($query) . '"', 1310027740);
                                break;
                        }
                        // Setting query array (for other applications to access if needed)
@@ -2816,7 +2816,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                break;
                        }
                } else {
-                       throw new \InvalidArgumentException(((('ERROR: Query could not be parsed: "' . htmlspecialchars($parsedQuery)) . '". Query: "') . htmlspecialchars($query)) . '"', 1310027793);
+                       throw new \InvalidArgumentException('ERROR: Query could not be parsed: "' . htmlspecialchars($parsedQuery) . '". Query: "' . htmlspecialchars($query) . '"', 1310027793);
                }
        }
 
@@ -2846,7 +2846,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                        $handlerKey = $this->table2handlerKeys[$vArray['table']] ? $this->table2handlerKeys[$vArray['table']] : '_DEFAULT';
                                        // In case of separate handler keys for joined tables:
                                        if ($outputHandlerKey && $handlerKey != $outputHandlerKey) {
-                                               throw new \RuntimeException(('DBAL fatal error: Tables in this list "' . $tableList) . '" didn\'t use the same DB handler!', 1310027833);
+                                               throw new \RuntimeException('DBAL fatal error: Tables in this list "' . $tableList . '" didn\'t use the same DB handler!', 1310027833);
                                        }
                                        $outputHandlerKey = $handlerKey;
                                }
@@ -2857,7 +2857,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                // Return handler key:
                                $this->cache_handlerKeyFromTableList[$key] = $outputHandlerKey;
                        } else {
-                               throw new \RuntimeException(((('DBAL fatal error: No handler found in handler_getFromTableList() for: "' . $tableList) . '" (') . $tableArray) . ')', 1310027933);
+                               throw new \RuntimeException('DBAL fatal error: No handler found in handler_getFromTableList() for: "' . $tableList . '" (' . $tableArray . ')', 1310027933);
                        }
                }
                return $this->cache_handlerKeyFromTableList[$key];
@@ -2902,11 +2902,11 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                        $setDBinit = \TYPO3\CMS\Core\Utility\GeneralUtility::trimExplode(LF, str_replace('\' . LF . \'', LF, $GLOBALS['TYPO3_CONF_VARS']['SYS']['setDBinit']), TRUE);
                                        foreach ($setDBinit as $v) {
                                                if (mysql_query($v, $link) === FALSE) {
-                                                       \TYPO3\CMS\Core\Utility\GeneralUtility::sysLog(('Could not initialize DB connection with query "' . $v) . '".', 'Core', 3);
+                                                       \TYPO3\CMS\Core\Utility\GeneralUtility::sysLog('Could not initialize DB connection with query "' . $v . '".', 'Core', 3);
                                                }
                                        }
                                } else {
-                                       \TYPO3\CMS\Core\Utility\GeneralUtility::sysLog(((('Could not connect to MySQL server ' . $cfgArray['config']['host']) . ' with user ') . $cfgArray['config']['username']) . '.', 'Core', 4);
+                                       \TYPO3\CMS\Core\Utility\GeneralUtility::sysLog('Could not connect to MySQL server ' . $cfgArray['config']['host'] . ' with user ' . $cfgArray['config']['username'] . '.', 'Core', 4);
                                }
                                break;
                        case 'adodb':
@@ -2935,9 +2935,9 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                        $this->handlerInstance[$handlerKey]->PConnect($cfgArray['config']['host'] . (isset($cfgArray['config']['port']) ? ':' . $cfgArray['config']['port'] : ''), $cfgArray['config']['username'], $cfgArray['config']['password'], $cfgArray['config']['database']);
                                }
                                if (!$this->handlerInstance[$handlerKey]->isConnected()) {
-                                       $dsn = ((((((($cfgArray['config']['driver'] . '://') . $cfgArray['config']['username']) . (strlen($cfgArray['config']['password']) ? ':XXXX@' : '')) . $cfgArray['config']['host']) . (isset($cfgArray['config']['port']) ? ':' . $cfgArray['config']['port'] : '')) . '/') . $cfgArray['config']['database']) . ($GLOBALS['TYPO3_CONF_VARS']['SYS']['no_pconnect'] ? '' : '?persistent=1');
-                                       \TYPO3\CMS\Core\Utility\GeneralUtility::sysLog(((('Could not connect to DB server using ADOdb on ' . $cfgArray['config']['host']) . ' with user ') . $cfgArray['config']['username']) . '.', 'Core', 4);
-                                       error_log(('DBAL error: Connection to ' . $dsn) . ' failed. Maybe PHP doesn\'t support the database?');
+                                       $dsn = $cfgArray['config']['driver'] . '://' . $cfgArray['config']['username'] . (strlen($cfgArray['config']['password']) ? ':XXXX@' : '') . $cfgArray['config']['host'] . (isset($cfgArray['config']['port']) ? ':' . $cfgArray['config']['port'] : '') . '/' . $cfgArray['config']['database'] . ($GLOBALS['TYPO3_CONF_VARS']['SYS']['no_pconnect'] ? '' : '?persistent=1');
+                                       \TYPO3\CMS\Core\Utility\GeneralUtility::sysLog('Could not connect to DB server using ADOdb on ' . $cfgArray['config']['host'] . ' with user ' . $cfgArray['config']['username'] . '.', 'Core', 4);
+                                       error_log('DBAL error: Connection to ' . $dsn . ' failed. Maybe PHP doesn\'t support the database?');
                                        $output = FALSE;
                                } else {
                                        $this->handlerInstance[$handlerKey]->DataDictionary = NewDataDictionary($this->handlerInstance[$handlerKey]);
@@ -2955,7 +2955,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                if (@is_file($fileName)) {
                                        require_once $fileName;
                                } else {
-                                       throw new \RuntimeException(('DBAL error: "' . $fileName) . '" was not a file to include.', 1310027975);
+                                       throw new \RuntimeException('DBAL error: "' . $fileName . '" was not a file to include.', 1310027975);
                                }
                                // Initialize:
                                $this->handlerInstance[$handlerKey] = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance($cfgArray['config']['class']);
@@ -2965,12 +2965,12 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                }
                                break;
                        default:
-                               throw new \RuntimeException(('ERROR: Invalid handler type: "' . $cfgArray['type']) . '"', 1310027995);
+                               throw new \RuntimeException('ERROR: Invalid handler type: "' . $cfgArray['type'] . '"', 1310027995);
                                break;
                        }
                        return $output;
                } else {
-                       throw new \RuntimeException(('ERROR: No handler for key "' . $handlerKey) . '"', 1310028018);
+                       throw new \RuntimeException('ERROR: No handler for key "' . $handlerKey . '"', 1310028018);
                }
        }
 
@@ -3027,7 +3027,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
         * @return      mixed           Returns an array of table names (parsed version of input table) if mapping is needed, otherwise just FALSE.
         */
        protected function map_needMapping($tableList, $fieldMappingOnly = FALSE, array &$parsedTableList = array()) {
-               $key = ($tableList . '|') . $fieldMappingOnly;
+               $key = $tableList . '|' . $fieldMappingOnly;
                if (!isset($this->cache_mappingFromTableList[$key])) {
                        $this->cache_mappingFromTableList[$key] = FALSE;
                        // Default:
@@ -3250,7 +3250,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                        case 'CASE':
                                                if (isset($sqlPartArray[$k]['case_field'])) {
                                                        $fieldArray = explode('.', $sqlPartArray[$k]['case_field']);
-                                                       if ((count($fieldArray) == 1 && is_array($this->mapping[$defaultTableKey]['mapFieldNames'])) && isset($this->mapping[$defaultTableKey]['mapFieldNames'][$fieldArray[0]])) {
+                                                       if (count($fieldArray) == 1 && is_array($this->mapping[$defaultTableKey]['mapFieldNames']) && isset($this->mapping[$defaultTableKey]['mapFieldNames'][$fieldArray[0]])) {
                                                                $sqlPartArray[$k]['case_field'] = $this->mapping[$defaultTableKey]['mapFieldNames'][$fieldArray[0]];
                                                        } elseif (count($fieldArray) == 2) {
                                                                // Map the external table
@@ -3264,7 +3264,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                                                if (is_array($this->mapping[$tableKey]['mapFieldNames']) && isset($this->mapping[$tableKey]['mapFieldNames'][$fieldArray[1]])) {
                                                                        $field = $this->mapping[$tableKey]['mapFieldNames'][$fieldArray[1]];
                                                                }
-                                                               $sqlPartArray[$k]['case_field'] = ($table . '.') . $field;
+                                                               $sqlPartArray[$k]['case_field'] = $table . '.' . $field;
                                                        }
                                                }
                                                foreach ($sqlPartArray[$k]['when'] as $key => $when) {
@@ -3308,7 +3308,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                        // Mapping field name in SQL-functions like MIN(), MAX() or SUM()
                                        if ($this->mapping[$t]['mapFieldNames']) {
                                                $fieldArray = explode('.', $sqlPartArray[$k]['func_content']);
-                                               if ((count($fieldArray) == 1 && is_array($this->mapping[$t]['mapFieldNames'])) && isset($this->mapping[$t]['mapFieldNames'][$fieldArray[0]])) {
+                                               if (count($fieldArray) == 1 && is_array($this->mapping[$t]['mapFieldNames']) && isset($this->mapping[$t]['mapFieldNames'][$fieldArray[0]])) {
                                                        $sqlPartArray[$k]['func_content.'][0]['func_content'] = $this->mapping[$t]['mapFieldNames'][$fieldArray[0]];
                                                        $sqlPartArray[$k]['func_content'] = $this->mapping[$t]['mapFieldNames'][$fieldArray[0]];
                                                } elseif (count($fieldArray) == 2) {
@@ -3323,8 +3323,8 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                                        if (is_array($this->mapping[$tableKey]['mapFieldNames']) && isset($this->mapping[$tableKey]['mapFieldNames'][$fieldArray[1]])) {
                                                                $field = $this->mapping[$tableKey]['mapFieldNames'][$fieldArray[1]];
                                                        }
-                                                       $sqlPartArray[$k]['func_content.'][0]['func_content'] = ($table . '.') . $field;
-                                                       $sqlPartArray[$k]['func_content'] = ($table . '.') . $field;
+                                                       $sqlPartArray[$k]['func_content.'][0]['func_content'] = $table . '.' . $field;
+                                                       $sqlPartArray[$k]['func_content'] = $table . '.' . $field;
                                                }
                                                // Mapping flow-control statements
                                                if (isset($sqlPartArray[$k]['flow-control'])) {
@@ -3356,7 +3356,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                        // this is a very simplistic check, beware
                                        if (!is_numeric($sqlPartArray[$k]['value'][0]) && !isset($sqlPartArray[$k]['value'][1])) {
                                                $fieldArray = explode('.', $sqlPartArray[$k]['value'][0]);
-                                               if ((count($fieldArray) == 1 && is_array($this->mapping[$t]['mapFieldNames'])) && isset($this->mapping[$t]['mapFieldNames'][$fieldArray[0]])) {
+                                               if (count($fieldArray) == 1 && is_array($this->mapping[$t]['mapFieldNames']) && isset($this->mapping[$t]['mapFieldNames'][$fieldArray[0]])) {
                                                        $sqlPartArray[$k]['value'][0] = $this->mapping[$t]['mapFieldNames'][$fieldArray[0]];
                                                } elseif (count($fieldArray) == 2) {
                                                        // Map the external table
@@ -3370,7 +3370,7 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                                                        if (is_array($this->mapping[$tableKey]['mapFieldNames']) && isset($this->mapping[$tableKey]['mapFieldNames'][$fieldArray[1]])) {
                                                                $field = $this->mapping[$tableKey]['mapFieldNames'][$fieldArray[1]];
                                                        }
-                                                       $sqlPartArray[$k]['value'][0] = ($table . '.') . $field;
+                                                       $sqlPartArray[$k]['value'][0] = $table . '.' . $field;
                                                }
                                        }
                                        // Map table?
@@ -3630,10 +3630,10 @@ class DatabaseConnection extends \TYPO3\CMS\Core\Database\DatabaseConnection {
                if (is_array($query)) {
                        $queryToLog = $query[0] . ' --  ';
                        if (count($query[1])) {
-                               $queryToLog .= (count($query[1]) . ' BLOB FIELDS: ') . implode(', ', array_keys($query[1]));
+                               $queryToLog .= count($query[1]) . ' BLOB FIELDS: ' . implode(', ', array_keys($query[1]));
                        }
                        if (count($query[2])) {
-                               $queryToLog .= (count($query[2]) . ' CLOB FIELDS: ') . implode(', ', array_keys($query[2]));
+                               $queryToLog .= count($query[2]) . ' CLOB FIELDS: ' . implode(', ', array_keys($query[2]));
                        }
                } else {
                        $queryToLog = $query;
index 2d2e7e4..3dec038 100644 (file)
@@ -135,7 +135,7 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
                                        // Detecting type:
                                        switch ($v['type']) {
                                        case 'function':
-                                               $outputParts[$k] = (($v['function'] . '(') . $v['func_content']) . ')';
+                                               $outputParts[$k] = $v['function'] . '(' . $v['func_content'] . ')';
                                                break;
                                        case 'flow-control':
                                                if ($v['flow-control']['type'] === 'CASE') {
@@ -143,12 +143,12 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
                                                }
                                                break;
                                        case 'field':
-                                               $outputParts[$k] = (($v['distinct'] ? $v['distinct'] : '') . ($v['table'] ? $v['table'] . '.' : '')) . $v['field'];
+                                               $outputParts[$k] = ($v['distinct'] ? $v['distinct'] : '') . ($v['table'] ? $v['table'] . '.' : '') . $v['field'];
                                                break;
                                        }
                                        // Alias:
                                        if ($v['as']) {
-                                               $outputParts[$k] .= ((' ' . $v['as_keyword']) . ' ') . $v['as'];
+                                               $outputParts[$k] .= ' ' . $v['as_keyword'] . ' ' . $v['as'];
                                        }
                                        // Specifically for ORDER BY and GROUP BY field lists:
                                        if ($v['sortDir']) {
@@ -184,17 +184,17 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
                        if (isset($components['case_field'])) {
                                $statement .= ' ' . $components['case_field'];
                        } elseif (isset($components['case_value'])) {
-                               $statement .= ((' ' . $components['case_value'][1]) . $components['case_value'][0]) . $components['case_value'][1];
+                               $statement .= ' ' . $components['case_value'][1] . $components['case_value'][0] . $components['case_value'][1];
                        }
                        foreach ($components['when'] as $when) {
                                $statement .= ' WHEN ';
                                $statement .= $this->compileWhereClause($when['when_value'], $functionMapping);
                                $statement .= ' THEN ';
-                               $statement .= ($when['then_value'][1] . $when['then_value'][0]) . $when['then_value'][1];
+                               $statement .= $when['then_value'][1] . $when['then_value'][0] . $when['then_value'][1];
                        }
                        if (isset($components['else'])) {
                                $statement .= ' ELSE ';
-                               $statement .= ($components['else'][1] . $components['else'][0]) . $components['else'][1];
+                               $statement .= $components['else'][1] . $components['else'][0] . $components['else'][1];
                        }
                        $statement .= ' END';
                        $output = $statement;
@@ -265,7 +265,7 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
        private function compileDROPTABLE($components) {
                switch ((string) $GLOBALS['TYPO3_DB']->handlerCfg[$GLOBALS['TYPO3_DB']->lastHandlerKey]['type']) {
                case 'native':
-                       $query = (('DROP TABLE' . ($components['ifExists'] ? ' IF EXISTS' : '')) . ' ') . $components['TABLE'];
+                       $query = 'DROP TABLE' . ($components['ifExists'] ? ' IF EXISTS' : '') . ' ' . $components['TABLE'];
                        break;
                case 'adodb':
                        $handlerKey = $GLOBALS['TYPO3_DB']->handler_getFromTableList($components['TABLE']);
@@ -295,7 +295,7 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
                        $indexKeys = array();
                        foreach ($components['FIELDS'] as $fN => $fCfg) {
                                $handlerKey = $GLOBALS['TYPO3_DB']->handler_getFromTableList($components['TABLE']);
-                               $fieldsKeys[$fN] = ($GLOBALS['TYPO3_DB']->quoteName($fN, $handlerKey, TRUE) . ' ') . $this->compileFieldCfg($fCfg['definition']);
+                               $fieldsKeys[$fN] = $GLOBALS['TYPO3_DB']->quoteName($fN, $handlerKey, TRUE) . ' ' . $this->compileFieldCfg($fCfg['definition']);
                        }
                        if (isset($components['KEYS']) && is_array($components['KEYS'])) {
                                foreach ($components['KEYS'] as $kN => $kCfg) {
@@ -308,7 +308,7 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
                                                        $indexKeys = array_merge($indexKeys, $GLOBALS['TYPO3_DB']->handlerInstance[$GLOBALS['TYPO3_DB']->handler_getFromTableList($components['TABLE'])]->DataDictionary->CreateIndexSQL($n, $components['TABLE'], $field, array('UNIQUE')));
                                                }
                                        } else {
-                                               $indexKeys = array_merge($indexKeys, $GLOBALS['TYPO3_DB']->handlerInstance[$GLOBALS['TYPO3_DB']->handler_getFromTableList($components['TABLE'])]->DataDictionary->CreateIndexSQL(($components['TABLE'] . '_') . $kN, $components['TABLE'], $kCfg));
+                                               $indexKeys = array_merge($indexKeys, $GLOBALS['TYPO3_DB']->handlerInstance[$GLOBALS['TYPO3_DB']->handler_getFromTableList($components['TABLE'])]->DataDictionary->CreateIndexSQL($components['TABLE'] . '_' . $kN, $components['TABLE'], $kCfg));
                                        }
                                }
                        }
@@ -340,10 +340,10 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
                        $fieldName = $GLOBALS['TYPO3_DB']->quoteName($components['FIELD'], NULL, TRUE);
                        switch (strtoupper(str_replace(array(' ', "\n", "\r", "\t"), '', $components['action']))) {
                        case 'ADD':
-                               $query = $GLOBALS['TYPO3_DB']->handlerInstance[$GLOBALS['TYPO3_DB']->lastHandlerKey]->DataDictionary->AddColumnSQL($tableName, ($fieldName . ' ') . $this->compileFieldCfg($components['definition']));
+                               $query = $GLOBALS['TYPO3_DB']->handlerInstance[$GLOBALS['TYPO3_DB']->lastHandlerKey]->DataDictionary->AddColumnSQL($tableName, $fieldName . ' ' . $this->compileFieldCfg($components['definition']));
                                break;
                        case 'CHANGE':
-                               $query = $GLOBALS['TYPO3_DB']->handlerInstance[$GLOBALS['TYPO3_DB']->lastHandlerKey]->DataDictionary->AlterColumnSQL($tableName, ($fieldName . ' ') . $this->compileFieldCfg($components['definition']));
+                               $query = $GLOBALS['TYPO3_DB']->handlerInstance[$GLOBALS['TYPO3_DB']->lastHandlerKey]->DataDictionary->AlterColumnSQL($tableName, $fieldName . ' ' . $this->compileFieldCfg($components['definition']));
                                break;
                        case 'DROP':
 
@@ -354,7 +354,7 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
                        case 'ADDPRIMARYKEY':
 
                        case 'ADDUNIQUE':
-                               $query .= (' (' . implode(',', $components['fields'])) . ')';
+                               $query .= ' (' . implode(',', $components['fields']) . ')';
                                break;
                        case 'DEFAULTCHARACTERSET':
 
@@ -393,7 +393,7 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
                        if (is_array($fieldCfg['featureIndex'])) {
                                // MySQL assigns DEFAULT value automatically if NOT NULL, fake this here
                                // numeric fields get 0 as default, other fields an empty string
-                               if ((isset($fieldCfg['featureIndex']['NOTNULL']) && !isset($fieldCfg['featureIndex']['DEFAULT'])) && !isset($fieldCfg['featureIndex']['AUTO_INCREMENT'])) {
+                               if (isset($fieldCfg['featureIndex']['NOTNULL']) && !isset($fieldCfg['featureIndex']['DEFAULT']) && !isset($fieldCfg['featureIndex']['AUTO_INCREMENT'])) {
                                        switch ($type) {
                                        case 'I8':
 
@@ -425,7 +425,7 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
                                                if ($featureDef['value'][0] === '') {
                                                        $cfg .= ' "\'\'"';
                                                } else {
-                                                       $cfg .= ((' ' . $featureDef['value'][1]) . $this->compileAddslashes($featureDef['value'][0])) . $featureDef['value'][1];
+                                                       $cfg .= ' ' . $featureDef['value'][1] . $this->compileAddslashes($featureDef['value'][0]) . $featureDef['value'][1];
                                                        if (!is_numeric($featureDef['value'][0])) {
                                                                $noQuote = FALSE;
                                                        }
@@ -490,37 +490,37 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
                                        $output .= $v['operator'] ? ' ' . $v['operator'] : '';
                                        // Look for sublevel:
                                        if (is_array($v['sub'])) {
-                                               $output .= (' (' . trim($this->compileWhereClause($v['sub'], $functionMapping))) . ')';
+                                               $output .= ' (' . trim($this->compileWhereClause($v['sub'], $functionMapping)) . ')';
                                        } elseif (isset($v['func']) && $v['func']['type'] === 'EXISTS') {
-                                               $output .= (((' ' . trim($v['modifier'])) . ' EXISTS (') . $this->compileSELECT($v['func']['subquery'])) . ')';
+                                               $output .= ' ' . trim($v['modifier']) . ' EXISTS (' . $this->compileSELECT($v['func']['subquery']) . ')';
                                        } else {
                                                if (isset($v['func']) && $v['func']['type'] === 'LOCATE') {
                                                        $output .= ' ' . trim($v['modifier']);
                                                        switch (TRUE) {
                                                        case $GLOBALS['TYPO3_DB']->runningADOdbDriver('mssql') && $functionMapping:
                                                                $output .= ' CHARINDEX(';
-                                                               $output .= ($v['func']['substr'][1] . $v['func']['substr'][0]) . $v['func']['substr'][1];
-                                                               $output .= (', ' . ($v['func']['table'] ? $v['func']['table'] . '.' : '')) . $v['func']['field'];
+                                                               $output .= $v['func']['substr'][1] . $v['func']['substr'][0] . $v['func']['substr'][1];
+                                                               $output .= ', ' . ($v['func']['table'] ? $v['func']['table'] . '.' : '') . $v['func']['field'];
                                                                $output .= isset($v['func']['pos']) ? ', ' . $v['func']['pos'][0] : '';
                                                                $output .= ')';
                                                                break;
                                                        case $GLOBALS['TYPO3_DB']->runningADOdbDriver('oci8') && $functionMapping:
                                                                $output .= ' INSTR(';
                                                                $output .= ($v['func']['table'] ? $v['func']['table'] . '.' : '') . $v['func']['field'];
-                                                               $output .= ((', ' . $v['func']['substr'][1]) . $v['func']['substr'][0]) . $v['func']['substr'][1];
+                                                               $output .= ', ' . $v['func']['substr'][1] . $v['func']['substr'][0] . $v['func']['substr'][1];
                                                                $output .= isset($v['func']['pos']) ? ', ' . $v['func']['pos'][0] : '';
                                                                $output .= ')';
                                                                break;
                                                        default:
                                                                $output .= ' LOCATE(';
-                                                               $output .= ($v['func']['substr'][1] . $v['func']['substr'][0]) . $v['func']['substr'][1];
-                                                               $output .= (', ' . ($v['func']['table'] ? $v['func']['table'] . '.' : '')) . $v['func']['field'];
+                                                               $output .= $v['func']['substr'][1] . $v['func']['substr'][0] . $v['func']['substr'][1];
+                                                               $output .= ', ' . ($v['func']['table'] ? $v['func']['table'] . '.' : '') . $v['func']['field'];
                                                                $output .= isset($v['func']['pos']) ? ', ' . $v['func']['pos'][0] : '';
                                                                $output .= ')';
                                                                break;
                                                        }
                                                } elseif (isset($v['func']) && $v['func']['type'] === 'IFNULL') {
-                                                       $output .= (' ' . trim($v['modifier'])) . ' ';
+                                                       $output .= ' ' . trim($v['modifier']) . ' ';
                                                        switch (TRUE) {
                                                        case $GLOBALS['TYPO3_DB']->runningADOdbDriver('mssql') && $functionMapping:
                                                                $output .= 'ISNULL';
@@ -534,10 +534,10 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
                                                        }
                                                        $output .= '(';
                                                        $output .= ($v['func']['table'] ? $v['func']['table'] . '.' : '') . $v['func']['field'];
-                                                       $output .= ((', ' . $v['func']['default'][1]) . $this->compileAddslashes($v['func']['default'][0])) . $v['func']['default'][1];
+                                                       $output .= ', ' . $v['func']['default'][1] . $this->compileAddslashes($v['func']['default'][0]) . $v['func']['default'][1];
                                                        $output .= ')';
                                                } elseif (isset($v['func']) && $v['func']['type'] === 'FIND_IN_SET') {
-                                                       $output .= (' ' . trim($v['modifier'])) . ' ';
+                                                       $output .= ' ' . trim($v['modifier']) . ' ';
                                                        if ($functionMapping) {
                                                                switch (TRUE) {
                                                                case $GLOBALS['TYPO3_DB']->runningADOdbDriver('mssql'):
@@ -545,19 +545,19 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
                                                                        if (!isset($v['func']['str_like'])) {
                                                                                $v['func']['str_like'] = $v['func']['str'][0];
                                                                        }
-                                                                       $output .= ((('\',\'+' . $field) . '+\',\' LIKE \'%,') . $v['func']['str_like']) . ',%\'';
+                                                                       $output .= '\',\'+' . $field . '+\',\' LIKE \'%,' . $v['func']['str_like'] . ',%\'';
                                                                        break;
                                                                case $GLOBALS['TYPO3_DB']->runningADOdbDriver('oci8'):
                                                                        $field = ($v['func']['table'] ? $v['func']['table'] . '.' : '') . $v['func']['field'];
                                                                        if (!isset($v['func']['str_like'])) {
                                                                                $v['func']['str_like'] = $v['func']['str'][0];
                                                                        }
-                                                                       $output .= ((('\',\'||' . $field) . '||\',\' LIKE \'%,') . $v['func']['str_like']) . ',%\'';
+                                                                       $output .= '\',\'||' . $field . '||\',\' LIKE \'%,' . $v['func']['str_like'] . ',%\'';
                                                                        break;
                                                                case $GLOBALS['TYPO3_DB']->runningADOdbDriver('postgres'):
                                                                        $output .= ' FIND_IN_SET(';
-                                                                       $output .= ($v['func']['str'][1] . $v['func']['str'][0]) . $v['func']['str'][1];
-                                                                       $output .= (', ' . ($v['func']['table'] ? $v['func']['table'] . '.' : '')) . $v['func']['field'];
+                                                                       $output .= $v['func']['str'][1] . $v['func']['str'][0] . $v['func']['str'][1];
+                                                                       $output .= ', ' . ($v['func']['table'] ? $v['func']['table'] . '.' : '') . $v['func']['field'];
                                                                        $output .= ') != 0';
                                                                        break;
                                                                default:
@@ -565,7 +565,7 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
                                                                        if (!isset($v['func']['str_like'])) {
                                                                                $v['func']['str_like'] = $v['func']['str'][0];
                                                                        }
-                                                                       $output .= (((((((((((((((((((('(' . $field) . ' LIKE \'%,') . $v['func']['str_like']) . ',%\'') . ' OR ') . $field) . ' LIKE \'') . $v['func']['str_like']) . ',%\'') . ' OR ') . $field) . ' LIKE \'%,') . $v['func']['str_like']) . '\'') . ' OR ') . $field) . '= ') . $v['func']['str'][1]) . $v['func']['str'][0]) . $v['func']['str'][1]) . ')';
+                                                                       $output .= '(' . $field . ' LIKE \'%,' . $v['func']['str_like'] . ',%\'' . ' OR ' . $field . ' LIKE \'' . $v['func']['str_like'] . ',%\'' . ' OR ' . $field . ' LIKE \'%,' . $v['func']['str_like'] . '\'' . ' OR ' . $field . '= ' . $v['func']['str'][1] . $v['func']['str'][0] . $v['func']['str'][1] . ')';
                                                                        break;
                                                                }
                                                        } else {
@@ -576,8 +576,8 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
 
                                                                case $GLOBALS['TYPO3_DB']->runningADOdbDriver('postgres'):
                                                                        $output .= ' FIND_IN_SET(';
-                                                                       $output .= ($v['func']['str'][1] . $v['func']['str'][0]) . $v['func']['str'][1];
-                                                                       $output .= (', ' . ($v['func']['table'] ? $v['func']['table'] . '.' : '')) . $v['func']['field'];
+                                                                       $output .= $v['func']['str'][1] . $v['func']['str'][0] . $v['func']['str'][1];
+                                                                       $output .= ', ' . ($v['func']['table'] ? $v['func']['table'] . '.' : '') . $v['func']['field'];
                                                                        $output .= ')';
                                                                        break;
                                                                default:
@@ -585,23 +585,23 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
                                                                        if (!isset($v['func']['str_like'])) {
                                                                                $v['func']['str_like'] = $v['func']['str'][0];
                                                                        }
-                                                                       $output .= (((((((((((((((((((('(' . $field) . ' LIKE \'%,') . $v['func']['str_like']) . ',%\'') . ' OR ') . $field) . ' LIKE \'') . $v['func']['str_like']) . ',%\'') . ' OR ') . $field) . ' LIKE \'%,') . $v['func']['str_like']) . '\'') . ' OR ') . $field) . '= ') . $v['func']['str'][1]) . $v['func']['str'][0]) . $v['func']['str'][1]) . ')';
+                                                                       $output .= '(' . $field . ' LIKE \'%,' . $v['func']['str_like'] . ',%\'' . ' OR ' . $field . ' LIKE \'' . $v['func']['str_like'] . ',%\'' . ' OR ' . $field . ' LIKE \'%,' . $v['func']['str_like'] . '\'' . ' OR ' . $field . '= ' . $v['func']['str'][1] . $v['func']['str'][0] . $v['func']['str'][1] . ')';
                                                                        break;
                                                                }
                                                        }
                                                } else {
                                                        // Set field/table with modifying prefix if any:
-                                                       $output .= (' ' . trim($v['modifier'])) . ' ';
+                                                       $output .= ' ' . trim($v['modifier']) . ' ';
                                                        // DBAL-specific: Set calculation, if any:
                                                        if ($v['calc'] === '&' && $functionMapping) {
                                                                switch (TRUE) {
                                                                case $GLOBALS['TYPO3_DB']->runningADOdbDriver('oci8'):
                                                                        // Oracle only knows BITAND(x,y) - sigh
-                                                                       $output .= ((((('BITAND(' . trim((($v['table'] ? $v['table'] . '.' : '') . $v['field']))) . ',') . $v['calc_value'][1]) . $this->compileAddslashes($v['calc_value'][0])) . $v['calc_value'][1]) . ')';
+                                                                       $output .= 'BITAND(' . trim((($v['table'] ? $v['table'] . '.' : '') . $v['field'])) . ',' . $v['calc_value'][1] . $this->compileAddslashes($v['calc_value'][0]) . $v['calc_value'][1] . ')';
                                                                        break;
                                                                default:
                                                                        // MySQL, MS SQL Server, PostgreSQL support the &-syntax
-                                                                       $output .= (((trim((($v['table'] ? $v['table'] . '.' : '') . $v['field'])) . $v['calc']) . $v['calc_value'][1]) . $this->compileAddslashes($v['calc_value'][0])) . $v['calc_value'][1];
+                                                                       $output .= trim((($v['table'] ? $v['table'] . '.' : '') . $v['field'])) . $v['calc'] . $v['calc_value'][1] . $this->compileAddslashes($v['calc_value'][0]) . $v['calc_value'][1];
                                                                        break;
                                                                }
                                                        } elseif ($v['calc']) {
@@ -609,9 +609,9 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
                                                                if (isset($v['calc_table'])) {
                                                                        $output .= trim(($v['calc_table'] ? $v['calc_table'] . '.' : '') . $v['calc_field']);
                                                                } else {
-                                                                       $output .= ($v['calc_value'][1] . $this->compileAddslashes($v['calc_value'][0])) . $v['calc_value'][1];
+                                                                       $output .= $v['calc_value'][1] . $this->compileAddslashes($v['calc_value'][0]) . $v['calc_value'][1];
                                                                }
-                                                       } elseif (!(($GLOBALS['TYPO3_DB']->runningADOdbDriver('oci8') && preg_match('/(NOT )?LIKE( BINARY)?/', $v['comparator'])) && $functionMapping)) {
+                                                       } elseif (!($GLOBALS['TYPO3_DB']->runningADOdbDriver('oci8') && preg_match('/(NOT )?LIKE( BINARY)?/', $v['comparator']) && $functionMapping)) {
                                                                $output .= trim(($v['table'] ? $v['table'] . '.' : '') . $v['field']);
                                                        }
                                                }
@@ -619,16 +619,16 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
                                                if ($v['comparator']) {
                                                        $isLikeOperator = preg_match('/(NOT )?LIKE( BINARY)?/', $v['comparator']);
                                                        switch (TRUE) {
-                                                       case ($GLOBALS['TYPO3_DB']->runningADOdbDriver('oci8') && $isLikeOperator) && $functionMapping:
+                                                       case $GLOBALS['TYPO3_DB']->runningADOdbDriver('oci8') && $isLikeOperator && $functionMapping:
                                                                // Oracle cannot handle LIKE on CLOB fields - sigh
                                                                if (isset($v['value']['operator'])) {
                                                                        $values = array();
                                                                        foreach ($v['value']['args'] as $fieldDef) {
                                                                                $values[] = ($fieldDef['table'] ? $fieldDef['table'] . '.' : '') . $fieldDef['field'];
                                                                        }
-                                                                       $compareValue = (((' ' . $v['value']['operator']) . '(') . implode(',', $values)) . ')';
+                                                                       $compareValue = ' ' . $v['value']['operator'] . '(' . implode(',', $values) . ')';
                                                                } else {
-                                                                       $compareValue = ($v['value'][1] . $this->compileAddslashes(trim($v['value'][0], '%'))) . $v['value'][1];
+                                                                       $compareValue = $v['value'][1] . $this->compileAddslashes(trim($v['value'][0], '%')) . $v['value'][1];
                                                                }
                                                                if (\TYPO3\CMS\Core\Utility\GeneralUtility::isFirstPartOfStr($v['comparator'], 'NOT')) {
                                                                        $output .= 'NOT ';
@@ -644,21 +644,21 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
                                                                }
                                                                if (strtoupper(substr($v['comparator'], -6)) === 'BINARY') {
                                                                        if ($isLob) {
-                                                                               $output .= ((('(dbms_lob.instr(' . trim((($v['table'] ? $v['table'] . '.' : '') . $v['field']))) . ', ') . $compareValue) . ',1,1) > 0)';
+                                                                               $output .= '(dbms_lob.instr(' . trim((($v['table'] ? $v['table'] . '.' : '') . $v['field'])) . ', ' . $compareValue . ',1,1) > 0)';
                                                                        } else {
-                                                                               $output .= ((('(instr(' . trim((($v['table'] ? $v['table'] . '.' : '') . $v['field']))) . ', ') . $compareValue) . ',1,1) > 0)';
+                                                                               $output .= '(instr(' . trim((($v['table'] ? $v['table'] . '.' : '') . $v['field'])) . ', ' . $compareValue . ',1,1) > 0)';
                                                                        }
                                                                } else {
                                                                        if ($isLob) {
-                                                                               $output .= ((('(dbms_lob.instr(LOWER(' . trim((($v['table'] ? $v['table'] . '.' : '') . $v['field']))) . '), ') . \TYPO3\CMS\Core\Utility\GeneralUtility::strtolower($compareValue)) . ',1,1) > 0)';
+                                                                               $output .= '(dbms_lob.instr(LOWER(' . trim((($v['table'] ? $v['table'] . '.' : '') . $v['field'])) . '), ' . \TYPO3\CMS\Core\Utility\GeneralUtility::strtolower($compareValue) . ',1,1) > 0)';
                                                                        } else {
-                                                                               $output .= ((('(instr(LOWER(' . trim((($v['table'] ? $v['table'] . '.' : '') . $v['field']))) . '), ') . \TYPO3\CMS\Core\Utility\GeneralUtility::strtolower($compareValue)) . ',1,1) > 0)';
+                                                                               $output .= '(instr(LOWER(' . trim((($v['table'] ? $v['table'] . '.' : '') . $v['field'])) . '), ' . \TYPO3\CMS\Core\Utility\GeneralUtility::strtolower($compareValue) . ',1,1) > 0)';
                                                                        }
                                                                }
                                                                break;
                                                        default:
                                                                if ($isLikeOperator && $functionMapping) {
-                                                                       if ((($GLOBALS['TYPO3_DB']->runningADOdbDriver('postgres') || $GLOBALS['TYPO3_DB']->runningADOdbDriver('postgres64')) || $GLOBALS['TYPO3_DB']->runningADOdbDriver('postgres7')) || $GLOBALS['TYPO3_DB']->runningADOdbDriver('postgres8')) {
+                                                                       if ($GLOBALS['TYPO3_DB']->runningADOdbDriver('postgres') || $GLOBALS['TYPO3_DB']->runningADOdbDriver('postgres64') || $GLOBALS['TYPO3_DB']->runningADOdbDriver('postgres7') || $GLOBALS['TYPO3_DB']->runningADOdbDriver('postgres8')) {
                                                                                // Remap (NOT)? LIKE to (NOT)? ILIKE
                                                                                // and (NOT)? LIKE BINARY to (NOT)? LIKE
                                                                                switch ($v['comparator']) {
@@ -681,28 +681,28 @@ class SqlParser extends \TYPO3\CMS\Core\Database\SqlParser {
                                                                // Detecting value type; list or plain:
                                                                if (\TYPO3\CMS\Core\Utility\GeneralUtility::inList('NOTIN,IN', strtoupper(str_replace(array(' ', TAB, CR, LF), '', $v['comparator'])))) {
                                                                        if (isset($v['subquery'])) {
-                                                                               $output .= (' (' . $this->compileSELECT($v['subquery'])) . ')';
+                                                                               $output .= ' (' . $this->compileSELECT($v['subquery']) . ')';
                                                                        } else {
                                                                                $valueBuffer = array();
                                                                                foreach ($v['value'] as $realValue) {
-                                                                                       $valueBuffer[] = ($realValue[1] . $this->compileAddslashes($realValue[0])) . $realValue[1];
+                                                                                       $valueBuffer[] = $realValue[1] . $this->compileAddslashes($realValue[0]) . $realValue[1];
                                                                                }
-                                                                               $output .= (' (' . trim(implode(',', $valueBuffer))) . ')';
+                                                                               $output .= ' (' . trim(implode(',', $valueBuffer)) . ')';
                                                                        }
                                                                } elseif (\TYPO3\CMS\Core\Utility\GeneralUtility::inList('BETWEEN,NOT BETWEEN', $v['comparator'])) {
                                                                        $lbound = $v['values'][0];
                                                                        $ubound = $v['values'][1];
-                                                                       $output .= ((' ' . $lbound[1]) . $this->compileAddslashes($lbound[0])) . $lbound[1];
+                                                                       $output .= ' ' . $lbound[1] . $this->compileAddslashes($lbound[0]) . $lbound[1];
                                                                        $output .= ' AND ';
-                                                                       $output .= ($ubound[1] . $this->compileAddslashes($ubound[0])) . $ubound[1];
+                                                                       $output .= $ubound[1] . $this->compileAddslashes($ubound[0]) . $ubound[1];
                                                                } elseif (isset($v['value']['operator'])) {
                                                                        $values = array();
                                                                        foreach ($v['value']['args'] as $fieldDef) {
                                                                                $values[] = ($fieldDef['table'] ? $fieldDef['table'] . '.' : '') . $fieldDef['field'];
                                                                        }
-                                                                       $output .= (((' ' . $v['value']['operator']) . '(') . implode(',', $values)) . ')';
+                                                                       $output .= ' ' . $v['value']['operator'] . '(' . implode(',', $values) . ')';
                                                                } else {
-                                                                       $output .= ((' ' . $v['value'][1]) . $this->compileAddslashes($v['value'][0])) . $v['value'][1];
+                                                                       $output .= ' ' . $v['value'][1] . $this->compileAddslashes($v['value'][0]) . $v['value'][1];
                                                                }
                                                                break;
                                                        }
index 7783708..2c553c4 100644 (file)
@@ -44,13 +44,13 @@ class MessageDisplay {
         * @todo Define visibility
         */
        public function displayMessage(array &$params, \TYPO3\CMS\Core\TypoScript\ConfigurationForm $tsObj) {
-               $out = ('
+               $out = '
                        <div>
                                <div class="typo3-message message-information">
                                        <div class="message-header">PostgreSQL</div>
                                        <div class="message-body">
                                                If you use a PostgreSQL database, make sure to run SQL scripts located in<br />
-                                               <tt>' . \TYPO3\CMS\Core\Extension\ExtensionManager::extPath('dbal')) . 'res/postgresql/</tt><br />
+                                               <tt>' . \TYPO3\CMS\Core\Extension\ExtensionManager::extPath('dbal') . 'res/postgresql/</tt><br />
                                                to ensure best compatibility with TYPO3.
                                        </div>
                                </div>
index 95771a4..f6c1f9e 100644 (file)
@@ -150,7 +150,7 @@ class ExtensionManagerHooks implements tx_em_Index_CheckDatabaseUpdatesHook {
                } elseif (!$isKeyField) {
                        $needsRemapping = strlen($field) > $this->maxIdentifierLength;
                } else {
-                       $needsRemapping = strlen(($table . '_') . $field) > $this->maxIdentifierLength;
+                       $needsRemapping = strlen($table . '_' . $field) > $this->maxIdentifierLength;
                }
                return $needsRemapping;
        }
@@ -231,18 +231,18 @@ class ExtensionManagerHooks implements tx_em_Index_CheckDatabaseUpdatesHook {
                $out = array();
                $tableId = uniqid('table');
                $label = 'DBAL Mapping';
-               $description = sprintf(((((('Some table names are longer than %s characters and/or some field names are longer than %s characters.' . ' This is incompatible with your database:') . ' <ul style="list-style: square; margin: 3px 1em; padding: 3px 1em;">') . '                <li>Table names should be short enough to let ADOdb generates a sequence of the form {table}_uid for the') . '                  auto-increment "uid" field within %s characters;</li>') . '             <li>Field names may not contain more than %s characters.</li>') . ' </ul>', $this->maxIdentifierLength - $this->tableNameCharacterReservation, $this->maxIdentifierLength, $this->maxIdentifierLength, $this->maxIdentifierLength);
+               $description = sprintf('Some table names are longer than %s characters and/or some field names are longer than %s characters.' . ' This is incompatible with your database:' . ' <ul style="list-style: square; margin: 3px 1em; padding: 3px 1em;">' . '               <li>Table names should be short enough to let ADOdb generates a sequence of the form {table}_uid for the' . '                   auto-increment "uid" field within %s characters;</li>' . '              <li>Field names may not contain more than %s characters.</li>' . ' </ul>', $this->maxIdentifierLength - $this->tableNameCharacterReservation, $this->maxIdentifierLength, $this->maxIdentifierLength, $this->maxIdentifierLength);
                $tables = array_unique(array_merge($tables, array_keys($fields)));
                foreach ($tables as $table) {
                        $newTableName = $table;
                        if (isset($suggestions[$table]) && isset($suggestions[$table]['mapTableName'])) {
                                $newTableName = $suggestions[$table]['mapTableName'];
                        }
-                       $out[] = (((((((((((('
+                       $out[] = '
                                <tr>
-                                       <td style="padding-top: 1em;"><label for="table-' . $table) . '">') . $table) . '</label></td>
+                                       <td style="padding-top: 1em;"><label for="table-' . $table . '">' . $table . '</label></td>
                                        <td style="padding-top: 1em;">=&gt;</td>
-                                       <td style="padding-top: 1em;"><input type="text" size="35" id="table-') . $table) . '" name="dbal[tables][') . $table) . ']" value="') . $newTableName) . '" /> ') . strlen($newTableName)) . ' characters') . '</td>
+                                       <td style="padding-top: 1em;"><input type="text" size="35" id="table-' . $table . '" name="dbal[tables][' . $table . ']" value="' . $newTableName . '" /> ' . strlen($newTableName) . ' characters' . '</td>
                                </tr>';
                        if (isset($fields[$table])) {
                                foreach ($fields[$table] as $field => $info) {
@@ -252,23 +252,23 @@ class ExtensionManagerHooks implements tx_em_Index_CheckDatabaseUpdatesHook {
                                                        $newFieldName = $suggestions[$table]['mapFieldNames'][$field];
                                                }
                                        }
-                                       $newFieldFullName = preg_replace(('/^' . $table) . '/', $newTableName, $info['fullName']);
-                                       $newFieldFullName = preg_replace(('/' . $field) . '$/', $newFieldName, $newFieldFullName);
-                                       $out[] = ((((((((((((((((('
+                                       $newFieldFullName = preg_replace('/^' . $table . '/', $newTableName, $info['fullName']);
+                                       $newFieldFullName = preg_replace('/' . $field . '$/', $newFieldName, $newFieldFullName);
+                                       $out[] = '
                                                <tr>
-                                                       <td>&nbsp;&nbsp;&nbsp;&nbsp;<label for="field-' . $table) . '_') . $field) . '">') . $field) . '</label></td>
+                                                       <td>&nbsp;&nbsp;&nbsp;&nbsp;<label for="field-' . $table . '_' . $field . '">' . $field . '</label></td>
                                                        <td>=&gt;</td>
-                                                       <td><input type="text" size="35" id="field-') . $table) . '_') . $field) . '" name="dbal[fields][') . $table) . '][') . $field) . ']" value="') . $newFieldName) . '" /> ') . ($info['fullname'] !== $field ? (strlen($newFieldFullName) . ' characters: ') . $newFieldFullName : '')) . '</td>
+                                                       <td><input type="text" size="35" id="field-' . $table . '_' . $field . '" name="dbal[fields][' . $table . '][' . $field . ']" value="' . $newFieldName . '" /> ' . ($info['fullname'] !== $field ? strlen($newFieldFullName) . ' characters: ' . $newFieldFullName : '') . '</td>
                                                </tr>';
                                }
                        }
                }
                // Compile rows:
-               $content = ((((((('
+               $content = '
                        <!-- Remapping database fields / tables -->
-                       <h3>' . $label) . '</h3>
-                       <p>') . $description) . '</p>
-                       <table border="0" cellpadding="2" cellspacing="2" id="') . $tableId) . '" class="remap-db-table-fields">') . implode('', $out)) . '
+                       <h3>' . $label . '</h3>
+                       <p>' . $description . '</p>
+                       <table border="0" cellpadding="2" cellspacing="2" id="' . $tableId . '" class="remap-db-table-fields">' . implode('', $out) . '
                        </table>';
                return $content;
        }
index d91bf5c..b4b1a3f 100644 (file)
@@ -174,7 +174,7 @@ class InstallToolHooks {
                        $this->driver = $this->getDefaultDriver();
                }
                // Get the template file
-               $templateFile = @file_get_contents(((\TYPO3\CMS\Core\Extension\ExtensionManager::extPath('dbal') . $this->templateFilePath) . 'install.html'));
+               $templateFile = @file_get_contents((\TYPO3\CMS\Core\Extension\ExtensionManager::extPath('dbal') . $this->templateFilePath . 'install.html'));
                // Get the template part from the file
                $template = \TYPO3\CMS\Core\Html\HtmlParser::getSubpart($templateFile, '###TEMPLATE###');
                // Get the subpart for the connection form
@@ -193,7 +193,7 @@ class InstallToolHooks {
                if ($this->driver === '' || $this->driver === 'mysql') {
                        $driverOptionsSubPart = \TYPO3\CMS\Core\Html\HtmlParser::getSubpart($template, '###DRIVER_MYSQL###');
                } else {
-                       $driverOptionsSubPart = \TYPO3\CMS\Core\Html\HtmlParser::getSubpart($template, ('###DRIVER_' . \TYPO3\CMS\Core\Utility\GeneralUtility::strtoupper($this->driver)) . '###');
+                       $driverOptionsSubPart = \TYPO3\CMS\Core\Html\HtmlParser::getSubpart($template, '###DRIVER_' . \TYPO3\CMS\Core\Utility\GeneralUtility::strtoupper($this->driver) . '###');
                        if ($driverOptionsSubPart === '') {
                                $driverOptionsSubPart = \TYPO3\CMS\Core\Html\HtmlParser::getSubpart($template, '###DRIVER_DEFAULT###');
                        }
@@ -305,7 +305,7 @@ class InstallToolHooks {
                                $markers = array(
                                        'driver' => $driver,
                                        'labelvendor' => $label,
-                                       'onclick' => ('document.location=\'index.php?TYPO3_INSTALL[type]=config&mode=123&step=2&driver=' . $driver) . '\';',
+                                       'onclick' => 'document.location=\'index.php?TYPO3_INSTALL[type]=config&mode=123&step=2&driver=' . $driver . '\';',
                                        'selected' => ''
                                );
                                if ($driver === $this->driver) {
@@ -428,14 +428,14 @@ class InstallToolHooks {
         * @param \TYPO3\CMS\Install\Installer $instObj
         */
        protected function createDatabaseForm(array &$markers, \TYPO3\CMS\Install\Installer $instObj) {
-               $error_missingConnect = ((('
+               $error_missingConnect = '
                        <p class="typo3-message message-error">
                                <strong>
                                        There is no connection to the database!
                                </strong>
                                <br />
-                               (Username: <em>' . TYPO3_db_username) . '</em>,
-                               Host: <em>') . TYPO3_db_host) . '</em>,
+                               (Username: <em>' . TYPO3_db_username . '</em>,
+                               Host: <em>' . TYPO3_db_host . '</em>,
                                Using Password: YES)
                                <br />
                                Go to Step 1 and enter a valid username and password!
@@ -446,7 +446,7 @@ class InstallToolHooks {
                // There should be a database host connection at this point
                if ($result = $GLOBALS['TYPO3_DB']->sql_pconnect(TYPO3_db_host, TYPO3_db_username, TYPO3_db_password)) {
                        // Get the template file
-                       $templateFile = @file_get_contents(((\TYPO3\CMS\Core\Extension\ExtensionManager::extPath('dbal') . $this->templateFilePath) . 'install.html'));
+                       $templateFile = @file_get_contents((\TYPO3\CMS\Core\Extension\ExtensionManager::extPath('dbal') . $this->templateFilePath . 'install.html'));
                        // Get the template part from the file
                        $template = \TYPO3\CMS\Core\Html\HtmlParser::getSubpart($templateFile, '###TEMPLATE###');
                        // Get the subpart for the database choice step
index 8fb4deb..3bed2cf 100644 (file)
@@ -63,7 +63,7 @@ class RecordList extends \TYPO3\CMS\Recordlist\RecordList\DatabaseRecordList {
                        $or = '';
                        // add the uid only if input is numeric, cast to int
                        if (is_numeric($this->searchString)) {
-                               $queryPart = (' AND (uid=' . (int) $this->searchString) . ' OR ';
+                               $queryPart = ' AND (uid=' . (int) $this->searchString . ' OR ';
                        } else {
                                $queryPart = ' AND (';
                        }
@@ -72,7 +72,7 @@ class RecordList extends \TYPO3\CMS\Recordlist\RecordList\DatabaseRecordList {
                                        if ($GLOBALS['TYPO3_DB']->cache_fieldType[$table][$fieldName]['metaType'] === 'B') {
 
                                        } elseif ($info['config']['type'] === 'text' || $info['config']['type'] === 'input' && !preg_match('/date|time|int/', $info['config']['eval'])) {
-                                               $queryPart .= ((($or . $fieldName) . ' LIKE \'%') . $GLOBALS['TYPO3_DB']->quoteStr($this->searchString, $table)) . '%\'';
+                                               $queryPart .= $or . $fieldName . ' LIKE \'%' . $GLOBALS['TYPO3_DB']->quoteStr($this->searchString, $table) . '%\'';
                                                $or = ' OR ';
                                        }
                                }
@@ -85,7 +85,7 @@ class RecordList extends \TYPO3\CMS\Recordlist\RecordList\DatabaseRecordList {
                                }
                                // If search-fields were defined (and there always are) we create the query:
                                if (count($sfields)) {
-                                       $like = (' LIKE \'%' . $GLOBALS['TYPO3_DB']->quoteStr($this->searchString, $table)) . '%\'';
+                                       $like = ' LIKE \'%' . $GLOBALS['TYPO3_DB']->quoteStr($this->searchString, $table) . '%\'';
                                        // Free-text
                                        $queryPart .= implode(($like . ' OR '), $sfields) . $like;
                                }
index 8c5d834..bf0c436 100644 (file)
@@ -48,8 +48,8 @@ abstract class BaseTestCase extends \tx_phpunit_testcase {
                $accessibleClassName = uniqid('AccessibleTestProxy');
                $class = new \ReflectionClass($className);
                $abstractModifier = $class->isAbstract() ? 'abstract ' : '';
-               eval(((((('
-                       ' . $abstractModifier) . 'class ') . $accessibleClassName) . ' extends ') . $className) . ' {
+               eval('
+                       ' . $abstractModifier . 'class ' . $accessibleClassName . ' extends ' . $className . ' {
                                public function _call($methodName) {
                                        return call_user_func_array(array($this, $methodName), array_slice(func_get_args(), 1));
                                }
index 28658d4..0c94b4d 100644 (file)
@@ -86,9 +86,9 @@ class DatabaseMssqlTest extends BaseTestCase {
                foreach ($GLOBALS['TYPO3_DB']->cache_fieldType as $table => $fieldTypes) {
                        $tableDef = $GLOBALS['TYPO3_DB']->_call('map_needMapping', $table);
                        if (in_array($table, $tablesWithMapping)) {
-                               self::assertTrue(is_array($tableDef), ('Table ' . $table) . ' was expected to need mapping');
+                               self::assertTrue(is_array($tableDef), 'Table ' . $table . ' was expected to need mapping');
                        } else {
-                               self::assertFalse($tableDef, ('Table ' . $table) . ' was not expected to need mapping');
+                               self::assertFalse($tableDef, 'Table ' . $table . ' was not expected to need mapping');
                        }
                }
        }
@@ -142,7 +142,7 @@ class DatabaseMssqlTest extends BaseTestCase {
         * @see http://bugs.typo3.org/view.php?id=13134
         */
        public function locateStatementIsProperlyQuoted() {
-               $query = $this->cleanSql($GLOBALS['TYPO3_DB']->SELECTquery((((('*, CASE WHEN' . ' LOCATE(') . $GLOBALS['TYPO3_DB']->fullQuoteStr('(fce)', 'tx_templavoila_tmplobj')) . ', datastructure)>0 THEN 2') . ' ELSE 1') . ' END AS scope', 'tx_templavoila_tmplobj', '1=1'));
+               $query = $this->cleanSql($GLOBALS['TYPO3_DB']->SELECTquery('*, CASE WHEN' . ' LOCATE(' . $GLOBALS['TYPO3_DB']->fullQuoteStr('(fce)', 'tx_templavoila_tmplobj') . ', datastructure)>0 THEN 2' . ' ELSE 1' . ' END AS scope', 'tx_templavoila_tmplobj', '1=1'));
                $expected = 'SELECT *, CASE WHEN CHARINDEX(\'(fce)\', "datastructure") > 0 THEN 2 ELSE 1 END AS "scope" FROM "tx_templavoila_tmplobj" WHERE 1 = 1';
                $this->assertEquals($expected, $query);
        }
@@ -152,7 +152,7 @@ class DatabaseMssqlTest extends BaseTestCase {
         * @see http://bugs.typo3.org/view.php?id=13134
         */
        public function locateStatementWithPositionIsProperlyQuoted() {
-               $query = $this->cleanSql($GLOBALS['TYPO3_DB']->SELECTquery((((('*, CASE WHEN' . ' LOCATE(') . $GLOBALS['TYPO3_DB']->fullQuoteStr('(fce)', 'tx_templavoila_tmplobj')) . ', datastructure, 4)>0 THEN 2') . ' ELSE 1') . ' END AS scope', 'tx_templavoila_tmplobj', '1=1'));
+               $query = $this->cleanSql($GLOBALS['TYPO3_DB']->SELECTquery('*, CASE WHEN' . ' LOCATE(' . $GLOBALS['TYPO3_DB']->fullQuoteStr('(fce)', 'tx_templavoila_tmplobj') . ', datastructure, 4)>0 THEN 2' . ' ELSE 1' . ' END AS scope', 'tx_templavoila_tmplobj', '1=1'));
                $expected = 'SELECT *, CASE WHEN CHARINDEX(\'(fce)\', "datastructure", 4) > 0 THEN 2 ELSE 1 END AS "scope" FROM "tx_templavoila_tmplobj" WHERE 1 = 1';
                $this->assertEquals($expected, $query);
        }
@@ -162,7 +162,7 @@ class DatabaseMssqlTest extends BaseTestCase {
         * @see http://bugs.typo3.org/view.php?id=13134
         */
        public function locateStatementIsProperlyRemapped() {
-               $selectFields = (((('*, CASE WHEN' . ' LOCATE(') . $GLOBALS['TYPO3_DB']->fullQuoteStr('(fce)', 'tx_templavoila_tmplobj')) . ', datastructure, 4)>0 THEN 2') . ' ELSE 1') . ' END AS scope';
+               $selectFields = '*, CASE WHEN' . ' LOCATE(' . $GLOBALS['TYPO3_DB']->fullQuoteStr('(fce)', 'tx_templavoila_tmplobj') . ', datastructure, 4)>0 THEN 2' . ' ELSE 1' . ' END AS scope';
                $fromTables = 'tx_templavoila_tmplobj';
                $whereClause = '1=1';
                $groupBy = '';
@@ -178,7 +178,7 @@ class DatabaseMssqlTest extends BaseTestCase {
         * @see http://bugs.typo3.org/view.php?id=13134
         */
        public function locateStatementWithExternalTableIsProperlyRemapped() {
-               $selectFields = (((('*, CASE WHEN' . ' LOCATE(') . $GLOBALS['TYPO3_DB']->fullQuoteStr('(fce)', 'tx_templavoila_tmplobj')) . ', tx_templavoila_tmplobj.datastructure, 4)>0 THEN 2') . ' ELSE 1') . ' END AS scope';
+               $selectFields = '*, CASE WHEN' . ' LOCATE(' . $GLOBALS['TYPO3_DB']->fullQuoteStr('(fce)', 'tx_templavoila_tmplobj') . ', tx_templavoila_tmplobj.datastructure, 4)>0 THEN 2' . ' ELSE 1' . ' END AS scope';
                $fromTables = 'tx_templavoila_tmplobj';
                $whereClause = '1=1';
                $groupBy = '';
index b70731f..0ea66a0 100644 (file)
@@ -86,9 +86,9 @@ class DatabaseOracleTest extends BaseTestCase {
                foreach ($GLOBALS['TYPO3_DB']->cache_fieldType as $table => $fieldTypes) {
                        $tableDef = $GLOBALS['TYPO3_DB']->_call('map_needMapping', $table);
                        if (in_array($table, $tablesWithMapping)) {
-                               self::assertTrue(is_array($tableDef), ('Table ' . $table) . ' was expected to need mapping');
+                               self::assertTrue(is_array($tableDef), 'Table ' . $table . ' was expected to need mapping');
                        } else {
-                               self::assertFalse($tableDef, ('Table ' . $table) . ' was not expected to need mapping');
+                               self::assertFalse($tableDef, 'Table ' . $table . ' was not expected to need mapping');
                        }
                }
        }
@@ -133,7 +133,7 @@ class DatabaseOracleTest extends BaseTestCase {
                $this->assertEquals(4, count($insert));
                for ($i = 0; $i < count($insert); $i++) {
                        foreach (\TYPO3\CMS\Core\Utility\GeneralUtility::trimExplode(',', 'uid,pid,tr_iso_nr,tr_parent_iso_nr,tr_name_en') as $field) {
-                               $this->assertTrue(isset($insert[$i][$field]), ('Could not find ' . $field) . ' column');
+                               $this->assertTrue(isset($insert[$i][$field]), 'Could not find ' . $field . ' column');
                        }
                }
        }
@@ -165,8 +165,8 @@ class DatabaseOracleTest extends BaseTestCase {
         * @see http://bugs.typo3.org/view.php?id=15535
         */
        public function groupConditionsAreProperlyTransformed() {
-               $query = $this->cleanSql($GLOBALS['TYPO3_DB']->SELECTquery('*', 'pages', (('pid=0 AND pages.deleted=0 AND pages.hidden=0 AND pages.starttime<=1281620460 ' . 'AND (pages.endtime=0 OR pages.endtime>1281620460) AND NOT pages.t3ver_state>0 ') . 'AND pages.doktype<200 AND (pages.fe_group=\'\' OR pages.fe_group IS NULL OR ') . 'pages.fe_group=\'0\' OR FIND_IN_SET(\'0\',pages.fe_group) OR FIND_IN_SET(\'-1\',pages.fe_group))'));
-               $expected = ((('SELECT * FROM "pages" WHERE "pid" = 0 AND "pages"."deleted" = 0 AND "pages"."hidden" = 0 ' . 'AND "pages"."starttime" <= 1281620460 AND ("pages"."endtime" = 0 OR "pages"."endtime" > 1281620460) ') . 'AND NOT "pages"."t3ver_state" > 0 AND "pages"."doktype" < 200 AND ("pages"."fe_group" = \'\' ') . 'OR "pages"."fe_group" IS NULL OR "pages"."fe_group" = \'0\' OR \',\'||"pages"."fe_group"||\',\' LIKE \'%,0,%\' ') . 'OR \',\'||"pages"."fe_group"||\',\' LIKE \'%,-1,%\')';
+               $query = $this->cleanSql($GLOBALS['TYPO3_DB']->SELECTquery('*', 'pages', 'pid=0 AND pages.deleted=0 AND pages.hidden=0 AND pages.starttime<=1281620460 ' . 'AND (pages.endtime=0 OR pages.endtime>1281620460) AND NOT pages.t3ver_state>0 ' . 'AND pages.doktype<200 AND (pages.fe_group=\'\' OR pages.fe_group IS NULL OR ' . 'pages.fe_group=\'0\' OR FIND_IN_SET(\'0\',pages.fe_group) OR FIND_IN_SET(\'-1\',pages.fe_group))'));
+               $expected = 'SELECT * FROM "pages" WHERE "pid" = 0 AND "pages"."deleted" = 0 AND "pages"."hidden" = 0 ' . 'AND "pages"."starttime" <= 1281620460 AND ("pages"."endtime" = 0 OR "pages"."endtime" > 1281620460) ' . 'AND NOT "pages"."t3ver_state" > 0 AND "pages"."doktype" < 200 AND ("pages"."fe_group" = \'\' ' . 'OR "pages"."fe_group" IS NULL OR "pages"."fe_group" = \'0\' OR \',\'||"pages"."fe_group"||\',\' LIKE \'%,0,%\' ' . 'OR \',\'||"pages"."fe_group"||\',\' LIKE \'%,-1,%\')';
                $this->assertEquals($expected, $query);
        }
 
@@ -244,8 +244,8 @@ class DatabaseOracleTest extends BaseTestCase {
         */
        public function cachingFrameworkQueryIsProperlyQuoted() {
                $currentTime = time();
-               $query = $this->cleanSql($GLOBALS['TYPO3_DB']->SELECTquery('content', 'cache_hash', ((('identifier = ' . $GLOBALS['TYPO3_DB']->fullQuoteStr('abbbabaf2d4b3f9a63e8dde781f1c106', 'cache_hash')) . ' AND (crdate + lifetime >= ') . $currentTime) . ' OR lifetime = 0)'));
-               $expected = ('SELECT "content" FROM "cache_hash" WHERE "identifier" = \'abbbabaf2d4b3f9a63e8dde781f1c106\' AND ("crdate"+"lifetime" >= ' . $currentTime) . ' OR "lifetime" = 0)';
+               $query = $this->cleanSql($GLOBALS['TYPO3_DB']->SELECTquery('content', 'cache_hash', 'identifier = ' . $GLOBALS['TYPO3_DB']->fullQuoteStr('abbbabaf2d4b3f9a63e8dde781f1c106', 'cache_hash') . ' AND (crdate + lifetime >= ' . $currentTime . ' OR lifetime = 0)'));
+               $expected = 'SELECT "content" FROM "cache_hash" WHERE "identifier" = \'abbbabaf2d4b3f9a63e8dde781f1c106\' AND ("crdate"+"lifetime" >= ' . $currentTime . ' OR "lifetime" = 0)';
                $this->assertEquals($expected, $query);
        }
 
@@ -255,8 +255,8 @@ class DatabaseOracleTest extends BaseTestCase {
         */
        public function calculatedFieldsAreProperlyQuoted() {
                $currentTime = time();
-               $query = $this->cleanSql($GLOBALS['TYPO3_DB']->SELECTquery('identifier', 'cachingframework_cache_pages', ('crdate + lifetime < ' . $currentTime) . ' AND lifetime > 0'));
-               $expected = ('SELECT "identifier" FROM "cachingframework_cache_pages" WHERE "crdate"+"lifetime" < ' . $currentTime) . ' AND "lifetime" > 0';
+               $query = $this->cleanSql($GLOBALS['TYPO3_DB']->SELECTquery('identifier', 'cachingframework_cache_pages', 'crdate + lifetime < ' . $currentTime . ' AND lifetime > 0'));
+               $expected = 'SELECT "identifier" FROM "cachingframework_cache_pages" WHERE "crdate"+"lifetime" < ' . $currentTime . ' AND "lifetime" > 0';
                $this->assertEquals($expected, $query);
        }
 
@@ -425,7 +425,7 @@ class DatabaseOracleTest extends BaseTestCase {
         */
        public function fieldFromAliasInJoinIsRemapped() {
                $selectFields = 'cat.uid, cat_mm.uid_local, news.uid';
-               $fromTables = ('tt_news_cat AS cat' . ' INNER JOIN tt_news_cat_mm AS cat_mm ON cat.uid = cat_mm.uid_foreign') . ' INNER JOIN tt_news AS news ON news.uid = cat_mm.uid_local';
+               $fromTables = 'tt_news_cat AS cat' . ' INNER JOIN tt_news_cat_mm AS cat_mm ON cat.uid = cat_mm.uid_foreign' . ' INNER JOIN tt_news AS news ON news.uid = cat_mm.uid_local';
                $whereClause = '1=1';
                $groupBy = '';
                $orderBy = '';
@@ -488,7 +488,7 @@ class DatabaseOracleTest extends BaseTestCase {
        public function aliasRemappingSupportsNestedSubqueries() {
                $selectFields = 'foo.uid';
                $fromTables = 'tt_news AS foo';
-               $whereClause = (('uid IN (' . 'SELECT foobar.uid_local FROM tt_news_cat_mm AS foobar WHERE uid_foreign IN (') . 'SELECT uid FROM tt_news_cat WHERE deleted = 0') . '))';
+               $whereClause = 'uid IN (' . 'SELECT foobar.uid_local FROM tt_news_cat_mm AS foobar WHERE uid_foreign IN (' . 'SELECT uid FROM tt_news_cat WHERE deleted = 0' . '))';
                $groupBy = '';
                $orderBy = '';
                $remappedParameters = $GLOBALS['TYPO3_DB']->_call('map_remapSELECTQueryParts', $selectFields, $fromTables, $whereClause, $groupBy, $orderBy);
@@ -554,7 +554,7 @@ class DatabaseOracleTest extends BaseTestCase {
         * @see http://bugs.typo3.org/view.php?id=14479
         */
        public function instrIsUsedForCEOnPages() {
-               $select = $this->cleanSql($GLOBALS['TYPO3_DB']->SELECTquery('*', 'tt_content', (((((('uid IN (62) AND tt_content.deleted=0 AND tt_content.t3ver_state<=0' . ' AND tt_content.hidden=0 AND (tt_content.starttime<=1264487640)') . ' AND (tt_content.endtime=0 OR tt_content.endtime>1264487640)') . ' AND (tt_content.fe_group=\'\' OR tt_content.fe_group IS NULL OR tt_content.fe_group=\'0\'') . ' OR (tt_content.fe_group LIKE \'%,0,%\' OR tt_content.fe_group LIKE \'0,%\' OR tt_content.fe_group LIKE \'%,0\'') . ' OR tt_content.fe_group=\'0\')') . ' OR (tt_content.fe_group LIKE\'%,-1,%\' OR tt_content.fe_group LIKE \'-1,%\' OR tt_content.fe_group LIKE \'%,-1\'') . ' OR tt_content.fe_group=\'-1\'))'));
+               $select = $this->cleanSql($GLOBALS['TYPO3_DB']->SELECTquery('*', 'tt_content', 'uid IN (62) AND tt_content.deleted=0 AND tt_content.t3ver_state<=0' . ' AND tt_content.hidden=0 AND (tt_content.starttime<=1264487640)' . ' AND (tt_content.endtime=0 OR tt_content.endtime>1264487640)' . ' AND (tt_content.fe_group=\'\' OR tt_content.fe_group IS NULL OR tt_content.fe_group=\'0\'' . ' OR (tt_content.fe_group LIKE \'%,0,%\' OR tt_content.fe_group LIKE \'0,%\' OR tt_content.fe_group LIKE \'%,0\'' . ' OR tt_content.fe_group=\'0\')' . ' OR (tt_content.fe_group LIKE\'%,-1,%\' OR tt_content.fe_group LIKE \'-1,%\' OR tt_content.fe_group LIKE \'%,-1\'' . ' OR tt_content.fe_group=\'-1\'))'));
                $expected = 'SELECT * FROM "tt_content"';
                $expected .= ' WHERE "uid" IN (62) AND "tt_content"."deleted" = 0 AND "tt_content"."t3ver_state" <= 0';
                $expected .= ' AND "tt_content"."hidden" = 0 AND ("tt_content"."starttime" <= 1264487640)';
@@ -690,9 +690,9 @@ class DatabaseOracleTest extends BaseTestCase {
         */
        public function cachingFrameworkQueryIsSupported() {
                $currentTime = time();
-               $query = $this->cleanSql($GLOBALS['TYPO3_DB']->DELETEquery('cachingframework_cache_hash_tags', ('identifier IN (' . $GLOBALS['TYPO3_DB']->SELECTsubquery('identifier', 'cachingframework_cache_pages', (('crdate + lifetime < ' . $currentTime) . ' AND lifetime > 0'))) . ')'));
+               $query = $this->cleanSql($GLOBALS['TYPO3_DB']->DELETEquery('cachingframework_cache_hash_tags', 'identifier IN (' . $GLOBALS['TYPO3_DB']->SELECTsubquery('identifier', 'cachingframework_cache_pages', ('crdate + lifetime < ' . $currentTime . ' AND lifetime > 0')) . ')'));
                $expected = 'DELETE FROM "cachingframework_cache_hash_tags" WHERE "identifier" IN (';
-               $expected .= ('SELECT "identifier" FROM "cachingframework_cache_pages" WHERE "crdate"+"lifetime" < ' . $currentTime) . ' AND "lifetime" > 0';
+               $expected .= 'SELECT "identifier" FROM "cachingframework_cache_pages" WHERE "crdate"+"lifetime" < ' . $currentTime . ' AND "lifetime" > 0';
                $expected .= ')';
                $this->assertEquals($expected, $query);
        }
@@ -704,7 +704,7 @@ class DatabaseOracleTest extends BaseTestCase {
        public function cachingFrameworkQueryIsRemapped() {
                $currentTime = time();
                $table = 'cachingframework_cache_hash_tags';
-               $where = ('identifier IN (' . $GLOBALS['TYPO3_DB']->SELECTsubquery('identifier', 'cachingframework_cache_pages', (('crdate + lifetime < ' . $currentTime) . ' AND lifetime > 0'))) . ')';
+               $where = 'identifier IN (' . $GLOBALS['TYPO3_DB']->SELECTsubquery('identifier', 'cachingframework_cache_pages', ('crdate + lifetime < ' . $currentTime . ' AND lifetime > 0')) . ')';
                // Perform remapping (as in method exec_DELETEquery)
                if ($tableArray = $GLOBALS['TYPO3_DB']->_call('map_needMapping', $table)) {
                        // Where clause:
@@ -718,7 +718,7 @@ class DatabaseOracleTest extends BaseTestCase {
                }
                $query = $this->cleanSql($GLOBALS['TYPO3_DB']->DELETEquery($table, $where));
                $expected = 'DELETE FROM "cf_cache_hash_tags" WHERE "identifier" IN (';
-               $expected .= ('SELECT "identifier" FROM "cf_cache_pages" WHERE "crdate"+"lifetime" < ' . $currentTime) . ' AND "lifetime" > 0';
+               $expected .= 'SELECT "identifier" FROM "cf_cache_pages" WHERE "crdate"+"lifetime" < ' . $currentTime . ' AND "lifetime" > 0';
                $expected .= ')';
                $this->assertEquals($expected, $query);
        }
@@ -728,7 +728,7 @@ class DatabaseOracleTest extends BaseTestCase {
         * @see http://bugs.typo3.org/view.php?id=12758
         */
        public function existsWhereClauseIsProperlyQuoted() {
-               $query = $this->cleanSql($GLOBALS['TYPO3_DB']->SELECTquery('*', 'tx_crawler_process', ('active = 0 AND NOT EXISTS (' . $GLOBALS['TYPO3_DB']->SELECTsubquery('*', 'tx_crawler_queue', 'tx_crawler_queue.process_id = tx_crawler_process.process_id AND tx_crawler_queue.exec_time = 0)')) . ')'));
+               $query = $this->cleanSql($GLOBALS['TYPO3_DB']->SELECTquery('*', 'tx_crawler_process', 'active = 0 AND NOT EXISTS (' . $GLOBALS['TYPO3_DB']->SELECTsubquery('*', 'tx_crawler_queue', 'tx_crawler_queue.process_id = tx_crawler_process.process_id AND tx_crawler_queue.exec_time = 0)') . ')'));
                $expected = 'SELECT * FROM "tx_crawler_process" WHERE "active" = 0 AND NOT EXISTS (';
                $expected .= 'SELECT * FROM "tx_crawler_queue" WHERE "tx_crawler_queue"."process_id" = "tx_crawler_process"."process_id" AND "tx_crawler_queue"."exec_time" = 0';
                $expected .= ')';
@@ -742,7 +742,7 @@ class DatabaseOracleTest extends BaseTestCase {
        public function subqueryIsRemappedForExistsWhereClause() {
                $selectFields = '*';
                $fromTables = 'tx_crawler_process';
-               $whereClause = ('active = 0 AND NOT EXISTS (' . $GLOBALS['TYPO3_DB']->SELECTsubquery('*', 'tx_crawler_queue', 'tx_crawler_queue.process_id = tx_crawler_process.process_id AND tx_crawler_queue.exec_time = 0')) . ')';
+               $whereClause = 'active = 0 AND NOT EXISTS (' . $GLOBALS['TYPO3_DB']->SELECTsubquery('*', 'tx_crawler_queue', 'tx_crawler_queue.process_id = tx_crawler_process.process_id AND tx_crawler_queue.exec_time = 0') . ')';
                $groupBy = '';
                $orderBy = '';
                $remappedParameters = $GLOBALS['TYPO3_DB']->_call('map_remapSELECTQueryParts', $selectFields, $fromTables, $whereClause, $groupBy, $orderBy);
@@ -761,7 +761,7 @@ class DatabaseOracleTest extends BaseTestCase {
         * @see http://bugs.typo3.org/view.php?id=13135
         */
        public function caseStatementIsProperlyQuoted() {
-               $query = $this->cleanSql($GLOBALS['TYPO3_DB']->SELECTquery((((((('process_id, CASE active' . ' WHEN 1 THEN ') . $GLOBALS['TYPO3_DB']->fullQuoteStr('one', 'tx_crawler_process')) . ' WHEN 2 THEN ') . $GLOBALS['TYPO3_DB']->fullQuoteStr('two', 'tx_crawler_process')) . ' ELSE ') . $GLOBALS['TYPO3_DB']->fullQuoteStr('out of range', 'tx_crawler_process')) . ' END AS number', 'tx_crawler_process', '1=1'));
+               $query = $this->cleanSql($GLOBALS['TYPO3_DB']->SELECTquery('process_id, CASE active' . ' WHEN 1 THEN ' . $GLOBALS['TYPO3_DB']->fullQuoteStr('one', 'tx_crawler_process') . ' WHEN 2 THEN ' . $GLOBALS['TYPO3_DB']->fullQuoteStr('two', 'tx_crawler_process') . ' ELSE ' . $GLOBALS['TYPO3_DB']->fullQuoteStr('out of range', 'tx_crawler_process') . ' END AS number', 'tx_crawler_process', '1=1'));
                $expected = 'SELECT "process_id", CASE "active" WHEN 1 THEN \'one\' WHEN 2 THEN \'two\' ELSE \'out of range\' END AS "number" FROM "tx_crawler_process" WHERE 1 = 1';
                $this->assertEquals($expected, $query);
        }
@@ -771,7 +771,7 @@ class DatabaseOracleTest extends BaseTestCase {
         * @see http://bugs.typo3.org/view.php?id=13135
         */
        public function caseStatementIsProperlyRemapped() {
-               $selectFields = (((((('process_id, CASE active' . ' WHEN 1 THEN ') . $GLOBALS['TYPO3_DB']->fullQuoteStr('one', 'tx_crawler_process')) . ' WHEN 2 THEN ') . $GLOBALS['TYPO3_DB']->fullQuoteStr('two', 'tx_crawler_process')) . ' ELSE ') . $GLOBALS['TYPO3_DB']->fullQuoteStr('out of range', 'tx_crawler_process')) . ' END AS number';
+               $selectFields = 'process_id, CASE active' . ' WHEN 1 THEN ' . $GLOBALS['TYPO3_DB']->fullQuoteStr('one', 'tx_crawler_process') . ' WHEN 2 THEN ' . $GLOBALS['TYPO3_DB']->fullQuoteStr('two', 'tx_crawler_process') . ' ELSE ' . $GLOBALS['TYPO3_DB']->fullQuoteStr('out of range', 'tx_crawler_process') . ' END AS number';
                $fromTables = 'tx_crawler_process';
                $whereClause = '1=1';
                $groupBy = '';
@@ -788,7 +788,7 @@ class DatabaseOracleTest extends BaseTestCase {
         * @see http://bugs.typo3.org/view.php?id=13135
         */
        public function caseStatementWithExternalTableIsProperlyRemapped() {
-               $selectFields = (((((('process_id, CASE tt_news.uid' . ' WHEN 1 THEN ') . $GLOBALS['TYPO3_DB']->fullQuoteStr('one', 'tt_news')) . ' WHEN 2 THEN ') . $GLOBALS['TYPO3_DB']->fullQuoteStr('two', 'tt_news')) . ' ELSE ') . $GLOBALS['TYPO3_DB']->fullQuoteStr('out of range', 'tt_news')) . ' END AS number';
+               $selectFields = 'process_id, CASE tt_news.uid' . ' WHEN 1 THEN ' . $GLOBALS['TYPO3_DB']->fullQuoteStr('one', 'tt_news') . ' WHEN 2 THEN ' . $GLOBALS['TYPO3_DB']->fullQuoteStr('two', 'tt_news') . ' ELSE ' . $GLOBALS['TYPO3_DB']->fullQuoteStr('out of range', 'tt_news') . ' END AS number';
                $fromTables = 'tx_crawler_process, tt_news';
                $whereClause = '1=1';
                $groupBy = '';
@@ -805,7 +805,7 @@ class DatabaseOracleTest extends BaseTestCase {
         * @see http://bugs.typo3.org/view.php?id=13134
         */
        public function locateStatementIsProperlyQuoted() {
-               $query = $this->cleanSql($GLOBALS['TYPO3_DB']->SELECTquery((((('*, CASE WHEN' . ' LOCATE(') . $GLOBALS['TYPO3_DB']->fullQuoteStr('(fce)', 'tx_templavoila_tmplobj')) . ', datastructure)>0 THEN 2') . ' ELSE 1') . ' END AS scope', 'tx_templavoila_tmplobj', '1=1'));
+               $query = $this->cleanSql($GLOBALS['TYPO3_DB']->SELECTquery('*, CASE WHEN' . ' LOCATE(' . $GLOBALS['TYPO3_DB']->fullQuoteStr('(fce)', 'tx_templavoila_tmplobj') . ', datastructure)>0 THEN 2' . ' ELSE 1' . ' END AS scope', 'tx_templavoila_tmplobj', '1=1'));
                $expected = 'SELECT *, CASE WHEN INSTR("datastructure", \'(fce)\') > 0 THEN 2 ELSE 1 END AS "scope" FROM "tx_templavoila_tmplobj" WHERE 1 = 1';
                $this->assertEquals($expected, $query);
        }
@@ -815,7 +815,7 @@ class DatabaseOracleTest extends BaseTestCase {
         * @see http://bugs.typo3.org/view.php?id=13134
         */
        public function locateStatementWithPositionIsProperlyQuoted() {
-               $query = $this->cleanSql($GLOBALS['TYPO3_DB']->SELECTquery((((('*, CASE WHEN' . ' LOCATE(') . $GLOBALS['TYPO3_DB']->fullQuoteStr('(fce)', 'tx_templavoila_tmplobj')) . ', datastructure, 4)>0 THEN 2') . ' ELSE 1') . ' END AS scope', 'tx_templavoila_tmplobj', '1=1'));
+               $query = $this->cleanSql($GLOBALS['TYPO3_DB']->SELECTquery('*, CASE WHEN' . ' LOCATE(' . $GLOBALS['TYPO3_DB']->fullQuoteStr('(fce)', 'tx_templavoila_tmplobj') . ', datastructure, 4)>0 THEN 2' . ' ELSE 1' . ' END AS scope', 'tx_templavoila_tmplobj', '1=1'));
                $expected = 'SELECT *, CASE WHEN INSTR("datastructure", \'(fce)\', 4) > 0 THEN 2 ELSE 1 END AS "scope" FROM "tx_templavoila_tmplobj" WHERE 1 = 1';
                $this->assertEquals($expected, $query);
        }
index d4246bc..5d01b92 100644 (file)
@@ -86,9 +86,9 @@ class DatabasePostgresqlTest extends BaseTestCase {
                foreach ($GLOBALS['TYPO3_DB']->cache_fieldType as $table => $fieldTypes) {
                        $tableDef = $GLOBALS['TYPO3_DB']->_call('map_needMapping', $table);
                        if (in_array($table, $tablesWithMapping)) {
-                               self::assertTrue(is_array($tableDef), ('Table ' . $table) . ' was expected to need mapping');
+                               self::assertTrue(is_array($tableDef), 'Table ' . $table . ' was expected to need mapping');
                        } else {
-                               self::assertFalse($tableDef, ('Table ' . $table) . ' was not expected to need mapping');
+                               self::assertFalse($tableDef, 'Table ' . $table . ' was not expected to need mapping');
                        }
                }
        }
index ef93924..e977f1b 100644 (file)
@@ -44,7 +44,7 @@ class FakeDatabaseConnection {
                $db->clearCachedFieldInfo();
                $db->_call('initInternalVariables');
                require_once \TYPO3\CMS\Core\Extension\ExtensionManager::extPath('adodb') . 'adodb/adodb.inc.php';
-               require_once ((\TYPO3\CMS\Core\Extension\ExtensionManager::extPath('adodb') . 'adodb/drivers/adodb-') . $driver) . '.inc.php';
+               require_once \TYPO3\CMS\Core\Extension\ExtensionManager::extPath('adodb') . 'adodb/drivers/adodb-' . $driver . '.inc.php';
                $handlerKey = '_DEFAULT';
                $db->lastHandlerKey = $handlerKey;
                $db->handlerInstance[$handlerKey] = \TYPO3\CMS\Core\Utility\GeneralUtility::makeInstance('ADODB_' . $driver);
index c5bee58..42566d5 100644 (file)
@@ -149,7 +149,7 @@ FROM pages WHERE pid IN (1,2,3,4)';
         * @test
         */
        public function parseWhereClauseReturnsArray() {
-               $parseString = ('uid IN (1,2) AND (starttime < ' . time()) . ' OR cruser_id + 10 < 20)';
+               $parseString = 'uid IN (1,2) AND (starttime < ' . time() . ' OR cruser_id + 10 < 20)';
                $where = $this->fixture->parseWhereClause($parseString);
                $this->assertTrue(is_array($where), $where);
                $this->assertTrue(empty($parseString), 'parseString is not empty');
@@ -710,7 +710,7 @@ FROM pages WHERE pid IN (1,2,3,4)';
                $components = $this->fixture->_callRef('parseSELECT', $sql);
                $components['parameters'][':pid'][0] = $pid;
                $query = $this->cleanSql($this->fixture->_callRef('compileSELECT', $components));
-               $expected = ('SELECT * FROM pages WHERE pid = ' . $pid) . ' AND title NOT LIKE \':pid\'';
+               $expected = 'SELECT * FROM pages WHERE pid = ' . $pid . ' AND title NOT LIKE \':pid\'';
                $this->assertEquals($expected, $query);
        }