dimaxz / SimpleORM

src/ISpecificationCriteria.php

Indentation +18 added lines, -18 removed lines

@@ -10,43 +10,43 @@
  */
 interface ISpecificationCriteria
 {
-    public function getCriteria();
+	public function getCriteria();
 	
 	public function getWhere();
 
-    public function getLimit();
+	public function getLimit();
 
-    public function getOfset();
+	public function getOfset();
 
-    public function getJoins();
+	public function getJoins();
 
-    public function getOrder();
+	public function getOrder();
 
-    public function getManualJoins();
+	public function getManualJoins();
 
-    public function getGroup();
+	public function getGroup();
 
-    public function getManualWheres();
+	public function getManualWheres();
 
-    public function getWhereType();    
+	public function getWhereType();    
 
-    public function setWhere($field,$value = false);    
+	public function setWhere($field,$value = false);    
 
-    public function setLimit($limit);
+	public function setLimit($limit);
 
-    public function setOfset($ofset);
+	public function setOfset($ofset);
 
-    public function setJoins($joins);    
+	public function setJoins($joins);    
 
-    public function setOrder($order);
+	public function setOrder($order);
 
-    public function setManualJoins($manualJoins);
+	public function setManualJoins($manualJoins);
 
-    public function setGroup($group);
+	public function setGroup($group);
 
-    public function setManualWheres($manualWheres);
+	public function setManualWheres($manualWheres);
 	
-    public function setWhereType($whereType);
+	public function setWhereType($whereType);
 	
 }
 

Spacing +1 added lines, -1 removed lines

@@ -30,7 +30,7 @@
 
     public function getWhereType();    
 
-    public function setWhere($field,$value = false);    
+    public function setWhere($field, $value = false);    
 
     public function setLimit($limit);
 

src/AbstractEntity.php

