dimaxz / SimpleORM

src/ISpecificationCriteria.php

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;
 	}
 	

src/Specification.php

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;
         }
         

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/Adapter/CodeigniterQueryBuilder.php

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;
 		}
 

tests/src/AbstractDataMapperTest.php

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');	
 	}	
 	

tests/bootstrap.php

Spacing +1 added lines, -1 removed lines

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

example/Domain/City/CityMapper.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
 					'primary'	 => true
 					]
 				)
-				->addMappingField('name','cty_name')
+				->addMappingField('name', 'cty_name')
 			;
 
 	}	

example/Domain/User/UserMapper.php

Spacing +2 added lines, -2 removed lines

@@ -31,7 +31,7 @@ 
 	 */
 	public function createEntity(array $row) {
 		$Group = $this->DI->get('UserGroupMapper')->createEntity([]);
-		return $this->buildEntity(new User('testname','mail@mail.ru','asdftyasdvh21267g',$Group), $row);
+		return $this->buildEntity(new User('testname', 'mail@mail.ru', 'asdftyasdvh21267g', $Group), $row);
 	}	
 	
 	/**
@@ -46,7 +46,7 @@ 
 					'primary'	 => true
 					]
 				)
-				->addMappingField('name','usr_name')
+				->addMappingField('name', 'usr_name')
 				->addMappingField('email', 'usr_email')
 				->addMappingField('password', 'usr_password')
 				->addMappingField('group', [