andela-tolotin / Potato-ORM

src/Model/BaseModel.php

Unused Use Statements -1 removed lines

@@ -14,7 +14,6 @@
 use Laztopaz\potatoORM\NoRecordFoundException;
 use Laztopaz\potatoORM\NoRecordInsertionException;
 use Laztopaz\potatoORM\NullArgumentPassedToFunction;
-use Laztopaz\potatoORM\WrongArgumentException;
 use Laztopaz\potatoORM\NoArgumentPassedToFunctionException;
 use Laztopaz\potatoORM\EmptyArrayException;
 

Doc Comments +2 added lines, -2 removed lines

@@ -99,7 +99,7 @@ 
     /**
      * This method find a record by id
      * @params int id
-     * @return Object
+     * @return BaseClass
      * @throws NoArgumentPassedToFunctionException
      */
     public static function find($id)
@@ -142,7 +142,7 @@ 
    /**
     * This method return the current class name
     * $params void
-    * @return classname
+    * @return string|false
     */
    public static function getClassName()
    {

Indentation +113 added lines, -113 removed lines

@@ -20,124 +20,124 @@ 
 
 class BaseClass  implements InterfaceBaseClass
 {
-    protected $databaseModel; // Private variable that contains instance of database
-    protected $tableName; // Class variable holding class name pluralized
-    protected $properties = []; // Properties will later contain key, value pairs from the magic setter, getter methods
-    use Inflector; // Inject the inflector trait
+	protected $databaseModel; // Private variable that contains instance of database
+	protected $tableName; // Class variable holding class name pluralized
+	protected $properties = []; // Properties will later contain key, value pairs from the magic setter, getter methods
+	use Inflector; // Inject the inflector trait
 
-    public function  __construct()
-    {
-        $this->tableName = $this->getClassName();
+	public function  __construct()
+	{
+		$this->tableName = $this->getClassName();
 	$this->databaseModel = new DatabaseHandler($this->tableName);
 	$this->properties['id'] = 0;
-    }
+	}
     
-    /**
-     * The magic getter method
-     * @params key
-     * @return array key
-     */
-    public function __get($key)
-    {
-    	$this->properties[$key];
-    }
+	/**
+	 * The magic getter method
+	 * @params key
+	 * @return array key
+	 */
+	public function __get($key)
+	{
+		$this->properties[$key];
+	}
     
-    /**
-     * The magic setter method
-     * @params property, key
-     * @return array associative array properties
-     */
-    public function  __set($property, $value)
-    {
-    	$this->properties[$property] = $value;
-    }
+	/**
+	 * The magic setter method
+	 * @params property, key
+	 * @return array associative array properties
+	 */
+	public function  __set($property, $value)
+	{
+		$this->properties[$property] = $value;
+	}
     
-    /**
-     * This method gets all the record from a particular table
-     * @params void
-     * @return associative array
-     * @throws NoRecordFoundException
-     */
-    public static function getAll()
-    {
-    	$allData = DatabaseHandler::read($id = false, self::getClassName());
-    	if (count($allData) > 0) {
-    	    return $allData;
-    	}
-    	throw NoRecordFoundException::checkNoRecordFoundException("There is no record to display");
-    }
+	/**
+	 * This method gets all the record from a particular table
+	 * @params void
+	 * @return associative array
+	 * @throws NoRecordFoundException
+	 */
+	public static function getAll()
+	{
+		$allData = DatabaseHandler::read($id = false, self::getClassName());
+		if (count($allData) > 0) {
+			return $allData;
+		}
+		throw NoRecordFoundException::checkNoRecordFoundException("There is no record to display");
+	}
     
-    /**
-     * This method create or update record in a database table
-     * @params void
-     * @return bool true or false;
-     * @throws EmptyArrayException
-     * @throws NoRecordInsertionException
-     * @throws NoRecordUpdateException
-     */
-    public function save()
-    {
-    	$boolCommit = false;
-    	if ($this->properties['id']) {
-    	    $allData = DatabaseHandler::read($id = $this->properties['id'], self::getClassName());
-    	    if ($this->checkIfRecordIsEmpty($allData)) {
-    	    	$boolCommit = $this->databaseModel->update(['id' => $this->properties['id']], $this->tableName, $this->properties);
-    	    	if ($boolCommit) {
-    	    	    return true;
-    	    	}
-    	    	throw NoRecordUpdateException::checkNoRecordUpdateException("Record not updated successfully");
-    	    }
-    	    throw EmptyArrayException::checkEmptyArrayException("Value passed didn't match any record");
-    	}
-    	$boolCommit = $this->databaseModel->create($this->properties, $this->tableName);
-    	if ($boolCommit) {
-    	    return true;
-    	}
-    	throw NoRecordInsertionException::checkNoRecordAddedException("Record not created successfully");
-    }
+	/**
+	 * This method create or update record in a database table
+	 * @params void
+	 * @return bool true or false;
+	 * @throws EmptyArrayException
+	 * @throws NoRecordInsertionException
+	 * @throws NoRecordUpdateException
+	 */
+	public function save()
+	{
+		$boolCommit = false;
+		if ($this->properties['id']) {
+			$allData = DatabaseHandler::read($id = $this->properties['id'], self::getClassName());
+			if ($this->checkIfRecordIsEmpty($allData)) {
+				$boolCommit = $this->databaseModel->update(['id' => $this->properties['id']], $this->tableName, $this->properties);
+				if ($boolCommit) {
+					return true;
+				}
+				throw NoRecordUpdateException::checkNoRecordUpdateException("Record not updated successfully");
+			}
+			throw EmptyArrayException::checkEmptyArrayException("Value passed didn't match any record");
+		}
+		$boolCommit = $this->databaseModel->create($this->properties, $this->tableName);
+		if ($boolCommit) {
+			return true;
+		}
+		throw NoRecordInsertionException::checkNoRecordAddedException("Record not created successfully");
+	}
 
-    /**
-     * This method find a record by id
-     * @params int id
-     * @return Object
-     * @throws NoArgumentPassedToFunctionException
-     */
-    public static function find($id)
-    {
-    	$num_args = (int) func_num_args(); // get number of arguments passed to
-    	if ($num_args == 0 || $num_args > 1) {
-    	    throw NoArgumentPassedToFunctionException::checkNoArgumentPassedToFunction("Argument missing: only one argument is allowed");
-    	}
-    	if ($id == "") {
-    		throw NullArgumentPassedToFunction::ckeckNullArgumentPassedToFunction("This function expect a value");
-    	}
-    	$staticFindInstance = new static();
-    	$staticFindInstance->id = $id == "" ? false : $id;
-    	return $staticFindInstance;
+	/**
+	 * This method find a record by id
+	 * @params int id
+	 * @return Object
+	 * @throws NoArgumentPassedToFunctionException
+	 */
+	public static function find($id)
+	{
+		$num_args = (int) func_num_args(); // get number of arguments passed to
+		if ($num_args == 0 || $num_args > 1) {
+			throw NoArgumentPassedToFunctionException::checkNoArgumentPassedToFunction("Argument missing: only one argument is allowed");
+		}
+		if ($id == "") {
+			throw NullArgumentPassedToFunction::ckeckNullArgumentPassedToFunction("This function expect a value");
+		}
+		$staticFindInstance = new static();
+		$staticFindInstance->id = $id == "" ? false : $id;
+		return $staticFindInstance;
     	
-    }
+	}
     
-    /**
-     * This method delete a row from the table by the row id
-     * @params int id
-     * @return boolean true or false
-     * @throws NoRecordDeletionException;
-     */
-    public static function destroy($id)
-    {
-    	$boolDeleted = false;
-    	$num_args = (int) func_num_args(); // get number of arguments passed to
+	/**
+	 * This method delete a row from the table by the row id
+	 * @params int id
+	 * @return boolean true or false
+	 * @throws NoRecordDeletionException;
+	 */
+	public static function destroy($id)
+	{
+		$boolDeleted = false;
+		$num_args = (int) func_num_args(); // get number of arguments passed to
     	
-    	if ($num_args == 0 || $num_args > 1) {
-    	    throw NoArgumentPassedToFunctionException::checkNoArgumentPassedToFunction("Argument missing: only one argument is allowed");
-    	}
-    	$boolDeleted = DatabaseHandler::delete($id, self::getClassName());
-    	if ($boolDeleted) {
-    	    return true;
-    	}
-    	throw NoRecordDeletionException::checkNoRecordUpdateException("Record deletion unsuccessful because id does not match any record");
+		if ($num_args == 0 || $num_args > 1) {
+			throw NoArgumentPassedToFunctionException::checkNoArgumentPassedToFunction("Argument missing: only one argument is allowed");
+		}
+		$boolDeleted = DatabaseHandler::delete($id, self::getClassName());
+		if ($boolDeleted) {
+			return true;
+		}
+		throw NoRecordDeletionException::checkNoRecordUpdateException("Record deletion unsuccessful because id does not match any record");
     	
-    }
+	}
 
    /**
     * This method return the current class name
@@ -156,12 +156,12 @@ 
     * @param $arrayOfRecord
     * @return bool
     */
-    public function checkIfRecordIsEmpty($arrayOfRecord)
-    {
-    	if (count($arrayOfRecord) > 0) {
-    	    return true;
-    	}
-    	return false;
-    }
+	public function checkIfRecordIsEmpty($arrayOfRecord)
+	{
+		if (count($arrayOfRecord) > 0) {
+			return true;
+		}
+		return false;
+	}
 
 }

src/Helper/Inflector.php

Spacing +9 added lines, -9 removed lines

@@ -74,14 +74,14 @@ 
 			'sex' => 'sexes',
 			'move' => 'moves');
 		$lowercased_word = strtolower($word);
