Completed
Push — master ( ac18ff...92b49e )
by Lars
01:41
created
src/voku/db/DB.php 1 patch
Spacing   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -438,9 +438,9 @@  discard block
 block discarded – undo
438 438
         }
439 439
 
440 440
         if ($_connector === 'NOT IN' || $_connector === 'IN') {
441
-          $_value = '(' . implode(',', $_value) . ')';
441
+          $_value = '('.implode(',', $_value).')';
442 442
         } elseif ($_connector === 'NOT BETWEEN' || $_connector === 'BETWEEN') {
443
-          $_value = '(' . implode(' AND ', $_value) . ')';
443
+          $_value = '('.implode(' AND ', $_value).')';
444 444
         }
445 445
 
446 446
       } else {
@@ -485,7 +485,7 @@  discard block
 block discarded – undo
485 485
           $_glueHelperInner = 'AND ('; // inner-loop "OR"-query glue
486 486
         }
487 487
 
488
-        $sql .= ' ' . $_glueHelperInner . ' ' . $quoteString . ' ' . $_connector . ' ' . $valueInner . " \n";
488
+        $sql .= ' '.$_glueHelperInner.' '.$quoteString.' '.$_connector.' '.$valueInner." \n";
489 489
         $tmpCounter++;
490 490
       }
491 491
 
@@ -679,7 +679,7 @@  discard block
 block discarded – undo
679 679
       );
680 680
 
681 681
     } catch (\Exception $e) {
682
-      $error = 'Error connecting to mysql server: ' . $e->getMessage();
682
+      $error = 'Error connecting to mysql server: '.$e->getMessage();
683 683
       $this->_debug->displayError($error, true);
684 684
       throw new DBConnectException($error, 100, $e);
685 685
     }
@@ -687,7 +687,7 @@  discard block
 block discarded – undo
687 687
 
688 688
     $errno = mysqli_connect_errno();
689 689
     if (!$this->connected || $errno) {
690
-      $error = 'Error connecting to mysql server: ' . \mysqli_connect_error() . ' (' . $errno . ')';
690
+      $error = 'Error connecting to mysql server: '.\mysqli_connect_error().' ('.$errno.')';
691 691
       $this->_debug->displayError($error, true);
692 692
       throw new DBConnectException($error, 101);
693 693
     }
@@ -728,10 +728,10 @@  discard block
 block discarded – undo
728 728
     }
729 729
 
730 730
     if ($databaseName) {
731
-      $databaseName = $this->quote_string(trim($databaseName)) . '.';
731
+      $databaseName = $this->quote_string(trim($databaseName)).'.';
732 732
     }
733 733
 
734
-    $sql = 'DELETE FROM ' . $databaseName . $this->quote_string($table) . " WHERE ($WHERE);";
734
+    $sql = 'DELETE FROM '.$databaseName.$this->quote_string($table)." WHERE ($WHERE);";
735 735
 
736 736
     return $this->query($sql);
737 737
   }
@@ -801,8 +801,8 @@  discard block
 block discarded – undo
801 801
         $nameTmp = substr($name, 1);
802 802
       }
803 803
 
804
-      $parseKeyInner = $nameTmp . '-' . $parseKey;
805
-      $sql = str_replace(':' . $nameTmp, $parseKeyInner, $sql);
804
+      $parseKeyInner = $nameTmp.'-'.$parseKey;
805
+      $sql = str_replace(':'.$nameTmp, $parseKeyInner, $sql);
806 806
     }
807 807
 
808 808
     foreach ($params as $name => $value) {
@@ -811,7 +811,7 @@  discard block
 block discarded – undo
811 811
         $nameTmp = substr($name, 1);
812 812
       }
813 813
 
814
-      $parseKeyInner = $nameTmp . '-' . $parseKey;
814
+      $parseKeyInner = $nameTmp.'-'.$parseKey;
815 815
       $sqlBefore = $sql;
816 816
 
