Completed
Push — master ( 059279...e0d76f )
by Lars
19:42 queued 17:36
created
src/voku/db/Debug.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -133,9 +133,9 @@  discard block
 block discarded – undo
133 133
     $this->logger(
134 134
         array(
135 135
             'error',
136
-            '<strong>' . date(
136
+            '<strong>'.date(
137 137
                 'd. m. Y G:i:s'
138
-            ) . ' (' . $fileInfo['file'] . ' line: ' . $fileInfo['line'] . ') (sql-error):</strong> ' . $error . '<br>',
138
+            ).' ('.$fileInfo['file'].' line: '.$fileInfo['line'].') (sql-error):</strong> '.$error.'<br>',
139 139
         )
140 140
     );
141 141
 
@@ -148,11 +148,11 @@  discard block
 block discarded – undo
148 148
         $box_bg = $this->css_mysql_box_bg;
149 149
 
150 150
         echo '
151
-        <div class="OBJ-mysql-box" style="border:' . $box_border . '; background:' . $box_bg . '; padding:10px; margin:10px;">
151
+        <div class="OBJ-mysql-box" style="border:' . $box_border.'; background:'.$box_bg.'; padding:10px; margin:10px;">
152 152
           <b style="font-size:14px;">MYSQL Error:</b>
153 153
           <code style="display:block;">
154
-            file / line: ' . $fileInfo['file'] . ' / ' . $fileInfo['line'] . '
155
-            ' . $error . '
154
+            file / line: ' . $fileInfo['file'].' / '.$fileInfo['line'].'
155
+            ' . $error.'
156 156
           </code>
157 157
         </div>
158 158
         ';
@@ -293,21 +293,21 @@  discard block
 block discarded – undo
293 293
     // get extra info
294 294
     $infoExtra = \mysqli_info($this->_db->getLink());
295 295
     if ($infoExtra) {
296
-      $infoExtra = ' | info => ' . $infoExtra;
296
+      $infoExtra = ' | info => '.$infoExtra;
297 297
     }
298 298
 
299 299
     //
300 300
     // logging
301 301
     //
302 302
 
303
-    $info = 'time => ' . round($duration, 5) . ' | results => ' . (int)$results . $infoExtra . ' | SQL => ' . UTF8::htmlentities($sql);
303
+    $info = 'time => '.round($duration, 5).' | results => '.(int)$results.$infoExtra.' | SQL => '.UTF8::htmlentities($sql);
304 304
 
305 305
     $fileInfo = $this->getFileAndLineFromSql();
306 306
 
307 307
     return $this->logger(
308 308
         array(
309 309
             $logLevel,
310
-            '<strong>' . date('d. m. Y G:i:s') . ' (' . $fileInfo['file'] . ' line: ' . $fileInfo['line'] . '):</strong> ' . $info . '<br>',
310
+            '<strong>'.date('d. m. Y G:i:s').' ('.$fileInfo['file'].' line: '.$fileInfo['line'].'):</strong> '.$info.'<br>',
311 311
             'sql',
312 312
         )
313 313
     );
@@ -373,21 +373,21 @@  discard block
 block discarded – undo
373 373
         $this->logger(
374 374
             array(
375 375
                 'debug',
376
-                $subject . ' | ' . $htmlBody,
376
+                $subject.' | '.$htmlBody,
377 377
             )
378 378
         );
379 379
       } elseif ($priority > 3) {
380 380
         $this->logger(
381 381
             array(
382 382
                 'error',
383
-                $subject . ' | ' . $htmlBody,
383
+                $subject.' | '.$htmlBody,
384 384
             )
385 385
         );
386 386
       } elseif ($priority < 3) {
387 387
         $this->logger(
388 388
             array(
389 389
                 'info',
390
-                $subject . ' | ' . $htmlBody,
390
+                $subject.' | '.$htmlBody,
391 391
             )
392 392
         );
393 393
       }
Please login to merge, or discard this patch.
src/voku/db/Helper.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -120,23 +120,23 @@  discard block
 block discarded – undo
120 120
 
121 121
     $whereSQL = $dbConnection->_parseArrayPair($whereArray, 'AND');
