Completed
Push — master ( 90dd46...423a40 )
by Lars
02:17
created
src/voku/db/ActiveRecord.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -659,7 +659,7 @@
 block discarded – undo
659 659
    * @param array     $param          <p>
660 660
    *                                  The param will be bind to the $sql query.
661 661
    *                                  </p>
662
-   * @param null|self $obj            <p>
662
+   * @param null|\self $obj            <p>
663 663
    *                                  The object, if find record in database, we will assign the attributes into
664 664
    *                                  this object.
665 665
    *                                  </p>
Please login to merge, or discard this patch.
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace voku\db;
6 6
 
@@ -583,7 +583,7 @@  discard block
 block discarded – undo
583 583
     $value = $this->_filterParam($this->dirty);
584 584
     $this->insert = new ActiveRecordExpressions(
585 585
         [
586
-            'operator' => 'INSERT INTO ' . $this->table,
586
+            'operator' => 'INSERT INTO '.$this->table,
587 587
             'target'   => new ActiveRecordExpressionsWrap(['target' => \array_keys($this->dirty)]),
588 588
         ]
589 589
     );
@@ -794,7 +794,7 @@  discard block
 block discarded – undo
794 794
         null === $active_record->{$sql_string_part}
795 795
     ) {
796 796
 
797
-      $sql_string_part = \strtoupper($sql_string_part) . ' ' . $active_record->table . '.*';
797
+      $sql_string_part = \strtoupper($sql_string_part).' '.$active_record->table.'.*';
798 798
 
799 799
     } elseif (
800 800
         (
@@ -806,15 +806,15 @@  discard block
 block discarded – undo
806 806
         null === $active_record->{$sql_string_part}
807 807
     ) {
808 808
 
809
-      $sql_string_part = \strtoupper($sql_string_part) . ' ' . $active_record->table;
809
+      $sql_string_part = \strtoupper($sql_string_part).' '.$active_record->table;
810 810
 
811 811
     } elseif ('delete' === $sql_string_part) {
812 812
 
813
-      $sql_string_part = \strtoupper($sql_string_part) . ' ';
813
+      $sql_string_part = \strtoupper($sql_string_part).' ';
814 814
 
815 815
     } else {
816 816
 
817
-      $sql_string_part = (null !== $active_record->{$sql_string_part}) ? $active_record->{$sql_string_part} . ' ' : '';
817
+      $sql_string_part = (null !== $active_record->{$sql_string_part}) ? $active_record->{$sql_string_part}.' ' : '';
818 818
 
819 819
     }
820 820
   }
@@ -928,10 +928,10 @@  discard block
 block discarded – undo