817 817
       while (strpos($sql, $parseKeyInner) !== false) {
@@ -988,7 +988,7 @@  discard block
 block discarded – undo
988 988
 
989 989
     if ($useCache === true) {
990 990
       $cache = new Cache(null, null, false, $useCache);
991
-      $cacheKey = 'sql-' . md5($query);
991
+      $cacheKey = 'sql-'.md5($query);
992 992
 
993 993
       if (
994 994
           $cache->getCacheIsReady() === true
@@ -1098,7 +1098,7 @@  discard block
 block discarded – undo
1098 1098
     static $firstInstance = null;
1099 1099
 
1100 1100
     if (
1101
-        $hostname . $username . $password . $database . $port . $charset == ''
1101
+        $hostname.$username.$password.$database.$port.$charset == ''
1102 1102
         &&
1103 1103
         null !== $firstInstance
1104 1104
     ) {
@@ -1108,7 +1108,7 @@  discard block
 block discarded – undo
1108 1108
     $extra_config_string = '';
1109 1109
     if (is_array($extra_config) === true) {
1110 1110
       foreach ($extra_config as $extra_config_key => $extra_config_value) {
1111
-        $extra_config_string .= $extra_config_key . (string)$extra_config_value;
1111
+        $extra_config_string .= $extra_config_key.(string)$extra_config_value;
1112 1112
       }
1113 1113
     } else {
1114 1114
       // only for backward compatibility
@@ -1116,7 +1116,7 @@  discard block
 block discarded – undo
1116 1116
     }
1117 1117
 
1118 1118
     $connection = md5(
1119
-        $hostname . $username . $password . $database . $port . $charset . (int)$exit_on_error . (int)$echo_on_error . $logger_class_name . $logger_level . $extra_config_string
1119
+        $hostname.$username.$password.$database.$port.$charset.(int)$exit_on_error.(int)$echo_on_error.$logger_class_name.$logger_level.$extra_config_string
1120 1120
     );
1121 1121
 
1122 1122
     if (!isset($instance[$connection])) {
@@ -1203,10 +1203,10 @@  discard block
 block discarded – undo
1203 1203
     $SET = $this->_parseArrayPair($data);
1204 1204
 
1205 1205
     if ($databaseName) {
1206
-      $databaseName = $this->quote_string(trim($databaseName)) . '.';
1206
+      $databaseName = $this->quote_string(trim($databaseName)).'.';
1207 1207
     }
1208 1208
 
1209
-    $sql = 'INSERT INTO ' . $databaseName . $this->quote_string($table) . " SET $SET;";
1209
+    $sql = 'INSERT INTO '.$databaseName.$this->quote_string($table)." SET $SET;";
1210 1210
 
1211 1211
     return $this->query($sql);
1212 1212
   }
@@ -1520,11 +1520,11 @@  discard block
 block discarded – undo
1520 1520
 
1521 1521
       // exit if we have more then 3 "DB server has gone away"-errors
1522 1522
       if ($RECONNECT_COUNTER > 3) {
1523
-        $this->_debug->mailToAdmin('DB-Fatal-Error', $errorMessage . '(' . $errorNumber . ') ' . ":\n<br />" . $sql, 5);
1523
+        $this->_debug->mailToAdmin('DB-Fatal-Error', $errorMessage.'('.$errorNumber.') '.":\n<br />".$sql, 5);
1524 1524
         throw new DBGoneAwayException($errorMessage);
1525 1525
       }
1526 1526
 
1527
-      $this->_debug->mailToAdmin('DB-Error', $errorMessage . '(' . $errorNumber . ') ' . ":\n<br />" . $sql);
1527
+      $this->_debug->mailToAdmin('DB-Error', $errorMessage.'('.$errorNumber.') '.":\n<br />".$sql);
1528 1528
 
1529 1529
       // reconnect
1530 1530
       $RECONNECT_COUNTER++;
@@ -1538,7 +1538,7 @@  discard block
 block discarded – undo
1538 1538
       return false;
1539 1539
     }
1540 1540
 
1541
-    $this->_debug->mailToAdmin('SQL-Error', $errorMessage . '(' . $errorNumber . ') ' . ":\n<br />" . $sql);
1541
+    $this->_debug->mailToAdmin('SQL-Error', $errorMessage.'('.$errorNumber.') '.":\n<br />".$sql);
1542 1542
 
1543 1543
     $force_exception_after_error = null; // auto
1544 1544
     if ($this->_in_transaction === true) {
@@ -1546,7 +1546,7 @@  discard block
 block discarded – undo
1546 1546
     }
1547 1547
     // this query returned an error, we must display it (only for dev) !!!
1548 1548
 
1549
-    $this->_debug->displayError($errorMessage . '(' . $errorNumber . ') ' . ' | ' . $sql, $force_exception_after_error);
1549
+    $this->_debug->displayError($errorMessage.'('.$errorNumber.') '.' | '.$sql, $force_exception_after_error);
1550 1550
 
1551 1551
     return false;
1552 1552
   }
@@ -1569,7 +1569,7 @@  discard block
 block discarded – undo
1569 1569
         )
1570 1570
     );
