Completed
Push — master ( cf6cbe...7c2b24 )
by Lars
04:28
created
src/voku/db/Result.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -229,7 +229,7 @@
 block discarded – undo
229 229
    * fetchObject
230 230
    *
231 231
    * @param string     $class
232
-   * @param null|array $params
232
+   * @param string $params
233 233
    * @param bool       $reset
234 234
    *
235 235
    * @return object|false false on error
Please login to merge, or discard this patch.
src/voku/db/DB.php 2 patches
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -795,7 +795,7 @@  discard block
 block discarded – undo
795 795
   /**
796 796
    * Escape
797 797
    *
798
-   * @param mixed $var boolean: convert into "integer"<br />
798
+   * @param string $var boolean: convert into "integer"<br />
799 799
    *                   int: convert into "integer"<br />
800 800
    *                   float: convert into "float" and replace "," with "."<br />
801 801
    *                   array: run escape() for every key => value<br />
@@ -804,7 +804,7 @@  discard block
 block discarded – undo
804 804
    * @param bool  $html_entity_decode
805 805
    * @param bool  $array_to_string
806 806
    *
807
-   * @return array|bool|float|int|string
807
+   * @return string
808 808
    */
809 809
   public function escape($var = '', $stripe_non_utf8 = true, $html_entity_decode = true, $array_to_string = false)
810 810
   {
@@ -923,7 +923,7 @@  discard block
 block discarded – undo
923 923
   /**
924 924
    * Returns the auto generated id used in the last query.
925 925
    *
926
-   * @return int|string
926
+   * @return integer
927 927
    */
928 928
   public function insert_id()
929 929
   {
Please login to merge, or discard this patch.
Spacing   +38 added lines, -38 removed lines patch added patch discarded remove patch
@@ -295,12 +295,12 @@  discard block
 block discarded – undo
295 295
           $this->socket
296 296
       );
297 297
     } catch (\Exception $e) {
298
-      $this->_displayError('Error connecting to mysql server: ' . $e->getMessage(), true);
298
+      $this->_displayError('Error connecting to mysql server: '.$e->getMessage(), true);
299 299
     }
300 300
     mysqli_report(MYSQLI_REPORT_OFF);
301 301
 
