rambler-digital-solutions / hydrogen

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

Doc Comments +1 added lines, -1 removed lines

@@ -149,7 +149,7 @@
      * Returns a set of pending operations (Deferred) and QueryBuilder.
      *
      * @param Query $query
-     * @return \Generator|\Closure[]|QueryBuilder
+     * @return \Generator
      */
     protected function toBuilder(Query $query): \Generator
     {

Spacing +4 added lines, -4 removed lines

@@ -39,7 +39,7 @@ 
      */
     public function getScalarResult(Query $query)
     {
-        return $this->exec($query, function (DoctrineQuery $query) {
+        return $this->exec($query, function(DoctrineQuery $query) {
             return $query->getSingleScalarResult();
         });
     }
@@ -93,7 +93,7 @@ 
      */
     private function applicator(QueryBuilder $builder): \Closure
     {
-        return function ($response) use ($builder) {
+        return function($response) use ($builder) {
             // Send the context (the builder) in case the
             // answer contains an empty value.
             if ($response === null) {
@@ -164,7 +164,7 @@ 
      */
     public function getResult(Query $query): iterable
     {
-        return $this->exec($query, function (DoctrineQuery $query) {
+        return $this->exec($query, function(DoctrineQuery $query) {
             return $query->getResult();
         });
     }
@@ -175,7 +175,7 @@ 
      */
     public function getArrayResult(Query $query): array
     {
-        return $this->exec($query, function (DoctrineQuery $query) {
+        return $this->exec($query, function(DoctrineQuery $query) {
             return $query->getArrayResult();
         });
     }

src/Processor/DatabaseProcessor/GroupBuilder.php

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 use Doctrine\ORM\QueryBuilder;
 use RDS\Hydrogen\Criteria\CriterionInterface;
 use RDS\Hydrogen\Criteria\Group;
-use RDS\Hydrogen\Criteria\Where;
 
 /**
  * Class GroupBuilder

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
         foreach ($inner->getCriteria() as $where) {
             $expression = parent::getDoctrineExpression($where, $builder->expr(), $where->getField());
 
-            yield from $this->extractResult($expression, function ($current) use ($expr, $where, $builder) {
+            yield from $this->extractResult($expression, function($current) use ($expr, $where, $builder) {
                 if ($where->isAnd()) {
                     $expr->add($current);
                 } else {

src/Processor/Processor.php

Doc Comments +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
     }
 
     /**
-     * @param mixed $context
+     * @param \Doctrine\ORM\QueryBuilder $context
      * @param Query $query
      * @return \Generator
      */

Spacing +1 added lines, -1 removed lines

@@ -85,7 +85,7 @@
      * @param Query $query
      * @return \Generator
      */
-    protected function forEach($context, Query $query): \Generator
+    protected function forEach ($context, Query $query): \Generator
     {
         foreach ($query->getCriteria() as $criterion) {
             $builder = $this->getBuilder($criterion);

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -85,7 +85,7 @@
      * @param Query $query
      * @return \Generator
      */
-    protected function forEach($context, Query $query): \Generator
+    protected function foreach($context, Query $query): \Generator
     {
         foreach ($query->getCriteria() as $criterion) {
             $builder = $this->getBuilder($criterion);

src/Query.php

Doc Comments +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
     /**
      * Returns a list of selection criteria.
      *
-     * @return \Generator|CriterionInterface[]
+     * @return \Generator
      */
     public function getCriteria(): \Generator
     {

src/Query/ExecutionsProvider.php

Doc Comments +3 added lines, -1 removed lines

@@ -20,6 +20,7 @@ 
 {
     /**
      * @param string ...$fields
+     * @param string[] $fields
      * @return object[]|iterable
      */
     public function get(string ...$fields): iterable
@@ -149,6 +150,7 @@ 
 
     /**
      * @param string ...$fields
+     * @param string[] $fields
      * @return Collection
      */
     public function collect(string ...$fields): Collection
@@ -157,7 +159,7 @@ 
     }
 
     /**
-     * @param string[] $fields
+     * @param string $fields
      * @return object|null
      * @throws \LogicException
      */

Spacing +12 added lines, -12 removed lines

@@ -31,7 +31,7 @@ 
         }
 
         return Collection::wrap($processor->getArrayResult($this))
-            ->map(function ($item) use ($fields) {
+            ->map(function($item) use ($fields) {
                 $result = [];
 
                 foreach ($fields as $field) {
@@ -71,22 +71,22 @@ 
 
         switch ($typeOf) {
             case 'callable':
-                return function (callable $applicator = null) use ($result) {
+                return function(callable $applicator = null) use ($result) {
                     return ($applicator ?? '\\value')($result);
                 };
 
             case 'object':
-                return (object)$result;
+                return (object) $result;
 
             case 'array':
             case 'iterable':
-                return (array)$result;
+                return (array) $result;
         }
 
-        $function = $typeOf . 'val';
+        $function = $typeOf.'val';
 
-        if (! \function_exists($function)) {
-            throw new \InvalidArgumentException('Could not cast to type ' . $typeOf);
+        if (!\function_exists($function)) {
+            throw new \InvalidArgumentException('Could not cast to type '.$typeOf);
         }
 
         return $function($result);
@@ -99,7 +99,7 @@ 
      */
     public function count(?string $field = 'id'): int
     {
-        return $this->select('COUNT(' . $field . ') AS __count')
+        return $this->select('COUNT('.$field.') AS __count')
             ->scalar('__count', 'int');
     }
 
@@ -110,7 +110,7 @@ 
      */
     public function sum(string $field = null): int
     {
-        return $this->select('SUM(' . $field . ') AS __sum')
+        return $this->select('SUM('.$field.') AS __sum')
             ->scalar('__sum', 'int');
     }
 
@@ -121,7 +121,7 @@ 
      */
     public function avg(string $field = null): int
     {
-        return $this->select('AVG(' . $field . ') AS __avg')
+        return $this->select('AVG('.$field.') AS __avg')
             ->scalar('__avg', 'int');
     }
 
@@ -132,7 +132,7 @@ 
      */
     public function max(string $field = null): int
     {
-        return $this->select('MAX(' . $field . ') AS __max')
+        return $this->select('MAX('.$field.') AS __max')
             ->scalar('__max', 'int');
     }
 
@@ -143,7 +143,7 @@ 
      */
     public function min(string $field = null): int
     {
-        return $this->select('MIN(' . $field . ') AS __min')
+        return $this->select('MIN('.$field.') AS __min')
             ->scalar('__min', 'int');
     }
 

src/Query/OrderProvider.php

Doc Comments +2 added lines

@@ -34,6 +34,7 @@ 
 
     /**
      * @param string ...$fields
+     * @param string[] $fields
      * @return Query|$this|self
      */
     public function asc(string ...$fields): self
@@ -70,6 +71,7 @@ 
 
     /**
      * @param string ...$fields
+     * @param string[] $fields
      * @return Query|$this|self
      */
     public function desc(string ...$fields): self

src/Query/RepositoryProvider.php

Doc Comments +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
     protected $repository;
 
     /**
-     * @param ObjectRepository|Hydrogen $repository
+     * @param ObjectRepository $repository
      * @return Query|$this
      */
     public function from(ObjectRepository $repository): self
@@ -34,7 +34,7 @@ 
     }
 
     /**
-     * @return ObjectRepository|Hydrogen
+     * @return ObjectRepository
      * @throws \LogicException
      */
     public function getRepository(): ObjectRepository