Completed
Push — master ( 58fd6d...05af0e )
by Lars
01:50
created
src/voku/db/DB.php 1 patch
Spacing   +36 added lines, -36 removed lines patch added patch discarded remove patch
@@ -433,19 +433,19 @@  discard block
 block discarded – undo
433 433
         }
434 434
 
435 435
         if ($_connector === 'NOT IN' || $_connector === 'IN') {
436
-          $_value = '(' . \implode(',', $_value) . ')';
436
+          $_value = '('.\implode(',', $_value).')';
437 437
         } elseif ($_connector === 'NOT BETWEEN' || $_connector === 'BETWEEN') {
438
-          $_value = '(' . \implode(' AND ', $_value) . ')';
438
+          $_value = '('.\implode(' AND ', $_value).')';
439 439
         } elseif ($firstKey && $firstValue) {
440 440
 
441 441
           if (\strpos($firstKey, ' +') !== false) {
442 442
             $firstKey = \str_replace(' +', '', $firstKey);
443
-            $_value = $firstKey . ' + ' . $firstValue;
443
+            $_value = $firstKey.' + '.$firstValue;
444 444
           }
445 445
 
446 446
           if (\strpos($firstKey, ' -') !== false) {
447 447
             $firstKey = \str_replace(' -', '', $firstKey);
448
-            $_value = $firstKey . ' - ' . $firstValue;
448
+            $_value = $firstKey.' - '.$firstValue;
449 449
           }
450 450
         }
451 451
 
@@ -493,7 +493,7 @@  discard block
 block discarded – undo
493 493
           $valueInner = "''";
494 494
         }
495 495
 
496
-        $sql .= ' ' . $_glueHelperInner . ' ' . $quoteString . ' ' . $_connector . ' ' . $valueInner . " \n";
496
+        $sql .= ' '.$_glueHelperInner.' '.$quoteString.' '.$_connector.' '.$valueInner." \n";
497 497
         $tmpCounter++;
498 498
       }
499 499
 
@@ -580,7 +580,7 @@  discard block
 block discarded – undo
580 580
 
581 581
       // add ":" if needed
582 582
       if (\strpos($name, ':') !== 0) {
583
-        $nameTmp = ':' . $name;
583
+        $nameTmp = ':'.$name;
584 584
       } else {
585 585
         $nameTmp = $name;
586 586
       }
@@ -773,7 +773,7 @@  discard block
 block discarded – undo
773 773
         $this->connected = $this->doctrine_connection->isConnected();
774 774
 
775 775
         if (!$this->connected) {
776
-          $error = 'Error connecting to mysql server: ' . $this->doctrine_connection->errorInfo();
776
+          $error = 'Error connecting to mysql server: '.$this->doctrine_connection->errorInfo();
777 777
           $this->debug->displayError($error, false);
778 778
           throw new DBConnectException($error, 101);
779 779
         }
@@ -790,7 +790,7 @@  discard block
 block discarded – undo
790 790
         $this->connected = $this->doctrine_connection->isConnected();
791 791
 
792 792
         if (!$this->connected) {
793
-          $error = 'Error connecting to mysql server: ' . $this->doctrine_connection->errorInfo();
793
+          $error = 'Error connecting to mysql server: '.$this->doctrine_connection->errorInfo();
794 794
           $this->debug->displayError($error, false);
795 795
           throw new DBConnectException($error, 101);
796 796
         }
@@ -853,7 +853,7 @@  discard block
 block discarded – undo
853 853
       );
854 854
 
855 855
     } catch (\Exception $e) {
856
-      $error = 'Error connecting to mysql server: ' . $e->getMessage();
856
+      $error = 'Error connecting to mysql server: '.$e->getMessage();
857 857
       $this->debug->displayError($error, false);
858 858
       throw new DBConnectException($error, 100, $e);
859 859
     }
@@ -861,7 +861,7 @@  discard block
 block discarded – undo
861 861
 