122 122
     if ($whereSQL) {
123
-      $whereSQL = 'AND ' . $whereSQL;
123
+      $whereSQL = 'AND '.$whereSQL;
124 124
     }
125 125
 
126 126
     if ($databaseName) {
127
-      $databaseName = $dbConnection->quote_string(trim($databaseName)) . '.';
127
+      $databaseName = $dbConnection->quote_string(trim($databaseName)).'.';
128 128
     }
129 129
 
130 130
     // get the row
131
-    $query = 'SELECT ' . $dbConnection->quote_string($searchFieldName) . ', ' . $dbConnection->quote_string($idFieldName) . ' 
132
-      FROM ' . $databaseName . $dbConnection->quote_string($table) . '
131
+    $query = 'SELECT '.$dbConnection->quote_string($searchFieldName).', '.$dbConnection->quote_string($idFieldName).' 
132
+      FROM ' . $databaseName.$dbConnection->quote_string($table).'
133 133
       WHERE 1 = 1
134
-      ' . $whereSQL . '
134
+      ' . $whereSQL.'
135 135
     ';
136 136
 
137 137
     if ($useCache === true) {
138 138
       $cache = new Cache(null, null, false, $useCache);
139
-      $cacheKey = 'sql-phonetic-search-' . md5($query);
139
+      $cacheKey = 'sql-phonetic-search-'.md5($query);
140 140
 
141 141
       if (
142 142
           $cache->getCacheIsReady() === true
@@ -266,10 +266,10 @@  discard block
 block discarded – undo
266 266
     }
267 267
 
268 268
     if ($databaseName) {
269
-      $databaseName = $dbConnection->quote_string(trim($databaseName)) . '.';
269
+      $databaseName = $dbConnection->quote_string(trim($databaseName)).'.';
270 270
     }
271 271
 
272
-    $sql = 'SHOW COLUMNS FROM ' . $databaseName . $dbConnection->escape($table);
272
+    $sql = 'SHOW COLUMNS FROM '.$databaseName.$dbConnection->escape($table);
273 273
     $result = $dbConnection->query($sql);
274 274
 
275 275
     if ($result && $result->num_rows > 0) {
@@ -315,17 +315,17 @@  discard block
 block discarded – undo
315 315
 
316 316
     $whereSQL = $dbConnection->_parseArrayPair($whereArray, 'AND');
317 317
     if ($whereSQL) {
318
-      $whereSQL = 'AND ' . $whereSQL;
318
+      $whereSQL = 'AND '.$whereSQL;
319 319
     }
320 320
 
321 321
     if ($databaseName) {
322
-      $databaseName = $dbConnection->quote_string(trim($databaseName)) . '.';
322
+      $databaseName = $dbConnection->quote_string(trim($databaseName)).'.';
323 323
     }
324 324
 
325 325
     // get the row
326
-    $query = 'SELECT * FROM ' . $databaseName . $dbConnection->quote_string($table) . '
326
+    $query = 'SELECT * FROM '.$databaseName.$dbConnection->quote_string($table).'
327 327
       WHERE 1 = 1
328
-      ' . $whereSQL . '
328
+      ' . $whereSQL.'
329 329
     ';
330 330
     $result = $dbConnection->query($query);
331 331
 
@@ -345,11 +345,11 @@  discard block
 block discarded – undo
345 345
 
346 346
           if (!in_array($fieldName, $ignoreArray, true)) {
347 347
             if (array_key_exists($fieldName, $updateArray)) {
348
-              $insert_keys .= ',' . $fieldName;
348
+              $insert_keys .= ','.$fieldName;
349 349
               $insert_values .= ',?';
350 350
               $bindings[] = $updateArray[$fieldName]; // INFO: do not escape non selected data
351 351
             } else {
352
-              $insert_keys .= ',' . $fieldName;
352
+              $insert_keys .= ','.$fieldName;
353 353
               $insert_values .= ',?';
354 354
               $bindings[] = $value; // INFO: do not escape non selected data
355 355
             }
@@ -360,10 +360,10 @@  discard block
 block discarded – undo
360 360
         $insert_values = ltrim($insert_values, ',');
361 361
 
362 362
         // insert the "copied" row
363
-        $new_query = 'INSERT INTO ' . $databaseName . $dbConnection->quote_string($table) . ' 
364
-          (' . $insert_keys . ')
363
+        $new_query = 'INSERT INTO '.$databaseName.$dbConnection->quote_string($table).' 
364
+          (' . $insert_keys.')
365 365
           VALUES 
366
-          (' . $insert_values . ')
366
+          (' . $insert_values.')
367 367
         ';
368 368
         return $dbConnection->query($new_query, $bindings);
369 369
       }
Please login to merge, or discard this patch.
src/voku/db/Prepare.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
 
82 82
     foreach ($this->_boundParams as $param) {
83 83
       $arguments[0] .= $param['type'];
84
-      $arguments[] =& $param['value'];
84
+      $arguments[] = & $param['value'];
85 85
     }
86 86
 
87 87
     return $arguments;
@@ -104,7 +104,7 @@  discard block
 block discarded – undo
104 104
     $value = $this->_db->escape($value);
105 105
 
106 106
     if ($type === 's') {
107
-      $valueForSqlWithBoundParameters = "'" . $value . "'";
107
+      $valueForSqlWithBoundParameters = "'".$value."'";
108 108
     } elseif ($type === 'i') {
109 109
       $valueForSqlWithBoundParameters = (int)$value;
110 110
     } elseif ($type === 'd') {
@@ -185,7 +185,7 @@  discard block
 block discarded – undo
185 185
       $trace = debug_backtrace();
186 186
     }
187 187
 
188
-    $args =& $trace[0]['args'];
188
+    $args = & $trace[0]['args'];
189 189
     $types = str_split($types);
190 190
 
191 191
     $args_count = count($args) - 1;
@@ -199,7 +199,7 @@  discard block
 block discarded – undo
199 199
 
200 200
     $arg = 1;
201 201
     foreach ($types as $typeInner) {
202
-      $val =& $args[$arg];
202
+      $val = & $args[$arg];
203 203
       $this->_boundParams[] = array(
204 204
           'type'  => $typeInner,
205 205
           'value' => &$val,
@@ -335,7 +335,7 @@  discard block
 block discarded – undo
335 335
     $bool = parent::prepare($query);
336 336
 
337 337
     if ($bool === false) {
338
-      $this->_debug->displayError('Can not prepare query: ' . $query . ' | ' . $this->error, false);
338
+      $this->_debug->displayError('Can not prepare query: '.$query.' | '.$this->error, false);
339 339
     }
340 340
 
341 341
     return $bool;
@@ -423,11 +423,11 @@  discard block
 block discarded – undo
423 423
 
424 424
       // exit if we have more then 3 "DB server has gone away"-errors
425 425
       if ($RECONNECT_COUNTER > 3) {
426
-        $this->_debug->mailToAdmin('DB-Fatal-Error', $errorMsg . ":\n<br />" . $sql, 5);
426
+        $this->_debug->mailToAdmin('DB-Fatal-Error', $errorMsg.":\n<br />".$sql, 5);
427 427
         throw new DBGoneAwayException($errorMsg);
428 428
       }
429 429
 
430
-      $this->_debug->mailToAdmin('DB-Error', $errorMsg . ":\n<br />" . $sql);
430
+      $this->_debug->mailToAdmin('DB-Error', $errorMsg.":\n<br />".$sql);
431 431
 
432 432
       // reconnect
433 433
       $RECONNECT_COUNTER++;
@@ -437,10 +437,10 @@  discard block
 block discarded – undo
437 437
       return $this->execute();
438 438
     }
439 439
 
440
-    $this->_debug->mailToAdmin('SQL-Error', $errorMsg . ":\n<br />" . $sql);
440
+    $this->_debug->mailToAdmin('SQL-Error', $errorMsg.":\n<br />".$sql);
441 441
 
442 442
     // this query returned an error, we must display it (only for dev) !!!
443
-    $this->_debug->displayError($errorMsg . ' | ' . $sql);
443
+    $this->_debug->displayError($errorMsg.' | '.$sql);
444 444
 
445 445
     return false;
446 446
   }
Please login to merge, or discard this patch.
src/voku/db/Result.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -622,7 +622,7 @@  discard block
 block discarded – undo
622 622
   {
623 623
     if ($as_array) {
624 624
       return array_map(
625
-          function ($object) {
625
+          function($object) {
626 626
             return (array)$object;
627 627
           },
628 628
           \mysqli_fetch_fields($this->_result)
@@ -734,7 +734,7 @@  discard block
 block discarded – undo
734 734
     }
735 735
 
736 736
     return array_map(
737
-        function ($values) use ($keys) {
737
+        function($values) use ($keys) {
738 738
           return array_combine($keys, $values);
739 739
         }, $rows
740 740
     );
Please login to merge, or discard this patch.
src/voku/db/DB.php 1 patch
Spacing   +28 added lines, -28 removed lines patch added patch discarded remove patch
@@ -434,9 +434,9 @@  discard block
 block discarded – undo
434 434
         }
435 435
 
436 436
         if ($_connector === 'NOT IN' || $_connector === 'IN') {
437
-          $_value = '(' . implode(',', $_value) . ')';
437
+          $_value = '('.implode(',', $_value).')';
438 438
         } elseif ($_connector === 'NOT BETWEEN' || $_connector === 'BETWEEN') {
439
-          $_value = '(' . implode(' AND ', $_value) . ')';
439
+          $_value = '('.implode(' AND ', $_value).')';
440 440
         }
441 441
 
442 442
       } else {
@@ -481,7 +481,7 @@  discard block
 block discarded – undo
481 481
           $_glueHelperInner = 'AND ('; // inner-loop "OR"-query glue
482 482
         }
483 483
 
484
-        $sql .= ' ' . $_glueHelperInner . ' ' . $quoteString . ' ' . $_connector . ' ' . $valueInner . " \n";
484
+        $sql .= ' '.$_glueHelperInner.' '.$quoteString.' '.$_connector.' '.$valueInner." \n";
485 485
         $tmpCounter++;
486 486
       }
487 487
 
@@ -669,7 +669,7 @@  discard block
 block discarded – undo
669 669
       );
670 670
 
671 671
     } catch (\Exception $e) {
672
-      $error = 'Error connecting to mysql server: ' . $e->getMessage();
672
+      $error = 'Error connecting to mysql server: '.$e->getMessage();
673 673
       $this->_debug->displayError($error, false);
674 674
       throw new DBConnectException($error, 100, $e);
675 675
     }
@@ -677,7 +677,7 @@  discard block
 block discarded – undo
677 677
 
678 678
     $errno = mysqli_connect_errno();
679 679
     if (!$this->connected || $errno) {
680
-      $error = 'Error connecting to mysql server: ' . \mysqli_connect_error() . ' (' . $errno . ')';
680
+      $error = 'Error connecting to mysql server: '.\mysqli_connect_error().' ('.$errno.')';
681 681
       $this->_debug->displayError($error, false);
682 682
       throw new DBConnectException($error, 101);
683 683
     }
@@ -718,10 +718,10 @@  discard block
 block discarded – undo
718 718
     }