928 928
   {
929 929
     if (\is_array($value)) {
930 930
       foreach ($value as $key => $val) {
931
-        $this->params[$value[$key] = self::PREFIX . ++self::$count] = $val;
931
+        $this->params[$value[$key] = self::PREFIX.++self::$count] = $val;
932 932
       }
933 933
     } elseif (\is_string($value)) {
934
-      $this->params[$ph = self::PREFIX . ++self::$count] = $value;
934
+      $this->params[$ph = self::PREFIX.++self::$count] = $value;
935 935
       $value = $ph;
936 936
     }
937 937
 
@@ -952,7 +952,7 @@  discard block
 block discarded – undo
952 952
     $value = $this->_filterParam($value);
953 953
     $expression = new ActiveRecordExpressions(
954 954
         [
955
-            'source'   => ('where' === strtolower($name) ? $this->table . '.' : '') . $field,
955
+            'source'   => ('where' === strtolower($name) ? $this->table.'.' : '').$field,
956 956
             'operator' => $operator,
957 957
             'target'   => \is_array($value)
958 958
                 ? new ActiveRecordExpressionsWrap(
@@ -986,7 +986,7 @@  discard block
 block discarded – undo
986 986
     $this->join = new ActiveRecordExpressions(
987 987
         [
988 988
             'source'   => $this->join ?: '',
989
-            'operator' => $type . ' JOIN',
989
+            'operator' => $type.' JOIN',
990 990
             'target'   => new ActiveRecordExpressions(
991 991
                 ['source' => $table, 'operator' => 'ON', 'target' => $on]
992 992
             ),
Please login to merge, or discard this patch.
src/voku/db/DB.php 2 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -995,7 +995,7 @@  discard block
 block discarded – undo
995 995
    * @param string    $query    <p>sql-query</p>
996 996
    * @param bool      $useCache <p>use cache?</p>
997 997
    * @param int       $cacheTTL <p>cache-ttl in seconds</p>
998
-   * @param self|null $db       optional <p>the database connection</p>
998
+   * @param null|\self $db       optional <p>the database connection</p>
999 999
    *
1000 1000
    * @return mixed "array" by "<b>SELECT</b>"-queries<br />
1001 1001
    *               "int" (insert_id) by "<b>INSERT</b>"-queries<br />
@@ -1741,7 +1741,7 @@  discard block
 block discarded – undo
1741 1741
    *
1742 1742
    * @param mixed $var
1743 1743
    *
1744
-   * @return mixed
1744
+   * @return string
1745 1745
    */
1746 1746
   public function secure($var)
1747 1747
   {
Please login to merge, or discard this patch.
Spacing   +33 added lines, -33 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace voku\db;
6 6
 
@@ -448,9 +448,9 @@  discard block
 block discarded – undo
448 448
         }
449 449
 
450 450
         if ($_connector === 'NOT IN' || $_connector === 'IN') {
451
-          $_value = '(' . \implode(',', $_value) . ')';
451
+          $_value = '('.\implode(',', $_value).')';
452 452
         } elseif ($_connector === 'NOT BETWEEN' || $_connector === 'BETWEEN') {
453
-          $_value = '(' . \implode(' AND ', $_value) . ')';
453
+          $_value = '('.\implode(' AND ', $_value).')';
454 454
         }
455 455
 
456 456
       } else {
@@ -497,7 +497,7 @@  discard block
 block discarded – undo
497 497
           $valueInner = "''";
498 498
         }
499 499
 
500
-        $sql .= ' ' . $_glueHelperInner . ' ' . $quoteString . ' ' . $_connector . ' ' . $valueInner . " \n";
500
+        $sql .= ' '.$_glueHelperInner.' '.$quoteString.' '.$_connector.' '.$valueInner." \n";
501 501
         $tmpCounter++;
502 502
       }
503 503
 
@@ -704,7 +704,7 @@  discard block
 block discarded – undo
704 704
       );
705 705
 
706 706
     } catch (\Exception $e) {
707
-      $error = 'Error connecting to mysql server: ' . $e->getMessage();
707
+      $error = 'Error connecting to mysql server: '.$e->getMessage();
708 708
       $this->_debug->displayError($error, true);
709 709
       throw new DBConnectException($error, 100, $e);
710 710
     }
@@ -712,7 +712,7 @@  discard block
 block discarded – undo
712 712
 
713 713
     $errno = \mysqli_connect_errno();
714 714
     if (!$this->connected || $errno) {
715
-      $error = 'Error connecting to mysql server: ' . \mysqli_connect_error() . ' (' . $errno . ')';
715
+      $error = 'Error connecting to mysql server: '.\mysqli_connect_error().' ('.$errno.')';
716 716
       $this->_debug->displayError($error, true);
717 717
       throw new DBConnectException($error, 101);
718 718
     }
@@ -753,10 +753,10 @@  discard block
 block discarded – undo
753 753
     }
754 754
 
755 755
     if ($databaseName) {
756
-      $databaseName = $this->quote_string(\trim($databaseName)) . '.';
756
+      $databaseName = $this->quote_string(\trim($databaseName)).'.';
757 757
     }
758 758
 
759
-    $sql = 'DELETE FROM ' . $databaseName . $this->quote_string($table) . " WHERE ($WHERE)";
759
+    $sql = 'DELETE FROM '.$databaseName.$this->quote_string($table)." WHERE ($WHERE)";
760 760
 
761 761
     return $this->query($sql);
762 762
   }
@@ -829,8 +829,8 @@  discard block
 block discarded – undo
829 829
         $nameTmp = \substr($name, 1);
830 830
       }
831 831
 
832
-      $parseKeyInner = $nameTmp . '-' . $PARSE_KEY_CACHE;
833
-      $sql = \str_replace(':' . $nameTmp, $parseKeyInner, $sql);
832
+      $parseKeyInner = $nameTmp.'-'.$PARSE_KEY_CACHE;
833
+      $sql = \str_replace(':'.$nameTmp, $parseKeyInner, $sql);
834 834
     }
835 835
 
836 836
     foreach ($params as $name => $value) {
@@ -839,7 +839,7 @@  discard block
 block discarded – undo
839 839
         $nameTmp = \substr($name, 1);
840 840
       }
841 841
 
842
-      $parseKeyInner = $nameTmp . '-' . $PARSE_KEY_CACHE;
842
+      $parseKeyInner = $nameTmp.'-'.$PARSE_KEY_CACHE;
843 843
       $sqlBefore = $sql;
844 844
       $secureParamValue = $this->secure($params[$name]);
845 845
 
@@ -1015,7 +1015,7 @@  discard block
 block discarded – undo
1015 1015
 