862 862
     $errno = \mysqli_connect_errno();
863 863
     if (!$this->connected || $errno) {
864
-      $error = 'Error connecting to mysql server: ' . \mysqli_connect_error() . ' (' . $errno . ')';
864
+      $error = 'Error connecting to mysql server: '.\mysqli_connect_error().' ('.$errno.')';
865 865
       $this->debug->displayError($error, false);
866 866
       throw new DBConnectException($error, 101);
867 867
     }
@@ -902,10 +902,10 @@  discard block
 block discarded – undo
902 902
     }
903 903
 
904 904
     if ($databaseName) {
905
-      $databaseName = $this->quote_string(\trim($databaseName)) . '.';
905
+      $databaseName = $this->quote_string(\trim($databaseName)).'.';
906 906
     }
907 907
 
908
-    $sql = 'DELETE FROM ' . $databaseName . $this->quote_string($table) . " WHERE ($WHERE)";
908
+    $sql = 'DELETE FROM '.$databaseName.$this->quote_string($table)." WHERE ($WHERE)";
909 909
 
910 910
     return $this->query($sql);
911 911
   }
@@ -1108,7 +1108,7 @@  discard block
 block discarded – undo
1108 1108
 
1109 1109
     if ($useCache === true) {
1110 1110
       $cache = new Cache(null, null, false, $useCache);
1111
-      $cacheKey = 'sql-' . \md5($query);
1111
+      $cacheKey = 'sql-'.\md5($query);
1112 1112
 
1113 1113
       if (
1114 1114
           $cache->getCacheIsReady() === true
@@ -1283,7 +1283,7 @@  discard block
 block discarded – undo
1283 1283
     }
1284 1284
 
1285 1285
     if (
1286
-        '' . $hostname . $username . $password . $database . $port . $charset == '' . $port . $charset
1286
+        ''.$hostname.$username.$password.$database.$port.$charset == ''.$port.$charset
1287 1287
         &&
1288 1288
         null !== $firstInstance
1289 1289
     ) {
@@ -1301,11 +1301,11 @@  discard block
 block discarded – undo
1301 1301
       } else {
1302 1302
         $extra_config_value_tmp = (string)$extra_config_value;
1303 1303
       }
1304
-      $extra_config_string .= $extra_config_key . $extra_config_value_tmp;
1304
+      $extra_config_string .= $extra_config_key.$extra_config_value_tmp;
1305 1305
     }
1306 1306
 
1307 1307
     $connection = \md5(
1308
-        $hostname . $username . $password . $database . $port . $charset . (int)$exit_on_error . (int)$echo_on_error . $logger_class_name . $logger_level . $extra_config_string
1308
+        $hostname.$username.$password.$database.$port.$charset.(int)$exit_on_error.(int)$echo_on_error.$logger_class_name.$logger_level.$extra_config_string
1309 1309
     );
1310 1310
 
1311 1311
     if (!isset($instance[$connection])) {
@@ -1445,10 +1445,10 @@  discard block
 block discarded – undo
1445 1445
     $SET = $this->_parseArrayPair($data);
1446 1446
 
1447 1447
     if ($databaseName) {
1448
-      $databaseName = $this->quote_string(\trim($databaseName)) . '.';
1448
+      $databaseName = $this->quote_string(\trim($databaseName)).'.';
1449 1449
     }
1450 1450
 
1451
-    $sql = 'INSERT INTO ' . $databaseName . $this->quote_string($table) . " SET $SET";
1451
+    $sql = 'INSERT INTO '.$databaseName.$this->quote_string($table)." SET $SET";
1452 1452
 
1453 1453
     return $this->query($sql);
1454 1454
   }
@@ -1980,11 +1980,11 @@  discard block
 block discarded – undo
1980 1980
 
1981 1981
       // exit if we have more then 3 "DB server has gone away"-errors