719 719
 
720 720
     if ($databaseName) {
721
-      $databaseName = $this->quote_string(trim($databaseName)) . '.';
721
+      $databaseName = $this->quote_string(trim($databaseName)).'.';
722 722
     }
723 723
 
724
-    $sql = 'DELETE FROM ' . $databaseName . $this->quote_string($table) . " WHERE ($WHERE);";
724
+    $sql = 'DELETE FROM '.$databaseName.$this->quote_string($table)." WHERE ($WHERE);";
725 725
 
726 726
     return $this->query($sql);
727 727
   }
@@ -784,7 +784,7 @@  discard block
 block discarded – undo
784 784
     $replace = array();
785 785
 
786 786
     foreach ($bindings as $name => $value) {
787
-      $search[] = '/:' . preg_quote($name, '/') . '\b/';
787
+      $search[] = '/:'.preg_quote($name, '/').'\b/';
788 788
       $replace[] = $this->secure($value);
789 789
     }
790 790
 
@@ -953,7 +953,7 @@  discard block
 block discarded – undo
953 953
 
954 954
     if ($useCache === true) {
955 955
       $cache = new Cache(null, null, false, $useCache);
956
-      $cacheKey = 'sql-' . md5($query);
956
+      $cacheKey = 'sql-'.md5($query);
957 957
 
958 958
       if (
959 959
           $cache->getCacheIsReady() === true
@@ -1061,7 +1061,7 @@  discard block
 block discarded – undo
1061 1061
     static $firstInstance = null;
1062 1062
 
1063 1063
     if (
1064
-        $hostname . $username . $password . $database . $port . $charset == ''
1064
+        $hostname.$username.$password.$database.$port.$charset == ''
1065 1065
         &&
1066 1066
         null !== $firstInstance
1067 1067
     ) {
@@ -1071,7 +1071,7 @@  discard block
 block discarded – undo
1071 1071
     $extra_config_string = '';
1072 1072
     if (is_array($extra_config) === true) {
1073 1073
       foreach ($extra_config as $extra_config_key => $extra_config_value) {
1074
-        $extra_config_string .= $extra_config_key . (string)$extra_config_value;
1074
+        $extra_config_string .= $extra_config_key.(string)$extra_config_value;
1075 1075
       }
1076 1076
     } else {
1077 1077
       // only for backward compatibility
@@ -1079,7 +1079,7 @@  discard block
 block discarded – undo
1079 1079
     }
1080 1080
 
1081 1081
     $connection = md5(
1082
-        $hostname . $username . $password . $database . $port . $charset . (int)$exit_on_error . (int)$echo_on_error . $logger_class_name . $logger_level . $extra_config_string
1082
+        $hostname.$username.$password.$database.$port.$charset.(int)$exit_on_error.(int)$echo_on_error.$logger_class_name.$logger_level.$extra_config_string
1083 1083
     );
1084 1084
 
1085 1085
     if (!isset($instance[$connection])) {
@@ -1166,10 +1166,10 @@  discard block
 block discarded – undo
1166 1166
     $SET = $this->_parseArrayPair($data);
1167 1167
 
1168 1168
     if ($databaseName) {
1169
-      $databaseName = $this->quote_string(trim($databaseName)) . '.';
1169
+      $databaseName = $this->quote_string(trim($databaseName)).'.';
1170 1170
     }
1171 1171
 
1172
-    $sql = 'INSERT INTO ' . $databaseName . $this->quote_string($table) . " SET $SET;";
1172
+    $sql = 'INSERT INTO '.$databaseName.$this->quote_string($table)." SET $SET;";
1173 1173
 
1174 1174
     return $this->query($sql);
1175 1175
   }
@@ -1478,11 +1478,11 @@  discard block
 block discarded – undo
1478 1478
 
1479 1479
       // exit if we have more then 3 "DB server has gone away"-errors
1480 1480
       if ($RECONNECT_COUNTER > 3) {
1481
-        $this->_debug->mailToAdmin('DB-Fatal-Error', $errorMessage . '(' . $errorNumber . ') ' . ":\n<br />" . $sql, 5);
1481
+        $this->_debug->mailToAdmin('DB-Fatal-Error', $errorMessage.'('.$errorNumber.') '.":\n<br />".$sql, 5);
1482 1482
         throw new DBGoneAwayException($errorMessage);
1483 1483
       }
1484 1484
 
1485
-      $this->_debug->mailToAdmin('DB-Error', $errorMessage . '(' . $errorNumber . ') ' . ":\n<br />" . $sql);
1485
+      $this->_debug->mailToAdmin('DB-Error', $errorMessage.'('.$errorNumber.') '.":\n<br />".$sql);
1486 1486
 
1487 1487
       // reconnect
1488 1488
       $RECONNECT_COUNTER++;
@@ -1496,10 +1496,10 @@  discard block
 block discarded – undo
1496 1496
       return false;
1497 1497
     }
1498 1498
 
1499
-    $this->_debug->mailToAdmin('SQL-Error', $errorMessage . '(' . $errorNumber . ') ' . ":\n<br />" . $sql);
1499
+    $this->_debug->mailToAdmin('SQL-Error', $errorMessage.'('.$errorNumber.') '.":\n<br />".$sql);
1500 1500
 
1501 1501
     // this query returned an error, we must display it (only for dev) !!!
1502
-    $this->_debug->displayError($errorMessage . '(' . $errorNumber . ') ' . ' | ' . $sql);
1502
+    $this->_debug->displayError($errorMessage.'('.$errorNumber.') '.' | '.$sql);
1503 1503
 
1504 1504
     return false;
1505 1505
   }
@@ -1522,7 +1522,7 @@  discard block
 block discarded – undo
1522 1522
         )