-		foreach ($uncountable as $_uncountable){
-			if(substr($lowercased_word,(-1*strlen($_uncountable))) == $_uncountable){
+		foreach ($uncountable as $_uncountable) {
+			if (substr($lowercased_word, (-1*strlen($_uncountable))) == $_uncountable) {
 				return $word;
 			}
 		}
-		foreach ($irregular as $_plural=> $_singular){
+		foreach ($irregular as $_plural=> $_singular) {
 			if (preg_match('/('.$_plural.')$/i', $word, $arr)) {
-				return preg_replace('/('.$_plural.')$/i', substr($arr[0],0,1).substr($_singular,1), $word);
+				return preg_replace('/('.$_plural.')$/i', substr($arr[0], 0, 1).substr($_singular, 1), $word);
 			}
 		}
 		foreach ($plural as $rule => $replacement) {
@@ -101,7 +101,7 @@ 
 	 */
 	public function singularize($word)
 	{
-		$singular = array (
+		$singular = array(
 			'/(quiz)zes$/i'             => "$1",
 			'/(matr)ices$/i'            => "$1ix",
 			'/(vert|ind)ices$/i'        => "$1ex",
@@ -139,14 +139,14 @@ 
 			'sex' => 'sexes',
 			'move' => 'moves');
 		$lowercased_word = strtolower($word);
-		foreach ($uncountable as $_uncountable){
-			if(substr($lowercased_word,(-1*strlen($_uncountable))) == $_uncountable){
+		foreach ($uncountable as $_uncountable) {
+			if (substr($lowercased_word, (-1*strlen($_uncountable))) == $_uncountable) {
 				return $word;
 			}
 		}
-		foreach ($irregular as $_plural=> $_singular){
+		foreach ($irregular as $_plural=> $_singular) {
 			if (preg_match('/('.$_singular.')$/i', $word, $arr)) {
-				return preg_replace('/('.$_singular.')$/i', substr($arr[0],0,1).substr($_plural,1), $word);
+				return preg_replace('/('.$_singular.')$/i', substr($arr[0], 0, 1).substr($_plural, 1), $word);
 			}
 		}
 		foreach ($singular as $rule => $replacement) {

src/Exceptions/NoRecordInsertionException.php

Spacing +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
  * @license  <https://opensource.org/license/MIT> MIT
  */
 
-namespace Laztopaz\potatoORM	;
+namespace Laztopaz\potatoORM;
 
 use Exception;
 

Indentation +4 added lines, -4 removed lines

@@ -12,8 +12,8 @@
 
 class NoRecordInsertionException extends Exception {
 
-    public static function checkNoRecordAddedException($mesaage)
-    {
-    	return new static($mesaage);
-    }
+	public static function checkNoRecordAddedException($mesaage)
+	{
+		return new static($mesaage);
+	}
 }

src/Database/DatabaseHandler.php

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 namespace Laztopaz\potatoORM;
 
 use PDO;
-use Laztopaz\potatoORM\DatabaseHelper;
 use Laztopaz\potatoORM\TableFieldUndefinedException;
 use Laztopaz\potatoORM\EmptyArrayException;
 

Spacing +17 added lines, -17 removed lines

@@ -41,9 +41,9 @@ 
     {
     	$tableFields = $this->getColumnNames($this->model, $this->dbConnection);
     	
-    	$unexpectedFields = self::checkIfMagicSetterContainsIsSameAsClassModel($tableFields,$associative1DArray);
+    	$unexpectedFields = self::checkIfMagicSetterContainsIsSameAsClassModel($tableFields, $associative1DArray);
     	if (count($unexpectedFields) > 0) {
-    	    throw TableFieldUndefinedException::fieldsNotDefinedException($unexpectedFields,"needs to be created as table field");
+    	    throw TableFieldUndefinedException::fieldsNotDefinedException($unexpectedFields, "needs to be created as table field");
     	}
     	unset($tableFields[0]);
     	if (is_null($dbConn)) {
@@ -55,15 +55,15 @@ 
     private function  insertRecord($dbConn, $tableName, $associative1DArray) 
     {
     	$insertQuery = 'INSERT INTO '.$tableName;
-    	$TableValues = implode(',',array_keys($associative1DArray));
+    	$TableValues = implode(',', array_keys($associative1DArray));
     	foreach ($associative1DArray as $field => $value) {
     	    $FormValues[] = "'".trim(addslashes($value))."'";
     	}
     	$splittedTableValues = implode(',', $FormValues);
-    	$insertQuery.= ' ('.$TableValues.')';
-    	$insertQuery.= ' VALUES ('.$splittedTableValues.')';
+    	$insertQuery .= ' ('.$TableValues.')';
+    	$insertQuery .= ' VALUES ('.$splittedTableValues.')';
     	$executeQuery = $dbConn->exec($insertQuery);
-    	return $executeQuery ? : false;
+    	return $executeQuery ?: false;
     	
 	}
 
@@ -80,12 +80,12 @@ 
     	}
     	$updateSql = "UPDATE `$tableName` SET ";
     	unset($associative1DArray['id']);
-    	$unexpectedFields = self::checkIfMagicSetterContainsIsSameAsClassModel($this->getColumnNames($this->model, $this->dbConnection),$associative1DArray);
+    	$unexpectedFields = self::checkIfMagicSetterContainsIsSameAsClassModel($this->getColumnNames($this->model, $this->dbConnection), $associative1DArray);
     	
     	if (count($unexpectedFields) > 0) {
     		throw TableFieldUndefinedException::fieldsNotDefinedException($unexpectedFields, "needs to be created as table field");
     	}
-    	foreach($associative1DArray as $field => $value) {
+    	foreach ($associative1DArray as $field => $value) {
     	    $sql .= "`$field` = '$value'".",";
     	}
     	
@@ -96,7 +96,7 @@ 
     	}
     	$stmt = $dbConn->prepare($updateSql);
     	$boolResponse = $stmt->execute();
-    	return $boolResponse ?  : false;
+    	return $boolResponse ?: false;
     }
     
   /**
@@ -111,7 +111,7 @@ 
    	if (is_null($dbConn)) {
    	    $dbConn = new DatabaseConnection();
    	}
-   	$sql = $id  ? 'SELECT * FROM '.$tableName.' WHERE id = '.$id : 'SELECT * FROM '.$tableName;
+   	$sql = $id ? 'SELECT * FROM '.$tableName.' WHERE id = '.$id : 'SELECT * FROM '.$tableName;
    	
    	try {
    	    $stmt = $dbConn->prepare($sql);
@@ -123,7 +123,7 @@ 
    	}
    	$results = $stmt->fetchAll(PDO::FETCH_ASSOC);
    	
-   	foreach($results as $result) {
+   	foreach ($results as $result) {
    	    array_push($tableData, $result);
    	}
    	return $tableData;
@@ -142,7 +142,7 @@ 
       
       $sql = 'DELETE FROM '.$tableName.' WHERE id = '.$id;
       $boolResponse = $dbConn->exec($sql);
-      return $boolResponse ? : false;
+      return $boolResponse ?: false;
   }
   
   /**
@@ -155,7 +155,7 @@ 
   {
       $unexpectedFields = [];
       foreach ($userSetterArray as $key => $val) {
-          if (!in_array($key,$tableColumn)) {
+          if (!in_array($key, $tableColumn)) {
               $unexpectedFields[] = $key;
           }
       }
@@ -169,9 +169,9 @@ 
     */
   public function prepareUpdateQuery($sql)
   {
-      $splittedQuery = explode(",",$sql);
+      $splittedQuery = explode(",", $sql);
       array_pop($splittedQuery);
-      $mergeData = implode(",",$splittedQuery);
+      $mergeData = implode(",", $splittedQuery);
       return $mergeData;
   }
   
@@ -197,7 +197,7 @@ 
           $statement->execute();
           $returnedRowNumbers = $statement->rowCount();
           
-          return $returnedRowNumbers  ? true : false;
+          return $returnedRowNumbers ? true : false;
       }
       throw EmptyArrayException::checkEmptyArrayException("Array Expected: parameter passed to this function is not an array");
   }
@@ -221,7 +221,7 @@ 
      $stmt->execute();
      $results = $stmt->fetchAll(PDO::FETCH_ASSOC);
      
-     foreach($results as $result) {
+     foreach ($results as $result) {
          array_push($tableFields, $result['Field']);
      }
      return $tableFields;

Doc Comments +8 added lines, -2 removed lines

@@ -21,6 +21,7 @@ 
     
     /**
      * This is a constructor; a default method  that will be called automatically during class instantiation
+     * @param string|false $modelClassName
      */
      public function __construct($modelClassName, $dbConn = Null)
      {
@@ -35,7 +36,8 @@ 
     /**
      * This method create a record and store it in a table row
      * @params associative array, string tablename
-     * @return boolean true or false
+     * @param string|false $tableName
+     * @return boolean|null true or false
      */
     public function create($associative1DArray, $tableName, $dbConn = Null)
     {
@@ -70,6 +72,7 @@ 
     /**
      * This method updates any table by supplying 3 parameter
      * @params: $updateParams, $tableName, $associative1DArray
+     * @param string|false $tableName
      * @return boolean true or false
      */
     public function update(array $updateParams, $tableName, $associative1DArray, $dbConn = Null)
@@ -102,6 +105,7 @@ 
   /**
    * This method retrieves record from a table
    * @params int id, string tableName
+   * @param string|false $tableName
    * @return array
    */
    public static function read($id, $tableName, $dbConn = Null)
@@ -132,6 +136,7 @@ 
   /**
    * This method deletes a record  from a table row
    * @params int id, string tableName
+   * @param string|false $tableName
    * @return boolean true or false
    */
   public static function delete($id, $tableName, $dbConn = Null)
@@ -164,7 +169,7 @@ 
   
   /**
    * This method returns sql query
-   * @param $sql
+   * @param string $sql
    * @return string
    */
   public function prepareUpdateQuery($sql)
@@ -206,6 +211,7 @@ 
    * This method returns column fields of a particular table
    * @param $table
    * @param $conn
+   * @param DatabaseConnection $dbConn
    * @return array
    */
  public function getColumnNames($table, $dbConn = Null) {

Indentation +125 added lines, -125 removed lines

@@ -14,90 +14,90 @@ 
 use Laztopaz\potatoORM\EmptyArrayException;
 
 class DatabaseHandler {
-    private $tableFields;
-    private $dbHelperInstance;
-    private $dbConnection;
-    private $model;
+	private $tableFields;
+	private $dbHelperInstance;
+	private $dbConnection;
+	private $model;
     
-    /**
-     * This is a constructor; a default method  that will be called automatically during class instantiation
-     */
-     public function __construct($modelClassName, $dbConn = Null)
-     {
-     	if (is_null($dbConn)) {
-     		$this->dbConnection = new DatabaseConnection();
-     	} else {
-     	    $this->dbConnection = $dbConn;
-     	}
-     	$this->model = $modelClassName;
-     }
+	/**
+	 * This is a constructor; a default method  that will be called automatically during class instantiation
+	 */
+	 public function __construct($modelClassName, $dbConn = Null)
+	 {
+	 	if (is_null($dbConn)) {
+	 		$this->dbConnection = new DatabaseConnection();
+	 	} else {
+	 		$this->dbConnection = $dbConn;
+	 	}
+	 	$this->model = $modelClassName;
+	 }
      
-    /**
-     * This method create a record and store it in a table row
-     * @params associative array, string tablename
-     * @return boolean true or false
-     */
-    public function create($associative1DArray, $tableName, $dbConn = Null)
-    {
-    	$tableFields = $this->getColumnNames($this->model, $this->dbConnection);
+	/**
+	 * This method create a record and store it in a table row
+	 * @params associative array, string tablename
+	 * @return boolean true or false
+	 */
+	public function create($associative1DArray, $tableName, $dbConn = Null)
+	{
+		$tableFields = $this->getColumnNames($this->model, $this->dbConnection);
     	
-    	$unexpectedFields = self::checkIfMagicSetterContainsIsSameAsClassModel($tableFields,$associative1DArray);
-    	if (count($unexpectedFields) > 0) {
-    	    throw TableFieldUndefinedException::fieldsNotDefinedException($unexpectedFields,"needs to be created as table field");
-    	}
-    	unset($tableFields[0]);
-    	if (is_null($dbConn)) {
-    	    $dbConn = $this->dbConnection;
-    	}
-    	$this->insertRecord($dbConn, $tableName, $associative1DArray);
-    }
+		$unexpectedFields = self::checkIfMagicSetterContainsIsSameAsClassModel($tableFields,$associative1DArray);
+		if (count($unexpectedFields) > 0) {
+			throw TableFieldUndefinedException::fieldsNotDefinedException($unexpectedFields,"needs to be created as table field");
+		}
+		unset($tableFields[0]);
+		if (is_null($dbConn)) {
+			$dbConn = $this->dbConnection;
+		}
+		$this->insertRecord($dbConn, $tableName, $associative1DArray);
+	}
     
-    private function  insertRecord($dbConn, $tableName, $associative1DArray) 
-    {
-    	$insertQuery = 'INSERT INTO '.$tableName;
-    	$TableValues = implode(',',array_keys($associative1DArray));
-    	foreach ($associative1DArray as $field => $value) {
-    	    $FormValues[] = "'".trim(addslashes($value))."'";
-    	}
-    	$splittedTableValues = implode(',', $FormValues);
-    	$insertQuery.= ' ('.$TableValues.')';
-    	$insertQuery.= ' VALUES ('.$splittedTableValues.')';
-    	$executeQuery = $dbConn->exec($insertQuery);
-    	return $executeQuery ? : false;
+	private function  insertRecord($dbConn, $tableName, $associative1DArray) 
+	{
+		$insertQuery = 'INSERT INTO '.$tableName;
+		$TableValues = implode(',',array_keys($associative1DArray));
+		foreach ($associative1DArray as $field => $value) {
+			$FormValues[] = "'".trim(addslashes($value))."'";
+		}
+		$splittedTableValues = implode(',', $FormValues);
+		$insertQuery.= ' ('.$TableValues.')';
+		$insertQuery.= ' VALUES ('.$splittedTableValues.')';
+		$executeQuery = $dbConn->exec($insertQuery);
+		return $executeQuery ? : false;
     	
 	}
 
-    /**
-     * This method updates any table by supplying 3 parameter
-     * @params: $updateParams, $tableName, $associative1DArray
-     * @return boolean true or false
-     */
-    public function update(array $updateParams, $tableName, $associative1DArray, $dbConn = Null)
-    {
-    	$sql = "";
-    	if (is_null($dbConn)) {
-    	    $dbConn = $this->dbConnection;
-    	}
-    	$updateSql = "UPDATE `$tableName` SET ";
-    	unset($associative1DArray['id']);
-    	$unexpectedFields = self::checkIfMagicSetterContainsIsSameAsClassModel($this->getColumnNames($this->model, $this->dbConnection),$associative1DArray);
+	/**
+	 * This method updates any table by supplying 3 parameter
+	 * @params: $updateParams, $tableName, $associative1DArray
+	 * @return boolean true or false
+	 */
+	public function update(array $updateParams, $tableName, $associative1DArray, $dbConn = Null)
+	{
+		$sql = "";
+		if (is_null($dbConn)) {
+			$dbConn = $this->dbConnection;
+		}
+		$updateSql = "UPDATE `$tableName` SET ";
+		unset($associative1DArray['id']);
+		$unexpectedFields = self::checkIfMagicSetterContainsIsSameAsClassModel($this->getColumnNames($this->model, $this->dbConnection),$associative1DArray);
     	
-    	if (count($unexpectedFields) > 0) {
-    		throw TableFieldUndefinedException::fieldsNotDefinedException($unexpectedFields, "needs to be created as table field");
-    	}
-    	foreach($associative1DArray as $field => $value) {
-    	    $sql .= "`$field` = '$value'".",";
-    	}
+		if (count($unexpectedFields) > 0) {
+			throw TableFieldUndefinedException::fieldsNotDefinedException($unexpectedFields, "needs to be created as table field");
+		}
+		foreach($associative1DArray as $field => $value) {
+			$sql .= "`$field` = '$value'".",";
+		}
     	
-    	$updateSql .= $this->prepareUpdateQuery($sql);
+		$updateSql .= $this->prepareUpdateQuery($sql);
     	
-    	foreach ($updateParams as $key => $val) {
-    	    $updateSql .= " WHERE $key = $val";
-    	}
-    	$stmt = $dbConn->prepare($updateSql);
-    	$boolResponse = $stmt->execute();
-    	return $boolResponse ?  : false;
-    }
+		foreach ($updateParams as $key => $val) {
+			$updateSql .= " WHERE $key = $val";
+		}
+		$stmt = $dbConn->prepare($updateSql);
+		$boolResponse = $stmt->execute();
+		return $boolResponse ?  : false;
+	}
     
   /**
    * This method retrieves record from a table
@@ -109,22 +109,22 @@ 
    	$tableData = [];
    	
    	if (is_null($dbConn)) {
-   	    $dbConn = new DatabaseConnection();
+   		$dbConn = new DatabaseConnection();
    	}
    	$sql = $id  ? 'SELECT * FROM '.$tableName.' WHERE id = '.$id : 'SELECT * FROM '.$tableName;
    	
    	try {
-   	    $stmt = $dbConn->prepare($sql);
-   	    $stmt->bindValue(':table', $tableName);
-   	    $stmt->bindValue(':id', $id);
-   	    $stmt->execute();
+   		$stmt = $dbConn->prepare($sql);
+   		$stmt->bindValue(':table', $tableName);
+   		$stmt->bindValue(':id', $id);
+   		$stmt->execute();
    	} catch (PDOException $e) {
-   	    return  $e->getMessage();
+   		return  $e->getMessage();
    	}
    	$results = $stmt->fetchAll(PDO::FETCH_ASSOC);
    	
    	foreach($results as $result) {
-   	    array_push($tableData, $result);
+   		array_push($tableData, $result);
    	}
    	return $tableData;
    }
@@ -136,13 +136,13 @@ 
    */
   public static function delete($id, $tableName, $dbConn = Null)
   {
-      if (is_null($dbConn)) {
-          $dbConn = new DatabaseConnection();
-      }
+	  if (is_null($dbConn)) {
+		  $dbConn = new DatabaseConnection();
+	  }
       
-      $sql = 'DELETE FROM '.$tableName.' WHERE id = '.$id;
-      $boolResponse = $dbConn->exec($sql);
-      return $boolResponse ? : false;
+	  $sql = 'DELETE FROM '.$tableName.' WHERE id = '.$id;
+	  $boolResponse = $dbConn->exec($sql);
+	  return $boolResponse ? : false;
   }
   
   /**
@@ -153,13 +153,13 @@ 
    */
   public static function checkIfMagicSetterContainsIsSameAsClassModel(array $tableColumn, array $userSetterArray)
   {
-      $unexpectedFields = [];
-      foreach ($userSetterArray as $key => $val) {
-          if (!in_array($key,$tableColumn)) {
-              $unexpectedFields[] = $key;
-          }
-      }
-      return $unexpectedFields;
+	  $unexpectedFields = [];
+	  foreach ($userSetterArray as $key => $val) {
+		  if (!in_array($key,$tableColumn)) {
+			  $unexpectedFields[] = $key;
+		  }
+	  }
+	  return $unexpectedFields;
   }
   
   /**
@@ -169,10 +169,10 @@ 
    */
   public function prepareUpdateQuery($sql)
   {
-      $splittedQuery = explode(",",$sql);
-      array_pop($splittedQuery);
-      $mergeData = implode(",",$splittedQuery);
-      return $mergeData;
+	  $splittedQuery = explode(",",$sql);
+	  array_pop($splittedQuery);
+	  $mergeData = implode(",",$splittedQuery);
+	  return $mergeData;
   }
   
   /**
@@ -184,22 +184,22 @@ 
    */
   public function findAndWhere($params, $tableName, $dbConn)
   {
-      if (is_null($dbConn)) {
-          $dbConn = $this->dbConnection;
-      }
-      if (is_array($params) && !empty($params)) {
-          $sql = "SELECT * FROM ".$tableName;
+	  if (is_null($dbConn)) {
+		  $dbConn = $this->dbConnection;
+	  }
+	  if (is_array($params) && !empty($params)) {
+		  $sql = "SELECT * FROM ".$tableName;
           
-          foreach ($params as $key => $val) {
-              $sql .= " WHERE `$key` = '$val'";
-          }
-          $statement = $dbConn->prepare($sql);
-          $statement->execute();
-          $returnedRowNumbers = $statement->rowCount();
+		  foreach ($params as $key => $val) {
+			  $sql .= " WHERE `$key` = '$val'";
+		  }
+		  $statement = $dbConn->prepare($sql);
+		  $statement->execute();
+		  $returnedRowNumbers = $statement->rowCount();
           
-          return $returnedRowNumbers  ? true : false;
-      }
-      throw EmptyArrayException::checkEmptyArrayException("Array Expected: parameter passed to this function is not an array");
+		  return $returnedRowNumbers  ? true : false;
+	  }
+	  throw EmptyArrayException::checkEmptyArrayException("Array Expected: parameter passed to this function is not an array");
   }
   
   /**
@@ -209,22 +209,22 @@ 
    * @return array
    */
  public function getColumnNames($table, $dbConn = Null) {
-     $tableFields = [];
+	 $tableFields = [];
      
-     if (is_null($dbConn)) {
-         $dbConn = $this->dbConnection;
-     }
-     $sql = "SHOW COLUMNS FROM ".$table;
+	 if (is_null($dbConn)) {
+		 $dbConn = $this->dbConnection;
+	 }
+	 $sql = "SHOW COLUMNS FROM ".$table;
      
-     $stmt = $dbConn->prepare($sql);
-     $stmt->bindValue(':table', $table, PDO::PARAM_STR);
-     $stmt->execute();
-     $results = $stmt->fetchAll(PDO::FETCH_ASSOC);
+	 $stmt = $dbConn->prepare($sql);
+	 $stmt->bindValue(':table', $table, PDO::PARAM_STR);
+	 $stmt->execute();
+	 $results = $stmt->fetchAll(PDO::FETCH_ASSOC);
      
-     foreach($results as $result) {
-         array_push($tableFields, $result['Field']);
-     }
-     return $tableFields;
+	 foreach($results as $result) {
+		 array_push($tableFields, $result['Field']);
+	 }
+	 return $tableFields;
  }
 
 }

src/Database/DatabaseHelper.php

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 
 namespace Laztopaz\potatoORM;
 
-use PDO;
 use Laztopaz\potatoORM\TableNotCreatedException;
 
 class DatabaseHelper {

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
    	}
    	
    	$sql = 'CREATE TABLE IF NOT EXISTS '.$tableName.'(';
-   	$sql.= ' id INT( 11 ) AUTO_INCREMENT PRIMARY KEY, name VARCHAR( 100 ), gender VARCHAR( 10 ), alias VARCHAR( 150 ) NOT NULL, class VARCHAR( 150 ), stack VARCHAR( 50 ) )';
+   	$sql .= ' id INT( 11 ) AUTO_INCREMENT PRIMARY KEY, name VARCHAR( 100 ), gender VARCHAR( 10 ), alias VARCHAR( 150 ) NOT NULL, class VARCHAR( 150 ), stack VARCHAR( 50 ) )';
    	
    	return $conn->exec($sql);
    	throw TableNotCreatedException::tableNotCreatedException("Check your database connection");

Indentation +14 added lines, -14 removed lines

@@ -13,25 +13,25 @@
 
 class DatabaseHelper {
 	
-    public $dbConn;
+	public $dbConn;
     
-    /**
-     * This is a constructor; a default method  that will be called automatically during class instantiation
-     */
-    public function __construct($dbConnect)
-    {
-    	$this->dbConn = $dbConnect;
-    }
+	/**
+	 * This is a constructor; a default method  that will be called automatically during class instantiation
+	 */
+	public function __construct($dbConnect)
+	{
+		$this->dbConn = $dbConnect;
+	}
     
-    /**
-     * This method creates a particular table
-     * @param tableName
-     * $return boolean true or false
-     */
+	/**
+	 * This method creates a particular table
+	 * @param tableName
+	 * $return boolean true or false
+	 */
    public function createTable($tableName, $conn = NULL)
    {
    	if (is_null($conn)) {
-   	    $conn = $this->dbConn;
+   		$conn = $this->dbConn;
    	}
    	
    	$sql = 'CREATE TABLE IF NOT EXISTS '.$tableName.'(';

src/Exceptions/NoRecordDeletionException.php

Indentation +4 added lines, -4 removed lines

@@ -12,8 +12,8 @@
 
 class NoRecordDeletionException extends  Exception
 {
-    public static function checkNoRecordUpdateException($message)
-    {
-        return new static($message);
-    }
+	public static function checkNoRecordUpdateException($message)
+	{
+		return new static($message);
+	}
 }

src/Exceptions/EmptyArrayException.php

Indentation +4 added lines, -4 removed lines

@@ -12,9 +12,9 @@
 
 class EmptyArrayException extends Exception {
 	
-    public static function checkEmptyArrayException($message)
-    {
-    	return new static($message);
-    }
+	public static function checkEmptyArrayException($message)
+	{
+		return new static($message);
+	}
 
 }

src/Exceptions/NoArgumentPassedToFunctionException.php

Indentation +4 added lines, -4 removed lines

@@ -12,9 +12,9 @@
 
 class NoArgumentPassedToFunctionException extends Exception {
 	
-    public static function checkNoArgumentPassedToFunction($message)
-    {
-    	return new static($message);
-    }
+	public static function checkNoArgumentPassedToFunction($message)
+	{
+		return new static($message);
+	}
 
 }

src/Exceptions/NoRecordUpdateException.php

Indentation +4 added lines, -4 removed lines

@@ -12,9 +12,9 @@
 
 class NoRecordUpdateException extends  Exception {
 	
-    public static function checkNoRecordUpdateException($message)
-    {
-    	return new static($message);
-    }
+	public static function checkNoRecordUpdateException($message)
+	{
+		return new static($message);
+	}
 
 }