@@ -881,7 +881,7 @@ |
||
881 | 881 | * Parse single result row to generate data tree. Must pass parsing to evert nested loader. |
882 | 882 | * |
883 | 883 | * @param array $row |
884 | - * @return bool |
|
884 | + * @return boolean|null |
|
885 | 885 | */ |
886 | 886 | private function parseRow(array $row) |
887 | 887 | { |
@@ -280,7 +280,7 @@ discard block |
||
280 | 280 | } |
281 | 281 | |
282 | 282 | //New alias is pretty simple and short |
283 | - return $this->alias = 'd' . decoct(++self::$counter); |
|
283 | + return $this->alias = 'd'.decoct(++self::$counter); |
|
284 | 284 | } |
285 | 285 | |
286 | 286 | if (empty($this->parent)) { |
@@ -290,7 +290,7 @@ discard block |
||
290 | 290 | $alias = $this->container; |
291 | 291 | } else { |
292 | 292 | //Let's use parent alias to continue chain |
293 | - $alias = $this->parent->getAlias() . '_' . $this->container; |
|
293 | + $alias = $this->parent->getAlias().'_'.$this->container; |
|
294 | 294 | } |
295 | 295 | |
296 | 296 | return $alias; |
@@ -328,7 +328,7 @@ discard block |
||
328 | 328 | return null; |
329 | 329 | } |
330 | 330 | |
331 | - return $this->getAlias() . '.' . $this->schema[ORM::M_PRIMARY_KEY]; |
|
331 | + return $this->getAlias().'.'.$this->schema[ORM::M_PRIMARY_KEY]; |
|
332 | 332 | } |
333 | 333 | |
334 | 334 | /** |
@@ -852,7 +852,7 @@ discard block |
||
852 | 852 | return null; |
853 | 853 | } |
854 | 854 | |
855 | - return $this->getAlias() . '.' . $this->definition[$key]; |
|
855 | + return $this->getAlias().'.'.$this->definition[$key]; |
|
856 | 856 | } |
857 | 857 | |
858 | 858 | /** |
@@ -867,7 +867,7 @@ discard block |
||
867 | 867 | throw new LoaderException("Unable to get parent key, no parent loader provided."); |
868 | 868 | } |
869 | 869 | |
870 | - return $this->parent->getAlias() . '.' . $this->definition[RecordEntity::INNER_KEY]; |
|
870 | + return $this->parent->getAlias().'.'.$this->definition[RecordEntity::INNER_KEY]; |
|
871 | 871 | } |
872 | 872 | |
873 | 873 | /** |
@@ -28,7 +28,7 @@ discard block |
||
28 | 28 | * Get instance of container associated with given object, uses global container as fallback |
29 | 29 | * if not. Method generally used by traits. |
30 | 30 | * |
31 | - * @return ContainerInterface|null |
|
31 | + * @return ContainerInterface |
|
32 | 32 | */ |
33 | 33 | protected function container() |
34 | 34 | { |
@@ -50,7 +50,7 @@ discard block |
||
50 | 50 | * |
51 | 51 | * @internal Do not use for business logic. |
52 | 52 | * @param ContainerInterface $container Can be set to null. |
53 | - * @return ContainerInterface|null |
|
53 | + * @return ContainerInterface |
|
54 | 54 | */ |
55 | 55 | final protected static function staticContainer(ContainerInterface $container = null) |
56 | 56 | { |
@@ -257,7 +257,7 @@ |
||
257 | 257 | /** |
258 | 258 | * {@inheritdoc} |
259 | 259 | * |
260 | - * @return DocumentCursor|Document[] |
|
260 | + * @return DocumentCursor |
|
261 | 261 | */ |
262 | 262 | public function getIterator() |
263 | 263 | { |
@@ -151,7 +151,7 @@ discard block |
||
151 | 151 | */ |
152 | 152 | public function query(array $query = []) |
153 | 153 | { |
154 | - array_walk_recursive($query, function (&$value) { |
|
154 | + array_walk_recursive($query, function(&$value) { |
|
155 | 155 | if ($value instanceof \DateTime) { |
156 | 156 | //MongoDate is always UTC, which is good :) |
157 | 157 | $value = new \MongoDate($value->getTimestamp()); |
@@ -334,7 +334,7 @@ discard block |
||
334 | 334 | public function __debugInfo() |
335 | 335 | { |
336 | 336 | return [ |
337 | - 'collection' => $this->database . '/' . $this->name, |
|
337 | + 'collection' => $this->database.'/'.$this->name, |
|
338 | 338 | 'query' => $this->query, |
339 | 339 | 'limit' => $this->limit, |
340 | 340 | 'offset' => $this->offset, |
@@ -392,7 +392,7 @@ discard block |
||
392 | 392 | } |
393 | 393 | |
394 | 394 | $this->logger()->debug( |
395 | - "{database}/{collection}: " . json_encode($queryInfo, JSON_PRETTY_PRINT), |
|
395 | + "{database}/{collection}: ".json_encode($queryInfo, JSON_PRETTY_PRINT), |
|
396 | 396 | [ |
397 | 397 | 'collection' => $this->name, |
398 | 398 | 'database' => $this->database, |
@@ -457,7 +457,7 @@ discard block |
||
457 | 457 | * |
458 | 458 | * @see findOne() |
459 | 459 | * @param mixed $id Primary key value. |
460 | - * @return RecordEntity|null |
|
460 | + * @return null|RecordInterface |
|
461 | 461 | * @throws SelectorException |
462 | 462 | */ |
463 | 463 | public function findByPK($id) |
@@ -482,7 +482,7 @@ discard block |
||
482 | 482 | * @see findByPK() |
483 | 483 | * @param array $where Selection WHERE statement. |
484 | 484 | * @param bool $setLimit Use limit 1. |
485 | - * @return RecordEntity|null |
|
485 | + * @return null|RecordInterface |
|
486 | 486 | */ |
487 | 487 | public function findOne(array $where = [], $setLimit = true) |
488 | 488 | { |
@@ -152,8 +152,8 @@ discard block |
||
152 | 152 | { |
153 | 153 | $offset = count($this->dataColumns); |
154 | 154 | foreach ($columns as $column) { |
155 | - $columnAlias = 'c' . (++$this->countColumns); |
|
156 | - $this->dataColumns[] = $table . '.' . $column . ' AS ' . $columnAlias; |
|
155 | + $columnAlias = 'c'.(++$this->countColumns); |
|
156 | + $this->dataColumns[] = $table.'.'.$column.' AS '.$columnAlias; |
|
157 | 157 | } |
158 | 158 | |
159 | 159 | return $offset; |
@@ -529,7 +529,7 @@ discard block |
||
529 | 529 | public function count($column = self::DEFAULT_COUNTING_FIELD) |
530 | 530 | { |
531 | 531 | if ($column == self::DEFAULT_COUNTING_FIELD && !empty($this->loader->getPrimaryKey())) { |
532 | - $column = 'DISTINCT(' . $this->loader->getPrimaryKey().')'; |
|
532 | + $column = 'DISTINCT('.$this->loader->getPrimaryKey().')'; |
|
533 | 533 | } |
534 | 534 | |
535 | 535 | return parent::count($column); |
@@ -162,7 +162,7 @@ |
||
162 | 162 | * @param array $directories |
163 | 163 | * @param array $exclude |
164 | 164 | * @param Finder $finder |
165 | - * @return ClassLocator |
|
165 | + * @return InvocationLocator |
|
166 | 166 | */ |
167 | 167 | public function invocationLocator( |
168 | 168 | array $directories = [], |
@@ -146,7 +146,7 @@ |
||
146 | 146 | */ |
147 | 147 | protected function classReflection($class) |
148 | 148 | { |
149 | - $loader = function ($class) { |
|
149 | + $loader = function($class) { |
|
150 | 150 | throw new LocatorException("Class '{$class}' can not be loaded."); |
151 | 151 | }; |
152 | 152 |
@@ -106,7 +106,7 @@ |
||
106 | 106 | * @see AbstractWhere |
107 | 107 | * @param string|mixed $identifier Column or expression. |
108 | 108 | * @param mixed $variousA Operator or value. |
109 | - * @param mixed $variousB Value, if operator specified. |
|
109 | + * @param Parameter $variousB Value, if operator specified. |
|
110 | 110 | * @param mixed $variousC Required only in between statements. |
111 | 111 | * @return $this |
112 | 112 | * @throws BuilderException |
@@ -419,7 +419,7 @@ |
||
419 | 419 | */ |
420 | 420 | private function havingWrapper() |
421 | 421 | { |
422 | - return function ($parameter) { |
|
422 | + return function($parameter) { |
|
423 | 423 | if ($parameter instanceof FragmentInterface) { |
424 | 424 | //We are only not creating bindings for plan fragments |
425 | 425 | if (!$parameter instanceof ParameterInterface && !$parameter instanceof QueryBuilder) { |
@@ -419,7 +419,7 @@ |
||
419 | 419 | */ |
420 | 420 | private function havingWrapper() |
421 | 421 | { |
422 | - return function ($parameter) { |
|
422 | + return function($parameter) { |
|
423 | 423 | if ($parameter instanceof FragmentInterface) { |
424 | 424 | //We are only not creating bindings for plan fragments |
425 | 425 | if (!$parameter instanceof ParameterInterface && !$parameter instanceof QueryBuilder) { |
@@ -429,7 +429,7 @@ |
||
429 | 429 | $class, |
430 | 430 | array $parameters, |
431 | 431 | $context = null, |
432 | - \ReflectionClass &$reflection = null |
|
432 | + \ReflectionClass & $reflection = null |
|
433 | 433 | ) { |
434 | 434 | try { |
435 | 435 | $reflection = new \ReflectionClass($class); |