1982 1982
       if ($RECONNECT_COUNTER > 3) {
1983
-        $this->debug->mailToAdmin('DB-Fatal-Error', $errorMessage . '(' . $errorNumber . ') ' . ":\n<br />" . $sql, 5);
1983
+        $this->debug->mailToAdmin('DB-Fatal-Error', $errorMessage.'('.$errorNumber.') '.":\n<br />".$sql, 5);
1984 1984
         throw new DBGoneAwayException($errorMessage);
1985 1985
       }
1986 1986
 
1987
-      $this->debug->mailToAdmin('DB-Error', $errorMessage . '(' . $errorNumber . ') ' . ":\n<br />" . $sql);
1987
+      $this->debug->mailToAdmin('DB-Error', $errorMessage.'('.$errorNumber.') '.":\n<br />".$sql);
1988 1988
 
1989 1989
       // reconnect
1990 1990
       $RECONNECT_COUNTER++;
@@ -1998,7 +1998,7 @@  discard block
 block discarded – undo
1998 1998
       return false;
1999 1999
     }
2000 2000
 
2001
-    $this->debug->mailToAdmin('SQL-Error', $errorMessage . '(' . $errorNumber . ') ' . ":\n<br />" . $sql);
2001
+    $this->debug->mailToAdmin('SQL-Error', $errorMessage.'('.$errorNumber.') '.":\n<br />".$sql);
2002 2002
 
2003 2003
     $force_exception_after_error = null; // auto
2004 2004
     if ($this->in_transaction === true) {
@@ -2006,7 +2006,7 @@  discard block
 block discarded – undo
2006 2006
     }
2007 2007
     // this query returned an error, we must display it (only for dev) !!!
2008 2008
 
2009
-    $this->debug->displayError($errorMessage . '(' . $errorNumber . ') ' . ' | ' . $sql, $force_exception_after_error);
2009
+    $this->debug->displayError($errorMessage.'('.$errorNumber.') '.' | '.$sql, $force_exception_after_error);
2010 2010
 
2011 2011
     return false;
2012 2012
   }
@@ -2029,7 +2029,7 @@  discard block
 block discarded – undo
2029 2029
         )
2030 2030
     );
2031 2031
 
2032
-    return '`' . $str . '`';
2032
+    return '`'.$str.'`';
2033 2033
   }
2034 2034
 
2035 2035
   /**
@@ -2099,10 +2099,10 @@  discard block
 block discarded – undo
2099 2099
     $values = \implode(',', $data);
2100 2100
 
2101 2101
     if ($databaseName) {
2102
-      $databaseName = $this->quote_string(\trim($databaseName)) . '.';
2102
+      $databaseName = $this->quote_string(\trim($databaseName)).'.';
2103 2103
     }
2104 2104
 
2105
-    $sql = 'REPLACE INTO ' . $databaseName . $this->quote_string($table) . " ($columns) VALUES ($values)";
2105
+    $sql = 'REPLACE INTO '.$databaseName.$this->quote_string($table)." ($columns) VALUES ($values)";
2106 2106
 
2107 2107
     return $this->query($sql);
2108 2108
   }
@@ -2246,7 +2246,7 @@  discard block
 block discarded – undo
2246 2246
     $var = $this->escape($var, false, false, null);
2247 2247
 
2248 2248
     if (\is_string($var)) {
2249
-      $var = "'" . \trim($var, "'") . "'";
2249
+      $var = "'".\trim($var, "'")."'";
2250 2250
     }
2251 2251
 
2252 2252
     return $var;
@@ -2283,10 +2283,10 @@  discard block
 block discarded – undo
2283 2283
     }
2284 2284
 
2285 2285
     if ($databaseName) {
2286
-      $databaseName = $this->quote_string(\trim($databaseName)) . '.';
2286
+      $databaseName = $this->quote_string(\trim($databaseName)).'.';
2287 2287
     }
2288 2288
 
2289
-    $sql = 'SELECT * FROM ' . $databaseName . $this->quote_string($table) . " WHERE ($WHERE)";
2289
+    $sql = 'SELECT * FROM '.$databaseName.$this->quote_string($table)." WHERE ($WHERE)";
2290 2290
 
2291 2291
     return $this->query($sql);
2292 2292
   }
@@ -2388,16 +2388,16 @@  discard block
 block discarded – undo
2388 2388
       $return = mysqli_set_charset($this->mysqli_link, $charset);
2389 2389
 
2390 2390
       /** @noinspection PhpUsageOfSilenceOperatorInspection */
