byjg / anydataset

src/Database/DBBaseFunctions.php

Doc Comments +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
 
     /**
      * Return if the database provider have a top or similar function
-     * @return unknown_type
+     * @return boolean
      */
     function hasTop()
     {

src/Database/DBDblibFunctions.php

Doc Comments +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
 
     /**
      * Return if the database provider have a top or similar function
-     * @return unknown_type
+     * @return boolean
      */
     function hasTop()
     {

Switch Indentation +44 added lines, -44 removed lines

@@ -76,50 +76,50 @@
             if ($s) $s .= '+';
             $ch = $fmt[$i];
             switch ($ch) {
-                case 'Y':
-                case 'y':
-                    $s .= "datename(yyyy,$col)";
-                    break;
-                case 'M':
-                    $s .= "convert(char(3),$col,0)";
-                    break;
-                case 'm':
-                    $s .= "replace(str(month($col),2),' ','0')";
-                    break;
-                case 'Q':
-                case 'q':
-                    $s .= "datename(quarter,$col)";
-                    break;
-                case 'D':
-                case 'd':
-                    $s .= "replace(str(day($col),2),' ','0')";
-                    break;
-                case 'h':
-                    $s .= "substring(convert(char(14),$col,0),13,2)";
-                    break;
-
-                case 'H':
-                    $s .= "replace(str(datepart(hh,$col),2),' ','0')";
-                    break;
-
-                case 'i':
-                    $s .= "replace(str(datepart(mi,$col),2),' ','0')";
-                    break;
-                case 's':
-                    $s .= "replace(str(datepart(ss,$col),2),' ','0')";
-                    break;
-                case 'a':
-                case 'A':
-                    $s .= "substring(convert(char(19),$col,0),18,2)";
-                    break;
-
-                default:
-                    if ($ch == '\\') {
-                        $i++;
-                        $ch = substr($fmt, $i, 1);
-                    }
-                    $s .= $this->qstr($ch);
-                    break;
+            case 'Y':
+            case 'y':
+                $s .= "datename(yyyy,$col)";
+                break;
+            case 'M':
+                $s .= "convert(char(3),$col,0)";
+                break;
+            case 'm':
+                $s .= "replace(str(month($col),2),' ','0')";
+                break;
+            case 'Q':
+            case 'q':
+                $s .= "datename(quarter,$col)";
+                break;
+            case 'D':
+            case 'd':
+                $s .= "replace(str(day($col),2),' ','0')";
+                break;
+            case 'h':
+                $s .= "substring(convert(char(14),$col,0),13,2)";
+                break;
+
+            case 'H':
+                $s .= "replace(str(datepart(hh,$col),2),' ','0')";
+                break;
+
+            case 'i':
+                $s .= "replace(str(datepart(mi,$col),2),' ','0')";
+                break;
+            case 's':
+                $s .= "replace(str(datepart(ss,$col),2),' ','0')";
+                break;
+            case 'a':
+            case 'A':
+                $s .= "substring(convert(char(19),$col,0),18,2)";
+                break;
+
+            default:
+                if ($ch == '\\') {
+                    $i++;
+                    $ch = substr($fmt, $i, 1);
+                }
+                $s .= $this->qstr($ch);
+                break;
             }
         }
         return $s;

Braces +6 added lines, -2 removed lines

@@ -68,12 +68,16 @@
      */
     function sqlDate($fmt, $col = false)
     {
-        if (!$col) $col = "getdate()";
+        if (!$col) {
+            $col = "getdate()";
+        }
         $s = '';
 
         $len = strlen($fmt);
         for ($i = 0; $i < $len; $i++) {
-            if ($s) $s .= '+';
+            if ($s) {
+                $s .= '+';
+            }
             $ch = $fmt[$i];
             switch ($ch) {
                 case 'Y':

src/Database/DBMysqlFunctions.php

Doc Comments +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 
     /**
      * Return if the database provider have a top or similar function
-     * @return unknown_type
+     * @return boolean
      */
     function hasTop()
     {

Switch Indentation +53 added lines, -53 removed lines

@@ -82,59 +82,59 @@
         for ($i = 0; $i < $len; $i++) {
             $ch = $fmt[$i];
             switch ($ch) {
-                case 'Y':
-                case 'y':
-                    $s .= '%Y';
-                    break;
-                case 'Q':
-                case 'q':
-                    $s .= "'),Quarter($col)";
-
-                    if ($len > $i + 1) $s .= ",DATE_FORMAT($col,'";
-                    else $s .= ",('";
-                    $concat = true;
-                    break;
-                case 'M':
-                    $s .= '%b';
-                    break;
-
-                case 'm':
-                    $s .= '%m';
-                    break;
-                case 'D':
-                case 'd':
-                    $s .= '%d';
-                    break;
-
-                case 'H':
-                    $s .= '%H';
-                    break;
-
-                case 'h':
-                    $s .= '%I';
-                    break;
-
-                case 'i':
-                    $s .= '%i';
-                    break;
-
-                case 's':
-                    $s .= '%s';
-                    break;
-
-                case 'a':
-                case 'A':
-                    $s .= '%p';
-                    break;
-
-                default:
-
-                    if ($ch == '\\') {
-                        $i++;
-                        $ch = substr($fmt, $i, 1);
-                    }
-                    $s .= $ch;
-                    break;
+            case 'Y':
+            case 'y':
+                $s .= '%Y';
+                break;
+            case 'Q':
+            case 'q':
+                $s .= "'),Quarter($col)";
+
+                if ($len > $i + 1) $s .= ",DATE_FORMAT($col,'";
+                else $s .= ",('";
+                $concat = true;
+                break;
+            case 'M':
+                $s .= '%b';
+                break;
+
+            case 'm':
+                $s .= '%m';
+                break;
+            case 'D':
+            case 'd':
+                $s .= '%d';
+                break;
+
+            case 'H':
+                $s .= '%H';
+                break;
+
+            case 'h':
+                $s .= '%I';
+                break;
+
+            case 'i':
+                $s .= '%i';
+                break;
+
+            case 's':
+                $s .= '%s';
+                break;
+
+            case 'a':
+            case 'A':
+                $s .= '%p';
+                break;
+
+            default:
+
+                if ($ch == '\\') {
+                    $i++;
+                    $ch = substr($fmt, $i, 1);
+                }
+                $s .= $ch;
+                break;
             }
         }
         $s.="')";

Spacing +1 added lines, -1 removed lines

@@ -137,7 +137,7 @@
                     break;
             }
         }
-        $s.="')";
+        $s .= "')";
         if ($concat) $s = "CONCAT($s)";
         return $s;
     }

Braces +11 added lines, -4 removed lines

@@ -75,7 +75,9 @@ 
      */
     function sqlDate($fmt, $col = false)
     {
-        if (!$col) $col = $this->sysTimeStamp;
+        if (!$col) {
+            $col = $this->sysTimeStamp;
+        }
         $s = 'DATE_FORMAT(' . $col . ",'";
         $concat = false;
         $len = strlen($fmt);
@@ -90,8 +92,11 @@ 
                 case 'q':
                     $s .= "'),Quarter($col)";
 
-                    if ($len > $i + 1) $s .= ",DATE_FORMAT($col,'";
-                    else $s .= ",('";
+                    if ($len > $i + 1) {
+                        $s .= ",DATE_FORMAT($col,'";
+                    } else {
+                        $s .= ",('";
+                    }
                     $concat = true;
                     break;
                 case 'M':
@@ -138,7 +143,9 @@ 
             }
         }
         $s.="')";
-        if ($concat) $s = "CONCAT($s)";
+        if ($concat) {
+            $s = "CONCAT($s)";
+        }
         return $s;
     }
 

