andela-joyebanji / PotatoORM

src/DatabaseConnectionStringFactory.php

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

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()
     {

Braces +3 added lines, -1 removed lines

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

test/DatabaseConnectionTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
 	 * Reference: https://jtreminio.com/2013/03/unit-testing-tutorial-part-3-testing-protected-private-methods-coverage-reports-and-crap/
 	* Call protected/private method of a class.
 	*
-	* @param object &$object    Instantiated object that we will run method on.
+	* @param object DatabaseConnection    Instantiated object that we will run method on.
 	* @param string $methodName Method name to call
 	* @param array  $parameters Array of parameters to pass into method.
 	*

src/Exception/ModelNotFoundException.php

Doc Comments +3 added lines

@@ -5,6 +5,9 @@
 class ModelNotFoundException extends \Exception 
 {
 
+    /**
+     * @param integer $id
+     */
     function __construct($id)
     {
         parent::__construct('The requested Model with ' . $id . ' does not exist');