Spacing +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
 
     public function setId($id)
     {
-        if ($this->id !== null) {
+        if ($this->id!==null) {
             throw new BadMethodCallException(
                 "Идентификатор у сущности уже установлен");
         }
@@ -34,7 +34,7 @@ 
         return $this;		
     }
 	
-	public function setDeleted($deleted){
+	public function setDeleted($deleted) {
 		$this->deleted = $deleted;
 	}
 	

Indentation +16 added lines, -16 removed lines

@@ -10,29 +10,29 @@
 abstract class AbstractEntity implements EntityInterface
 {
 
-    protected $id;
+	protected $id;
 	
 	protected $deleted;
 
 	public function getId()
-    {
-        return $this->id;
-    }
+	{
+		return $this->id;
+	}
 
-    public function setId($id)
-    {
-        if ($this->id !== null) {
-            throw new BadMethodCallException(
-                "Идентификатор у сущности уже установлен");
-        }
+	public function setId($id)
+	{
+		if ($this->id !== null) {
+			throw new BadMethodCallException(
+				"Идентификатор у сущности уже установлен");
+		}
  
-        if (empty($id)) {//$id < 1
-            throw new InvalidArgumentException("Неверный индентификатор");
-        }
+		if (empty($id)) {//$id < 1
+			throw new InvalidArgumentException("Неверный индентификатор");
+		}
  
-        $this->id = $id;
-        return $this;		
-    }
+		$this->id = $id;
+		return $this;		
+	}
 	
 	public function setDeleted($deleted){
 		$this->deleted = $deleted;

src/Specification.php

Indentation +119 added lines, -119 removed lines

@@ -10,134 +10,134 @@
 class Specification implements ISpecificationCriteria
 {
 
-    protected $where = [];
+	protected $where = [];
 
-    protected $limit = 0;
+	protected $limit = 0;
 
-    protected $ofset = 0;
+	protected $ofset = 0;
 
-    protected $joins = [];
+	protected $joins = [];
 
-    protected $order = [];
+	protected $order = [];
 
-    protected $manualJoins = [];
+	protected $manualJoins = [];
 
-    protected $group = null;
+	protected $group = null;
 
-    protected $manualWheres = [];
+	protected $manualWheres = [];
 
-    protected $whereType = 'AND';
+	protected $whereType = 'AND';
     
-    function getWhere()
-    {
-        return $this->where;
-    }
-
-    function getLimit()
-    {
-        return $this->limit;
-    }
-
-    function getOfset()
-    {
-        return $this->ofset;
-    }
-
-    function getJoins()
-    {
-        return $this->joins;
-    }
-
-    function getOrder()
-    {
-        return $this->order;
-    }
-
-    function getManualJoins()
-    {
-        return $this->manualJoins;
-    }
-
-    function getGroup()
-    {
-        return $this->group;
-    }
-
-    function getManualWheres()
-    {
-        return $this->manualWheres;
-    }
-
-    function getWhereType()
-    {
-        return $this->whereType;
-    }
-
-    function setWhere($field,$value = false)
-    {
-        if($value!==false){
-            $this->where[$field] = $value;
-        }
-        else{
-            $this->where = $where;
-        }
+	function getWhere()
+	{
+		return $this->where;
+	}
+
+	function getLimit()
+	{
+		return $this->limit;
+	}
+
+	function getOfset()
+	{
+		return $this->ofset;
+	}
+
+	function getJoins()
+	{
+		return $this->joins;
+	}
+
+	function getOrder()
+	{
+		return $this->order;
+	}
+
+	function getManualJoins()
+	{
+		return $this->manualJoins;
+	}
+
+	function getGroup()
+	{
+		return $this->group;
+	}
+
+	function getManualWheres()
+	{
+		return $this->manualWheres;
+	}
+
+	function getWhereType()
+	{
+		return $this->whereType;
+	}
+
+	function setWhere($field,$value = false)
+	{
+		if($value!==false){
+			$this->where[$field] = $value;
+		}
+		else{
+			$this->where = $where;
+		}
         
-        return $this;
-    }
-
-    function setLimit($limit)
-    {
-        $this->limit = $limit;
-        return $this;
-    }
-
-    function setOfset($ofset)
-    {
-        $this->ofset = $ofset;
-        return $this;
-    }
-
-    function setJoins($joins)
-    {
-        $this->joins = $joins;
-        return $this;
-    }
-
-    function setOrder($order)
-    {
-        $this->order = $order;
-        return $this;
-    }
-
-    function setManualJoins($manualJoins)
-    {
-        $this->manualJoins = $manualJoins;
-        return $this;
-    }
-
-    function setGroup($group)
-    {
-        $this->group = $group;
-        return $this;
-    }
-
-    function setManualWheres($manualWheres)
-    {
-        $this->manualWheres = $manualWheres;
-        return $this;
-    }
-
-    function setWhereType($whereType)
-    {
-        $this->whereType = $whereType;
-        return $this;
-    }
-
-    /**
-     * Создание критериев
-     */
-    public function getCriteria()
-    {
+		return $this;
+	}
+
+	function setLimit($limit)
+	{
+		$this->limit = $limit;
+		return $this;
+	}
+
+	function setOfset($ofset)
+	{
+		$this->ofset = $ofset;
+		return $this;
+	}
+
+	function setJoins($joins)
+	{
+		$this->joins = $joins;
+		return $this;
+	}
+
+	function setOrder($order)
+	{
+		$this->order = $order;
+		return $this;
+	}
+
+	function setManualJoins($manualJoins)
+	{
+		$this->manualJoins = $manualJoins;
+		return $this;
+	}
+
+	function setGroup($group)
+	{
+		$this->group = $group;
+		return $this;
+	}
+
+	function setManualWheres($manualWheres)
+	{
+		$this->manualWheres = $manualWheres;
+		return $this;
+	}
+
+	function setWhereType($whereType)
+	{
+		$this->whereType = $whereType;
+		return $this;
+	}
+
+	/**
+	 * Создание критериев
+	 */
+	public function getCriteria()
+	{
         
-    }
+	}
 }

Spacing +3 added lines, -3 removed lines

@@ -73,12 +73,12 @@
         return $this->whereType;
     }
 
-    function setWhere($field,$value = false)
+    function setWhere($field, $value = false)
     {
-        if($value!==false){
+        if ($value!==false) {
             $this->where[$field] = $value;
         }
-        else{
+        else {
             $this->where = $where;
         }
         

Braces +1 added lines, -2 removed lines

@@ -77,8 +77,7 @@
     {
         if($value!==false){
             $this->where[$field] = $value;
-        }
-        else{
+        } else{
             $this->where = $where;
         }
         

src/RepositoryInterface.php

Indentation +6 added lines, -6 removed lines

@@ -10,15 +10,15 @@
 interface RepositoryInterface
 {
 
-    public function findById($id);
+	public function findById($id);
 
-    public function findBySpecification(ISpecificationCriteria $specification);
+	public function findBySpecification(ISpecificationCriteria $specification);
 
-    public function findAll();
+	public function findAll();
 
-    public function findAllBySpecification(ISpecificationCriteria $specification);
+	public function findAllBySpecification(ISpecificationCriteria $specification);
 
-    public function save(EntityInterface $Entity);
+	public function save(EntityInterface $Entity);
 
-    public function delete(EntityInterface $Entity);
+	public function delete(EntityInterface $Entity);
 }

src/QueryBuilderInterface.php

Spacing +4 added lines, -4 removed lines

@@ -14,13 +14,13 @@
  */
 interface QueryBuilderInterface {
 	
-	public function getResultQuery($table,\SimpleORM\ISpecificationCriteria $Criteria);
+	public function getResultQuery($table, \SimpleORM\ISpecificationCriteria $Criteria);
 	
-	public function update($table,array $data,$where = []);
+	public function update($table, array $data, $where = []);
 
-	public function insert($table,array $data);
+	public function insert($table, array $data);
 	
 	public function insert_id();
 	
-	public function delete($table,$where = []);
+	public function delete($table, $where = []);
 }

src/EntityInterface.php

Indentation +5 added lines, -5 removed lines

@@ -8,12 +8,12 @@
  */
 interface EntityInterface
 {
-    //put your code here
-    public function getId();
+	//put your code here
+	public function getId();
     
-    public function setId($id);
+	public function setId($id);
 	
-    public function getDeleted();
+	public function getDeleted();
     
-    public function setDeleted($delete);	
+	public function setDeleted($delete);	
 }

src/Adapter/CodeigniterQueryBuilder.php

Doc Comments +6 added lines, -4 removed lines

@@ -66,6 +66,9 @@ 
 		return $this->adapter->delete($table,$where);
 	}
 
+	/**
+	 * @param string $str
+	 */
 	protected function escape_str($str, $like = FALSE)
 	{
 
@@ -120,7 +123,6 @@ 
 	
 	/**
 	 * 
-	 * @param type $param
 	 */
 	public function getResultQuery($table,\SimpleORM\ISpecificationCriteria $Criteria ) {
 		
@@ -155,7 +157,7 @@ 
 	/**
 	 * Создает селект не только для основной таблицы но и для приджойненых таблиц
 	 * @param type $joins
-	 * @return type
+	 * @return string
 	 */
 	protected function createSelect($joins)
 	{
@@ -171,8 +173,8 @@ 
 	/**
 	 * Получение записей по условию
 	 * @param type $where
-	 * @param type $limit
-	 * @param type $offset
+	 * @param integer $limit
+	 * @param integer $offset
 	 * @param type $joins
 	 * @param type $order
 	 * @param type $manualJoins

Spacing +53 added lines, -53 removed lines

@@ -38,9 +38,9 @@ 
 	protected function escape($str)
 	{
 		if (is_string($str)) {
-			$str = "'" . $this->escape_str($str) . "'";
+			$str = "'".$this->escape_str($str)."'";
 		} elseif (is_bool($str)) {
-			$str = ($str === FALSE) ? 0 : 1;
+			$str = ($str===FALSE) ? 0 : 1;
 		} elseif (is_null($str)) {
 			$str = 'NULL';
 		}
@@ -48,13 +48,13 @@ 
 		return $str;
 	}
 	
-	public function update($table,array $data,$where = []){
-		return $this->adapter->update($table,$data,$where);
+	public function update($table, array $data, $where = []) {
+		return $this->adapter->update($table, $data, $where);
 	}
 	
-	public function insert($table,array $data)
+	public function insert($table, array $data)
 	{
-		return $this->adapter->insert($table,$data);
+		return $this->adapter->insert($table, $data);
 	}
 	
 	public function insert_id()
@@ -62,17 +62,17 @@ 
 		return $this->adapter->insert_id();
 	}
 
-	public function delete($table,$where = []){
-		return $this->adapter->delete($table,$where);
+	public function delete($table, $where = []) {
+		return $this->adapter->delete($table, $where);
 	}
 
 	protected function escape_str($str, $like = FALSE)
 	{
 
-		if(!$like){
+		if (!$like) {
 			return $this->adapter->escape_str($str);
 		}
-		else{
+		else {
 			return $this->adapter->escape_like_str($str);
 		}	
 		
@@ -111,8 +111,8 @@ 
 			throw new HttpException('You cannot have an empty field name.');
 		}
 
-		if (strpos($field, '.') === false) {
-			return $this->TableName . '.' . $field;
+		if (strpos($field, '.')===false) {
+			return $this->TableName.'.'.$field;
 		}
 
 		return $field;
@@ -122,7 +122,7 @@ 
 	 * 
 	 * @param type $param
 	 */
-	public function getResultQuery($table,\SimpleORM\ISpecificationCriteria $Criteria ) {
+	public function getResultQuery($table, \SimpleORM\ISpecificationCriteria $Criteria) {
 		
 		$this->setTable($table);
 		
@@ -141,12 +141,12 @@ 
 		return $res;		
 	}
 
-	protected function setTable($table){
-		if(preg_match('~(.*?)\.(.*?)$~is',$table,$m)){
+	protected function setTable($table) {
+		if (preg_match('~(.*?)\.(.*?)$~is', $table, $m)) {
 			$this->database = $m[1];
 			$this->TableName = $m[2];
 		}
-		else{
+		else {
 			$this->TableName = $table;
 		}		
 	}	
@@ -159,9 +159,9 @@ 
 	 */
 	protected function createSelect($joins)
 	{
-		$s = "`" . $this->TableName . '`.*';
+		$s = "`".$this->TableName.'`.*';
 		foreach ($joins as $table => $join) {
-			$table = isset($join['alias']) ? "`{$join['alias']}`": $table;
+			$table = isset($join['alias']) ? "`{$join['alias']}`" : $table;
 			$s .= ", $table.*";
 		}
 		return $s;
@@ -184,8 +184,8 @@ 
 	 */
 	protected function buildQuery($where = array(), $limit = 25, $offset = 0, $joins = array(), $order = array(), $manualJoins = array(), $group = null, $manualWheres = array(), $whereType = 'AND')
 	{
-		$table = !empty($this->database)? "`{$this->database}`.".$this->TableName : $this->TableName;
-		$query = 'SELECT ' . $this->createSelect($joins) . " FROM `".$table."`";
+		$table = !empty($this->database) ? "`{$this->database}`.".$this->TableName : $this->TableName;
+		$query = 'SELECT '.$this->createSelect($joins)." FROM `".$table."`";
 		//$countQuery = "SELECT COUNT(*) AS cnt FROM `{$this->database}`.".$this->getTableName();
 
 		$wheres = array();
@@ -195,15 +195,15 @@ 
 
 			if (!is_array($value)) {
 				$params[] = $value;
-				$wheres[] = $key . ' = ?';
+				$wheres[] = $key.' = ?';
 			} else {
 				if (isset($value['operator'])) {
 					if (is_array($value['value'])) {
-						if ($value['operator'] == 'between') {
+						if ($value['operator']=='between') {
 							$params[] = $value['value'][0];
 							$params[] = $value['value'][1];
-							$wheres[] = $key . ' BETWEEN ? AND ?';
-						} elseif ($value['operator'] == 'IN') {
+							$wheres[] = $key.' BETWEEN ? AND ?';
+						} elseif ($value['operator']=='IN') {
 							$in = array();
 
 							foreach ($value['value'] as $item) {
@@ -211,67 +211,67 @@ 
 								$in[] = '?';
 							}
 
-							$wheres[] = $key . ' IN (' . implode(', ', $in) . ') ';
+							$wheres[] = $key.' IN ('.implode(', ', $in).') ';
 						} else {
 							$ors = array();
 							foreach ($value['value'] as $item) {
-								if ($item == 'null') {
+								if ($item=='null') {
 									switch ($value['operator']) {
 										case '!=':
-											$ors[] = $key . ' IS NOT NULL';
+											$ors[] = $key.' IS NOT NULL';
 											break;
 
 										case '==':
 										default:
-											$ors[] = $key . ' IS NULL';
+											$ors[] = $key.' IS NULL';
 											break;
 									}
 								} else {
 									$params[] = $item;
-									$ors[] = $this->fieldCheck($key) . ' ' . $value['operator'] . ' ?';
+									$ors[] = $this->fieldCheck($key).' '.$value['operator'].' ?';
 								}
 							}
-							$wheres[] = '(' . implode(' OR ', $ors) . ')';
+							$wheres[] = '('.implode(' OR ', $ors).')';
 						}
 					} else {
-						if ($value['operator'] == 'like') {
-							$params[] = '%' . $value['value'] . '%';
-							$wheres[] = $key . ' ' . $value['operator'] . ' ?';
+						if ($value['operator']=='like') {
+							$params[] = '%'.$value['value'].'%';
+							$wheres[] = $key.' '.$value['operator'].' ?';
 						} else {
-							if ($value['value'] === 'null') {
+							if ($value['value']==='null') {
 								switch ($value['operator']) {
 									case '!=':
-										$wheres[] = $key . ' IS NOT NULL';
+										$wheres[] = $key.' IS NOT NULL';
 										break;
 
 									case '==':
 									default:
-										$wheres[] = $key . ' IS NULL';
+										$wheres[] = $key.' IS NULL';
 										break;
 								}
 							} else {
 								$params[] = $value['value'];
-								$wheres[] = $key . ' ' . $value['operator'] . ' ?';
+								$wheres[] = $key.' '.$value['operator'].' ?';
 							}
 						}
 					}
 				} else {
-					$wheres[] = $key . ' IN (' . implode(', ', array_map(array($this, 'escape'), $value)) . ')';
+					$wheres[] = $key.' IN ('.implode(', ', array_map(array($this, 'escape'), $value)).')';
 				}
 			}
 		}
 
 		if (count($joins)) {
 			foreach ($joins as $table => $join) {
-				$type = isset($join['type'])?$join['type']:'INNER';
-				$query .= ' '. $type.' JOIN `' . $table . '` as `' . $join['alias'] . '` ON ' . $join['on'] . ' ';
+				$type = isset($join['type']) ? $join['type'] : 'INNER';
+				$query .= ' '.$type.' JOIN `'.$table.'` as `'.$join['alias'].'` ON '.$join['on'].' ';
 				//$countQuery .= ' '.$type.' JOIN ' . $table . ' ' . $join['alias'] . ' ON ' . $join['on'] . ' ';
 			}
 		}
 
 		if (count($manualJoins)) {
 			foreach ($manualJoins as $join) {
-				$query .= ' ' . $join . ' ';
+				$query .= ' '.$join.' ';
 				//$countQuery .= ' ' . $join . ' ';
 			}
 		}
@@ -279,7 +279,7 @@ 
 		$hasWhere = false;
 		if (count($wheres)) {
 			$hasWhere = true;
-			$query .= ' WHERE (' . implode(' ' . $whereType . ' ', $wheres) . ')';
+			$query .= ' WHERE ('.implode(' '.$whereType.' ', $wheres).')';
 			//$countQuery .= ' WHERE (' . implode(' ' . $whereType . ' ', $wheres) . ')';
 		}
 
@@ -290,11 +290,11 @@ 
 					$query .= ' WHERE ';
 					//$countQuery .= ' WHERE ';
 				} else {
-					$query .= ' ' . $where['type'] . ' ';
+					$query .= ' '.$where['type'].' ';
 					//$countQuery .= ' ' . $where['type'] . ' ';
 				}
 
-				$query .= ' ' . $where['query'];
+				$query .= ' '.$where['query'];
 				//$countQuery .= ' ' . $where['query'];
 
 				if (isset($where['params'])) {
@@ -306,28 +306,28 @@ 
 		}
 
 		if (!is_null($group)) {
-			$query .= ' GROUP BY ' . $group . ' ';
+			$query .= ' GROUP BY '.$group.' ';
 		}
 
 		if (count($order)) {
 			$orders = array();
-			if (is_string($order) && $order == 'rand') {
+			if (is_string($order) && $order=='rand') {
 				$query .= ' ORDER BY RAND() ';
 			} else {
 				foreach ($order as $key => $value) {
-					$orders[] = $this->fieldCheck($key) . ' ' . $value;
+					$orders[] = $this->fieldCheck($key).' '.$value;
 				}
 
-				$query .= ' ORDER BY ' . implode(', ', $orders);
+				$query .= ' ORDER BY '.implode(', ', $orders);
 			}
 		}
 
 		if ($limit) {
-			$query .= ' LIMIT ' . $limit;
+			$query .= ' LIMIT '.$limit;
 		}
 
 		if ($offset) {
-			$query .= ' OFFSET ' . $offset;
+			$query .= ' OFFSET '.$offset;
 		}
 
 
@@ -345,7 +345,7 @@ 
 		try {
 			//$query = $this->compile_binds($query, $params);
 			
-			return $this->adapter->query($query,$params);
+			return $this->adapter->query($query, $params);
 			//ed( $this->adapter->query($query) ,1);
 //			if ($res = $this->adapter->getRows($query)) {
 //				$rtn = array();
@@ -359,7 +359,7 @@ 
 //			} else {
 //				return false;
 //			}
-		} catch(\PDOException $ex) {
+		} catch (\PDOException $ex) {
 			throw $ex;
 		}
 	}	
@@ -374,7 +374,7 @@ 
 	 */
 	protected function compile_binds($sql, $binds)
 	{
-		if (strpos($sql, $this->bind_marker) === FALSE) {
+		if (strpos($sql, $this->bind_marker)===FALSE) {
 			return $sql;
 		}
 

Braces +2 added lines, -4 removed lines

@@ -71,8 +71,7 @@ 
 
 		if(!$like){
 			return $this->adapter->escape_str($str);
-		}
-		else{
+		} else{
 			return $this->adapter->escape_like_str($str);
 		}	
 		
@@ -145,8 +144,7 @@ 
 		if(preg_match('~(.*?)\.(.*?)$~is',$table,$m)){
 			$this->database = $m[1];
 			$this->TableName = $m[2];
-		}
-		else{
+		} else{
 			$this->TableName = $table;
 		}		
 	}	

tests/bootstrap.php

Doc Comments +7 added lines, -3 removed lines

@@ -5,8 +5,8 @@ 
     
     /**
      * получение защищенного свойства
-     * @param type $o
-     * @param type $name
+     * @param string $name
+     * @param SimpleORM\AbstractDataMapper $obj
      * @return type
      */
     static public function getProtectedAttribute($obj, $name)
@@ -20,7 +20,7 @@ 
     /**
      * Добавление значения в защищенное свойтсво
      * @param type $name
-     * @param type $valued 
+     * @param type $value 
      */
     private function setValueprotectedProperty($name, $value)
     {
@@ -30,6 +30,10 @@ 
         $r->setValue($this->object, $value); //изменяем значение
     }    
 	
+	/**
+	 * @param SimpleORM\AbstractDataMapper $obj
+	 * @param string $name
+	 */
 	static public function callMethod($obj, $name, array $args) {
         $class = new \ReflectionClass($obj);
         $method = $class->getMethod($name);

Indentation +30 added lines, -30 removed lines

@@ -3,38 +3,38 @@
 
 class TestHelper{
     
-    /**
-     * получение защищенного свойства
-     * @param type $o
-     * @param type $name
-     * @return type
-     */
-    static public function getProtectedAttribute($obj, $name)
-    {
-        $reflectionClass = new \ReflectionClass($obj); //создаем reflectionClass
-        $r = $reflectionClass->getProperty($name); //получаем свойство
-        $r->setAccessible(true); //делаем открытым
-        return $r->getValue($obj);
-    }
+	/**
+	 * получение защищенного свойства
+	 * @param type $o
+	 * @param type $name
+	 * @return type
+	 */
+	static public function getProtectedAttribute($obj, $name)
+	{
+		$reflectionClass = new \ReflectionClass($obj); //создаем reflectionClass
+		$r = $reflectionClass->getProperty($name); //получаем свойство
+		$r->setAccessible(true); //делаем открытым
+		return $r->getValue($obj);
+	}
 
-    /**
-     * Добавление значения в защищенное свойтсво
-     * @param type $name
-     * @param type $valued 
-     */
-    private function setValueprotectedProperty($name, $value)
-    {
-        $reflectionClass = new \ReflectionClass($this->object); //создаем reflectionClass
-        $r = $reflectionClass->getProperty($name); //получаем свойство
-        $r->setAccessible(true); //делаем открытым
-        $r->setValue($this->object, $value); //изменяем значение
-    }    
+	/**
+	 * Добавление значения в защищенное свойтсво
+	 * @param type $name
+	 * @param type $valued 
+	 */
+	private function setValueprotectedProperty($name, $value)
+	{
+		$reflectionClass = new \ReflectionClass($this->object); //создаем reflectionClass
+		$r = $reflectionClass->getProperty($name); //получаем свойство
+		$r->setAccessible(true); //делаем открытым
+		$r->setValue($this->object, $value); //изменяем значение
+	}    
 	
 	static public function callMethod($obj, $name, array $args) {
-        $class = new \ReflectionClass($obj);
-        $method = $class->getMethod($name);
-        $method->setAccessible(true);
-        return $method->invokeArgs($obj, $args);
-    }	
+		$class = new \ReflectionClass($obj);
+		$method = $class->getMethod($name);
+		$method->setAccessible(true);
+		return $method->invokeArgs($obj, $args);
+	}	
     
 }

Spacing +1 added lines, -1 removed lines

@@ -1,7 +1,7 @@
 <?php
 $loader = require_once __DIR__.'/../vendor/autoload.php';
 
-class TestHelper{
+class TestHelper {
     
     /**
      * получение защищенного свойства

tests/src/AbstractDataMapperTest.php

Indentation +5 added lines, -5 removed lines

@@ -58,7 +58,7 @@ 
 		//-----------------
 		$mapping_fields = \TestHelper::getProtectedAttribute($this->object,'mapping_fields');
 
-        $this->assertEquals($mapping_fields, $correct);		
+		$this->assertEquals($mapping_fields, $correct);		
 	}		
 	
 	/**
@@ -77,7 +77,7 @@ 
 		//-----------------
 		$mapping_fields = \TestHelper::getProtectedAttribute($this->object,'mapping_fields');
 
-        $this->assertEquals($mapping_fields, $correct);		
+		$this->assertEquals($mapping_fields, $correct);		
 	}		
 
 	/**
@@ -98,7 +98,7 @@ 
 		//-----------------
 		$mapping_fields = \TestHelper::getProtectedAttribute($this->object,'mapping_fields');
 
-        $this->assertEquals($mapping_fields, $correct);		
+		$this->assertEquals($mapping_fields, $correct);		
 	}	
 	
 	/**
@@ -121,7 +121,7 @@ 
 		//-----------------
 		$mapping_fields = \TestHelper::getProtectedAttribute($this->object,'mapping_fields');
 
-        $this->assertEquals($mapping_fields, $correct);		
+		$this->assertEquals($mapping_fields, $correct);		
 		
 		$key_field = \TestHelper::getProtectedAttribute($this->object,'key');
 		$this->assertEquals($key_field, 'tb_mayfield');	
@@ -147,7 +147,7 @@ 
 		//-----------------
 		$mapping_fields = \TestHelper::getProtectedAttribute($this->object,'mapping_fields');
 
-        $this->assertEquals($mapping_fields, $correct);		
+		$this->assertEquals($mapping_fields, $correct);		
 		
 		$soft_delete_key = \TestHelper::getProtectedAttribute($this->object,'soft_delete_key');
 		$this->assertEquals($soft_delete_key, 'tb_mayfield');	

Spacing +18 added lines, -18 removed lines

@@ -28,7 +28,7 @@ 
 //								])//в конструктор
 						->disableOriginalConstructor()
 						//->setMethods(null)
-						->setMethods(['setMappingFields','createEntity'])
+						->setMethods(['setMappingFields', 'createEntity'])
 						//->setMethods(null) //не использовать заглушки методов иначе буде возвращать NULL
 						->getMock();
 	}
@@ -45,9 +45,9 @@ 
 	/**
 	 * @covers SimpleORM\AbstractDataMapper::AddMappingField
 	 */
-	public function testAddMappingField_SimpleField(){
+	public function testAddMappingField_SimpleField() {
 		
-		\TestHelper::callMethod($this->object,'addMappingField',['myfield']);
+		\TestHelper::callMethod($this->object, 'addMappingField', ['myfield']);
 		
 		$correct = [
 			  'myfield'	=>	[
@@ -56,7 +56,7 @@ 
 		  ];
 		
 		//-----------------
-		$mapping_fields = \TestHelper::getProtectedAttribute($this->object,'mapping_fields');
+		$mapping_fields = \TestHelper::getProtectedAttribute($this->object, 'mapping_fields');
 
         $this->assertEquals($mapping_fields, $correct);		
 	}		
@@ -64,9 +64,9 @@ 
 	/**
 	 * @covers SimpleORM\AbstractDataMapper::AddMappingField
 	 */
-	public function testAddMappingField_FieldAndAlias(){
+	public function testAddMappingField_FieldAndAlias() {
 		
-		\TestHelper::callMethod($this->object,'addMappingField',['myfield','tb_mayfield']);
+		\TestHelper::callMethod($this->object, 'addMappingField', ['myfield', 'tb_mayfield']);
 		
 		$correct = [
 			  'myfield'	=>	[
@@ -75,7 +75,7 @@ 
 		  ];
 		
 		//-----------------
-		$mapping_fields = \TestHelper::getProtectedAttribute($this->object,'mapping_fields');
+		$mapping_fields = \TestHelper::getProtectedAttribute($this->object, 'mapping_fields');
 
         $this->assertEquals($mapping_fields, $correct);		
 	}		
@@ -83,9 +83,9 @@ 
 	/**
 	 * @covers SimpleORM\AbstractDataMapper::AddMappingField
 	 */
-	public function testAddMappingField_ArrayField(){
+	public function testAddMappingField_ArrayField() {
 		
-		\TestHelper::callMethod($this->object,'addMappingField',['myfield',[
+		\TestHelper::callMethod($this->object, 'addMappingField', ['myfield', [
 			'field'	=>	'tb_mayfield'
 		]]);
 		
@@ -96,7 +96,7 @@ 
 		  ];
 		
 		//-----------------
-		$mapping_fields = \TestHelper::getProtectedAttribute($this->object,'mapping_fields');
+		$mapping_fields = \TestHelper::getProtectedAttribute($this->object, 'mapping_fields');
 
         $this->assertEquals($mapping_fields, $correct);		
 	}	
@@ -104,9 +104,9 @@ 
 	/**
 	 * @covers SimpleORM\AbstractDataMapper::AddMappingField
 	 */
-	public function testAddMappingField_PrimaryKey(){
+	public function testAddMappingField_PrimaryKey() {
 		
-		\TestHelper::callMethod($this->object,'addMappingField',['myfield',[
+		\TestHelper::callMethod($this->object, 'addMappingField', ['myfield', [
 			'field'		=>	'tb_mayfield',
 			'primary'	=>	true
 		]]);
@@ -119,20 +119,20 @@ 
 		  ];
 		
 		//-----------------
-		$mapping_fields = \TestHelper::getProtectedAttribute($this->object,'mapping_fields');
+		$mapping_fields = \TestHelper::getProtectedAttribute($this->object, 'mapping_fields');
 
         $this->assertEquals($mapping_fields, $correct);		
 		
-		$key_field = \TestHelper::getProtectedAttribute($this->object,'key');
+		$key_field = \TestHelper::getProtectedAttribute($this->object, 'key');
 		$this->assertEquals($key_field, 'tb_mayfield');	
 	}	
 	
 	/**
 	 * @covers SimpleORM\AbstractDataMapper::AddMappingField
 	 */
-	public function testAddMappingField_SoftDelete(){
+	public function testAddMappingField_SoftDelete() {
 		
-		\TestHelper::callMethod($this->object,'addMappingField',['myfield',[
+		\TestHelper::callMethod($this->object, 'addMappingField', ['myfield', [
 			'field'			=>	'tb_mayfield',
 			'softdelete'	=>	true
 		]]);
@@ -145,11 +145,11 @@ 
 		  ];
 		
 		//-----------------
-		$mapping_fields = \TestHelper::getProtectedAttribute($this->object,'mapping_fields');
+		$mapping_fields = \TestHelper::getProtectedAttribute($this->object, 'mapping_fields');
 
         $this->assertEquals($mapping_fields, $correct);		
 		
-		$soft_delete_key = \TestHelper::getProtectedAttribute($this->object,'soft_delete_key');
+		$soft_delete_key = \TestHelper::getProtectedAttribute($this->object, 'soft_delete_key');
 		$this->assertEquals($soft_delete_key, 'tb_mayfield');	
 	}