src/Database/DBOci8Driver.php

Doc Comments +1 added lines, -1 removed lines

@@ -185,7 +185,7 @@
 
     /**
      *
-     * @return handle
+     * @return resource|null
      */
     public function getDbConnection()
     {

src/Database/DBPgsqlFunctions.php

Doc Comments +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
 
     /**
      * Return if the database provider have a top or similar function
-     * @return unknown_type
+     * @return boolean
      */
     function hasTop()
     {

Switch Indentation +53 added lines, -53 removed lines

@@ -76,59 +76,59 @@
         for ($i = 0; $i < $len; $i++) {
             $ch = $fmt[$i];
             switch ($ch) {
-                case 'Y':
-                case 'y':
-                    $s .= 'YYYY';
-                    break;
-                case 'Q':
-                case 'q':
-                    $s .= 'Q';
-                    break;
-
-                case 'M':
-                    $s .= 'Mon';
-                    break;
-
-                case 'm':
-                    $s .= 'MM';
-                    break;
-                case 'D':
-                case 'd':
-                    $s .= 'DD';
-                    break;
-
-                case 'H':
-                    $s.= 'HH24';
-                    break;
-
-                case 'h':
-                    $s .= 'HH';
-                    break;
-
-                case 'i':
-                    $s .= 'MI';
-                    break;
-
-                case 's':
-                    $s .= 'SS';
-                    break;
-
-                case 'a':
-                case 'A':
-                    $s .= 'AM';
-                    break;
-
-                default:
-                    // handle escape characters...
-                    if ($ch == '\\') {
-                        $i++;
-                        $ch = substr($fmt, $i, 1);
-                    }
-                    if (strpos('-/.:;, ', $ch) !== false) {
-                        $s .= $ch;
-                    } else {
-                        $s .= '"' . $ch . '"';
-                    }
+            case 'Y':
+            case 'y':
+                $s .= 'YYYY';
+                break;
+            case 'Q':
+            case 'q':
+                $s .= 'Q';
+                break;
+
+            case 'M':
+                $s .= 'Mon';
+                break;
+
+            case 'm':
+                $s .= 'MM';
+                break;
+            case 'D':
+            case 'd':
+                $s .= 'DD';
+                break;
+
+            case 'H':
+                $s.= 'HH24';
+                break;
+
+            case 'h':
+                $s .= 'HH';
+                break;
+
+            case 'i':
+                $s .= 'MI';
+                break;
+
+            case 's':
+                $s .= 'SS';
+                break;
+
+            case 'a':
+            case 'A':
+                $s .= 'AM';
+                break;
+
+            default:
+                // handle escape characters...
+                if ($ch == '\\') {
+                    $i++;
+                    $ch = substr($fmt, $i, 1);
+                }
+                if (strpos('-/.:;, ', $ch) !== false) {
+                    $s .= $ch;
+                } else {
+                    $s .= '"' . $ch . '"';
+                }
             }
         }
         return $s . "')";

Spacing +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
                     break;
 
                 case 'H':
-                    $s.= 'HH24';
+                    $s .= 'HH24';
                     break;
 
                 case 'h':

Braces +3 added lines, -1 removed lines

@@ -69,7 +69,9 @@
      */
     function sqlDate($fmt, $col = false)
     {
-        if (!$col) $col = $this->sysTimeStamp;
+        if (!$col) {
+            $col = $this->sysTimeStamp;
+        }
         $s = 'TO_CHAR(' . $col . ",'";
 
         $len = strlen($fmt);

src/Database/DBSQLRelayDriver.php

Doc Comments +3 added lines

@@ -25,6 +25,9 @@
     protected $_conn;
     protected $_transaction = false;
 
+    /**
+     * @param ConnectionManagement $connMngt
+     */
     public function __construct($connMngt)
     {
         $this->_connectionManagement = $connMngt;

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
             sqlrcur_prepareQuery($cur, $sql);
             $bindCount = 1;
             foreach ($array as $key => $value) {
-                $field = strval($bindCount ++);
+                $field = strval($bindCount++);
                 sqlrcur_inputBind($cur, $field, $value);
             }
             $success = sqlrcur_executeQuery($cur);

src/Database/SQLBind.php

Doc Comments +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
      *
      * @param ConnectionManagement $connData
      * @param string $sql
-     * @param array $param
+     * @param array $params
      * @return array An array with the adjusted SQL and PARAMs
      */
     public static function parseSQL(ConnectionManagement $connData, $sql, $params = null)

src/Database/SQLHelper.php

Doc Comments +3 added lines, -1 removed lines

@@ -32,7 +32,6 @@ 
     /**
      * Generate and Execute UPDATE and INSERTS
      *
-     * @param DBDataset $db
      * @param string $table
      * @param array $fields
      * @param SQLType $type
@@ -148,6 +147,9 @@ 
         $this->_fieldDeliRight = $right;
     }
 
+    /**
+     * @param string $sql
+     */
     public static function createSafeSQL($sql, $list)
     {
         foreach ($list as $key => $value) {

Spacing +1 added lines, -1 removed lines

@@ -109,7 +109,7 @@
         } elseif ($valores[0] == SQLFieldType::DATE) {
             $date = ($valores[1] instanceof DateTime ? $valores[1]->format(DBBaseFunctions::YMDH) : $valores[1]);
             $param[$name] = $date;
-            if (($this->_db->getDbType() == 'oci8') || ( ($this->_db->getDbType() == 'dsn') && (strpos($this->_db->getDbConnectionString(),
+            if (($this->_db->getDbType() == 'oci8') || (($this->_db->getDbType() == 'dsn') && (strpos($this->_db->getDbConnectionString(),
                     "oci8")))) {
                 return "TO_DATE($paramName, 'YYYY-MM-DD')";
             } else {

src/Enum/FixedTextDefinition.php

Doc Comments +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
      * @param string $fieldName
      * @param int $startPos
      * @param int $length
-     * @param bool $requiredValue
+     * @param string|boolean $requiredValue
      * @param array_of_FixedTextDefinition $subTypes
      */
     public function __construct($fieldName, $startPos, $length, $requiredValue = "", $subTypes = null)