rambler-digital-solutions / hydrogen

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/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/Criteria/Where.php

Spacing +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
                 return \iterator_to_array($value);
 
             case \is_object($value) && \method_exists($value, '__toString'):
-                return (string)$value;
+                return (string) $value;
         }
 
         return $value;

src/Query/RelationProvider.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
             }
 
             $error = 'Relation should be string ("relation_name") '.
-                'or array (["relation" => function]), ' .
+                'or array (["relation" => function]), '.
                 'but %s given';
 
             throw new \InvalidArgumentException(\sprintf($error, \gettype($relation)));

src/Hydrogen.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
      */
     public function query(): Query
     {
-        if (! $this instanceof EntityRepository) {
+        if (!$this instanceof EntityRepository) {
             $error = 'Could not use %s under non-repository class, but %s given';
             throw new \LogicException(\sprintf($error, Hydrogen::class, static::class));
         }

src/Query/ExecutionsProvider.php

Doc Comments +2 added lines

@@ -21,6 +21,7 @@ 
 {
     /**
      * @param string ...$fields
+     * @param string[] $fields
      * @return object[]|iterable
      */
     public function get(string ...$fields): iterable
@@ -167,6 +168,7 @@ 
 
     /**
      * @param string ...$fields
+     * @param string[] $fields
      * @return Collection
      */
     public function collect(string ...$fields): Collection

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 
 namespace RDS\Hydrogen\Query;
 
-use Illuminate\Support\Arr;
 use RDS\Hydrogen\Query;
 use RDS\Hydrogen\Collection\Collection;
 

Spacing +12 added lines, -12 removed lines

@@ -75,25 +75,25 @@ 
 
         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;
 
             case 'string':
-                return (string)$result;
+                return (string) $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);
@@ -111,7 +111,7 @@ 
         }
 
         return $this
-            ->select('COUNT(' . $field . ') AS __count')
+            ->select('COUNT('.$field.') AS __count')
             ->scalar('__count', 'int');
     }
 
@@ -123,7 +123,7 @@ 
     public function sum(string $field = null): int
     {
         return $this
-            ->select('SUM(' . $field . ') AS __sum')
+            ->select('SUM('.$field.') AS __sum')
             ->scalar('__sum', 'int');
     }
 
@@ -135,7 +135,7 @@ 
     public function avg(string $field = null): int
     {
         return $this
-            ->select('AVG(' . $field . ') AS __avg')
+            ->select('AVG('.$field.') AS __avg')
             ->scalar('__avg', 'int');
     }
 
@@ -147,7 +147,7 @@ 
     public function max(string $field = null): int
     {
         return $this
-            ->select('MAX(' . $field . ') AS __max')
+            ->select('MAX('.$field.') AS __max')
             ->scalar('__max', 'int');
     }
 
@@ -159,7 +159,7 @@ 
     public function min(string $field = null): int
     {
         return $this
-            ->select('MIN(' . $field . ') AS __min')
+            ->select('MIN('.$field.') AS __min')
             ->scalar('__min', 'int');
     }
 

src/Criteria/Common/Field.php

Doc Comments +2 added lines, -2 removed lines

@@ -78,7 +78,7 @@ 
 
     /**
      * @param Lexer $lexer
-     * @return \Generator|string[]
+     * @return \Generator
      */
     private function analyse(Lexer $lexer): \Generator
     {
@@ -182,7 +182,7 @@ 
     }
 
     /**
-     * @return iterable|Field[]
+     * @return \Generator
      */
     public function getIterator(): iterable
     {

Spacing +2 added lines, -2 removed lines

@@ -73,7 +73,7 @@ 
         $before = \substr($query, 0, $analyzed->getReturn());
         $after  = \substr($query, $analyzed->getReturn() + \max(0, $haystack - 1));
 
-        $this->wrapper = $before . '%s' . $after;
+        $this->wrapper = $before.'%s'.$after;
     }
 
     /**
@@ -117,7 +117,7 @@ 
             }
         }
 
-        return (int)$offset;
+        return (int) $offset;
     }
 
     /**

src/Processor/DatabaseProcessor.php

Doc Comments +2 added lines, -1 removed lines

@@ -128,6 +128,7 @@ 
     /**
      * @param Query $query
      * @param string ...$fields
+     * @param string[] $fields
      * @return iterable
      */
     public function getResult(Query $query, string ...$fields): iterable
@@ -151,7 +152,7 @@ 
 
     /**
      * @param QueryBuilder $builder
-     * @param array $fields
+     * @param string[] $fields
      * @return array
      */
     private function executeFetchFields(QueryBuilder $builder, array $fields): array

Spacing +3 added lines, -3 removed lines

@@ -113,7 +113,7 @@ 
 
                         if ($result === null) {
                             $stmt = \is_object($value) ? \get_class($value) : \gettype($value);
-                            $error = 'Unrecognized coroutine\'s return statement: ' . $stmt;
+                            $error = 'Unrecognized coroutine\'s return statement: '.$stmt;
                             $context->throw(new \InvalidArgumentException($error));
                         }
 
@@ -134,8 +134,8 @@ 
     {
         $query->from($this->repository);
 
-        if (! $query->has(Criteria\Selection::class)) {
-            $query->select(':' . $query->getAlias());
+        if (!$query->has(Criteria\Selection::class)) {
+            $query->select(':'.$query->getAlias());
         }
 
         /**

src/Processor/DatabaseProcessor/GroupBuilder.php

Doc Comments +2 added lines, -2 removed lines

@@ -34,7 +34,7 @@ 
     /**
      * @param QueryBuilder $builder
      * @param CriterionInterface|WhereGroup $group
-     * @return iterable|null
+     * @return \Generator
      */
     public function apply($builder, CriterionInterface $group): ?iterable
     {
@@ -80,7 +80,7 @@ 
 
     /**
      * @param WhereGroup $group
-     * @return iterable|callable[]
+     * @return \Generator
      */
     protected function getInnerSelections(WhereGroup $group): iterable
     {