andela-joyebanji / PotatoORM

src/DatabaseConnectionStringFactory.php

Spacing +4 added lines, -4 removed lines

@@ -18,13 +18,13 @@
 
         switch ($driver) {
             case 'sqlite':
-                $dsn = $driver.'::memory:';
+                $dsn = $driver . '::memory:';
                 break;
             case 'mysql':
             case 'postgres':
-                if(strcasecmp($driver, 'postgres') == 0) $driver="pgsql";
-                $dsn = $driver.':host='.$config['HOSTNAME'].';dbname='.$config['DBNAME'];
-                if(isset($config['PORT'])) $dsn .= ';port='.$config['PORT'];
+                if (strcasecmp($driver, 'postgres') == 0) $driver = "pgsql";
+                $dsn = $driver . ':host=' . $config['HOSTNAME'] . ';dbname=' . $config['DBNAME'];
+                if (isset($config['PORT'])) $dsn .= ';port=' . $config['PORT'];
                 break;
             default:
                 throw new DatabaseDriverNotSupportedException;

Braces +6 added lines, -2 removed lines

@@ -22,9 +22,13 @@
                 break;
             case 'mysql':
             case 'postgres':
-                if(strcasecmp($driver, 'postgres') == 0) $driver="pgsql";
+                if(strcasecmp($driver, 'postgres') == 0) {
+                    $driver="pgsql";
+                }
                 $dsn = $driver.':host='.$config['HOSTNAME'].';dbname='.$config['DBNAME'];
-                if(isset($config['PORT'])) $dsn .= ';port='.$config['PORT'];
+                if(isset($config['PORT'])) {
+                    $dsn .= ';port='.$config['PORT'];
+                }
                 break;
             default:
                 throw new DatabaseDriverNotSupportedException;

src/Model.php

Spacing +13 added lines, -13 removed lines

@@ -55,7 +55,7 @@ 
     public function getTableName()
     {
         $className = explode('\\', get_called_class());
-        $table = strtolower(end($className) .'s');
+        $table = strtolower(end($className) . 's');
         return $table;
     }
     /**
@@ -81,7 +81,7 @@ 
         $sqlStatement = $this->databaseConnection->prepare($sql);
         $sqlStatement->setFetchMode($this->databaseConnection::FETCH_CLASS, get_called_class());
         $sqlStatement->execute();
-        if($sqlStatement->rowCount() < 1){
+        if ($sqlStatement->rowCount() < 1) {
             throw new ModelNotFoundException($id);
         }
         return $sqlStatement->fetch();
@@ -111,20 +111,20 @@ 
         $columnNames = "";
         $columnValues = "";
         $count = 0;
-        $update = "UPDATE " . $this->getTableName() . " SET " ;
+        $update = "UPDATE " . $this->getTableName() . " SET ";
         foreach ($this->properties as $key => $val) {
             $count++;
-            if(($key == 'id')) continue;
+            if (($key == 'id')) continue;
             $update .= "$key = '$val'";
-            if ($count < count($this->properties) )
+            if ($count < count($this->properties))
             {
-                $update .=",";
+                $update .= ",";
             }
         }
         $update .= " WHERE id = " . $this->properties['id'];
         $stmt = $connection->prepare($update);
             foreach ($this->properties as $key => $val) {
-                if($key == 'id') continue;
+                if ($key == 'id') continue;
             }
         $stmt->execute();
         return $stmt->rowCount();
@@ -138,7 +138,7 @@ 
         $columnNames = "";
         $columnValues = "";
         $count = 0;
-        $create = "INSERT" . " INTO " . $this->getTableName()." (";
+        $create = "INSERT" . " INTO " . $this->getTableName() . " (";
             foreach ($this->properties as $key => $val) {
                 $columnNames .= $key;
                 $columnValues .= ':' . $key;
@@ -149,15 +149,15 @@ 
                     $columnValues .= ', ';
                 }
             }
-        $create .= $columnNames.') VALUES (' .$columnValues.')';
+        $create .= $columnNames . ') VALUES (' . $columnValues . ')';
         $stmt = $connection->prepare($create);
             foreach ($this->properties as $key => $val) {
-                $stmt->bindValue(':'.$key, $val);
+                $stmt->bindValue(':' . $key, $val);
             }
             try {
                 // if prop returned and props from db differ throw exception
                 $stmt->execute();
-            } catch(PDOException $e){
+            } catch (PDOException $e) {
                 return $e->getExceptionMessage();
             }
         return $stmt->rowCount();
@@ -167,7 +167,7 @@ 
     */
     public function getConnection($connection = null)
     {
-        if(is_null($connection))
+        if (is_null($connection))
         {
             return new Connection();
         }
@@ -193,7 +193,7 @@ 
     public static function destroy($id)
     {
         
-        $sql = "DELETE" . " FROM " . self::getTableName()." WHERE id = ". $id;
+        $sql = "DELETE" . " FROM " . self::getTableName() . " WHERE id = " . $id;
         $delete = $connection->prepare($sql);
         $delete->execute();
         $count = $delete->rowCount();

Braces +6 added lines, -2 removed lines

@@ -114,7 +114,9 @@ 
         $update = "UPDATE " . $this->getTableName() . " SET " ;
         foreach ($this->properties as $key => $val) {
             $count++;
-            if(($key == 'id')) continue;
+            if(($key == 'id')) {
+                continue;
+            }
             $update .= "$key = '$val'";
             if ($count < count($this->properties) )
             {
@@ -124,7 +126,9 @@ 
         $update .= " WHERE id = " . $this->properties['id'];
         $stmt = $connection->prepare($update);
             foreach ($this->properties as $key => $val) {
-                if($key == 'id') continue;
+                if($key == 'id') {
+                    continue;
+                }
             }
         $stmt->execute();
         return $stmt->rowCount();

Indentation +44 added lines, -44 removed lines

@@ -146,28 +146,28 @@ 
 
     /**
      * Store instance of database connection used.
-    * @var [type]
-    */
+     * @var [type]
+     */
     protected  $databaseConnection;
 
-     public function __construct()
+        public function __construct()
     {
         $this->databaseConnection = DatabaseConnection::getInstance()->databaseConnection;
         //$databaseConnection->databaseConnection->connect();
     }
     /**
-    * @param string $key rep column name
-    * @param string $val rep column value
-    * sets into $propertie the $key => $value pairs
-    */
+     * @param string $key rep column name
+     * @param string $val rep column value
+     * sets into $propertie the $key => $value pairs
+     */
     public  function __set($key, $val)
     {
         $this->properties[$key] = $val;
     }
     /**
-    * @param string $key reps the column name
-    * @return $key and $value
-    */
+     * @param string $key reps the column name
+     * @return $key and $value
+     */
     public function __get($key)
     {
         return $this->properties[$key];
@@ -177,17 +177,17 @@ 
      *
      * @return array
      */
-     public function getProperties()
-     {
-         return $this->properties;
-     }
+        public function getProperties()
+        {
+            return $this->properties;
+        }
     /**
-    * Gets the name of the child class only
-    * without the namespace
-    * @var $className
-    * @var $table
-    * @return $table
-    */
+     * Gets the name of the child class only
+     * without the namespace
+     * @var $className
+     * @var $table
+     * @return $table
+     */
     public function getTableName()
     {
         $className = explode('\\', get_called_class());
@@ -195,22 +195,22 @@ 
         return $table;
     }
     /**
-    * returns a particular record
-    * @param $id reps the record id
-    * @param $connection initialised to null
-    * @return object
-    */
+     * returns a particular record
+     * @param $id reps the record id
+     * @param $connection initialised to null
+     * @return object
+     */
     public static function find($id)
     {
         $model = new static;
         return $model->get($id); 
     }
     /**
-    * returns a particular record
-    * @param $id reps the record id
-    * @param $connection initialised to null
-    * @return object
-    */
+     * returns a particular record
+     * @param $id reps the record id
+     * @param $connection initialised to null
+     * @return object
+     */
     public function get($id)
     {
         $sql = "SELECT * FROM {$this->getTableName()} WHERE id={$id}";
@@ -239,8 +239,8 @@ 
 
     }
     /** update table with instance properties
-    *
-    */
+     *
+     */
     private function update()
     {
         $connection = $this->getConnection();
@@ -266,8 +266,8 @@ 
         return $stmt->rowCount();
     }
     /**
-    * insert instance data into the table
-    */
+     * insert instance data into the table
+     */
     private function create()
     {
         $connection = $this->getConnection();
@@ -299,8 +299,8 @@ 
         return $stmt->rowCount();
     }
     /**
-    * get db connection
-    */
+     * get db connection
+     */
     public function getConnection($connection = null)
     {
         if(is_null($connection))
@@ -309,10 +309,10 @@ 
         }
     }
     /**
-    * checks if the id exists
-    * update if exist
-    * create if not exist
-    */
+     * checks if the id exists
+     * update if exist
+     * create if not exist
+     */
     public function save()
     {
         if ($this->id) {
@@ -322,10 +322,10 @@ 
         }
     }
     /**
-    * @param row reps record id
-    * @param $connection initialised to null
-    * @return boolean
-    */
+     * @param row reps record id
+     * @param $connection initialised to null
+     * @return boolean
+     */
     public static function destroy($id)
     {
         

test/DatabaseConnectionStringFactoryTest.php

Indentation +5 added lines, -5 removed lines

@@ -4,11 +4,11 @@
 
 class DatabaseConnectionStringFactoryTest extends PHPUnit_Framework_TestCase
 {
-     /**
-     * The instance of DatabaseConnectionStringFactory used in the test.
-     *
-     * @var Pyjac\ORM\DatabaseConnectionStringFactory
-     */
+        /**
+         * The instance of DatabaseConnectionStringFactory used in the test.
+         *
+         * @var Pyjac\ORM\DatabaseConnectionStringFactory
+         */
     protected $openSourceEvangelistFactory;
 
     /**