kristuff / patabase

lib/Query/Insert.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
         foreach ($this->parameters as $key => $val) {
             $arg = $this->getArgName($key);
             $columnsNames[]    = $this->escape($key); 
-            $columnsValues[]   = ':' . $arg;
+            $columnsValues[]   = ':'.$arg;
         }        
         return trim(sprintf(
             'INSERT INTO %s (%s) VALUES (%s)',

lib/Query/QueryBase.php

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
      */
     public function errorCode()
     {
-        return !empty($this->error) ? $this->error['code']: null;
+        return !empty($this->error) ? $this->error['code'] : null;
     }
 
     /**

lib/Query/QueryFilter.php

Indentation +3 added lines, -3 removed lines

@@ -204,7 +204,7 @@ 
      */
     protected function returnFunction()
     {
-         return $this->isGroupOpen === true ? $this : $this->query;
+            return $this->isGroupOpen === true ? $this : $this->query;
     }
 
     /**
@@ -262,7 +262,7 @@ 
                         $sql .= ')';
                         break;
 
-                   case 'NULL':
+                    case 'NULL':
                    case 'NOT_NULL':
                         $sql .=  $isSqlNeedOperator ? ' '.$currentOperator.' ' : '';
                         $sql .=  $item['sql'];
@@ -279,7 +279,7 @@ 
                         if (is_string($item['value']) && 
                             strlen($item['value']) >  strlen(Patabase\Constants::COLUMN_LITERALL) &&
                             substr($item['value'], 0, strlen(Patabase\Constants::COLUMN_LITERALL)) === 
-                                                             Patabase\Constants::COLUMN_LITERALL) {
+                                                                Patabase\Constants::COLUMN_LITERALL) {
 
                             $arg = substr($item['value'], strlen(Patabase\Constants::COLUMN_LITERALL));
                             $sql .=  $isSqlNeedOperator ? ' '.$currentOperator.' ' : '';

Braces +1 added lines, -1 removed lines

@@ -285,7 +285,7 @@
                             $sql .=  $isSqlNeedOperator ? ' '.$currentOperator.' ' : '';
                             $sql .=  $item['sql'] . $this->query->escape($arg);
 
-                        }   else {
+                        } else {
                                 // *normal* value 
                                 $arg = $this->getArgumentName($item['column']);
                                 $sql .=  $isSqlNeedOperator ? ' '.$currentOperator.' ' : '';

Spacing +18 added lines, -18 removed lines

@@ -112,8 +112,8 @@ 
      */
     private function getArgumentName(string $column): string
     {
-        $arg = ':__' . str_replace('.', '_', $column); 
-        return $this->topQuery->sqlParameterExists($arg) ? $arg . uniqid() : $arg;
+        $arg = ':__'.str_replace('.', '_', $column); 
+        return $this->topQuery->sqlParameterExists($arg) ? $arg.uniqid() : $arg;
     }
 
     /**
@@ -215,7 +215,7 @@ 
     {
         // define argument for each values
         $valueArgs = array();
-        foreach($item['value'] as $value){
+        foreach ($item['value'] as $value) {
             $arg = $this->getArgumentName($item['column']); 
             $valueArgs[] = $arg;
 
@@ -224,7 +224,7 @@ 
         }
         
         //build and return sql
-        return $item['sql']. '(' . implode(', ', $valueArgs) .')';
+        return $item['sql'].'('.implode(', ', $valueArgs).')';
     }
 
     /**
@@ -237,20 +237,20 @@ 
     {
         $sql = '';
         if (!empty($this->conditions)) {
-            $sql = ' '. $this->sqlBase  .' ';  // start the SQL WHERE or HAVING clause
-            $currentOperator = 'AND';          // current condition operator
+            $sql = ' '.$this->sqlBase.' '; // start the SQL WHERE or HAVING clause
+            $currentOperator = 'AND'; // current condition operator
             
             foreach ($this->conditions as $key => $item) {
 
                 // need operator AND or OR, except for the first or if 
                 // previous item is a begin group item
-                $isSqlNeedOperator = $key > 0  && $this->conditions[$key -1]['type'] != 'group_start';
+                $isSqlNeedOperator = $key > 0 && $this->conditions[$key-1]['type'] != 'group_start';
 
                 switch ($item['type']) {
                     
                     case'group_start':
                         $currentOperator = $item['operator']; // register operator
-                        $sql .= '(' ;
+                        $sql .= '(';
                         break;
                     
                     case'group_end':
@@ -260,32 +260,32 @@ 
 
                    case 'NULL':
                    case 'NOT_NULL':
-                        $sql .=  $isSqlNeedOperator ? ' '.$currentOperator.' ' : '';
-                        $sql .=  $item['sql'];
+                        $sql .= $isSqlNeedOperator ? ' '.$currentOperator.' ' : '';
+                        $sql .= $item['sql'];
                         break;
 
                     case 'IN':
                     case 'NOT_IN':
-                        $sql .=  $isSqlNeedOperator ? ' '.$currentOperator.' ' : '';
-                        $sql .=  $this->getSqlInOrNotIn($item);
+                        $sql .= $isSqlNeedOperator ? ' '.$currentOperator.' ' : '';
+                        $sql .= $this->getSqlInOrNotIn($item);
                         break;
 
                     default:
                         // support for column literral 
                         if (is_string($item['value']) && 
-                            strlen($item['value']) >  strlen(Patabase\Constants::COLUMN_LITERALL) &&
+                            strlen($item['value']) > strlen(Patabase\Constants::COLUMN_LITERALL) &&
                             substr($item['value'], 0, strlen(Patabase\Constants::COLUMN_LITERALL)) === 
                                                              Patabase\Constants::COLUMN_LITERALL) {
 
                             $arg = substr($item['value'], strlen(Patabase\Constants::COLUMN_LITERALL));
-                            $sql .=  $isSqlNeedOperator ? ' '.$currentOperator.' ' : '';
-                            $sql .=  $item['sql'] . $this->query->escape($arg);
+                            $sql .= $isSqlNeedOperator ? ' '.$currentOperator.' ' : '';
+                            $sql .= $item['sql'].$this->query->escape($arg);
 
-                        }   else {
+                        } else {
                                 // *normal* value 
                                 $arg = $this->getArgumentName($item['column']);
-                                $sql .=  $isSqlNeedOperator ? ' '.$currentOperator.' ' : '';
-                                $sql .=  $item['sql'] . $arg;
+                                $sql .= $isSqlNeedOperator ? ' '.$currentOperator.' ' : '';
+                                $sql .= $item['sql'].$arg;
                                
                                 // set parameters
                                 $this->topQuery->setSqlParameter($arg, $item['value']); 

lib/Query/Update.php

Indentation +3 added lines, -3 removed lines

@@ -143,9 +143,9 @@
         $sqlWhere = (isset($this->where)) ? $this->where->sql() : '';
         $sqlTableName = $this->escape($this->tableName);
         return trim(sprintf('UPDATE %s SET %s %s',
-           $sqlTableName,
-           $this->sqlColumns(),
-           $sqlWhere
+            $sqlTableName,
+            $this->sqlColumns(),
+            $sqlWhere
         ));
     }
 }
\ No newline at end of file

Spacing +5 added lines, -5 removed lines

@@ -58,15 +58,15 @@ 
     {
         $columns = array();
         foreach ($this->parameters as $key => $val) {
-            $arg       = ':_' . str_replace('.', '_', $key); 
-            $columns[] = $this->escape($key) . ' =' .$arg; 
+            $arg       = ':_'.str_replace('.', '_', $key); 
+            $columns[] = $this->escape($key).' ='.$arg; 
             $this->pdoParameters[$arg] = $val; 
         }
         foreach ($this->incrementColumns as $key => $val) {
-            $columns[] = $this->escape($key) . ' =' . $this->escape($key) . '+' .$val ; 
+            $columns[] = $this->escape($key).' ='.$this->escape($key).'+'.$val; 
         }
         foreach ($this->decrementColumns as $key => $val) {
-            $columns[] = $this->escape($key) . ' =' . $this->escape($key) . '-' .$val ; 
+            $columns[] = $this->escape($key).' ='.$this->escape($key).'-'.$val; 
         }
         return implode(', ', $columns);
     }
@@ -79,7 +79,7 @@ 
      */
     public function where(): Where
     {
-        if (!isset($this->where)){
+        if (!isset($this->where)) {
             $this->where = new Query\Where($this, $this->driver);
         }
         return $this->where; 

lib/Driver/DriverFactory.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
      */
     public static function getInstance(array $settings, $isServerConnection = false)
     {
-        if (! isset($settings['driver'])) {
+        if (!isset($settings['driver'])) {
             throw new Exception\MissingArgException('You must define a driver');
         }
 

lib/Database.php

Indentation +1 added lines, -1 removed lines

@@ -97,7 +97,7 @@
      * @param string    $tableName      The name of the table
      * 
      * @return bool     True if the table has been dropped, otherwise false
-      */
+     */
     public function dropTable(string $tableName): bool
     {
         $sql = trim(sprintf('DROP TABLE %s', $tableName));

Spacing +1 added lines, -1 removed lines

@@ -261,7 +261,7 @@
      */
     public function beginTransaction(): void
     {
-        if (! $this->inTransaction()) {
+        if (!$this->inTransaction()) {
             $this->getConnection()->beginTransaction();
         }
     }

lib/Driver/DatabaseDriver.php

Indentation +2 added lines, -2 removed lines

@@ -131,8 +131,8 @@
      */
     public function escape(string $str): string
     {
-       $list = explode('.', $str);
-       return implode('.', $this->escapeList($list));
+        $list = explode('.', $str);
+        return implode('.', $this->escapeList($list));
     }
 
     /**

Spacing +6 added lines, -6 removed lines

@@ -162,15 +162,15 @@ 
     {
         // check for required attributes
         foreach ($this->dsnAttributes as $attribute) {
-            if (! array_key_exists($attribute, $settings)) {
+            if (!array_key_exists($attribute, $settings)) {
                 throw new Exception\MissingArgException('This configuration parameter is missing: "'.$attribute.'"');
             }
         }
 
         // defaut output format
-        if (array_key_exists('default_output_format', $settings)){
+        if (array_key_exists('default_output_format', $settings)) {
             $format = $settings['default_output_format'];
-            if (!in_array($format, $this->outputFormats)){
+            if (!in_array($format, $this->outputFormats)) {
                 throw new Exception\InvalidArgException('The default output format specified is invalid.');
             } 
             $this->defaultOutputFormat = $format;
@@ -178,7 +178,7 @@ 
 
         $this->createConnection($settings);
         $this->pdo->setAttribute(\PDO::ATTR_ERRMODE, \PDO::ERRMODE_EXCEPTION);
-        $this->hostname = array_key_exists('hostname',$settings) && $settings['hostname'] ? $settings['hostname'] : '';
+        $this->hostname = array_key_exists('hostname', $settings) && $settings['hostname'] ? $settings['hostname'] : '';
         $this->driverName = $settings['driver'];
     }
 
@@ -212,7 +212,7 @@ 
      */
     public function errorCode(): int
     {
-        return !empty($this->error) ? $this->error['code']: '';
+        return !empty($this->error) ? $this->error['code'] : '';
     }
 
     /**
@@ -256,7 +256,7 @@ 
             return $pdoStatement->execute($parameters);
         } catch (\PDOException $e) {
             // register error
-            $this->error['code'] = (int)$e->getCode();
+            $this->error['code'] = (int) $e->getCode();
             $this->error['message'] = $e->getMessage();
             return false;
         }

lib/Driver/Postgres/PostgresDriver.php

Spacing +8 added lines, -8 removed lines

@@ -98,9 +98,9 @@ 
      */
     public function createConnection(array $settings): void
     {
-        $port    = !empty($settings['port'])     ?  ';port='.$settings['port']        : '';
-        $dbname  = !empty($settings['database']) ?  ';dbname='.$settings['database']  : '';
-        $dsn     = 'pgsql:host='.$settings['hostname'] .$port .$dbname ;
+        $port    = !empty($settings['port']) ? ';port='.$settings['port'] : '';
+        $dbname  = !empty($settings['database']) ? ';dbname='.$settings['database'] : '';
+        $dsn     = 'pgsql:host='.$settings['hostname'].$port.$dbname;
 
         $this->pdo = new \PDO(
             $dsn,
@@ -222,7 +222,7 @@ 
     {
         $sql = 'SELECT COUNT(*) FROM pg_database WHERE datname = :dbName'; 
         $query = $this->pdo->prepare($sql);
-        $query->bindValue(':dbName',  $databaseName, \PDO::PARAM_STR);
+        $query->bindValue(':dbName', $databaseName, \PDO::PARAM_STR);
         $query->execute();
         return (bool) $query->fetchColumn();
     }
@@ -237,11 +237,11 @@ 
      *
      * @return bool     True if the database has been created, otherwise false.
      */
-    public function createDatabase(string $databaseName, ?string $owner= null, ?string $template = 'template0'): bool
+    public function createDatabase(string $databaseName, ?string $owner = null, ?string $template = 'template0'): bool
     {
         $sql = trim(sprintf('CREATE DATABASE %s %s TEMPLATE %s', 
             $this->escape($databaseName),
-            isset($owner) ? 'OWNER '. $this->escape($owner) : '', 
+            isset($owner) ? 'OWNER '.$this->escape($owner) : '', 
             $template
         ));
         return $this->prepareAndExecuteSql($sql);
@@ -260,7 +260,7 @@ 
     {
         $sql = trim(sprintf('CREATE USER %s PASSWORD %s', 
                     $this->escape($userName), 
-                    "'" . $userPassword ."'"
+                    "'".$userPassword."'"
         ));
         return $this->prepareAndExecuteSql($sql);
     }
@@ -277,7 +277,7 @@ 
     public function dropUser(string $userName, bool $ifExists = false): bool
     {
         $sql = trim(sprintf('DROP USER %s %s', 
-                    $ifExists === true ? 'IF EXISTS': '',
+                    $ifExists === true ? 'IF EXISTS' : '',
                     $this->escape($userName)
         ));
         return $this->prepareAndExecuteSql($sql);

Braces +1 added lines, -2 removed lines

@@ -129,8 +129,7 @@
             $rq->execute();
             // return string 
             return strval($rq->fetchColumn());
-        }
-        catch (\PDOException $e) {
+        } catch (\PDOException $e) {
             return 0;
         }
     }

Indentation +11 added lines, -12 removed lines

@@ -122,9 +122,9 @@ 
      */
     public function lastInsertedId(): string
     {
-       // Postgres does not set pdo->lastInsertedId
-       // use sequence
-       try {
+        // Postgres does not set pdo->lastInsertedId
+        // use sequence
+        try {
             $rq = $this->pdo->prepare('SELECT LASTVAL()');
             $rq->execute();
             // return string 
@@ -158,7 +158,6 @@ 
     /**
      * Get whether foreign keys are enabled or not
      * For compatibility with Sqlite, not implemented in that driver (allways enabled), return true 
-
      * @access public
      * @return bool     true if foreign keys are enabled, otherwise false
      */
@@ -182,11 +181,11 @@ 
     public function addForeignKey(string $fkName, string $srcTable, string $srcColumn, string $refTable, string $refColumn): bool
     {
         $sql = sprintf('ALTER TABLE %s ADD CONSTRAINT %s FOREIGN KEY (%s) REFERENCES %s(%s)',
-                       $this->escape($srcTable),
-                       $fkName,
-                       $this->escape($srcColumn),
-                       $this->escape($refTable),
-                       $this->escape($refColumn)
+                        $this->escape($srcTable),
+                        $fkName,
+                        $this->escape($srcColumn),
+                        $this->escape($refTable),
+                        $this->escape($refColumn)
         );
         return $this->prepareAndExecuteSql($sql);
     }
@@ -203,9 +202,9 @@ 
     public function dropForeignKey(string $fkName, string $tableName, bool $ifExists = false): bool
     {
         $sql = sprintf('ALTER TABLE %s DROP CONSTRAINT %s %s',
-                       $this->escape($tableName),
-                       $ifExists ? 'IF EXISTS' : '',
-                       $fkName
+                        $this->escape($tableName),
+                        $ifExists ? 'IF EXISTS' : '',
+                        $fkName
         );
         return $this->prepareAndExecuteSql($sql);
     }

lib/Driver/ServerDriver.php

Spacing +2 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
     public function __construct(array $settings, bool $isServerConnection = false)
     {
         // remove database attribute for server connection
-        if ($isServerConnection){
+        if ($isServerConnection) {
             $dbKey = array_search('database', $this->dsnAttributes);
             if ($dbKey !== false) {
                 unset($this->dsnAttributes[$dbKey]);
@@ -66,7 +66,7 @@ 
      *
      * @return bool     True if the database has been created, otherwise false.
      */
-    abstract public function createDatabase(string $databaseName, ?string $owner= null, ?string $template = null): bool;
+    abstract public function createDatabase(string $databaseName, ?string $owner = null, ?string $template = null): bool;
 
     /**
      * Create a user