rambler-digital-solutions / hydrogen

src/Query.php

Doc Comments +5 added lines, -3 removed lines

@@ -337,6 +337,7 @@ 
 
     /**
      * @param string ...$fields
+     * @param string[] $fields
      * @return Query|$this|self
      */
     public function asc(string ...$fields): self
@@ -370,6 +371,7 @@ 
 
     /**
      * @param string ...$fields
+     * @param string[] $fields
      * @return Query|$this|self
      */
     public function desc(string ...$fields): self
@@ -531,7 +533,7 @@ 
     }
 
     /**
-     * @return \Traversable|CriterionInterface[]
+     * @return \Generator
      */
     public function getCriteria(): iterable
     {
@@ -549,7 +551,7 @@ 
     }
 
     /**
-     * @param Selectable[] $contexts
+     * @param ObjectRepository $contexts
      * @return Query|$this|self
      */
     public function scope(Selectable ...$contexts): self
@@ -587,7 +589,7 @@ 
     }
 
     /**
-     * @return \Traversable|Selectable[]
+     * @return \Generator
      */
     public function getScopes(): \Traversable
     {

Spacing +2 added lines, -2 removed lines

@@ -132,7 +132,7 @@ 
      */
     private function mode(): bool
     {
-        return \tap($this->conjunction, function () {
+        return \tap($this->conjunction, function() {
             $this->conjunction = true;
         });
     }
@@ -636,7 +636,7 @@ 
         $scopes = \implode(', ', \array_map('\\class_basename', $this->scopes));
         $error  = \vsprintf('Can not find method %s in %s%s', [
             $method,
-            $scopes ? $scopes . ' or ' : '',
+            $scopes ? $scopes.' or ' : '',
             \class_basename($this),
         ]);
 

src/Processor/DatabaseProcessor/GroupProcessor.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
                 continue;
             }
 
-            throw new \InvalidArgumentException('Inner selection does not support the ' . \class_basename($where));
+            throw new \InvalidArgumentException('Inner selection does not support the '.\class_basename($where));
         }
 
         return $group->isAnd()

src/Collection.php

Spacing +2 added lines, -2 removed lines

@@ -40,7 +40,7 @@ 
      */
     final public function __get($key)
     {
-        if (! \in_array($key, static::$proxies, true)) {
+        if (!\in_array($key, static::$proxies, true)) {
             $error = \sprintf('Property [%s] does not exist on this collection instance.', $key);
             throw new \InvalidArgumentException($error);
         }
@@ -217,7 +217,7 @@ 
      */
     public function indexOf($element)
     {
-        foreach($this->items as $key => $value) {
+        foreach ($this->items as $key => $value) {
             if ($value === $element) {
                 return $key;
             }

src/helpers.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
      *  }, ...);
      * </code>
      */
-    if (! \defined('_')) {
+    if (!\defined('_')) {
         \define('_', \RDS\Hydrogen\Collection\HigherOrderCollectionProxy::PATTERN);
     }
 }

src/Criteria/OrderBy.php

Spacing +1 added lines, -1 removed lines

@@ -64,6 +64,6 @@
      */
     public function isDesc(): bool
     {
-        return ! $this->asc;
+        return !$this->asc;
     }
 }

src/Processor/DatabaseProcessor.php

Spacing +3 added lines, -3 removed lines

@@ -133,7 +133,7 @@ 
         foreach ($query->getCriteria() as $criterion) {
             $identifier = \get_class($criterion);
 
-            if (! \array_key_exists($identifier, $criteria)) {
+            if (!\array_key_exists($identifier, $criteria)) {
                 $criteria[$identifier] = $this->criterion($criterion, $alias);
             }
 
@@ -212,7 +212,7 @@ 
      */
     public function count(Query $query): int
     {
-        return $this->scalar($query, function (QueryBuilder $builder, string $field) {
+        return $this->scalar($query, function(QueryBuilder $builder, string $field) {
             return $builder->expr()->count($field);
         });
     }
@@ -228,7 +228,7 @@ 
         $builder = $this->toBuilder($query);
 
         try {
-            return (int)$builder->select($expr($builder, $this->getPrimary()->withAlias($this->alias)))
+            return (int) $builder->select($expr($builder, $this->getPrimary()->withAlias($this->alias)))
                 ->getQuery()
                 ->getSingleScalarResult();
         } catch (NoResultException | \InvalidArgumentException $e) {

src/Processor/DatabaseProcessor/RelationProcessor.php

Spacing +3 added lines, -3 removed lines

@@ -42,7 +42,7 @@ 
             $exists     = $this->hasAlias($parent);
             $childAlias = $this->fetchAlias($parent, $relation->getName());
 
-            if (! $exists) {
+            if (!$exists) {
                 $builder->leftJoin($parent, $childAlias);
                 $builder->addSelect($childAlias);
             }
@@ -71,7 +71,7 @@ 
      */
     private function fetchAlias(string $parent, string $class): string
     {
-        if (! $this->hasAlias($parent)) {
+        if (!$this->hasAlias($parent)) {
             $this->relations[$parent] = $this->createAlias($class);
         }
 
@@ -84,6 +84,6 @@ 
      */
     private function createAlias(string $class): string
     {
-        return 'ref_' . \snake_case(\class_basename($class)) . '_' . ++self::$relationId;
+        return 'ref_'.\snake_case(\class_basename($class)).'_'.++self::$relationId;
     }
 }

src/Processor/DatabaseProcessor/CriterionProcessor.php

Spacing +2 added lines, -2 removed lines

@@ -72,7 +72,7 @@ 
 
         $this->parameters[$alias] = $value;
 
-        return ':' . $alias;
+        return ':'.$alias;
     }
 
     /**
@@ -85,7 +85,7 @@ 
     {
         $prefix = $field ? str_replace('.', '_', $field->getName()) : 'value';
 
-        return $prefix . self::$parameterId++;
+        return $prefix.self::$parameterId++;
     }
 
     /**