1016 1016
     if ($useCache === true) {
1017 1017
       $cache = new Cache(null, null, false, $useCache);
1018
-      $cacheKey = 'sql-' . \md5($query);
1018
+      $cacheKey = 'sql-'.\md5($query);
1019 1019
 
1020 1020
       if (
1021 1021
           $cache->getCacheIsReady() === true
@@ -1126,7 +1126,7 @@  discard block
 block discarded – undo
1126 1126
     static $firstInstance = null;
1127 1127
 
1128 1128
     if (
1129
-        $hostname . $username . $password . $database . $port . $charset == '3306utf8'
1129
+        $hostname.$username.$password.$database.$port.$charset == '3306utf8'
1130 1130
         &&
1131 1131
         null !== $firstInstance
1132 1132
     ) {
@@ -1140,7 +1140,7 @@  discard block
 block discarded – undo
1140 1140
     $extra_config_string = '';
1141 1141
     if (\is_array($extra_config) === true) {
1142 1142
       foreach ($extra_config as $extra_config_key => $extra_config_value) {
1143
-        $extra_config_string .= $extra_config_key . (string)$extra_config_value;
1143
+        $extra_config_string .= $extra_config_key.(string)$extra_config_value;
1144 1144
       }
1145 1145
     } else {
1146 1146
       // only for backward compatibility
@@ -1148,7 +1148,7 @@  discard block
 block discarded – undo
1148 1148
     }
1149 1149
 
1150 1150
     $connection = \md5(
1151
-        $hostname . $username . $password . $database . $port . $charset . (int)$exit_on_error . (int)$echo_on_error . $logger_class_name . $logger_level . $extra_config_string
1151
+        $hostname.$username.$password.$database.$port.$charset.(int)$exit_on_error.(int)$echo_on_error.$logger_class_name.$logger_level.$extra_config_string
1152 1152
     );
1153 1153
 
1154 1154
     if (!isset($instance[$connection])) {
@@ -1239,10 +1239,10 @@  discard block
 block discarded – undo
1239 1239
     $SET = $this->_parseArrayPair($data);
1240 1240
 
1241 1241
     if ($databaseName) {
1242
-      $databaseName = $this->quote_string(\trim($databaseName)) . '.';
1242
+      $databaseName = $this->quote_string(\trim($databaseName)).'.';
1243 1243
     }
1244 1244
 
1245
-    $sql = 'INSERT INTO ' . $databaseName . $this->quote_string($table) . " SET $SET";
1245
+    $sql = 'INSERT INTO '.$databaseName.$this->quote_string($table)." SET $SET";
1246 1246
 
1247 1247
     return $this->query($sql);
1248 1248
   }
@@ -1553,11 +1553,11 @@  discard block
 block discarded – undo
1553 1553
 
1554 1554
       // exit if we have more then 3 "DB server has gone away"-errors
1555 1555
       if ($RECONNECT_COUNTER > 3) {
1556
-        $this->_debug->mailToAdmin('DB-Fatal-Error', $errorMessage . '(' . $errorNumber . ') ' . ":\n<br />" . $sql, 5);
1556
+        $this->_debug->mailToAdmin('DB-Fatal-Error', $errorMessage.'('.$errorNumber.') '.":\n<br />".$sql, 5);
1557 1557
         throw new DBGoneAwayException($errorMessage);
1558 1558
       }
1559 1559
 
1560
-      $this->_debug->mailToAdmin('DB-Error', $errorMessage . '(' . $errorNumber . ') ' . ":\n<br />" . $sql);
1560
+      $this->_debug->mailToAdmin('DB-Error', $errorMessage.'('.$errorNumber.') '.":\n<br />".$sql);
1561 1561
 
1562 1562
       // reconnect
1563 1563
       $RECONNECT_COUNTER++;
@@ -1571,7 +1571,7 @@  discard block
 block discarded – undo
1571 1571
       return false;
1572 1572
     }
1573 1573
 
1574
-    $this->_debug->mailToAdmin('SQL-Error', $errorMessage . '(' . $errorNumber . ') ' . ":\n<br />" . $sql);
1574
+    $this->_debug->mailToAdmin('SQL-Error', $errorMessage.'('.$errorNumber.') '.":\n<br />".$sql);
1575 1575
 
1576 1576
     $force_exception_after_error = null; // auto
1577 1577
     if ($this->_in_transaction === true) {
@@ -1579,7 +1579,7 @@  discard block
 block discarded – undo
1579 1579
     }
1580 1580
     // this query returned an error, we must display it (only for dev) !!!
1581 1581
 
1582
-    $this->_debug->displayError($errorMessage . '(' . $errorNumber . ') ' . ' | ' . $sql, $force_exception_after_error);
1582
+    $this->_debug->displayError($errorMessage.'('.$errorNumber.') '.' | '.$sql, $force_exception_after_error);
1583 1583
 
1584 1584
     return false;
1585 1585
   }
@@ -1602,7 +1602,7 @@  discard block
 block discarded – undo
1602 1602
         )