1571 1571
 
1572
-    return '`' . $str . '`';
1572
+    return '`'.$str.'`';
1573 1573
   }
1574 1574
 
1575 1575
   /**
@@ -1640,10 +1640,10 @@  discard block
 block discarded – undo
1640 1640
     $values = implode(',', $data);
1641 1641
 
1642 1642
     if ($databaseName) {
1643
-      $databaseName = $this->quote_string(trim($databaseName)) . '.';
1643
+      $databaseName = $this->quote_string(trim($databaseName)).'.';
1644 1644
     }
1645 1645
 
1646
-    $sql = 'REPLACE INTO ' . $databaseName . $this->quote_string($table) . " ($columns) VALUES ($values);";
1646
+    $sql = 'REPLACE INTO '.$databaseName.$this->quote_string($table)." ($columns) VALUES ($values);";
1647 1647
 
1648 1648
     return $this->query($sql);
1649 1649
   }
@@ -1731,7 +1731,7 @@  discard block
 block discarded – undo
1731 1731
     $var = $this->escape($var, false, false, null);
1732 1732
 
1733 1733
     if (is_string($var)) {
1734
-      $var = "'" . trim($var, "'") . "'";
1734
+      $var = "'".trim($var, "'")."'";
1735 1735
     }
1736 1736
 
1737 1737
     return $var;
@@ -1768,10 +1768,10 @@  discard block
 block discarded – undo
1768 1768
     }
1769 1769
 
1770 1770
     if ($databaseName) {
1771
-      $databaseName = $this->quote_string(trim($databaseName)) . '.';
1771
+      $databaseName = $this->quote_string(trim($databaseName)).'.';
1772 1772
     }
1773 1773
 
1774
-    $sql = 'SELECT * FROM ' . $databaseName . $this->quote_string($table) . " WHERE ($WHERE);";
1774
+    $sql = 'SELECT * FROM '.$databaseName.$this->quote_string($table)." WHERE ($WHERE);";
1775 1775
 
1776 1776
     return $this->query($sql);
1777 1777
   }
@@ -1814,10 +1814,10 @@  discard block
 block discarded – undo
1814 1814
     $return = mysqli_set_charset($this->link, $charset);
1815 1815
     /** @noinspection PhpUsageOfSilenceOperatorInspection */
1816 1816
     /** @noinspection UsageOfSilenceOperatorInspection */
1817
-    @\mysqli_query($this->link, 'SET CHARACTER SET ' . $charset);
1817
+    @\mysqli_query($this->link, 'SET CHARACTER SET '.$charset);
1818 1818
     /** @noinspection PhpUsageOfSilenceOperatorInspection */
1819 1819
     /** @noinspection UsageOfSilenceOperatorInspection */