2391
-      @\mysqli_query($this->mysqli_link, 'SET CHARACTER SET ' . $charset);
2391
+      @\mysqli_query($this->mysqli_link, 'SET CHARACTER SET '.$charset);
2392 2392
       /** @noinspection PhpUsageOfSilenceOperatorInspection */
2393
-      @\mysqli_query($this->mysqli_link, "SET NAMES '" . $charset . "'");
2393
+      @\mysqli_query($this->mysqli_link, "SET NAMES '".$charset."'");
2394 2394
 
2395 2395
     } elseif ($this->isDoctrinePDOConnection() === true) {
2396 2396
 
2397 2397
       $doctrineWrappedConnection = $this->getDoctrinePDOConnection();
2398 2398
 
2399
-      $doctrineWrappedConnection->exec('SET CHARACTER SET ' . $charset);
2400
-      $doctrineWrappedConnection->exec("SET NAMES '" . $charset . "'");
2399
+      $doctrineWrappedConnection->exec('SET CHARACTER SET '.$charset);
2400
+      $doctrineWrappedConnection->exec("SET NAMES '".$charset."'");
2401 2401
 
2402 2402
       $return = true;
2403 2403
 
@@ -2542,7 +2542,7 @@  discard block
 block discarded – undo
2542 2542
    */
2543 2543
   public function table_exists(string $table): bool
2544 2544
   {
2545
-    $check = $this->query('SELECT 1 FROM ' . $this->quote_string($table));
2545
+    $check = $this->query('SELECT 1 FROM '.$this->quote_string($table));
2546 2546
 
2547 2547
     return $check !== false
2548 2548
            &&
@@ -2573,7 +2573,7 @@  discard block
 block discarded – undo
2573 2573
       $result = $callback($this);
2574 2574
       if ($result === false) {
2575 2575
         /** @noinspection ThrowRawExceptionInspection */
2576
-        throw new \Exception('call_user_func [' . $callback . '] === false');
2576
+        throw new \Exception('call_user_func ['.$callback.'] === false');
2577 2577
       }
2578 2578
 
2579 2579
       return $this->commit();
@@ -2632,10 +2632,10 @@  discard block
 block discarded – undo
2632 2632
     }
2633 2633
 
2634 2634
     if ($databaseName) {
2635
-      $databaseName = $this->quote_string(\trim($databaseName)) . '.';
2635
+      $databaseName = $this->quote_string(\trim($databaseName)).'.';
2636 2636
     }
2637 2637
 
2638
-    $sql = 'UPDATE ' . $databaseName . $this->quote_string($table) . " SET $SET WHERE ($WHERE)";
2638
+    $sql = 'UPDATE '.$databaseName.$this->quote_string($table)." SET $SET WHERE ($WHERE)";
2639 2639
 
2640 2640
     return $this->query($sql);
2641 2641
   }
Please login to merge, or discard this patch.
src/voku/db/Debug.php 1 patch
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -139,9 +139,9 @@  discard block
 block discarded – undo
139 139
     $this->logger(
140 140
         [
141 141
             'error',
142
-            '<strong>' . \date(
142
+            '<strong>'.\date(
143 143
                 'd. m. Y G:i:s'
144
-            ) . ' (' . $fileInfo['file'] . ' line: ' . $fileInfo['line'] . ') (sql-error):</strong> ' . $error . '<br>',
144
+            ).' ('.$fileInfo['file'].' line: '.$fileInfo['line'].') (sql-error):</strong> '.$error.'<br>',
145 145
         ]
146 146
     );