1603 1603
     );
1604 1604
 
1605
-    return '`' . $str . '`';
1605
+    return '`'.$str.'`';
1606 1606
   }
1607 1607
 
1608 1608
   /**
@@ -1673,10 +1673,10 @@  discard block
 block discarded – undo
1673 1673
     $values = \implode(',', $data);
1674 1674
 
1675 1675
     if ($databaseName) {
1676
-      $databaseName = $this->quote_string(\trim($databaseName)) . '.';
1676
+      $databaseName = $this->quote_string(\trim($databaseName)).'.';
1677 1677
     }
1678 1678
 
1679
-    $sql = 'REPLACE INTO ' . $databaseName . $this->quote_string($table) . " ($columns) VALUES ($values)";
1679
+    $sql = 'REPLACE INTO '.$databaseName.$this->quote_string($table)." ($columns) VALUES ($values)";
1680 1680
 
1681 1681
     return $this->query($sql);
1682 1682
   }
@@ -1772,7 +1772,7 @@  discard block
 block discarded – undo
1772 1772
     $var = $this->escape($var, false, false, null);
1773 1773
 
1774 1774
     if (\is_string($var)) {
1775
-      $var = "'" . \trim($var, "'") . "'";
1775
+      $var = "'".\trim($var, "'")."'";
1776 1776
     }
1777 1777
 
1778 1778
     return $var;
@@ -1809,10 +1809,10 @@  discard block
 block discarded – undo
1809 1809
     }
1810 1810
 
1811 1811
     if ($databaseName) {
1812
-      $databaseName = $this->quote_string(\trim($databaseName)) . '.';
1812
+      $databaseName = $this->quote_string(\trim($databaseName)).'.';
1813 1813
     }
1814 1814
 
1815
-    $sql = 'SELECT * FROM ' . $databaseName . $this->quote_string($table) . " WHERE ($WHERE)";
1815
+    $sql = 'SELECT * FROM '.$databaseName.$this->quote_string($table)." WHERE ($WHERE)";
1816 1816
 
1817 1817
     return $this->query($sql);
1818 1818
   }
@@ -1854,9 +1854,9 @@  discard block
 block discarded – undo
1854 1854
 
1855 1855
     $return = mysqli_set_charset($this->link, $charset);
1856 1856
     /** @noinspection PhpUsageOfSilenceOperatorInspection */
1857
-    @\mysqli_query($this->link, 'SET CHARACTER SET ' . $charset);
1857
+    @\mysqli_query($this->link, 'SET CHARACTER SET '.$charset);
1858 1858
     /** @noinspection PhpUsageOfSilenceOperatorInspection */
1859
-    @\mysqli_query($this->link, "SET NAMES '" . $charset . "'");
1859
+    @\mysqli_query($this->link, "SET NAMES '".$charset."'");
1860 1860
 
1861 1861
     return $return;
1862 1862
   }
@@ -1990,7 +1990,7 @@  discard block
 block discarded – undo
1990 1990
       $result = $callback($this);
1991 1991
       if ($result === false) {
1992 1992
         /** @noinspection ThrowRawExceptionInspection */
1993
-        throw new \Exception('call_user_func [' . $callback . '] === false');
1993
+        throw new \Exception('call_user_func ['.$callback.'] === false');
1994 1994
       }
1995 1995
 
1996 1996
       return $this->commit();
@@ -2043,10 +2043,10 @@  discard block
 block discarded – undo
2043 2043
     }
2044 2044
 
2045 2045
     if ($databaseName) {
2046
-      $databaseName = $this->quote_string(\trim($databaseName)) . '.';
2046
+      $databaseName = $this->quote_string(\trim($databaseName)).'.';
2047 2047
     }
2048 2048
 
2049
-    $sql = 'UPDATE ' . $databaseName . $this->quote_string($table) . " SET $SET WHERE ($WHERE)";
2049
+    $sql = 'UPDATE '.$databaseName.$this->quote_string($table)." SET $SET WHERE ($WHERE)";
2050 2050
 
2051 2051
     return $this->query($sql);
2052 2052
   }
@@ -2060,7 +2060,7 @@  discard block
 block discarded – undo
2060 2060
    */
2061 2061
   public function table_exists(string $table): bool
2062 2062
   {
2063
-    $check = $this->query('SELECT 1 FROM ' . $this->quote_string($table));
2063
+    $check = $this->query('SELECT 1 FROM '.$this->quote_string($table));
2064 2064
 
2065 2065
     return $check !== false
2066 2066
            &&
Please login to merge, or discard this patch.
src/voku/db/ActiveRecordExpressionsWrap.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace voku\db;
6 6
 
Please login to merge, or discard this patch.