1820
-    @\mysqli_query($this->link, "SET NAMES '" . $charset . "'");
1820
+    @\mysqli_query($this->link, "SET NAMES '".$charset."'");
1821 1821
 
1822 1822
     return $return;
1823 1823
   }
@@ -1945,7 +1945,7 @@  discard block
 block discarded – undo
1945 1945
       $result = call_user_func($callback, $this);
1946 1946
       if ($result === false) {
1947 1947
         /** @noinspection ThrowRawExceptionInspection */
1948
-        throw new \Exception('call_user_func [' . $callback . '] === false');
1948
+        throw new \Exception('call_user_func ['.$callback.'] === false');
1949 1949
       }
1950 1950
 
1951 1951
       return $this->commit();
@@ -1998,10 +1998,10 @@  discard block
 block discarded – undo
1998 1998
     }
1999 1999
 
2000 2000
     if ($databaseName) {
2001
-      $databaseName = $this->quote_string(trim($databaseName)) . '.';
2001
+      $databaseName = $this->quote_string(trim($databaseName)).'.';
2002 2002
     }
2003 2003
 
2004
-    $sql = 'UPDATE ' . $databaseName . $this->quote_string($table) . " SET $SET WHERE ($WHERE);";
2004
+    $sql = 'UPDATE '.$databaseName.$this->quote_string($table)." SET $SET WHERE ($WHERE);";
2005 2005
 
2006 2006
     return $this->query($sql);
2007 2007
   }
@@ -2015,7 +2015,7 @@  discard block
 block discarded – undo
2015 2015
    */
2016 2016
   public function table_exists($table)