147 147
 
@@ -157,16 +157,16 @@  discard block
 block discarded – undo
157 157
 
158 158
       if (\PHP_SAPI === 'cli') {
159 159
         echo "\n";
160
-        echo 'file:line -> ' . $fileInfo['file'] . ':' . $fileInfo['line'] . "\n";
161
-        echo 'error: ' . \str_replace(  ["\r\n", "\n", "\r"], '', $error);
160
+        echo 'file:line -> '.$fileInfo['file'].':'.$fileInfo['line']."\n";
161
+        echo 'error: '.\str_replace(["\r\n", "\n", "\r"], '', $error);
162 162
         echo "\n";
163 163
       } else {
164 164
         echo '
165
-        <div class="OBJ-mysql-box" style="border: ' . $box_border . '; background: ' . $box_bg . '; padding: 10px; margin: 10px;">
165
+        <div class="OBJ-mysql-box" style="border: ' . $box_border.'; background: '.$box_bg.'; padding: 10px; margin: 10px;">
166 166
           <b style="font-size: 14px;">MYSQL Error:</b>
167 167
           <code style="display: block;">
168
-            file:line -> ' . $fileInfo['file'] . ':' . $fileInfo['line'] . '
169
-            ' . $error . '
168
+            file:line -> ' . $fileInfo['file'].':'.$fileInfo['line'].'
169
+            ' . $error.'
170 170
           </code>
171 171
         </div>
172 172
         ';
@@ -308,7 +308,7 @@  discard block
 block discarded – undo
308 308
       /** @noinspection PhpUsageOfSilenceOperatorInspection */
309 309
       $infoExtra = @\mysqli_info($tmpLink);
310 310
       if ($infoExtra) {
311
-        $infoExtra = ' | info => ' . $infoExtra;
311
+        $infoExtra = ' | info => '.$infoExtra;
312 312
       }
313 313
     }
314 314
 
@@ -316,14 +316,14 @@  discard block
 block discarded – undo
316 316
     // logging
317 317
     //
318 318
 
319
-    $info = 'time => ' . \round($duration, 5) . ' | results => ' . (int)$results . $infoExtra . ' | SQL => ' . UTF8::htmlentities($sql);
319
+    $info = 'time => '.\round($duration, 5).' | results => '.(int)$results.$infoExtra.' | SQL => '.UTF8::htmlentities($sql);
320 320
 
321 321
     $fileInfo = $this->getFileAndLineFromSql();
322 322
 
323 323
     return $this->logger(
324 324
         [
325 325
             $logLevel,
326
-            '<strong>' . \date('d. m. Y G:i:s') . ' (' . $fileInfo['file'] . ' line: ' . $fileInfo['line'] . '):</strong> ' . $info . '<br>',
326
+            '<strong>'.\date('d. m. Y G:i:s').' ('.$fileInfo['file'].' line: '.$fileInfo['line'].'):</strong> '.$info.'<br>',
327 327
             'sql',
328 328
         ]
329 329
     );
@@ -389,21 +389,21 @@  discard block
 block discarded – undo
389 389
         $this->logger(
390 390
             [
391 391
                 'debug',
392
-                $subject . ' | ' . $htmlBody,
392
+                $subject.' | '.$htmlBody,
393 393
             ]
394 394
         );
395 395
       } elseif ($priority > 3) {
396 396
         $this->logger(
397 397
             [
398 398
                 'error',
399
-                $subject . ' | ' . $htmlBody,
399
+                $subject.' | '.$htmlBody,
400 400
             ]
401 401
         );
402 402
       } elseif ($priority < 3) {
403 403
         $this->logger(
404 404
             [
405 405
                 'info',
406
-                $subject . ' | ' . $htmlBody,
406
+                $subject.' | '.$htmlBody,
407 407
             ]
408 408
         );
409 409
       }
Please login to merge, or discard this patch.