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;

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;
 
     /**

src/Model.php

Spacing +10 added lines, -10 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;
     }
     /**
@@ -82,7 +82,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();
@@ -114,15 +114,15 @@ 
         $columnNames = "";
         $columnValues = "";
         $bindNameParameters = [];
-        $sqlUpdate = "UPDATE " . $this->getTableName() . " SET " ;
+        $sqlUpdate = "UPDATE " . $this->getTableName() . " SET ";
         foreach ($this->properties as $columnName => $columnValue) {
-            if($key == 'id') continue;
+            if ($key == 'id') continue;
             $bindColumnName = ':' . $columnName;
             $sqlUpdate .= "$columnName = $bindColumnName,";
             $bindNameParameters[$bindColumnName] = $columnValue
         }
         //Remove the last comma in sql command then join it to the other query part.
-        $sqlUpdate = substr($sqlUpdate, 0, -1)." WHERE id = :id";
+        $sqlUpdate = substr($sqlUpdate, 0, -1) . " WHERE id = :id";
         $sqlStatement = $this->databaseConnection->prepare($sqlUpdate);
         $sqlStatement->bindValue(":id", $this->properties['id']);
         $sqlStatement->execute($bindNameParameters);
@@ -140,19 +140,19 @@ 
         $columnNames = "";
         $columnValues = "";
         $bindNameParameters = [];
-        $sqlCreate = "INSERT" . " INTO " . $this->getTableName()." (";
+        $sqlCreate = "INSERT" . " INTO " . $this->getTableName() . " (";
         foreach ($this->properties as $columnName => $columnValue) {
 
             $bindColumnName = ':' . $columnName;
-            $columnNames .= $columnName.",";
-            $columnValues .= $bindColumnName.",";
+            $columnNames .= $columnName . ",";
+            $columnValues .= $bindColumnName . ",";
             $bindNameParameters[$bindColumnName] = $columnValue
         }
         // Remove ending comma and whitespace.
         $columnNames = substr($columnNames, 0, -1);
         $columnValues = substr($columnValues, 0, -1);
 
-        $sqlCreate .= $columnNames.') VALUES (' .$columnValues.')';
+        $sqlCreate .= $columnNames . ') VALUES (' . $columnValues . ')';
         $sqlStatement = $this->databaseConnection->prepare($sqlCreate);
         $sqlStatement->execute($bindNameParameters);
         return $sqlStatement->rowCount();
@@ -187,7 +187,7 @@ 
      */
     public function delete($id)
     {
-        $sql = "DELETE" . " FROM " . self::getTableName()." WHERE id = ". $id;
+        $sql = "DELETE" . " FROM " . self::getTableName() . " WHERE id = " . $id;
         $sqlStatment = $this->databaseConnection->prepare($sql);
         $sqlStatment->execute();
         return ($sqlStatment->rowCount() > 0) ? true : false;

Braces +3 added lines, -1 removed lines

@@ -116,7 +116,9 @@
         $bindNameParameters = [];
         $sqlUpdate = "UPDATE " . $this->getTableName() . " SET " ;
         foreach ($this->properties as $columnName => $columnValue) {
-            if($key == 'id') continue;
+            if($key == 'id') {
+                continue;
+            }
             $bindColumnName = ':' . $columnName;
             $sqlUpdate .= "$columnName = $bindColumnName,";
             $bindNameParameters[$bindColumnName] = $columnValue

Indentation +49 added lines, -49 removed lines

@@ -10,15 +10,15 @@ 
 
     /**
      * Store instance of database connection used.
-    * @var [type]
-    */
+     * @var [type]
+     */
     protected  $databaseConnection;
 
     /**
      * Create a model instance.
      * 
      */
-     public function __construct()
+        public function __construct()
     {
         $this->databaseConnection = DatabaseConnection::getInstance()->databaseConnection;
        
@@ -26,19 +26,19 @@ 
     /**
      * Sets into $properties the $key => $value pairs
      * 
-    * @param string $key 
-    * @param string $val 
-    *
-    */
+     * @param string $key 
+     * @param string $val 
+     *
+     */
     public  function __set($key, $val)
     {
         $this->properties[$key] = $val;
     }
     /**
-    * @param string $key
-    * 
-    * @return array
-    */
+     * @param string $key
+     * 
+     * @return array
+     */
     public function __get($key)
     {
         return $this->properties[$key];
@@ -48,15 +48,15 @@ 
      *
      * @return array
      */
-     public function getProperties()
-     {
-         return $this->properties;
-     }
+        public function getProperties()
+        {
+            return $this->properties;
+        }
     /**
-    * Gets the name of the child class with a 's'.
-    *
-    * @return string
-    */
+     * Gets the name of the child class with a 's'.
+     *
+     * @return string
+     */
     public function getTableName()
     {
         $className = explode('\\', get_called_class());
@@ -64,12 +64,12 @@ 
         return $table;
     }
     /**
-    * Find the particular model with the passed id.
-    * 
-    * @param int $id
-    * 
-    * @return object
-    */
+     * Find the particular model with the passed id.
+     * 
+     * @param int $id
+     * 
+     * @return object
+     */
     public static function find($id)
     {
         $model = new static;
@@ -77,12 +77,12 @@ 
     }
 
     /**
-    * Get the particular model with the passed id.
-    * 
-    * @param int $id
-    * 
-    * @return object
-    */
+     * Get the particular model with the passed id.
+     * 
+     * @param int $id
+     * 
+     * @return object
+     */
     public function get($id)
     {
         $sql = "SELECT * FROM {$this->getTableName()} WHERE id={$id}";
@@ -96,10 +96,10 @@ 
     }
 
     /**
-    * Get all the models from the database.
-    * 
-    * @return array
-    */
+     * Get all the models from the database.
+     * 
+     * @return array
+     */
     public static function getAll()
     {
         $model = new static;
@@ -107,10 +107,10 @@ 
     }
 
     /**
-    * Returns all the models from the database.
-    * 
-    * @return array
-    */
+     * Returns all the models from the database.
+     * 
+     * @return array
+     */
     public function all()
     {
         $sql = "SELECT * FROM {$this->getTableName()}";
@@ -124,7 +124,7 @@ 
      * Update the model in the database.
      * 
      * @return int
-    */
+     */
     private function update()
     {
        
@@ -147,10 +147,10 @@ 
     }
 
     /**
-    * Insert the model values into the database.
-    *
-    * @return int
-    */
+     * Insert the model values into the database.
+     *
+     * @return int
+     */
     private function create()
     {
         
@@ -185,11 +185,11 @@ 
         return $this->id ? $this->update() : $this->create();
     }
 
-   /**
-    * Delete a model from the database. 
-    * @param  int $id 
-    * @return boolean
-    */
+    /**
+     * Delete a model from the database. 
+     * @param  int $id 
+     * @return boolean
+     */
     public static function destroy($id)
     {
         $model = new static;

Doc Comments +1 added lines, -1 removed lines

@@ -178,7 +178,7 @@
     /**
      * Save the model data to the database.
      * 
-     * @return boolean
+     * @return integer
      */
     public function save()
     {