Passed
Push — master ( 5928a0...6c7f1e )
by dima
02:51
created
src/ISpecificationCriteria.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@
 block discarded – undo
30 30
 
31 31
     public function getWhereType();    
32 32
 
33
-    public function setWhere($field,$value = false);    
33
+    public function setWhere($field, $value = false);    
34 34
 
35 35
     public function setLimit($limit);
36 36
 
Please login to merge, or discard this patch.
src/AbstractEntity.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
 
22 22
     public function setId($id)
23 23
     {
24
-        if ($this->id !== null) {
24
+        if ($this->id!==null) {
25 25
             throw new BadMethodCallException(
26 26
                 "Идентификатор у сущности уже установлен");
27 27
         }
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
         return $this;		
35 35
     }
36 36
 	
37
-	public function setDeleted($deleted){
37
+	public function setDeleted($deleted) {
38 38
 		$this->deleted = $deleted;
39 39
 	}
40 40
 	
Please login to merge, or discard this patch.
src/Specification.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -73,12 +73,12 @@
 block discarded – undo
73 73
         return $this->whereType;
74 74
     }
75 75
 
76
-    function setWhere($field,$value = false)
76
+    function setWhere($field, $value = false)
77 77
     {
78
-        if($value!==false){
78
+        if ($value!==false) {
79 79
             $this->where[$field] = $value;
80 80
         }
81
-        else{
81
+        else {
82 82
             $this->where = $where;
83 83
         }
84 84
         
Please login to merge, or discard this patch.
src/QueryBuilderInterface.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -14,13 +14,13 @@
 block discarded – undo
14 14
  */
15 15
 interface QueryBuilderInterface {
16 16
 	
17
-	public function getResultQuery($table,\SimpleORM\ISpecificationCriteria $Criteria);
17
+	public function getResultQuery($table, \SimpleORM\ISpecificationCriteria $Criteria);
18 18
 	
19
-	public function update($table,array $data,$where = []);
19
+	public function update($table, array $data, $where = []);
20 20
 
21
-	public function insert($table,array $data);
21
+	public function insert($table, array $data);
22 22
 	
23 23
 	public function insert_id();
24 24
 	
25
-	public function delete($table,$where = []);
25
+	public function delete($table, $where = []);
26 26
 }
Please login to merge, or discard this patch.
src/Adapter/CodeigniterQueryBuilder.php 2 patches
Doc Comments   +6 added lines, -4 removed lines patch added patch discarded remove patch
@@ -66,6 +66,9 @@  discard block
 block discarded – undo
66 66
 		return $this->adapter->delete($table,$where);
67 67
 	}
68 68
 
69
+	/**
70
+	 * @param string $str
71
+	 */
69 72
 	protected function escape_str($str, $like = FALSE)
70 73
 	{
71 74
 
@@ -120,7 +123,6 @@  discard block
 block discarded – undo
120 123
 	
121 124
 	/**
122 125
 	 * 
123
-	 * @param type $param
124 126
 	 */
125 127
 	public function getResultQuery($table,\SimpleORM\ISpecificationCriteria $Criteria ) {
126 128
 		
@@ -155,7 +157,7 @@  discard block
 block discarded – undo
155 157
 	/**
156 158
 	 * Создает селект не только для основной таблицы но и для приджойненых таблиц
157 159
 	 * @param type $joins
158
-	 * @return type
160
+	 * @return string
159 161
 	 */
160 162
 	protected function createSelect($joins)
161 163
 	{
@@ -171,8 +173,8 @@  discard block
 block discarded – undo
171 173
 	/**
172 174
 	 * Получение записей по условию
173 175
 	 * @param type $where
174
-	 * @param type $limit
175
-	 * @param type $offset
176
+	 * @param integer $limit
177
+	 * @param integer $offset
176 178
 	 * @param type $joins
177 179
 	 * @param type $order
178 180
 	 * @param type $manualJoins
Please login to merge, or discard this patch.
Spacing   +53 added lines, -53 removed lines patch added patch discarded remove patch
@@ -38,9 +38,9 @@  discard block
 block discarded – undo
38 38
 	protected function escape($str)
39 39
 	{
40 40
 		if (is_string($str)) {
41
-			$str = "'" . $this->escape_str($str) . "'";
41
+			$str = "'".$this->escape_str($str)."'";
42 42
 		} elseif (is_bool($str)) {
43
-			$str = ($str === FALSE) ? 0 : 1;
43
+			$str = ($str===FALSE) ? 0 : 1;
44 44
 		} elseif (is_null($str)) {
45 45
 			$str = 'NULL';
46 46
 		}
@@ -48,13 +48,13 @@  discard block
 block discarded – undo
48 48
 		return $str;
49 49
 	}
50 50
 	
51
-	public function update($table,array $data,$where = []){
52
-		return $this->adapter->update($table,$data,$where);
51
+	public function update($table, array $data, $where = []) {
52
+		return $this->adapter->update($table, $data, $where);
53 53
 	}
54 54
 	
55
-	public function insert($table,array $data)
55
+	public function insert($table, array $data)
56 56
 	{
57
-		return $this->adapter->insert($table,$data);
57
+		return $this->adapter->insert($table, $data);
58 58
 	}
59 59
 	
60 60
 	public function insert_id()
@@ -62,17 +62,17 @@  discard block
 block discarded – undo
62 62
 		return $this->adapter->insert_id();
63 63
 	}
64 64
 
65
-	public function delete($table,$where = []){
66
-		return $this->adapter->delete($table,$where);
65
+	public function delete($table, $where = []) {
66
+		return $this->adapter->delete($table, $where);
67 67
 	}
68 68
 
69 69
 	protected function escape_str($str, $like = FALSE)
70 70
 	{
71 71
 
72
-		if(!$like){
72
+		if (!$like) {
73 73
 			return $this->adapter->escape_str($str);
74 74
 		}
75
-		else{
75
+		else {
76 76
 			return $this->adapter->escape_like_str($str);
77 77
 		}	
78 78
 		
@@ -111,8 +111,8 @@  discard block
 block discarded – undo
111 111
 			throw new HttpException('You cannot have an empty field name.');
112 112
 		}
113 113
 
114
-		if (strpos($field, '.') === false) {
115
-			return $this->TableName . '.' . $field;
114
+		if (strpos($field, '.')===false) {
115
+			return $this->TableName.'.'.$field;
116 116
 		}
117 117
 
118 118
 		return $field;
@@ -122,7 +122,7 @@  discard block
 block discarded – undo
122 122
 	 * 
123 123
 	 * @param type $param
124 124
 	 */
125
-	public function getResultQuery($table,\SimpleORM\ISpecificationCriteria $Criteria ) {
125
+	public function getResultQuery($table, \SimpleORM\ISpecificationCriteria $Criteria) {
126 126
 		
127 127
 		$this->setTable($table);
128 128
 		
@@ -141,12 +141,12 @@  discard block
 block discarded – undo
141 141
 		return $res;		
142 142
 	}
143 143
 
144
-	protected function setTable($table){
145
-		if(preg_match('~(.*?)\.(.*?)$~is',$table,$m)){
144
+	protected function setTable($table) {
145
+		if (preg_match('~(.*?)\.(.*?)$~is', $table, $m)) {
146 146
 			$this->database = $m[1];
147 147
 			$this->TableName = $m[2];
148 148
 		}
149
-		else{
149
+		else {
150 150
 			$this->TableName = $table;
151 151
 		}		
152 152
 	}	
@@ -159,9 +159,9 @@  discard block
 block discarded – undo
159 159
 	 */
160 160
 	protected function createSelect($joins)
161 161
 	{
162
-		$s = "`" . $this->TableName . '`.*';
162
+		$s = "`".$this->TableName.'`.*';
163 163
 		foreach ($joins as $table => $join) {
164
-			$table = isset($join['alias']) ? "`{$join['alias']}`": $table;
164
+			$table = isset($join['alias']) ? "`{$join['alias']}`" : $table;
165 165
 			$s .= ", $table.*";
166 166
 		}
167 167
 		return $s;
@@ -184,8 +184,8 @@  discard block
 block discarded – undo
184 184
 	 */
185 185
 	protected function buildQuery($where = array(), $limit = 25, $offset = 0, $joins = array(), $order = array(), $manualJoins = array(), $group = null, $manualWheres = array(), $whereType = 'AND')
186 186
 	{
187
-		$table = !empty($this->database)? "`{$this->database}`.".$this->TableName : $this->TableName;
188
-		$query = 'SELECT ' . $this->createSelect($joins) . " FROM `".$table."`";
187
+		$table = !empty($this->database) ? "`{$this->database}`.".$this->TableName : $this->TableName;
188
+		$query = 'SELECT '.$this->createSelect($joins)." FROM `".$table."`";
189 189
 		//$countQuery = "SELECT COUNT(*) AS cnt FROM `{$this->database}`.".$this->getTableName();
190 190
 
191 191
 		$wheres = array();
@@ -195,15 +195,15 @@  discard block
 block discarded – undo
195 195
 
196 196
 			if (!is_array($value)) {
197 197
 				$params[] = $value;
198
-				$wheres[] = $key . ' = ?';
198
+				$wheres[] = $key.' = ?';
199 199
 			} else {
200 200
 				if (isset($value['operator'])) {
201 201
 					if (is_array($value['value'])) {
202
-						if ($value['operator'] == 'between') {
202
+						if ($value['operator']=='between') {
203 203
 							$params[] = $value['value'][0];
204 204
 							$params[] = $value['value'][1];
205
-							$wheres[] = $key . ' BETWEEN ? AND ?';
206
-						} elseif ($value['operator'] == 'IN') {
205
+							$wheres[] = $key.' BETWEEN ? AND ?';
206
+						} elseif ($value['operator']=='IN') {
207 207
 							$in = array();
208 208
 
209 209
 							foreach ($value['value'] as $item) {
@@ -211,67 +211,67 @@  discard block
 block discarded – undo
211 211
 								$in[] = '?';
212 212
 							}
213 213
 
214
-							$wheres[] = $key . ' IN (' . implode(', ', $in) . ') ';
214
+							$wheres[] = $key.' IN ('.implode(', ', $in).') ';
215 215
 						} else {
216 216
 							$ors = array();
217 217
 							foreach ($value['value'] as $item) {
218
-								if ($item == 'null') {
218
+								if ($item=='null') {
219 219
 									switch ($value['operator']) {
220 220
 										case '!=':
221
-											$ors[] = $key . ' IS NOT NULL';
221
+											$ors[] = $key.' IS NOT NULL';
222 222
 											break;
223 223
 
224 224
 										case '==':
225 225
 										default:
226
-											$ors[] = $key . ' IS NULL';
226
+											$ors[] = $key.' IS NULL';
227 227
 											break;
228 228
 									}
229 229
 								} else {
230 230
 									$params[] = $item;
231
-									$ors[] = $this->fieldCheck($key) . ' ' . $value['operator'] . ' ?';
231
+									$ors[] = $this->fieldCheck($key).' '.$value['operator'].' ?';
232 232
 								}
233 233
 							}
234
-							$wheres[] = '(' . implode(' OR ', $ors) . ')';
234
+							$wheres[] = '('.implode(' OR ', $ors).')';
235 235
 						}
236 236
 					} else {
237
-						if ($value['operator'] == 'like') {
238
-							$params[] = '%' . $value['value'] . '%';
239
-							$wheres[] = $key . ' ' . $value['operator'] . ' ?';
237
+						if ($value['operator']=='like') {
238
+							$params[] = '%'.$value['value'].'%';
239
+							$wheres[] = $key.' '.$value['operator'].' ?';
240 240
 						} else {
241
-							if ($value['value'] === 'null') {
241
+							if ($value['value']==='null') {
242 242
 								switch ($value['operator']) {
243 243
 									case '!=':
244
-										$wheres[] = $key . ' IS NOT NULL';
244
+										$wheres[] = $key.' IS NOT NULL';
245 245
 										break;
246 246
 
247 247
 									case '==':
248 248
 									default:
249
-										$wheres[] = $key . ' IS NULL';
249
+										$wheres[] = $key.' IS NULL';
250 250
 										break;
251 251
 								}
252 252
 							} else {
253 253
 								$params[] = $value['value'];
254
-								$wheres[] = $key . ' ' . $value['operator'] . ' ?';
254
+								$wheres[] = $key.' '.$value['operator'].' ?';
255 255
 							}
256 256
 						}
257 257
 					}
258 258
 				} else {
259
-					$wheres[] = $key . ' IN (' . implode(', ', array_map(array($this, 'escape'), $value)) . ')';
259
+					$wheres[] = $key.' IN ('.implode(', ', array_map(array($this, 'escape'), $value)).')';
260 260
 				}
261 261
 			}
262 262
 		}
263 263
 
264 264
 		if (count($joins)) {
265 265
 			foreach ($joins as $table => $join) {
266
-				$type = isset($join['type'])?$join['type']:'INNER';
267
-				$query .= ' '. $type.' JOIN `' . $table . '` as `' . $join['alias'] . '` ON ' . $join['on'] . ' ';
266
+				$type = isset($join['type']) ? $join['type'] : 'INNER';
267
+				$query .= ' '.$type.' JOIN `'.$table.'` as `'.$join['alias'].'` ON '.$join['on'].' ';
268 268
 				//$countQuery .= ' '.$type.' JOIN ' . $table . ' ' . $join['alias'] . ' ON ' . $join['on'] . ' ';
269 269
 			}
270 270
 		}
271 271
 
272 272
 		if (count($manualJoins)) {
273 273
 			foreach ($manualJoins as $join) {
274
-				$query .= ' ' . $join . ' ';
274
+				$query .= ' '.$join.' ';
275 275
 				//$countQuery .= ' ' . $join . ' ';
276 276
 			}
277 277
 		}
@@ -279,7 +279,7 @@  discard block
 block discarded – undo
279 279
 		$hasWhere = false;
280 280
 		if (count($wheres)) {
281 281
 			$hasWhere = true;
282
-			$query .= ' WHERE (' . implode(' ' . $whereType . ' ', $wheres) . ')';
282
+			$query .= ' WHERE ('.implode(' '.$whereType.' ', $wheres).')';
283 283
 			//$countQuery .= ' WHERE (' . implode(' ' . $whereType . ' ', $wheres) . ')';
284 284
 		}
285 285
 
@@ -290,11 +290,11 @@  discard block
 block discarded – undo
290 290
 					$query .= ' WHERE ';
291 291
 					//$countQuery .= ' WHERE ';
292 292
 				} else {
293
-					$query .= ' ' . $where['type'] . ' ';
293
+					$query .= ' '.$where['type'].' ';
294 294
 					//$countQuery .= ' ' . $where['type'] . ' ';
295 295
 				}
296 296
 
297
-				$query .= ' ' . $where['query'];
297
+				$query .= ' '.$where['query'];
298 298
 				//$countQuery .= ' ' . $where['query'];
299 299
 
300 300
 				if (isset($where['params'])) {
@@ -306,28 +306,28 @@  discard block
 block discarded – undo
306 306
 		}
307 307
 
308 308
 		if (!is_null($group)) {
309
-			$query .= ' GROUP BY ' . $group . ' ';
309
+			$query .= ' GROUP BY '.$group.' ';
310 310
 		}
311 311
 
312 312
 		if (count($order)) {
313 313
 			$orders = array();
314
-			if (is_string($order) && $order == 'rand') {
314
+			if (is_string($order) && $order=='rand') {
315 315
 				$query .= ' ORDER BY RAND() ';
316 316
 			} else {
317 317
 				foreach ($order as $key => $value) {
318
-					$orders[] = $this->fieldCheck($key) . ' ' . $value;
318
+					$orders[] = $this->fieldCheck($key).' '.$value;
319 319
 				}
320 320
 
321
-				$query .= ' ORDER BY ' . implode(', ', $orders);
321
+				$query .= ' ORDER BY '.implode(', ', $orders);
322 322
 			}
323 323
 		}
324 324
 
325 325
 		if ($limit) {
326
-			$query .= ' LIMIT ' . $limit;
326
+			$query .= ' LIMIT '.$limit;
327 327
 		}
328 328
 
329 329
 		if ($offset) {
330
-			$query .= ' OFFSET ' . $offset;
330
+			$query .= ' OFFSET '.$offset;
331 331
 		}
332 332
 
333 333
 
@@ -345,7 +345,7 @@  discard block
 block discarded – undo
345 345
 		try {
346 346
 			//$query = $this->compile_binds($query, $params);
347 347
 			
348
-			return $this->adapter->query($query,$params);
348
+			return $this->adapter->query($query, $params);
349 349
 			//ed( $this->adapter->query($query) ,1);
350 350
 //			if ($res = $this->adapter->getRows($query)) {
351 351
 //				$rtn = array();
@@ -359,7 +359,7 @@  discard block
 block discarded – undo
359 359
 //			} else {
360 360
 //				return false;
361 361
 //			}
362
-		} catch(\PDOException $ex) {
362
+		} catch (\PDOException $ex) {
363 363
 			throw $ex;
364 364
 		}
365 365
 	}	
@@ -374,7 +374,7 @@  discard block
 block discarded – undo
374 374
 	 */
375 375
 	protected function compile_binds($sql, $binds)
376 376
 	{
377
-		if (strpos($sql, $this->bind_marker) === FALSE) {
377
+		if (strpos($sql, $this->bind_marker)===FALSE) {
378 378
 			return $sql;
379 379
 		}
380 380
 
Please login to merge, or discard this patch.
tests/bootstrap.php 2 patches
Doc Comments   +7 added lines, -3 removed lines patch added patch discarded remove patch
@@ -5,8 +5,8 @@  discard block
 block discarded – undo
5 5
     
6 6
     /**
7 7
      * получение защищенного свойства
8
-     * @param type $o
9
-     * @param type $name
8
+     * @param string $name
9
+     * @param SimpleORM\AbstractDataMapper $obj
10 10
      * @return type
11 11
      */
12 12
     static public function getProtectedAttribute($obj, $name)
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
     /**
21 21
      * Добавление значения в защищенное свойтсво
22 22
      * @param type $name
23
-     * @param type $valued 
23
+     * @param type $value 
24 24
      */
25 25
     private function setValueprotectedProperty($name, $value)
26 26
     {
@@ -30,6 +30,10 @@  discard block
 block discarded – undo
30 30
         $r->setValue($this->object, $value); //изменяем значение
31 31
     }    
32 32
 	
33
+	/**
34
+	 * @param SimpleORM\AbstractDataMapper $obj
35
+	 * @param string $name
36
+	 */
33 37
 	static public function callMethod($obj, $name, array $args) {
34 38
         $class = new \ReflectionClass($obj);
35 39
         $method = $class->getMethod($name);
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,7 +1,7 @@
 block discarded – undo
1 1
 <?php
2 2
 $loader = require_once __DIR__.'/../vendor/autoload.php';
3 3
 
4
-class TestHelper{
4
+class TestHelper {
5 5
     
6 6
     /**
7 7
      * получение защищенного свойства
Please login to merge, or discard this patch.
tests/src/AbstractDataMapperTest.php 1 patch
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
 //								])//в конструктор
29 29
 						->disableOriginalConstructor()
30 30
 						//->setMethods(null)
31
-						->setMethods(['setMappingFields','createEntity'])
31
+						->setMethods(['setMappingFields', 'createEntity'])
32 32
 						//->setMethods(null) //не использовать заглушки методов иначе буде возвращать NULL
33 33
 						->getMock();
34 34
 	}
@@ -45,9 +45,9 @@  discard block
 block discarded – undo
45 45
 	/**
46 46
 	 * @covers SimpleORM\AbstractDataMapper::AddMappingField
47 47
 	 */
48
-	public function testAddMappingField_SimpleField(){
48
+	public function testAddMappingField_SimpleField() {
49 49
 		
50
-		\TestHelper::callMethod($this->object,'addMappingField',['myfield']);
50
+		\TestHelper::callMethod($this->object, 'addMappingField', ['myfield']);
51 51
 		
52 52
 		$correct = [
53 53
 			  'myfield'	=>	[
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
 		  ];
57 57
 		
58 58
 		//-----------------
59
-		$mapping_fields = \TestHelper::getProtectedAttribute($this->object,'mapping_fields');
59
+		$mapping_fields = \TestHelper::getProtectedAttribute($this->object, 'mapping_fields');
60 60
 
61 61
         $this->assertEquals($mapping_fields, $correct);		
62 62
 	}		
@@ -64,9 +64,9 @@  discard block
 block discarded – undo
64 64
 	/**
65 65
 	 * @covers SimpleORM\AbstractDataMapper::AddMappingField
66 66
 	 */
67
-	public function testAddMappingField_FieldAndAlias(){
67
+	public function testAddMappingField_FieldAndAlias() {
68 68
 		
69
-		\TestHelper::callMethod($this->object,'addMappingField',['myfield','tb_mayfield']);
69
+		\TestHelper::callMethod($this->object, 'addMappingField', ['myfield', 'tb_mayfield']);
70 70
 		
71 71
 		$correct = [
72 72
 			  'myfield'	=>	[
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
 		  ];
76 76
 		
77 77
 		//-----------------
78
-		$mapping_fields = \TestHelper::getProtectedAttribute($this->object,'mapping_fields');
78
+		$mapping_fields = \TestHelper::getProtectedAttribute($this->object, 'mapping_fields');
79 79
 
80 80
         $this->assertEquals($mapping_fields, $correct);		
81 81
 	}		
@@ -83,9 +83,9 @@  discard block
 block discarded – undo
83 83
 	/**
84 84
 	 * @covers SimpleORM\AbstractDataMapper::AddMappingField
85 85
 	 */
86
-	public function testAddMappingField_ArrayField(){
86
+	public function testAddMappingField_ArrayField() {
87 87
 		
88
-		\TestHelper::callMethod($this->object,'addMappingField',['myfield',[
88
+		\TestHelper::callMethod($this->object, 'addMappingField', ['myfield', [
89 89
 			'field'	=>	'tb_mayfield'
90 90
 		]]);
91 91
 		
@@ -96,7 +96,7 @@  discard block
 block discarded – undo
96 96
 		  ];
97 97
 		
98 98
 		//-----------------
99
-		$mapping_fields = \TestHelper::getProtectedAttribute($this->object,'mapping_fields');
99
+		$mapping_fields = \TestHelper::getProtectedAttribute($this->object, 'mapping_fields');
100 100
 
101 101
         $this->assertEquals($mapping_fields, $correct);		
102 102
 	}	
@@ -104,9 +104,9 @@  discard block
 block discarded – undo
104 104
 	/**
105 105
 	 * @covers SimpleORM\AbstractDataMapper::AddMappingField
106 106
 	 */
107
-	public function testAddMappingField_PrimaryKey(){
107
+	public function testAddMappingField_PrimaryKey() {
108 108
 		
109
-		\TestHelper::callMethod($this->object,'addMappingField',['myfield',[
109
+		\TestHelper::callMethod($this->object, 'addMappingField', ['myfield', [
110 110
 			'field'		=>	'tb_mayfield',
111 111
 			'primary'	=>	true
112 112
 		]]);
@@ -119,20 +119,20 @@  discard block
 block discarded – undo
119 119
 		  ];
120 120
 		
121 121
 		//-----------------
122
-		$mapping_fields = \TestHelper::getProtectedAttribute($this->object,'mapping_fields');
122
+		$mapping_fields = \TestHelper::getProtectedAttribute($this->object, 'mapping_fields');
123 123
 
124 124
         $this->assertEquals($mapping_fields, $correct);		
125 125
 		
126
-		$key_field = \TestHelper::getProtectedAttribute($this->object,'key');
126
+		$key_field = \TestHelper::getProtectedAttribute($this->object, 'key');
127 127
 		$this->assertEquals($key_field, 'tb_mayfield');	
128 128
 	}	
129 129
 	
130 130
 	/**
131 131
 	 * @covers SimpleORM\AbstractDataMapper::AddMappingField
132 132
 	 */
133
-	public function testAddMappingField_SoftDelete(){
133
+	public function testAddMappingField_SoftDelete() {
134 134
 		
135
-		\TestHelper::callMethod($this->object,'addMappingField',['myfield',[
135
+		\TestHelper::callMethod($this->object, 'addMappingField', ['myfield', [
136 136
 			'field'			=>	'tb_mayfield',
137 137
 			'softdelete'	=>	true
138 138
 		]]);
@@ -145,11 +145,11 @@  discard block
 block discarded – undo
145 145
 		  ];
146 146
 		
147 147
 		//-----------------
148
-		$mapping_fields = \TestHelper::getProtectedAttribute($this->object,'mapping_fields');
148
+		$mapping_fields = \TestHelper::getProtectedAttribute($this->object, 'mapping_fields');
149 149
 
150 150
         $this->assertEquals($mapping_fields, $correct);		
151 151
 		
152
-		$soft_delete_key = \TestHelper::getProtectedAttribute($this->object,'soft_delete_key');
152
+		$soft_delete_key = \TestHelper::getProtectedAttribute($this->object, 'soft_delete_key');
153 153
 		$this->assertEquals($soft_delete_key, 'tb_mayfield');	
154 154
 	}	
155 155
 	
Please login to merge, or discard this patch.
example/Domain/UserGroup/UserGroup.php 2 patches
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -18,6 +18,10 @@
 block discarded – undo
18 18
 	
19 19
 	protected $title;
20 20
 	
21
+	/**
22
+	 * @param string $code
23
+	 * @param string $title
24
+	 */
21 25
 	function __construct($code,$title){
22 26
 		$this->setCode($code);
23 27
 		$this->setTitle($title);
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@
 block discarded – undo
18 18
 	
19 19
 	protected $title;
20 20
 	
21
-	function __construct($code,$title){
21
+	function __construct($code, $title) {
22 22
 		$this->setCode($code);
23 23
 		$this->setTitle($title);
24 24
 	}
Please login to merge, or discard this patch.
src/AbstractDataMapper.php 2 patches
Doc Comments   +4 added lines, -3 removed lines patch added patch discarded remove patch
@@ -303,9 +303,9 @@  discard block
 block discarded – undo
303 303
 	/**
304 304
 	 * Подготавливаем конечный вариант Сущности
305 305
 	 * 
306
-	 * @param \Core\Infrastructure\EntityInterface $Entity
306
+	 * @param EntityInterface $Entity
307 307
 	 * @param array $row
308
-	 * @return \Core\Infrastructure\EntityInterface
308
+	 * @return EntityInterface
309 309
 	 * @throws BadMethodCallException
310 310
 	 */
311 311
 	protected function buildEntity(EntityInterface $Entity, array $row){
@@ -355,7 +355,7 @@  discard block
 block discarded – undo
355 355
 	
356 356
 	/**
357 357
 	 * из объекта формирует массив
358
-	 * @param \Core\Infrastructure\EntityInterface $Entity
358
+	 * @param EntityInterface $Entity
359 359
 	 * @return \Core\Infrastructure\EntityInterface
360 360
 	 * @throws BadMethodCallException
361 361
 	 */
@@ -400,6 +400,7 @@  discard block
 block discarded – undo
400 400
 	
401 401
 	/**
402 402
 	 * Установка поля для маппинга
403
+	 * @param string $alias
403 404
 	 */
404 405
 	protected function addMappingField($alias,$field = null){
405 406
 		
Please login to merge, or discard this patch.
Spacing   +77 added lines, -77 removed lines patch added patch discarded remove patch
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
 		
81 81
 		$this->setEntityTable($db_name);
82 82
 		
83
-		if($this->getEntityTable()=='' || $this->getPrimaryKey()==''){
83
+		if ($this->getEntityTable()=='' || $this->getPrimaryKey()=='') {
84 84
 			throw new InvalidEntityPropertyException('Свойства entityTable или key не заданы');
85 85
 		}		
86 86
 		
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
         return $this->adapter;
93 93
     }
94 94
 
95
-	public function setAdapter(QueryBuilderInterface $adapter){
95
+	public function setAdapter(QueryBuilderInterface $adapter) {
96 96
 		 $this->adapter = $adapter;
97 97
 	}
98 98
 			
@@ -105,7 +105,7 @@  discard block
 block discarded – undo
105 105
 	 * Уставнока таблицы
106 106
 	 */
107 107
 	protected function setEntityTable($db_name) {
108
-		$this->entityTable = !empty($db_name)? "$db_name.".$this->table : $this->table;
108
+		$this->entityTable = !empty($db_name) ? "$db_name.".$this->table : $this->table;
109 109
 	}	
110 110
 
111 111
 	/**
@@ -115,7 +115,7 @@  discard block
 block discarded – undo
115 115
 	 */
116 116
     public function findById($id)
117 117
     {
118
-		$Criteria = (new Specification())->setWhere($this->key , $id);
118
+		$Criteria = (new Specification())->setWhere($this->key, $id);
119 119
 		
120 120
         return $this->findBySpecification($Criteria);
121 121
     }	
@@ -136,7 +136,7 @@  discard block
 block discarded – undo
136 136
 			
137 137
 			unset($data[$this->setSoftDeleteKey()]);
138 138
 			
139
-			$this->getAdapter()->insert($this->getEntityTable(),$data);
139
+			$this->getAdapter()->insert($this->getEntityTable(), $data);
140 140
 			
141 141
 			if (!$id = $this->getAdapter()->insert_id()) {
142 142
 				return false;
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
 		//update
148 148
 		else {
149 149
 			
150
-			if(!$this->getAdapter()->update($this->getEntityTable(), $data, "{$this->getPrimaryKey()} = '{$id}'")){
150
+			if (!$this->getAdapter()->update($this->getEntityTable(), $data, "{$this->getPrimaryKey()} = '{$id}'")) {
151 151
 				return false;
152 152
 			}
153 153
 
@@ -162,13 +162,13 @@  discard block
 block discarded – undo
162 162
 	 */
163 163
 	public function save(EntityInterface $Entity)
164 164
 	{
165
-		if(method_exists($this, 'onAfterSave' )) $this->onAfterSave( $Entity );
165
+		if (method_exists($this, 'onAfterSave')) $this->onAfterSave($Entity);
166 166
 		
167
-		if(!$this->saveWithoutEvents($Entity)){
167
+		if (!$this->saveWithoutEvents($Entity)) {
168 168
 			return false;
169 169
 		}
170 170
 		
171
-		if(method_exists($this, 'onBeforeSave' )) $this->onBeforeSave( $Entity );
171
+		if (method_exists($this, 'onBeforeSave')) $this->onBeforeSave($Entity);
172 172
 
173 173
 		return true;
174 174
 	}
@@ -177,18 +177,18 @@  discard block
 block discarded – undo
177 177
 	 * Событие перед сохранением
178 178
 	 * @param \SimpleORM\EntityInterface $Entity
179 179
 	 */
180
-	public function onAfterSave(EntityInterface $Entity){
180
+	public function onAfterSave(EntityInterface $Entity) {
181 181
 		
182 182
 		$this->getAdapter()->startTransaction();
183 183
 		
184 184
 		$rel_list = $this->createListRelation();
185 185
 
186
-		foreach ($rel_list as $mapper => $obj_path){
186
+		foreach ($rel_list as $mapper => $obj_path) {
187 187
 
188
-			$get_path = str_replace('#','->get','$o = $Entity'.$obj_path.';');
189
-			$set_path = str_replace(['#','();'],['->set','($o);'],'$Entity'.$obj_path.';');
190
-			eval($get_path);//получаем объект таким образом дабы не гулять по корневому объекту
191
-			if($this->DI->get($mapper)->saveWithoutEvents($o)){
188
+			$get_path = str_replace('#', '->get', '$o = $Entity'.$obj_path.';');
189
+			$set_path = str_replace(['#', '();'], ['->set', '($o);'], '$Entity'.$obj_path.';');
190
+			eval($get_path); //получаем объект таким образом дабы не гулять по корневому объекту
191
+			if ($this->DI->get($mapper)->saveWithoutEvents($o)) {
192 192
 				eval($set_path);
193 193
 			}
194 194
 			unset($o);
@@ -200,7 +200,7 @@  discard block
 block discarded – undo
200 200
 	 * После успешного сохранения
201 201
 	 * @param \SimpleORM\EntityInterface $Entity
202 202
 	 */
203
-	protected function onBeforeSave(EntityInterface $Entity){
203
+	protected function onBeforeSave(EntityInterface $Entity) {
204 204
 		
205 205
 		$this->getAdapter()->endTransaction();
206 206
 		
@@ -223,12 +223,12 @@  discard block
 block discarded – undo
223 223
 	 * получение мапперов в порядке их использования с учетом вложенности
224 224
 	 * @return array
225 225
 	 */
226
-	protected function createListRelation(){
226
+	protected function createListRelation() {
227 227
 		$rel_list = [];
228 228
 		
229 229
 		$rel_map = $this->getRelations();
230 230
 		
231
-		$this->createListRelationReq($rel_map,$rel_list);
231
+		$this->createListRelationReq($rel_map, $rel_list);
232 232
 		
233 233
 		return $rel_list;
234 234
 	}
@@ -238,17 +238,17 @@  discard block
 block discarded – undo
238 238
 	 * @param array $rel_map
239 239
 	 * @param type $rel_list
240 240
 	 */
241
-	protected function createListRelationReq(array $rel_map,&$rel_list,$obj_parent_link = null) {
241
+	protected function createListRelationReq(array $rel_map, &$rel_list, $obj_parent_link = null) {
242 242
 		
243
-		foreach ($rel_map as $rel){
243
+		foreach ($rel_map as $rel) {
244 244
 			
245 245
 			$obj_link = $parent.'#'.$rel['alias'].'()';
246 246
 			
247
-			if(count($rel['relations'])>0){
248
-				$this->createListRelationReq($rel['relations'],$rel_list,$parent.'->get'.$rel['alias'].'()');
249
-				$rel_list [$rel['name']]= $obj_parent_link.$obj_link;
247
+			if (count($rel['relations']) > 0) {
248
+				$this->createListRelationReq($rel['relations'], $rel_list, $parent.'->get'.$rel['alias'].'()');
249
+				$rel_list [$rel['name']] = $obj_parent_link.$obj_link;
250 250
 			}
251
-			else{
251
+			else {
252 252
 				$rel_list [$rel['name']] = $obj_parent_link.$obj_link;
253 253
 			}
254 254
 		}
@@ -263,9 +263,9 @@  discard block
 block discarded – undo
263 263
 		foreach ($this->relations as $alias => $cfg) {
264 264
 			$mapper = $cfg['mapper'];
265 265
 			//если связь один к одному то удаляем сущность
266
-			if($cgg['reltype'] == 'has_one'){
266
+			if ($cgg['reltype']=='has_one') {
267 267
 				$Entity = $Entity->{'get'.$alias}();
268
-				if(!$mapper->delete($Entity)){
268
+				if (!$mapper->delete($Entity)) {
269 269
 					throw new \Autoprice\Exceptions\EntityNotDeleteException('Unable to delete Entity!');
270 270
 				}
271 271
 			}
@@ -279,12 +279,12 @@  discard block
 block discarded – undo
279 279
 	/**
280 280
 	 * получить связи
281 281
 	 */
282
-	protected function getRelations(){
282
+	protected function getRelations() {
283 283
 		$rel_map = [];
284
-		foreach ($this->mapping_fields as $field => $cfg){
285
-			if(isset($cfg['relation'])){		
284
+		foreach ($this->mapping_fields as $field => $cfg) {
285
+			if (isset($cfg['relation'])) {		
286 286
 				$rels = $this->DI->get($cfg['relation'])->getRelations();
287
-				$rel_map []= [
287
+				$rel_map [] = [
288 288
 					'name'		=>	$cfg['relation'],
289 289
 					'alias'		=>	$field,
290 290
 					'relations'	=>	$rels
@@ -308,43 +308,43 @@  discard block
 block discarded – undo
308 308
 	 * @return \Core\Infrastructure\EntityInterface
309 309
 	 * @throws BadMethodCallException
310 310
 	 */
311
-	protected function buildEntity(EntityInterface $Entity, array $row){
311
+	protected function buildEntity(EntityInterface $Entity, array $row) {
312 312
 		
313
-        foreach ($this->mapping_fields as $alias => $cfg ) {
313
+        foreach ($this->mapping_fields as $alias => $cfg) {
314 314
 			
315 315
 			$value = false;
316 316
 			
317 317
 			$field = $cfg['field'];
318 318
 			
319
-			$method_set = 'set' . ucfirst($alias);
319
+			$method_set = 'set'.ucfirst($alias);
320 320
 			
321
-			if(!method_exists($Entity, $method_set )){
321
+			if (!method_exists($Entity, $method_set)) {
322 322
 				throw new BadMethodCallException("Метод {$method_set}  не определен");
323 323
 			}			
324 324
 			
325 325
 			//событие на формирование поля
326
-			if( isset($cfg['build']) && is_object($cfg['build']) ){
326
+			if (isset($cfg['build']) && is_object($cfg['build'])) {
327 327
 				$value = call_user_func($cfg['build'], $row);
328 328
 			}
329 329
 			//на связь
330
-			elseif(isset($cfg['relation'])){
330
+			elseif (isset($cfg['relation'])) {
331 331
 				
332 332
 				$mapper = $this->DI->get($cfg['relation']);
333 333
 				
334
-				if($this->use_joins===true || empty($row[$field])){
334
+				if ($this->use_joins===true || empty($row[$field])) {
335 335
 					$value = $mapper->createEntity($row);
336 336
 				}
337
-				else{
338
-					$fkey = isset($cfg['on']) ? $cfg['on'] :$mapper->key;
337
+				else {
338
+					$fkey = isset($cfg['on']) ? $cfg['on'] : $mapper->key;
339 339
 					$value = $mapper->findBySpecification((new Specification)->setWhere($fkey, $row[$field]));
340 340
 				}				
341 341
 				
342 342
 			}
343
-			elseif(is_string($field) && isset($row[strtolower($field)])){
343
+			elseif (is_string($field) && isset($row[strtolower($field)])) {
344 344
 				$value = $row[strtolower($field)];
345 345
 			}
346 346
 			
347
-			if($value!==false)
347
+			if ($value!==false)
348 348
 				$Entity->{$method_set}($value);
349 349
 			
350 350
         }
@@ -359,27 +359,27 @@  discard block
 block discarded – undo
359 359
 	 * @return \Core\Infrastructure\EntityInterface
360 360
 	 * @throws BadMethodCallException
361 361
 	 */
362
-	protected function unbuildEntity(EntityInterface $Entity){
362
+	protected function unbuildEntity(EntityInterface $Entity) {
363 363
 		
364 364
 		$row = [];
365 365
 
366
-        foreach ($this->mapping_fields as $alias => $cfg ) {
366
+        foreach ($this->mapping_fields as $alias => $cfg) {
367 367
 			
368 368
 			$field = $cfg['field'];
369 369
 			
370
-			$method_get = 'get' . ucfirst($alias);
370
+			$method_get = 'get'.ucfirst($alias);
371 371
 			
372
-			if(!method_exists($Entity, $method_get )){
372
+			if (!method_exists($Entity, $method_get)) {
373 373
 				throw new BadMethodCallException("Метод {$method_get}  не определен");
374 374
 			}		
375 375
 			
376 376
 			//--------------------------------------------------------------------
377
-			if( isset($cfg['unbuild']) && is_object($cfg['unbuild']) ){
378
-				$value = call_user_func($cfg['unbuild'], $Entity->{$method_get}() );
377
+			if (isset($cfg['unbuild']) && is_object($cfg['unbuild'])) {
378
+				$value = call_user_func($cfg['unbuild'], $Entity->{$method_get}());
379 379
 			}
380
-			elseif(isset($cfg['relation'])){
380
+			elseif (isset($cfg['relation'])) {
381 381
 				
382
-				if(isset($cfg['on']))
382
+				if (isset($cfg['on']))
383 383
 					$fkey = $this->DI->get($cfg['relation'])->getFieldAlias($cfg['on']);
384 384
 				else
385 385
 					$fkey = 'id';
@@ -387,7 +387,7 @@  discard block
 block discarded – undo
387 387
 				$value = $Entity->{$method_get}()->{'get'.$fkey}();
388 388
 				
389 389
 			}			
390
-			else{
390
+			else {
391 391
 				$value = $Entity->{$method_get}();
392 392
 			}			
393 393
 						
@@ -401,22 +401,22 @@  discard block
 block discarded – undo
401 401
 	/**
402 402
 	 * Установка поля для маппинга
403 403
 	 */
404
-	protected function addMappingField($alias,$field = null){
404
+	protected function addMappingField($alias, $field = null) {
405 405
 		
406
-		if(is_string($field)){
406
+		if (is_string($field)) {
407 407
 			$field = ['field'	=>	$field];
408 408
 		}
409
-		elseif( (is_array($field) && !isset($field['field'])) || empty($field)){
410
-			$field['field']	= $alias;
409
+		elseif ((is_array($field) && !isset($field['field'])) || empty($field)) {
410
+			$field['field'] = $alias;
411 411
 		}
412 412
 	
413 413
 		$this->mapping_fields[$alias] = $field;
414 414
 
415
-		if(isset($field['primary']) && $field['primary']===true){
415
+		if (isset($field['primary']) && $field['primary']===true) {
416 416
 			$this->key = $field['field'];
417 417
 		}
418 418
 		
419
-		if(isset($field['softdelete']) && $field['softdelete']===true){
419
+		if (isset($field['softdelete']) && $field['softdelete']===true) {
420 420
 			$this->soft_delete_key = $field['field'];
421 421
 		}
422 422
 		
@@ -443,7 +443,7 @@  discard block
 block discarded – undo
443 443
 
444 444
 
445 445
 	
446
-	public function getFieldAlias($field){
446
+	public function getFieldAlias($field) {
447 447
 		
448 448
 		return $this->mapping_fields_aliases[$field];
449 449
 		
@@ -455,7 +455,7 @@  discard block
 block discarded – undo
455 455
 	 * @param ISpecificationCriteria $specification
456 456
 	 * @return type
457 457
 	 */
458
-	public function findBySpecification(ISpecificationCriteria $specification){
458
+	public function findBySpecification(ISpecificationCriteria $specification) {
459 459
 
460 460
 		//псеводо удаление
461 461
 		$this->setSoftDelete($specification);
@@ -465,7 +465,7 @@  discard block
 block discarded – undo
465 465
 		$specification->setLimit(1);
466 466
 		
467 467
 		//получение записей
468
-		$res = $this->getAdapter()->getResultQuery($this->getEntityTable(),$specification);
468
+		$res = $this->getAdapter()->getResultQuery($this->getEntityTable(), $specification);
469 469
 
470 470
         if (!$row = $res->row_array()) {
471 471
             return null;
@@ -486,13 +486,13 @@  discard block
 block discarded – undo
486 486
 		
487 487
 		if (
488 488
 				$delete_key > '' && 
489
-				$Entity->getId() > 0){
490
-				$result = $this->getAdapter()->update($this->getEntityTable(), [ $delete_key => 1 ], "{$this->getPrimaryKey()} = '{$Entity->getId()}'");
489
+				$Entity->getId() > 0) {
490
+				$result = $this->getAdapter()->update($this->getEntityTable(), [$delete_key => 1], "{$this->getPrimaryKey()} = '{$Entity->getId()}'");
491 491
 		}
492
-		elseif($Entity->getId() > 0){
492
+		elseif ($Entity->getId() > 0) {
493 493
 			
494
-			if($result = $this->getAdapter()->delete($this->getEntityTable(), $this->getPrimaryKey()."  = ".$Entity->getId())){
495
-				if(method_exists($this, 'onBeforeDelete' )){ $result = $this->onBeforeDelete( $Entity );}
494
+			if ($result = $this->getAdapter()->delete($this->getEntityTable(), $this->getPrimaryKey()."  = ".$Entity->getId())) {
495
+				if (method_exists($this, 'onBeforeDelete')) { $result = $this->onBeforeDelete($Entity); }
496 496
 			}
497 497
 		}
498 498
 		
@@ -509,13 +509,13 @@  discard block
 block discarded – undo
509 509
 		
510 510
 		$this->setRelations($specification);
511 511
 		
512
-		$res = $this->getAdapter()->getResultQuery($this->getEntityTable(),$specification);
512
+		$res = $this->getAdapter()->getResultQuery($this->getEntityTable(), $specification);
513 513
 		
514 514
 		if (!$rows = $res->result_array()) {
515 515
             return null;
516 516
         }	
517 517
 		
518
-		foreach($rows as $k =>  $row){
518
+		foreach ($rows as $k =>  $row) {
519 519
 			$rows[$k] = $this->createEntity($row);
520 520
 		}
521 521
 		
@@ -531,13 +531,13 @@  discard block
 block discarded – undo
531 531
 	 * Выборка удаленных моделей
532 532
 	 * @param ISpecificationCriteria $specification
533 533
 	 */
534
-	private function setSoftDelete(ISpecificationCriteria $specification){
535
-		if(
536
-				$this->use_delete === false &&
537
-				$this->setSoftDeleteKey()>'' 
534
+	private function setSoftDelete(ISpecificationCriteria $specification) {
535
+		if (
536
+				$this->use_delete===false &&
537
+				$this->setSoftDeleteKey() > '' 
538 538
 				&& !isset($specification->getWhere()[$this->setSoftDeleteKey()])
539 539
 				)
540
-		$specification->setWhere($this->setSoftDeleteKey(),0);
540
+		$specification->setWhere($this->setSoftDeleteKey(), 0);
541 541
 	}
542 542
 	
543 543
 	/**
@@ -548,12 +548,12 @@  discard block
 block discarded – undo
548 548
 	 * belongs_to - многие к многим (пользователь имет множество оплат одного заказа)
549 549
 	 * has_one - один к одному
550 550
 	 */
551
-	protected function setRelations(ISpecificationCriteria $Specification){
551
+	protected function setRelations(ISpecificationCriteria $Specification) {
552 552
 
553 553
 		$joins = [];
554 554
 
555
-		foreach ($this->mapping_fields as $field => $cfg){
556
-			if(isset($cfg['relation'])){
555
+		foreach ($this->mapping_fields as $field => $cfg) {
556
+			if (isset($cfg['relation'])) {
557 557
 				
558 558
 				$this->relations[$field] = [
559 559
 					'mapper'	=>	$mapper = $this->DI->get($cfg['relation']),
@@ -562,7 +562,7 @@  discard block
 block discarded – undo
562 562
 
563 563
 				$table = $mapper->getEntityTable();
564 564
 
565
-				$relation_key = isset($cfg['on'])? $cfg['on'] : $mapper->key;
565
+				$relation_key = isset($cfg['on']) ? $cfg['on'] : $mapper->key;
566 566
 				
567 567
 				$joins[$table] = [
568 568
 						'alias'	=> $field,
@@ -573,7 +573,7 @@  discard block
 block discarded – undo
573 573
 			}
574 574
 		}	
575 575
 
576
-		if($this->use_joins===true){
576
+		if ($this->use_joins===true) {
577 577
 			$Specification->setJoins($joins);
578 578
 		}			
579 579
 	}
@@ -588,7 +588,7 @@  discard block
 block discarded – undo
588 588
 		return $o;
589 589
 	}
590 590
 	
591
-	public function withDelete(){
591
+	public function withDelete() {
592 592
 		$o = clone $this;
593 593
 		$o->use_delete = true;
594 594
 		return $o;
Please login to merge, or discard this patch.