2017 2017
   {
2018
-    $check = $this->query('SELECT 1 FROM ' . $this->quote_string($table));
2018
+    $check = $this->query('SELECT 1 FROM '.$this->quote_string($table));
2019 2019
     if (
2020 2020
         $check !== false
2021 2021
         &&
Please login to merge, or discard this patch.
src/voku/db/Helper.php 1 patch
Spacing   +19 added lines, -19 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
     $optimized = 0;
30 30
     if (!empty($tables)) {
31 31
       foreach ($tables as $table) {
32
-        $optimize = 'OPTIMIZE TABLE ' . $dbConnection->quote_string($table);
32
+        $optimize = 'OPTIMIZE TABLE '.$dbConnection->quote_string($table);
33 33
         $result = $dbConnection->query($optimize);
34 34
         if ($result) {
35 35
           $optimized++;
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
     $optimized = 0;
58 58
     if (!empty($tables)) {
59 59
       foreach ($tables as $table) {
60
-        $optimize = 'REPAIR TABLE ' . $dbConnection->quote_string($table);
60
+        $optimize = 'REPAIR TABLE '.$dbConnection->quote_string($table);
61 61
         $result = $dbConnection->query($optimize);
62 62
         if ($result) {
63 63
           $optimized++;
@@ -176,23 +176,23 @@  discard block
 block discarded – undo
176 176
 
177 177
     $whereSQL = $dbConnection->_parseArrayPair($whereArray, 'AND');
178 178
     if ($whereSQL) {
179
-      $whereSQL = 'AND ' . $whereSQL;
179
+      $whereSQL = 'AND '.$whereSQL;
180 180
     }
181 181
 
182 182
     if ($databaseName) {
183
-      $databaseName = $dbConnection->quote_string(trim($databaseName)) . '.';
183
+      $databaseName = $dbConnection->quote_string(trim($databaseName)).'.';
184 184
     }
185 185
 
186 186
     // get the row
187
-    $query = 'SELECT ' . $dbConnection->quote_string($searchFieldName) . ', ' . $dbConnection->quote_string($idFieldName) . ' 
188
-      FROM ' . $databaseName . $dbConnection->quote_string($table) . '
187
+    $query = 'SELECT '.$dbConnection->quote_string($searchFieldName).', '.$dbConnection->quote_string($idFieldName).' 
188
+      FROM ' . $databaseName.$dbConnection->quote_string($table).'
189 189
       WHERE 1 = 1
190
-      ' . $whereSQL . '
190
+      ' . $whereSQL.'
191 191
     ';
192 192
 
193 193
     if ($useCache === true) {
194 194
       $cache = new Cache(null, null, false, $useCache);
195
-      $cacheKey = 'sql-phonetic-search-' . md5($query);
195
+      $cacheKey = 'sql-phonetic-search-'.md5($query);
196 196
 
197 197
       if (
198 198
           $cache->getCacheIsReady() === true
@@ -322,10 +322,10 @@  discard block
 block discarded – undo
322 322
     }
323 323
 
324 324
     if ($databaseName) {
325
-      $databaseName = $dbConnection->quote_string(trim($databaseName)) . '.';
325
+      $databaseName = $dbConnection->quote_string(trim($databaseName)).'.';
326 326
     }
327 327
 
328
-    $sql = 'SHOW COLUMNS FROM ' . $databaseName . $dbConnection->escape($table);
328
+    $sql = 'SHOW COLUMNS FROM '.$databaseName.$dbConnection->escape($table);
329 329
     $result = $dbConnection->query($sql);
330 330
 
331 331
     if ($result && $result->num_rows > 0) {
@@ -371,17 +371,17 @@  discard block
 block discarded – undo
371 371
 
372 372
     $whereSQL = $dbConnection->_parseArrayPair($whereArray, 'AND');
373 373
     if ($whereSQL) {
374
-      $whereSQL = 'AND ' . $whereSQL;
374
+      $whereSQL = 'AND '.$whereSQL;
375 375
     }
376 376
 
377 377
     if ($databaseName) {
378
-      $databaseName = $dbConnection->quote_string(trim($databaseName)) . '.';
378
+      $databaseName = $dbConnection->quote_string(trim($databaseName)).'.';
379 379
     }
380 380
 
381 381
     // get the row
382
-    $query = 'SELECT * FROM ' . $databaseName . $dbConnection->quote_string($table) . '
382
+    $query = 'SELECT * FROM '.$databaseName.$dbConnection->quote_string($table).'
383 383
       WHERE 1 = 1
384
-      ' . $whereSQL . '
384
+      ' . $whereSQL.'
385 385
     ';
386 386
     $result = $dbConnection->query($query);
387 387
 
@@ -401,11 +401,11 @@  discard block
 block discarded – undo
401 401
 
402 402
           if (!in_array($fieldName, $ignoreArray, true)) {
403 403
             if (array_key_exists($fieldName, $updateArray)) {
404
-              $insert_keys .= ',' . $fieldName;
404
+              $insert_keys .= ','.$fieldName;
405 405
               $insert_values .= ',?';
406 406
               $bindings[] = $updateArray[$fieldName]; // INFO: do not escape non selected data
407 407
             } else {
408
-              $insert_keys .= ',' . $fieldName;
408
+              $insert_keys .= ','.$fieldName;
409 409
               $insert_values .= ',?';
410 410
               $bindings[] = $value; // INFO: do not escape non selected data
411 411
             }
@@ -416,10 +416,10 @@  discard block
 block discarded – undo
416 416
         $insert_values = ltrim($insert_values, ',');
417 417
 
418 418
         // insert the "copied" row
419
-        $new_query = 'INSERT INTO ' . $databaseName . $dbConnection->quote_string($table) . ' 
420
-          (' . $insert_keys . ')
419
+        $new_query = 'INSERT INTO '.$databaseName.$dbConnection->quote_string($table).' 
420
+          (' . $insert_keys.')
421 421
           VALUES 
422
-          (' . $insert_values . ')
422
+          (' . $insert_values.')
423 423
         ';
424 424
         return $dbConnection->query($new_query, $bindings);
425 425
       }
Please login to merge, or discard this patch.