302 302
     if (!$this->link) {
303
-      $this->_displayError('Error connecting to mysql server: ' . mysqli_connect_error(), true);
303
+      $this->_displayError('Error connecting to mysql server: '.mysqli_connect_error(), true);
304 304
     } else {
305 305
       $this->set_charset($this->charset);
306 306
       $this->connected = true;
@@ -334,9 +334,9 @@  discard block
 block discarded – undo
334 334
     $this->logger(
335 335
         array(
336 336
             'error',
337
-            '<strong>' . date(
337
+            '<strong>'.date(
338 338
                 'd. m. Y G:i:s'
339
-            ) . ' (' . $fileInfo['file'] . ' line: ' . $fileInfo['line'] . ') (sql-error):</strong> ' . $error . '<br>',
339
+            ).' ('.$fileInfo['file'].' line: '.$fileInfo['line'].') (sql-error):</strong> '.$error.'<br>',
340 340
         )
341 341
     );
342 342
 
@@ -349,11 +349,11 @@  discard block
 block discarded – undo
349 349
         $box_bg = $this->css_mysql_box_bg;
350 350
 
351 351
         echo '
352
-        <div class="OBJ-mysql-box" style="border:' . $box_border . '; background:' . $box_bg . '; padding:10px; margin:10px;">
352
+        <div class="OBJ-mysql-box" style="border:' . $box_border.'; background:'.$box_bg.'; padding:10px; margin:10px;">
353 353
           <b style="font-size:14px;">MYSQL Error:</b>
354 354
           <code style="display:block;">
355
-            file / line: ' . $fileInfo['file'] . ' / ' . $fileInfo['line'] . '
356
-            ' . $error . '
355
+            file / line: ' . $fileInfo['file'].' / '.$fileInfo['line'].'
356
+            ' . $error.'
357 357
           </code>
358 358
         </div>
359 359
         ';
@@ -559,7 +559,7 @@  discard block
 block discarded – undo
559 559
     static $firstInstance = null;
560 560
 
561 561
     if (
562
-        $hostname . $username . $password . $database . $port . $charset == ''
562
+        $hostname.$username.$password.$database.$port.$charset == ''
563 563
         &&
564 564
         null !== $firstInstance
565 565
     ) {
@@ -567,7 +567,7 @@  discard block
 block discarded – undo
567 567
     }
568 568
 
569 569
     $connection = md5(
570
-        $hostname . $username . $password . $database . $port . $charset . (int)$exit_on_error . (int)$echo_on_error . $logger_class_name . $logger_level . (int)$session_to_db
570
+        $hostname.$username.$password.$database.$port.$charset.(int)$exit_on_error.(int)$echo_on_error.$logger_class_name.$logger_level.(int)$session_to_db
571 571
     );
572 572
 
573 573
     if (!isset($instance[$connection])) {
@@ -744,7 +744,7 @@  discard block
 block discarded – undo
744 744
       // "string"
745 745
 
746 746
       if (!in_array($var, $this->mysqlDefaultTimeFunctions, true)) {
747
-        $var = "'" . trim($this->escape(trim(trim((string)$var), "'")), "'") . "'";
747
+        $var = "'".trim($this->escape(trim(trim((string)$var), "'")), "'")."'";
748 748
       }
749 749
 
750 750
     } elseif (is_float($var)) {
@@ -764,7 +764,7 @@  discard block
 block discarded – undo
764 764
       // "DateTime"-object
765 765
 
766 766
       try {
767
-        $var = "'" . $this->escape($var->format('Y-m-d H:i:s'), false, false) . "'";
767
+        $var = "'".$this->escape($var->format('Y-m-d H:i:s'), false, false)."'";
768 768
       } catch (\Exception $e) {
769 769
         $var = null;
770 770
       }
@@ -773,7 +773,7 @@  discard block
 block discarded – undo
773 773
 
774 774
       // fallback ...
775 775
 
776
-      $var = "'" . trim($this->escape(trim(trim((string)$var), "'")), "'") . "'";
776
+      $var = "'".trim($this->escape(trim(trim((string)$var), "'")), "'")."'";
777 777
 
778 778
     }
779 779
 
@@ -889,18 +889,18 @@  discard block
 block discarded – undo
889 889
       return false;
890 890
     }
891 891
 
892
-    $info = 'time => ' . round(
892
+    $info = 'time => '.round(
893 893
             $duration,
894 894
             5
895
-        ) . ' - ' . 'results => ' . $results . ' - ' . 'SQL => ' . UTF8::htmlentities($sql);
895
+        ).' - '.'results => '.$results.' - '.'SQL => '.UTF8::htmlentities($sql);
896 896
 
897 897
     $fileInfo = $this->getFileAndLineFromSql();
898 898
     $this->logger(
899 899
         array(
900 900
             'debug',
901
-            '<strong>' . date(
901
+            '<strong>'.date(
902 902
                 'd. m. Y G:i:s'
903
-            ) . ' (' . $fileInfo['file'] . ' line: ' . $fileInfo['line'] . '):</strong> ' . $info . '<br>',
903
+            ).' ('.$fileInfo['file'].' line: '.$fileInfo['line'].'):</strong> '.$info.'<br>',
904 904
             'sql',
905 905
         )
906 906
     );
@@ -944,10 +944,10 @@  discard block
 block discarded – undo
944 944
 
945 945
       // exit if we have more then 3 "DB server has gone away"-errors
946 946
       if ($reconnectCounter > 3) {
947
-        $this->mailToAdmin('SQL-Fatal-Error', $errorMsg . ":\n<br />" . $sql, 5);
947
+        $this->mailToAdmin('SQL-Fatal-Error', $errorMsg.":\n<br />".$sql, 5);
948 948
         throw new \Exception($errorMsg);
949 949
       } else {
950
-        $this->mailToAdmin('SQL-Error', $errorMsg . ":\n<br />" . $sql);
950
+        $this->mailToAdmin('SQL-Error', $errorMsg.":\n<br />".$sql);
951 951
 
952 952
         // reconnect
953 953
         $reconnectCounter++;
@@ -957,10 +957,10 @@  discard block
 block discarded – undo
957 957
         $this->query($sql, $sqlParams);
958 958
       }