1523 1523
     );
1524 1524
 
1525
-    return '`' . $str . '`';
1525
+    return '`'.$str.'`';
1526 1526
   }
1527 1527
 
1528 1528
   /**
@@ -1593,10 +1593,10 @@  discard block
 block discarded – undo
1593 1593
     $values = implode(',', $data);
1594 1594
 
1595 1595
     if ($databaseName) {
1596
-      $databaseName = $this->quote_string(trim($databaseName)) . '.';
1596
+      $databaseName = $this->quote_string(trim($databaseName)).'.';
1597 1597
     }
1598 1598
 
1599
-    $sql = 'REPLACE INTO ' . $databaseName . $this->quote_string($table) . " ($columns) VALUES ($values);";
1599
+    $sql = 'REPLACE INTO '.$databaseName.$this->quote_string($table)." ($columns) VALUES ($values);";
1600 1600
 
1601 1601
     return $this->query($sql);
1602 1602
   }
@@ -1684,7 +1684,7 @@  discard block
 block discarded – undo
1684 1684
     $var = $this->escape($var, false, false, null);
1685 1685
 
1686 1686
     if (is_string($var)) {
1687
-      $var = "'" . trim($var, "'") . "'";
1687
+      $var = "'".trim($var, "'")."'";
1688 1688
     }
1689 1689
 
1690 1690
     return $var;
@@ -1721,10 +1721,10 @@  discard block
 block discarded – undo
1721 1721
     }
1722 1722
 
1723 1723
     if ($databaseName) {
1724
-      $databaseName = $this->quote_string(trim($databaseName)) . '.';
1724
+      $databaseName = $this->quote_string(trim($databaseName)).'.';
1725 1725
     }
1726 1726
 
1727
-    $sql = 'SELECT * FROM ' . $databaseName . $this->quote_string($table) . " WHERE ($WHERE);";
1727
+    $sql = 'SELECT * FROM '.$databaseName.$this->quote_string($table)." WHERE ($WHERE);";
1728 1728
 
1729 1729
     return $this->query($sql);
1730 1730
   }
@@ -1767,10 +1767,10 @@  discard block
 block discarded – undo
1767 1767
     $return = mysqli_set_charset($this->link, $charset);
1768 1768
     /** @noinspection PhpUsageOfSilenceOperatorInspection */
