andela-oogunjimi / Potato-ORM

src/Connections/MySqlConnection.php

Spacing +3 added lines, -3 removed lines

@@ -12,9 +12,9 @@
     {
         $this->useDbEnv();
 
-        $dsn = 'mysql:host='.$this->_host
-                .';port='.$this->_port
-                .';dbname='.$this->_database;
+        $dsn = 'mysql:host=' . $this->_host
+                .';port=' . $this->_port
+                .';dbname=' . $this->_database;
 
         try {
             $this->_pdo = new PDO($dsn, $this->_username, $this->_password);

Unused Use Statements +3 added lines, -3 removed lines

@@ -2,12 +2,12 @@
 
 namespace Opeyemiabiodun\PotatoORM\Connections;
 
-use PDO;
-use PDOException;
 use InvalidArgumentException;
 use Opeyemiabiodun\PotatoORM\Connections\Connection;
-use Opeyemiabiodun\PotatoORM\Connections\LoadEnvVariablesTrait;
 use Opeyemiabiodun\PotatoORM\Connections\DatabaseTransactionsTrait;
+use Opeyemiabiodun\PotatoORM\Connections\LoadEnvVariablesTrait;
+use PDO;
+use PDOException;
 
 final class MySqlConnection implements Connection
 {

src/Models/Model.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,12 +2,12 @@
 
 namespace Opeyemiabiodun\PotatoORM\Models;
 
-use RuntimeException;
 use InvalidArgumentException;
 use Opeyemiabiodun\PotatoORM\Connections\Connection;
 use Opeyemiabiodun\PotatoORM\Connections\PgSqlConnection;
 use Opeyemiabiodun\PotatoORM\Exceptions\AssignmentException;
 use Opeyemiabiodun\PotatoORM\Exceptions\PropertyNotFoundException;
+use RuntimeException;
 
 trait Model
 {

Doc Comments +2 added lines, -2 removed lines

@@ -123,7 +123,7 @@ 
      *
      * @param int $number Specifies which model instance to find; the 1st, 2nd, 3rd, .....
      *
-     * @return array Returns the particular instance of the model.
+     * @return Model Returns the particular instance of the model.
      */
     public static function find($number)
     {
@@ -206,7 +206,7 @@ 
     /**
      * Sets the model's connection.
      *
-     * @param Opeyemiabiodun\PotatoORM\Connections\Connection $connection An instance of Opeyemiabiodun\PotatoORM\Connections\Connection.
+     * @param Connection $connection An instance of Opeyemiabiodun\PotatoORM\Connections\Connection.
      */
     protected function setConnection(Connection $connection)
     {

Spacing +13 added lines, -13 removed lines

@@ -52,13 +52,13 @@ 
         }
 
         if (null === $table) {
-            $table = strtolower(substr(get_class($this),strripos(get_class($this), "\\") + 1))."_table";
+            $table = strtolower(substr(get_class($this), strripos(get_class($this), "\\") + 1)) . "_table";
         }
 
         $this->setConnection($connection);
         $this->setTable($table);
 
-        if (! empty($array)) {
+        if (!empty($array)) {
             $this->setProperties($array);
         }
     }
@@ -75,7 +75,7 @@ 
         if (array_key_exists($property, $this->_attributes)) {
             return $this->_attributes[$property];
         } else {
-            throw new PropertyNotFoundException("The ".get_class($this)." instance has no {$property} property.");
+            throw new PropertyNotFoundException("The " . get_class($this) . " instance has no {$property} property.");
         }
     }
 
@@ -87,14 +87,14 @@ 
      */
     public function __set($property, $value)
     {
-        if (! is_scalar($value)) {
+        if (!is_scalar($value)) {
             throw new AssignmentException("Only scalar values can be assigned to the {$property} property.");
         }
 
         if (array_key_exists($property, $this->_attributes)) {
             $this->_attributes[$property] = $value;
         } else {
-            throw new PropertyNotFoundException("The ".get_class($this)." instance has no {$property} property.");
+            throw new PropertyNotFoundException("The " . get_class($this) . " instance has no {$property} property.");
         }
     }
 
@@ -107,7 +107,7 @@ 
      */
     public static function destroy($number)
     {
-        if (! is_int($number)) {
+        if (!is_int($number)) {
             throw new InvalidArgumentException("The parameter {$number} is not an integer. An integer is required instead.");
         }
 
@@ -127,7 +127,7 @@ 
      */
     public static function find($number)
     {
-        if (! is_int($number)) {
+        if (!is_int($number)) {
             throw new InvalidArgumentException("The parameter {$number} is not an integer. An integer is required instead.");
         }
 
@@ -156,7 +156,7 @@ 
         $count = count($records);
         $result = [];
 
-        for ($i=0; $i < $count ; $i++) { 
+        for ($i = 0; $i < $count; $i++) { 
             array_push($result, new self($records[$i]));
         }
 
@@ -173,7 +173,7 @@ 
         $hasAttributes = false;
 
         foreach ($this->_attributes as $key => $value) {
-            if (! is_null($value)) {
+            if (!is_null($value)) {
                 $hasAttributes = true;
             }
         }
@@ -188,11 +188,11 @@ 
      */
     public function save()
     {
-        if (! $this->hasAttributes()) {
-            throw new RuntimeException(get_class($this)." model has nothing to persist to the database.");
+        if (!$this->hasAttributes()) {
+            throw new RuntimeException(get_class($this) . " model has nothing to persist to the database.");
         }
 
-        $pk = (empty($this->_attributes[self::$_primaryKey])) ? "NULL" :  $this->_attributes[self::$_primaryKey];
+        $pk = (empty($this->_attributes[self::$_primaryKey])) ? "NULL" : $this->_attributes[self::$_primaryKey];
 
         #$record = self::$_connection->findRecord(self::$_table, (string) $pk);
 
@@ -222,7 +222,7 @@ 
     {
         foreach ($this->_attributes as $key => $value) {
 
-            if (! empty($array[$key])) {
+            if (!empty($array[$key])) {
                 $this->_attributes[$key] = $array[$key];
             }
 

src/Connections/LoadEnvVariablesTrait.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -2,8 +2,8 @@
 
 namespace Opeyemiabiodun\PotatoORM\Connections;
 
-use Exception;
 use Dotenv\Dotenv;
+use Exception;
 use RuntimeException;
 
 trait LoadEnvVariablesTrait

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
      */
     private function loadDbEnv()
     {
-        $dotenv = new Dotenv(__DIR__.'/../..');
+        $dotenv = new Dotenv(__DIR__ . '/../..');
         $dotenv->load();
 
         $dotenv->required(['DB_HOST', 'DB_DATABASE', 'DB_USERNAME', 'DB_PASSWORD', 'DB_PORT'])->notEmpty();

src/Connections/PgSqlConnection.php

Unused Use Statements +4 added lines, -4 removed lines

@@ -2,13 +2,13 @@
 
 namespace Opeyemiabiodun\PotatoORM\Connections;
 
-use PDO;
-use PDOException;
-use RuntimeException;
 use InvalidArgumentException;
 use Opeyemiabiodun\PotatoORM\Connections\Connection;
-use Opeyemiabiodun\PotatoORM\Connections\LoadEnvVariablesTrait;
 use Opeyemiabiodun\PotatoORM\Connections\DatabaseTransactionsTrait;
+use Opeyemiabiodun\PotatoORM\Connections\LoadEnvVariablesTrait;
+use PDO;
+use PDOException;
+use RuntimeException;
 
 final class PgSqlConnection implements Connection
 {

src/Connections/DatabaseTransactionsTrait.php

Spacing +6 added lines, -6 removed lines

@@ -36,9 +36,9 @@ 
             }
 
             if ($count > 0) {
-                $sql = $sql."{$key}, ";
+                $sql = $sql . "{$key}, ";
             } else {
-                $sql = $sql."{$key}) ";
+                $sql = $sql . "{$key}) ";
             }
         }
 
@@ -54,9 +54,9 @@ 
             }
 
             if ($count > 0) {
-                $sql = (empty($value)) ? $sql."NULL, " : $sql."'{$value}', ";
+                $sql = (empty($value)) ? $sql . "NULL, " : $sql . "'{$value}', ";
             } else {
-                $sql = (empty($value)) ? $sql."NULL " : $sql."'{$value}') ";
+                $sql = (empty($value)) ? $sql . "NULL " : $sql . "'{$value}') ";
             }
         }
 
@@ -151,11 +151,11 @@ 
 
             if ($count > 0) {
 
-                $sql = $sql."{$key}='{$value}', ";
+                $sql = $sql . "{$key}='{$value}', ";
 
             } else {
 
-                $sql = $sql."{$key}='{$value}' ";
+                $sql = $sql . "{$key}='{$value}' ";
 
             }
         }

src/Connections/ConnectionFactory.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -3,8 +3,8 @@
 namespace Opeyemiabiodun\PotatoORM\Connections;
 
 use Dotenv\Dotenv;
-use Opeyemiabiodun\PotatoORM\Connections\PgSqlConnection;
 use Opeyemiabiodun\PotatoORM\Connections\MySqlConnection;
+use Opeyemiabiodun\PotatoORM\Connections\PgSqlConnection;
 use Opeyemiabiodun\PotatoORM\Connections\SqliteConnection;
 
 class ConnectionFactory

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
 	public static function load()
 	{
 
-                $dotenv = new Dotenv(__DIR__.'/../..');
+                $dotenv = new Dotenv(__DIR__ . '/../..');
                 $dotenv->load();
 
                 $dotenv->required(['DB_ENGINE'])->allowedValues(['mysql', 'pgsql', 'sqlite']);

Doc Comments +3 added lines

@@ -19,6 +19,9 @@
 	{
 	}
 
+	/**
+	 * @return Connection
+	 */
 	public static function load()
 	{
                 

Indentation +9 added lines, -9 removed lines

@@ -15,12 +15,12 @@ 
 
         private static $_sqliteConnection;
 
-	private function __construct()
-	{
-	}
+    private function __construct()
+    {
+    }
 
-	public static function load()
-	{
+    public static function load()
+    {
                 
                 $dotenv = new Dotenv(__DIR__.'/../..');
                 $dotenv->load();
@@ -29,13 +29,13 @@ 
 
                 switch (getenv('DB_ENGINE')) 
                 {
-                	case 'mysql':
+                    case 'mysql':
                                 if (self::$_mySqlConnection == null) {
                                         self::$_mySqlConnection = new MySqlConnection();        
                                 } 
-        		        return self::$_mySqlConnection;
+                        return self::$_mySqlConnection;
 
-        		case 'pgsql':
+                case 'pgsql':
                                 if (self::$_pgSqlConnection == null) {
                                         self::$_pgSqlConnection = new PgSqlConnection();        
                                 } 
@@ -47,5 +47,5 @@ 
                                 } 
                                 return self::$_sqliteConnection;
                 }		
-	}
+    }
 }
\ No newline at end of file

src/Connections/SqliteConnection.php

Unused Use Statements +2 added lines, -3 removed lines

@@ -2,12 +2,11 @@
 
 namespace Opeyemiabiodun\PotatoORM\Connections;
 
-use PDO;
-use PDOException;
-use RuntimeException;
 use InvalidArgumentException;
 use Opeyemiabiodun\PotatoORM\Connections\Connection;
 use Opeyemiabiodun\PotatoORM\Connections\DatabaseTransactionsTrait;
+use PDO;
+use PDOException;
 
 final class SqliteConnection implements Connection
 {

Doc Comments +3 added lines

@@ -95,6 +95,9 @@
         return $this->keys[$table][0];
     }
 
+    /**
+     * @param string $table
+     */
     private function loadColumnInfo($table)
     {
         $query     = "PRAGMA table_info('{$table}')";

src/Models/User.php

Indentation +2 added lines, -2 removed lines

@@ -7,9 +7,9 @@
 
 class User
 {
-	use Model;
+    use Model;
 
-	public static function createTable(PDO $pdo)
+    public static function createTable(PDO $pdo)
     {
         $pdo->query("CREATE TABLE user_table
         				(