959 959
     } else {
960
-      $this->mailToAdmin('SQL-Warning', $errorMsg . ":\n<br />" . $sql);
960
+      $this->mailToAdmin('SQL-Warning', $errorMsg.":\n<br />".$sql);
961 961
 
962 962
       // this query returned an error, we must display it (only for dev) !!!
963
-      $this->_displayError($errorMsg . ' | ' . $sql);
963
+      $this->_displayError($errorMsg.' | '.$sql);
964 964
     }
965 965
   }
966 966
 
@@ -981,21 +981,21 @@  discard block
 block discarded – undo
981 981
         $this->logger(
982 982
             array(
983 983
                 'debug',
984
-                $subject . ' | ' . $htmlBody,
984
+                $subject.' | '.$htmlBody,
985 985
             )
986 986
         );
987 987
       } elseif ($priority > 3) {
988 988
         $this->logger(
989 989
             array(
990 990
                 'error',
991
-                $subject . ' | ' . $htmlBody,
991
+                $subject.' | '.$htmlBody,
992 992
             )
993 993
         );
994 994
       } elseif ($priority < 3) {
995 995
         $this->logger(
996 996
             array(
997 997
                 'info',
998
-                $subject . ' | ' . $htmlBody,
998
+                $subject.' | '.$htmlBody,
999 999
             )
1000 1000
         );
1001 1001
       }
@@ -1066,7 +1066,7 @@  discard block
 block discarded – undo
1066 1066
 
1067 1067
     if ($useCache === true) {
1068 1068
       $cache = new Cache(null, null, false, $useCache);
1069
-      $cacheKey = 'sql-' . md5($query);
1069
+      $cacheKey = 'sql-'.md5($query);
1070 1070
 
1071 1071
       if (
1072 1072
           $cache->getCacheIsReady() === true
@@ -1128,9 +1128,9 @@  discard block
 block discarded – undo
1128 1128
 
1129 1129
     $return = mysqli_set_charset($this->link, $charset);
1130 1130
     /** @noinspection PhpUsageOfSilenceOperatorInspection */
1131
-    @mysqli_query($this->link, 'SET CHARACTER SET ' . $charset);
1131
+    @mysqli_query($this->link, 'SET CHARACTER SET '.$charset);
1132 1132
     /** @noinspection PhpUsageOfSilenceOperatorInspection */
1133
-    @mysqli_query($this->link, "SET NAMES '" . ($charset == 'utf8' ? 'utf8mb4' : $charset) . "'");
1133
+    @mysqli_query($this->link, "SET NAMES '".($charset == 'utf8' ? 'utf8mb4' : $charset)."'");
1134 1134
 
1135 1135
     return $return;
1136 1136
   }
@@ -1221,10 +1221,10 @@  discard block
 block discarded – undo
1221 1221
 
1222 1222
       if ($this->checkForDev() === true) {
1223 1223
         echo "Info: maybe you have to increase your 'max_allowed_packet = 30M' in the config: 'my.conf' \n<br />";
1224
-        echo 'Error:' . $errorMsg;
1224
+        echo 'Error:'.$errorMsg;
1225 1225
       }
1226 1226
 
1227
-      $this->mailToAdmin('SQL-Error in mysqli_multi_query', $errorMsg . ":\n<br />" . $sql);
1227
+      $this->mailToAdmin('SQL-Error in mysqli_multi_query', $errorMsg.":\n<br />".$sql);
1228 1228
     }
1229 1229
 
1230 1230
     // return the result only if there was a "SELECT"-query
@@ -1261,7 +1261,7 @@  discard block
 block discarded – undo
1261 1261
 
1262 1262
       return false;