1769 1769
     /** @noinspection UsageOfSilenceOperatorInspection */
1770
-    @\mysqli_query($this->link, 'SET CHARACTER SET ' . $charset);
1770
+    @\mysqli_query($this->link, 'SET CHARACTER SET '.$charset);
1771 1771
     /** @noinspection PhpUsageOfSilenceOperatorInspection */
1772 1772
     /** @noinspection UsageOfSilenceOperatorInspection */
1773
-    @\mysqli_query($this->link, "SET NAMES '" . $charset . "'");
1773
+    @\mysqli_query($this->link, "SET NAMES '".$charset."'");
1774 1774
 
1775 1775
     return $return;
1776 1776
   }
@@ -1937,10 +1937,10 @@  discard block
 block discarded – undo
1937 1937
     }
1938 1938
 
1939 1939
     if ($databaseName) {
1940
-      $databaseName = $this->quote_string(trim($databaseName)) . '.';
1940
+      $databaseName = $this->quote_string(trim($databaseName)).'.';
1941 1941
     }
1942 1942
 
1943
-    $sql = 'UPDATE ' . $databaseName . $this->quote_string($table) . " SET $SET WHERE ($WHERE);";
1943
+    $sql = 'UPDATE '.$databaseName.$this->quote_string($table)." SET $SET WHERE ($WHERE);";
1944 1944
 
1945 1945
     return $this->query($sql);
1946 1946
   }
Please login to merge, or discard this patch.