Completed
Pull Request — master (#16)
by James
03:36
created
src/Axolotl/Collection.php 1 patch
Spacing   +140 added lines, -140 removed lines patch added patch discarded remove patch
@@ -42,19 +42,19 @@  discard block
 block discarded – undo
42 42
 	 * @param string $type
43 43
 	 * @param array  $elements
44 44
 	 */
45
-	public function __construct( $type, array $elements = array() ) {
46
-		$this->type = new Type( $type );
45
+	public function __construct($type, array $elements = array()) {
46
+		$this->type = new Type($type);
47 47
 
48
-		if ( $this->type->is_model() ) {
49
-			foreach ( $elements as $idx => $element ) {
50
-				if ( is_array( $element ) ) {
51
-					$elements[ $idx ] = $this->type->create_model( $element );
48
+		if ($this->type->is_model()) {
49
+			foreach ($elements as $idx => $element) {
50
+				if (is_array($element)) {
51
+					$elements[$idx] = $this->type->create_model($element);
52 52
 				}
53 53
 			}
54 54
 		}
55 55
 
56
-		if ( $elements ) {
57
-			$this->type->validate_elements( $elements );
56
+		if ($elements) {
57
+			$this->type->validate_elements($elements);
58 58
 		}
59 59
 
60 60
 		$this->elements = $elements;
@@ -78,18 +78,18 @@  discard block
 block discarded – undo
78 78
 	 *
79 79
 	 * @throws InvalidArgumentException
80 80
 	 */
81
-	public function add( $element ) {
82
-		if ( $this->type->is_model() && is_array( $element ) ) {
83
-			$element = $this->type->create_model( $element );
81
+	public function add($element) {
82
+		if ($this->type->is_model() && is_array($element)) {
83
+			$element = $this->type->create_model($element);
84 84
 		}
85 85
 
86
-		$this->type->validate_element( $element );
86
+		$this->type->validate_element($element);
87 87
 
88 88
 		$elements   = $this->elements;
89 89
 		$elements[] = $element;
90 90
 
91
-		$collection = new static( $this->get_type() );
92
-		$collection->set_from_trusted( $elements );
91
+		$collection = new static($this->get_type());
92
+		$collection->set_from_trusted($elements);
93 93
 
94 94
 		return $collection;
95 95
 	}
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
 	 * @return Collection
101 101
 	 */
102 102
 	public function clear() {
103
-		return new static( $this->get_type() );
103
+		return new static($this->get_type());
104 104
 	}
105 105
 
106 106
 	/**
@@ -110,8 +110,8 @@  discard block
 block discarded – undo
110 110
 	 *
111 111
 	 * @return bool
112 112
 	 */
113
-	public function contains( $condition ) {
114
-		return (bool) $this->find( $condition );
113
+	public function contains($condition) {
114
+		return (bool) $this->find($condition);
115 115
 	}
116 116
 
117 117
 	/**
@@ -121,10 +121,10 @@  discard block
 block discarded – undo
121 121
 	 *
122 122
 	 * @return mixed
123 123
 	 */
124
-	public function find( $condition ) {
125
-		$index = $this->find_index( $condition );
124
+	public function find($condition) {
125
+		$index = $this->find_index($condition);
126 126
 
127
-		return -1 === $index ? false : $this->elements[ $index ];
127
+		return -1 === $index ? false : $this->elements[$index];
128 128
 	}
129 129
 
130 130
 	/**
@@ -134,11 +134,11 @@  discard block
 block discarded – undo
134 134
 	 *
135 135
 	 * @return int
136 136
 	 */
137
-	public function find_index( $condition ) {
137
+	public function find_index($condition) {
138 138
 		$index = -1;
139 139
 
140
-		for ( $i = 0, $count = count( $this->elements ); $i < $count; $i++ ) {
141
-			if ( call_user_func( $condition, ($this->at( $i ) ) ) ) {
140
+		for ($i = 0, $count = count($this->elements); $i < $count; $i++) {
141
+			if (call_user_func($condition, ($this->at($i)))) {
142 142
 				$index = $i;
143 143
 				break;
144 144
 			}
@@ -156,10 +156,10 @@  discard block
 block discarded – undo
156 156
 	 *
157 157
 	 * @throws OutOfRangeException
158 158
 	 */
159
-	public function at( $index ) {
160
-		$this->validate_index( $index );
159
+	public function at($index) {
160
+		$this->validate_index($index);
161 161
 
162
-		return $this->elements[ $index ];
162
+		return $this->elements[$index];
163 163
 	}
164 164
 
165 165
 	/**
@@ -171,13 +171,13 @@  discard block
 block discarded – undo
171 171
 	 *
172 172
 	 * @throws InvalidArgumentException
173 173
 	 */
174
-	public function index_exists( $index ) {
175
-		if ( ! is_int( $index ) ) {
176
-			throw new InvalidArgumentException( 'Index must be an integer' );
174
+	public function index_exists($index) {
175
+		if (!is_int($index)) {
176
+			throw new InvalidArgumentException('Index must be an integer');
177 177
 		}
178 178
 
179
-		if ( $index < 0 ) {
180
-			throw new InvalidArgumentException( 'Index must be a non-negative integer' );
179
+		if ($index < 0) {
180
+			throw new InvalidArgumentException('Index must be a non-negative integer');
181 181
 		}
182 182
 
183 183
 		return $index < $this->count();
@@ -190,16 +190,16 @@  discard block
 block discarded – undo
190 190
 	 *
191 191
 	 * @return mixed
192 192
 	 */
193
-	public function filter( $condition ) {
193
+	public function filter($condition) {
194 194
 		$elements = array();
195 195
 
196
-		foreach ( $this->elements as $element ) {
197
-			if ( call_user_func( $condition, $element ) ) {
196
+		foreach ($this->elements as $element) {
197
+			if (call_user_func($condition, $element)) {
198 198
 				$elements[] = $element;
199 199
 			}
200 200
 		}
201 201
 
202
-		return $this->new_from_trusted( $elements );
202
+		return $this->new_from_trusted($elements);
203 203
 	}
204 204
 	/**
205 205
 	 * {@inheritdoc}
@@ -208,10 +208,10 @@  discard block
 block discarded – undo
208 208
 	 *
209 209
 	 * @return mixed
210 210
 	 */
211
-	public function find_last( $condition ) {
212
-		$index = $this->find_last_index( $condition );
211
+	public function find_last($condition) {
212
+		$index = $this->find_last_index($condition);
213 213
 
214
-		return -1 === $index ? null : $this->elements[ $index ];
214
+		return -1 === $index ? null : $this->elements[$index];
215 215
 	}
216 216
 
217 217
 	/**
@@ -220,11 +220,11 @@  discard block
 block discarded – undo
220 220
 	 * @param  callable $condition
221 221
 	 * @return int
222 222
 	 */
223
-	public function find_last_index( $condition ) {
223
+	public function find_last_index($condition) {
224 224
 		$index = -1;
225 225
 
226
-		for ( $i = count( $this->elements ) - 1; $i >= 0; $i-- ) {
227
-			if ( call_user_func( $condition, $this->elements[ $i ] ) ) {
226
+		for ($i = count($this->elements) - 1; $i >= 0; $i--) {
227
+			if (call_user_func($condition, $this->elements[$i])) {
228 228
 				$index = $i;
229 229
 				break;
230 230
 			}
@@ -243,26 +243,26 @@  discard block
 block discarded – undo
243 243
 	 *
244 244
 	 * @throws InvalidArgumentException
245 245
 	 */
246
-	public function slice( $start, $end ) {
247
-		if ( $start < 0 || ! is_int( $start ) ) {
248
-			throw new InvalidArgumentException( 'Start must be a non-negative integer' );
246
+	public function slice($start, $end) {
247
+		if ($start < 0 || !is_int($start)) {
248
+			throw new InvalidArgumentException('Start must be a non-negative integer');
249 249
 		}
250 250
 
251
-		if ( $end < 0 || ! is_int( $end ) ) {
252
-			throw new InvalidArgumentException( 'End must be a positive integer' );
251
+		if ($end < 0 || !is_int($end)) {
252
+			throw new InvalidArgumentException('End must be a positive integer');
253 253
 		}
254 254
 
255
-		if ( $start > $end ) {
256
-			throw new InvalidArgumentException( 'End must be greater than start' );
255
+		if ($start > $end) {
256
+			throw new InvalidArgumentException('End must be greater than start');
257 257
 		}
258 258
 
259
-		if ( $end > $this->count() + 1 ) {
260
-			throw new InvalidArgumentException( 'End must be less than the count of the items in the Collection' );
259
+		if ($end > $this->count() + 1) {
260
+			throw new InvalidArgumentException('End must be less than the count of the items in the Collection');
261 261
 		}
262 262
 
263 263
 		$length = $end - $start + 1;
264 264
 
265
-		return $this->new_from_trusted( array_slice( $this->elements, $start, $length ) );
265
+		return $this->new_from_trusted(array_slice($this->elements, $start, $length));
266 266
 	}
267 267
 
268 268
 	/**
@@ -276,16 +276,16 @@  discard block
 block discarded – undo
276 276
 	 * @throws InvalidArgumentException
277 277
 	 * @throws OutOfRangeException
278 278
 	 */
279
-	public function insert( $index, $element ) {
280
-		$this->validate_index( $index );
281
-		$this->type->validate_element( $element );
279
+	public function insert($index, $element) {
280
+		$this->validate_index($index);
281
+		$this->type->validate_element($element);
282 282
 
283
-		$a = array_slice( $this->elements, 0, $index );
284
-		$b = array_slice( $this->elements, $index, count( $this->elements ) );
283
+		$a = array_slice($this->elements, 0, $index);
284
+		$b = array_slice($this->elements, $index, count($this->elements));
285 285
 
286 286
 		$a[] = $element;
287 287
 
288
-		return $this->new_from_trusted( array_merge( $a, $b ) );
288
+		return $this->new_from_trusted(array_merge($a, $b));
289 289
 	}
290 290
 
291 291
 	/**
@@ -298,19 +298,19 @@  discard block
 block discarded – undo
298 298
 	 *
299 299
 	 * @throws OutOfRangeException
300 300
 	 */
301
-	public function insert_range( $index, array $elements ) {
302
-		$this->validate_index( $index );
303
-		$this->type->validate_elements( $elements );
301
+	public function insert_range($index, array $elements) {
302
+		$this->validate_index($index);
303
+		$this->type->validate_elements($elements);
304 304
 
305
-		if ( $index < 0 ) {
305
+		if ($index < 0) {
306 306
 			$index = $this->count() + $index + 1;
307 307
 		}
308 308
 
309 309
 		return $this->new_from_trusted(
310 310
 			array_merge(
311
-				array_slice( $this->elements, 0, $index ),
311
+				array_slice($this->elements, 0, $index),
312 312
 				$elements,
313
-				array_slice( $this->elements, $index, count( $this->elements ) )
313
+				array_slice($this->elements, $index, count($this->elements))
314 314
 			)
315 315
 		);
316 316
 	}
@@ -322,12 +322,12 @@  discard block
 block discarded – undo
322 322
 	 *
323 323
 	 * @return Collection
324 324
 	 */
325
-	public function without( $condition ) {
326
-		$inverse = function ( $element ) use ( $condition ) {
327
-			return ! call_user_func( $condition, $element );
325
+	public function without($condition) {
326
+		$inverse = function($element) use ($condition) {
327
+			return !call_user_func($condition, $element);
328 328
 		};
329 329
 
330
-		return $this->filter( $inverse );
330
+		return $this->filter($inverse);
331 331
 	}
332 332
 
333 333
 	/**
@@ -339,15 +339,15 @@  discard block
 block discarded – undo
339 339
 	 *
340 340
 	 * @throws OutOfRangeException
341 341
 	 */
342
-	public function remove_at( $index ) {
343
-		$this->validate_index( $index );
342
+	public function remove_at($index) {
343
+		$this->validate_index($index);
344 344
 
345 345
 		$elements = $this->elements;
346 346
 
347 347
 		return $this->new_from_trusted(
348 348
 			array_merge(
349
-				array_slice( $elements, 0, $index ),
350
-				array_slice( $elements, $index + 1, count( $elements ) )
349
+				array_slice($elements, 0, $index),
350
+				array_slice($elements, $index + 1, count($elements))
351 351
 			)
352 352
 		);
353 353
 	}
@@ -358,7 +358,7 @@  discard block
 block discarded – undo
358 358
 	 */
359 359
 	public function reverse() {
360 360
 		return $this->new_from_trusted(
361
-			array_reverse( $this->elements )
361
+			array_reverse($this->elements)
362 362
 		);
363 363
 	}
364 364
 
@@ -369,10 +369,10 @@  discard block
 block discarded – undo
369 369
 	 *
370 370
 	 * @return Collection
371 371
 	 */
372
-	public function sort( $callback ) {
372
+	public function sort($callback) {
373 373
 		$elements = $this->elements;
374
-		usort( $elements, $callback );
375
-		return $this->new_from_trusted( $elements );
374
+		usort($elements, $callback);
375
+		return $this->new_from_trusted($elements);
376 376
 	}
377 377
 
378 378
 	/**
@@ -393,8 +393,8 @@  discard block
 block discarded – undo
393 393
 	 *
394 394
 	 * @return mixed
395 395
 	 */
396
-	public function reduce( $callable, $initial = null ) {
397
-		return array_reduce( $this->elements, $callable, $initial );
396
+	public function reduce($callable, $initial = null) {
397
+		return array_reduce($this->elements, $callable, $initial);
398 398
 	}
399 399
 
400 400
 	/**
@@ -404,13 +404,13 @@  discard block
 block discarded – undo
404 404
 	 *
405 405
 	 * @return bool
406 406
 	 */
407
-	public function every( $condition ) {
407
+	public function every($condition) {
408 408
 		$response = true;
409 409
 
410
-		foreach ( $this->elements as $element ) {
411
-			$result = call_user_func( $condition, $element );
410
+		foreach ($this->elements as $element) {
411
+			$result = call_user_func($condition, $element);
412 412
 
413
-			if ( false === $result ) {
413
+			if (false === $result) {
414 414
 				$response = false;
415 415
 				break;
416 416
 			}
@@ -428,8 +428,8 @@  discard block
 block discarded – undo
428 428
 	 *
429 429
 	 * @throws InvalidArgumentException
430 430
 	 */
431
-	public function drop( $num ) {
432
-		return $this->slice( $num, $this->count() );
431
+	public function drop($num) {
432
+		return $this->slice($num, $this->count());
433 433
 	}
434 434
 
435 435
 	/**
@@ -441,9 +441,9 @@  discard block
 block discarded – undo
441 441
 	 *
442 442
 	 * @throws InvalidArgumentException
443 443
 	 */
444
-	public function drop_right( $num ) {
444
+	public function drop_right($num) {
445 445
 		return $num !== $this->count()
446
-			? $this->slice( 0, $this->count() - $num - 1 )
446
+			? $this->slice(0, $this->count() - $num - 1)
447 447
 			: $this->clear();
448 448
 	}
449 449
 
@@ -454,9 +454,9 @@  discard block
 block discarded – undo
454 454
 	 *
455 455
 	 * @return Collection
456 456
 	 */
457
-	public function drop_while( $condition ) {
458
-		$count = $this->count_while_true( $condition );
459
-		return $count ? $this->drop( $count ) : $this;
457
+	public function drop_while($condition) {
458
+		$count = $this->count_while_true($condition);
459
+		return $count ? $this->drop($count) : $this;
460 460
 	}
461 461
 	/**
462 462
 	 * {@inheritdoc}
@@ -466,7 +466,7 @@  discard block
 block discarded – undo
466 466
 	 * @throws InvalidArgumentException
467 467
 	 */
468 468
 	public function tail() {
469
-		return $this->slice( 1, $this->count() );
469
+		return $this->slice(1, $this->count());
470 470
 	}
471 471
 
472 472
 	/**
@@ -478,8 +478,8 @@  discard block
 block discarded – undo
478 478
 	 *
479 479
 	 * @throws InvalidArgumentException
480 480
 	 */
481
-	public function take( $num ) {
482
-		return $this->slice( 0, $num - 1 );
481
+	public function take($num) {
482
+		return $this->slice(0, $num - 1);
483 483
 	}
484 484
 
485 485
 	/**
@@ -491,8 +491,8 @@  discard block
 block discarded – undo
491 491
 	 *
492 492
 	 * @throws InvalidArgumentException
493 493
 	 */
494
-	public function take_right( $num ) {
495
-		return $this->slice( $this->count() - $num, $this->count() );
494
+	public function take_right($num) {
495
+		return $this->slice($this->count() - $num, $this->count());
496 496
 	}
497 497
 
498 498
 	/**
@@ -502,10 +502,10 @@  discard block
 block discarded – undo
502 502
 	 *
503 503
 	 * @return Collection
504 504
 	 */
505
-	public function take_while( $condition ) {
506
-		$count = $this->count_while_true( $condition );
505
+	public function take_while($condition) {
506
+		$count = $this->count_while_true($condition);
507 507
 
508
-		return $count ? $this->take( $count ) : $this->clear();
508
+		return $count ? $this->take($count) : $this->clear();
509 509
 	}
510 510
 
511 511
 	/**
@@ -513,9 +513,9 @@  discard block
 block discarded – undo
513 513
 	 *
514 514
 	 * @param callable $callable Callback function.
515 515
 	 */
516
-	public function each( $callable ) {
517
-		foreach ( $this->elements as $element ) {
518
-			call_user_func( $callable, $element );
516
+	public function each($callable) {
517
+		foreach ($this->elements as $element) {
518
+			call_user_func($callable, $element);
519 519
 		}
520 520
 	}
521 521
 
@@ -526,24 +526,24 @@  discard block
 block discarded – undo
526 526
 	 *
527 527
 	 * @return Collection
528 528
 	 */
529
-	public function map( $callable ) {
529
+	public function map($callable) {
530 530
 		$elements = array();
531 531
 		$type = null;
532
-		foreach ( $this->elements as $element ) {
533
-			$result = call_user_func( $callable, $element );
532
+		foreach ($this->elements as $element) {
533
+			$result = call_user_func($callable, $element);
534 534
 
535
-			if ( null === $type ) {
536
-				$type = gettype( $result );
535
+			if (null === $type) {
536
+				$type = gettype($result);
537 537
 
538
-				if ( 'object' === $type ) {
539
-					$type = get_class( $result );
538
+				if ('object' === $type) {
539
+					$type = get_class($result);
540 540
 				}
541 541
 			}
542 542
 
543 543
 			$elements[] = $result;
544 544
 		}
545 545
 
546
-		return $this->new_from_trusted( $elements, $type ? : $this->get_type() );
546
+		return $this->new_from_trusted($elements, $type ?: $this->get_type());
547 547
 	}
548 548
 
549 549
 	/**
@@ -554,9 +554,9 @@  discard block
 block discarded – undo
554 554
 	 *
555 555
 	 * @return mixed
556 556
 	 */
557
-	public function reduce_right( $callable, $initial = null ) {
557
+	public function reduce_right($callable, $initial = null) {
558 558
 		return array_reduce(
559
-			array_reverse( $this->elements ),
559
+			array_reverse($this->elements),
560 560
 			$callable,
561 561
 			$initial
562 562
 		);
@@ -569,9 +569,9 @@  discard block
 block discarded – undo
569 569
 	 */
570 570
 	public function shuffle() {
571 571
 		$elements = $this->elements;
572
-		shuffle( $elements );
572
+		shuffle($elements);
573 573
 
574
-		return $this->new_from_trusted( $elements );
574
+		return $this->new_from_trusted($elements);
575 575
 	}
576 576
 
577 577
 	/**
@@ -583,19 +583,19 @@  discard block
 block discarded – undo
583 583
 	 *
584 584
 	 * @throws InvalidArgumentException
585 585
 	 */
586
-	public function merge( $elements ) {
587
-		if ( $elements instanceof static ) {
586
+	public function merge($elements) {
587
+		if ($elements instanceof static) {
588 588
 			$elements = $elements->to_array();
589 589
 		}
590 590
 
591
-		if ( ! is_array( $elements ) ) {
592
-			throw new InvalidArgumentException( 'Merge must be given array or Collection' );
591
+		if (!is_array($elements)) {
592
+			throw new InvalidArgumentException('Merge must be given array or Collection');
593 593
 		}
594 594
 
595
-		$this->type->validate_elements( $elements );
595
+		$this->type->validate_elements($elements);
596 596
 
597 597
 		return $this->new_from_trusted(
598
-			array_merge( $this->elements, $elements )
598
+			array_merge($this->elements, $elements)
599 599
 		);
600 600
 	}
601 601
 
@@ -607,11 +607,11 @@  discard block
 block discarded – undo
607 607
 	 * @throws OutOfBoundsException
608 608
 	 */
609 609
 	public function first() {
610
-		if ( empty( $this->elements ) ) {
611
-			throw new OutOfBoundsException( 'Cannot get first element of empty Collection' );
610
+		if (empty($this->elements)) {
611
+			throw new OutOfBoundsException('Cannot get first element of empty Collection');
612 612
 		}
613 613
 
614
-		return reset( $this->elements );
614
+		return reset($this->elements);
615 615
 	}
616 616
 
617 617
 	/**
@@ -622,11 +622,11 @@  discard block
 block discarded – undo
622 622
 	 * @throws OutOfBoundsException
623 623
 	 */
624 624
 	public function last() {
625
-		if ( empty( $this->elements ) ) {
626
-			throw new OutOfBoundsException( 'Cannot get last element of empty Collection' );
625
+		if (empty($this->elements)) {
626
+			throw new OutOfBoundsException('Cannot get last element of empty Collection');
627 627
 		}
628 628
 
629
-		return end( $this->elements );
629
+		return end($this->elements);
630 630
 	}
631 631
 
632 632
 	/**
@@ -635,7 +635,7 @@  discard block
 block discarded – undo
635 635
 	 * @return int
636 636
 	 */
637 637
 	public function count() {
638
-		return count( $this->elements );
638
+		return count($this->elements);
639 639
 	}
640 640
 
641 641
 	/**
@@ -644,8 +644,8 @@  discard block
 block discarded – undo
644 644
 	 * @return array
645 645
 	 */
646 646
 	public function serialize() {
647
-		return $this->map(function( $element ) {
648
-			if ( $element instanceof Serializes ) {
647
+		return $this->map(function($element) {
648
+			if ($element instanceof Serializes) {
649 649
 				return $element->serialize();
650 650
 			}
651 651
 
@@ -659,14 +659,14 @@  discard block
 block discarded – undo
659 659
 	 * @return mixed
660 660
 	 */
661 661
 	public function current() {
662
-		return $this->at( $this->position );
662
+		return $this->at($this->position);
663 663
 	}
664 664
 
665 665
 	/**
666 666
 	 * Move forward to next element.
667 667
 	 */
668 668
 	public function next() {
669
-		$this->position ++;
669
+		$this->position++;
670 670
 	}
671 671
 
672 672
 	/**
@@ -684,7 +684,7 @@  discard block
 block discarded – undo
684 684
 	 * @return bool
685 685
 	 */
686 686
 	public function valid() {
687
-		return isset( $this->elements[ $this->position ] );
687
+		return isset($this->elements[$this->position]);
688 688
 	}
689 689
 
690 690
 	/**
@@ -703,9 +703,9 @@  discard block
 block discarded – undo
703 703
 	 *
704 704
 	 * @return static
705 705
 	 */
706
-	protected function new_from_trusted( array $elements, $type = null ) {
707
-		$collection = new static( null !== $type ? $type : $this->get_type() );
708
-		$collection->set_from_trusted( $elements );
706
+	protected function new_from_trusted(array $elements, $type = null) {
707
+		$collection = new static(null !== $type ? $type : $this->get_type());
708
+		$collection->set_from_trusted($elements);
709 709
 
710 710
 		return $collection;
711 711
 	}
@@ -715,7 +715,7 @@  discard block
 block discarded – undo
715 715
 	 *
716 716
 	 * @param array $elements Pre-validated elements to set.
717 717
 	 */
718
-	protected function set_from_trusted( array $elements ) {
718
+	protected function set_from_trusted(array $elements) {
719 719
 		$this->elements = $elements;
720 720
 	}
721 721
 
@@ -725,11 +725,11 @@  discard block
 block discarded – undo
725 725
 	 * @param callable $condition Condition to check.
726 726
 	 * @return int
727 727
 	 */
728
-	protected function count_while_true( $condition ) {
728
+	protected function count_while_true($condition) {
729 729
 		$count = 0;
730 730
 
731
-		foreach ( $this->elements as $element ) {
732
-			if ( ! $condition($element) ) {
731
+		foreach ($this->elements as $element) {
732
+			if (!$condition($element)) {
733 733
 				break;
734 734
 			}
735 735
 			$count++;
@@ -745,11 +745,11 @@  discard block
 block discarded – undo
745 745
 	 *
746 746
 	 * @throws OutOfRangeException
747 747
 	 */
748
-	protected function validate_index( $index ) {
749
-		$exists = $this->index_exists( $index );
748
+	protected function validate_index($index) {
749
+		$exists = $this->index_exists($index);
750 750
 
751
-		if ( ! $exists ) {
752
-			throw new OutOfRangeException( 'Index out of bounds of collection' );
751
+		if (!$exists) {
752
+			throw new OutOfRangeException('Index out of bounds of collection');
753 753
 		}
754 754
 	}
755 755
 }
Please login to merge, or discard this patch.
src/Axolotl/Type.php 1 patch
Spacing   +33 added lines, -34 removed lines patch added patch discarded remove patch
@@ -27,8 +27,8 @@  discard block
 block discarded – undo
27 27
 	 *
28 28
 	 * @param string $type
29 29
 	 */
30
-	public function __construct( $type ) {
31
-		$this->type = $this->determine( $type );
30
+	public function __construct($type) {
31
+		$this->type = $this->determine($type);
32 32
 	}
33 33
 
34 34
 	/**
@@ -46,12 +46,12 @@  discard block
 block discarded – undo
46 46
 	 * @return bool
47 47
 	 */
48 48
 	public function is_model() {
49
-		if ( ! class_exists( $this->type ) ) {
49
+		if (!class_exists($this->type)) {
50 50
 			return false;
51 51
 		}
52 52
 
53
-		$reflection = new ReflectionClass( $this->type );
54
-		return $reflection->isSubclassOf( 'Intraxia\Jaxion\Axolotl\Model' );
53
+		$reflection = new ReflectionClass($this->type);
54
+		return $reflection->isSubclassOf('Intraxia\Jaxion\Axolotl\Model');
55 55
 	}
56 56
 
57 57
 	/**
@@ -61,8 +61,8 @@  discard block
 block discarded – undo
61 61
 	 *
62 62
 	 * @return Model
63 63
 	 */
64
-	public function create_model( array $data ) {
65
-		return new $this->type( $data );
64
+	public function create_model(array $data) {
65
+		return new $this->type($data);
66 66
 	}
67 67
 
68 68
 	/**
@@ -72,9 +72,9 @@  discard block
 block discarded – undo
72 72
 	 *
73 73
 	 * @throws InvalidArgumentException
74 74
 	 */
75
-	public function validate_elements( array $elements ) {
76
-		foreach ( $elements as $element ) {
77
-			$this->validate_element( $element );
75
+	public function validate_elements(array $elements) {
76
+		foreach ($elements as $element) {
77
+			$this->validate_element($element);
78 78
 		}
79 79
 	}
80 80
 
@@ -85,25 +85,25 @@  discard block
 block discarded – undo
85 85
 	 *
86 86
 	 * @throws InvalidArgumentException
87 87
 	 */
88
-	public function validate_element( $element ) {
89
-		$type = gettype( $element );
88
+	public function validate_element($element) {
89
+		$type = gettype($element);
90 90
 		$callable = $this->type === 'callable';
91 91
 		$is_object = 'object' === $type;
92 92
 		$loose_check = $this->type === 'object';
93 93
 
94 94
 		// callable must be callable
95
-		if ( $callable && ! is_callable( $element ) ) {
96
-			throw new InvalidArgumentException( 'Item must be callable' );
95
+		if ($callable && !is_callable($element)) {
96
+			throw new InvalidArgumentException('Item must be callable');
97 97
 		}
98 98
 
99 99
 		// target isn't callable, object must be an instance of target
100
-		if ( ! $loose_check && ! $callable && $is_object && ! is_a( $element, $this->type ) ) {
101
-			throw new InvalidArgumentException( "Item is not type or subtype of $this->type" );
100
+		if (!$loose_check && !$callable && $is_object && !is_a($element, $this->type)) {
101
+			throw new InvalidArgumentException("Item is not type or subtype of $this->type");
102 102
 		}
103 103
 
104 104
 		// a non callable, non object type should match the target string
105
-		if ( ! $callable && ! $is_object && $type !== $this->type ) {
106
-			throw new InvalidArgumentException( "Item is not of type: $this->type" );
105
+		if (!$callable && !$is_object && $type !== $this->type) {
106
+			throw new InvalidArgumentException("Item is not of type: $this->type");
107 107
 		}
108 108
 	}
109 109
 
@@ -117,20 +117,20 @@  discard block
 block discarded – undo
117 117
 	 *
118 118
 	 * @throws InvalidArgumentException
119 119
 	 */
120
-	private function determine( $type, $key_type = false ) {
121
-		if ( ! $key_type && $this->non_scalar_type_exists( $type ) ) {
120
+	private function determine($type, $key_type = false) {
121
+		if (!$key_type && $this->non_scalar_type_exists($type)) {
122 122
 			return $type;
123 123
 		}
124 124
 
125
-		if ( $scalar_type = $this->determine_scalar( $type ) ) {
126
-			if ( $key_type && (in_array( $scalar_type, array( 'double', 'boolean' ) )) ) {
127
-				throw new InvalidArgumentException( 'This type is not supported as a key.' );
125
+		if ($scalar_type = $this->determine_scalar($type)) {
126
+			if ($key_type && (in_array($scalar_type, array('double', 'boolean')))) {
127
+				throw new InvalidArgumentException('This type is not supported as a key.');
128 128
 			}
129 129
 
130 130
 			return $scalar_type;
131 131
 		}
132 132
 
133
-		throw new InvalidArgumentException( 'This type does not exist.' );
133
+		throw new InvalidArgumentException('This type does not exist.');
134 134
 	}
135 135
 
136 136
 	/**
@@ -140,10 +140,10 @@  discard block
 block discarded – undo
140 140
 	 *
141 141
 	 * @return bool
142 142
 	 */
143
-	private function non_scalar_type_exists( $type ) {
144
-		return class_exists( $type )
145
-				|| interface_exists( $type )
146
-				|| in_array( $type, array( 'array', 'object', 'callable' ) );
143
+	private function non_scalar_type_exists($type) {
144
+		return class_exists($type)
145
+				|| interface_exists($type)
146
+				|| in_array($type, array('array', 'object', 'callable'));
147 147
 	}
148 148
 
149 149
 	/**
@@ -153,19 +153,18 @@  discard block
 block discarded – undo
153 153
 	 *
154 154
 	 * @return string|null
155 155
 	 */
156
-	private function determine_scalar( $type ) {
156
+	private function determine_scalar($type) {
157 157
 		$synonyms = array(
158 158
 			'int' => 'integer',
159 159
 			'float' => 'double',
160 160
 			'bool' => 'boolean',
161 161
 		);
162 162
 
163
-		if ( array_key_exists( $type, $synonyms ) ) {
164
-			$type = $synonyms[ $type ];
163
+		if (array_key_exists($type, $synonyms)) {
164
+			$type = $synonyms[$type];
165 165
 		}
166 166
 
167
-		return in_array( $type, array( 'string', 'integer', 'double', 'boolean' ) ) ?
168
-			$type :
169
-			null;
167
+		return in_array($type, array('string', 'integer', 'double', 'boolean')) ?
168
+			$type : null;
170 169
 	}
171 170
 }
Please login to merge, or discard this patch.