1263 1263
     } elseif (mysqli_connect_errno()) {
1264
-      $this->_displayError('Error connecting to mysql server: ' . mysqli_connect_error(), true);
1264
+      $this->_displayError('Error connecting to mysql server: '.mysqli_connect_error(), true);
1265 1265
 
1266 1266
       return false;
1267 1267
     } else {
@@ -1370,7 +1370,7 @@  discard block
 block discarded – undo
1370 1370
 
1371 1371
     $SET = $this->_parseArrayPair($data);
1372 1372
 
1373
-    $sql = 'INSERT INTO ' . $this->quote_string($table) . " SET $SET;";
1373
+    $sql = 'INSERT INTO '.$this->quote_string($table)." SET $SET;";
1374 1374
 
1375 1375
     return $this->query($sql);
1376 1376
   }
@@ -1464,7 +1464,7 @@  discard block
 block discarded – undo
1464 1464
             /** @noinspection AlterInForeachInspection */
1465 1465
             $_value[$oldKey] = $this->secure($oldValue);
1466 1466
           }
1467
-          $_value = '(' . implode(',', $_value) . ')';
1467
+          $_value = '('.implode(',', $_value).')';
1468 1468
         } elseif (
1469 1469
             is_array($_value)
1470 1470
             &&
@@ -1478,13 +1478,13 @@  discard block
 block discarded – undo
1478 1478
             /** @noinspection AlterInForeachInspection */
1479 1479
             $_value[$oldKey] = $this->secure($oldValue);
1480 1480
           }
1481
-          $_value = '(' . implode(' AND ', $_value) . ')';
1481
+          $_value = '('.implode(' AND ', $_value).')';
1482 1482
         } else {
1483 1483
           $_value = $this->secure($_value);
1484 1484
         }
1485 1485
 
1486 1486
         $quoteString = $this->quote_string(trim(str_ireplace($_connector, '', $_key)));
1487
-        $pairs[] = ' ' . $quoteString . ' ' . $_connector . ' ' . $_value . " \n";
1487
+        $pairs[] = ' '.$quoteString.' '.$_connector.' '.$_value." \n";
1488 1488
       }
1489 1489
 
1490 1490
       $sql = implode($glue, $pairs);
@@ -1502,7 +1502,7 @@  discard block
 block discarded – undo
1502 1502
    */
1503 1503
   public function quote_string($str)
1504 1504
   {
1505
-    return '`' . $this->escape($str, false, false) . '`';
1505
+    return '`'.$this->escape($str, false, false).'`';
1506 1506
   }
1507 1507
 
1508 1508
   /**
@@ -1556,7 +1556,7 @@  discard block
 block discarded – undo
1556 1556
     }
1557 1557
     $values = implode(',', $data);
1558 1558
 
1559
-    $sql = 'REPLACE INTO ' . $this->quote_string($table) . " ($columns) VALUES ($values);";
1559
+    $sql = 'REPLACE INTO '.$this->quote_string($table)." ($columns) VALUES ($values);";
1560 1560
 
1561 1561
     return $this->query($sql);
1562 1562
   }
@@ -1596,7 +1596,7 @@  discard block
 block discarded – undo
1596 1596
       $WHERE = '';
1597 1597
     }
1598 1598
 
1599
-    $sql = 'UPDATE ' . $this->quote_string($table) . " SET $SET WHERE ($WHERE);";
1599
+    $sql = 'UPDATE '.$this->quote_string($table)." SET $SET WHERE ($WHERE);";
1600 1600
 
1601 1601
     return $this->query($sql);
1602 1602
   }
@@ -1628,7 +1628,7 @@  discard block
 block discarded – undo
1628 1628
       $WHERE = '';
1629 1629
     }
1630 1630
 
1631
-    $sql = 'DELETE FROM ' . $this->quote_string($table) . " WHERE ($WHERE);";
1631
+    $sql = 'DELETE FROM '.$this->quote_string($table)." WHERE ($WHERE);";
1632 1632
 
1633 1633
     return $this->query($sql);
1634 1634
   }
@@ -1658,7 +1658,7 @@  discard block
 block discarded – undo
1658 1658
       $WHERE = '';
1659 1659
     }
1660 1660
 
1661
-    $sql = 'SELECT * FROM ' . $this->quote_string($table) . " WHERE ($WHERE);";
1661
+    $sql = 'SELECT * FROM '.$this->quote_string($table)." WHERE ($WHERE);";
1662 1662
 
1663 1663
     return $this->query($sql);
1664 1664
   }
Please login to merge, or discard this patch.