limoncello-php-dist / flute

src/Http/Query/ParametersMapper.php

Doc Comments +1 added lines, -1 removed lines

@@ -408,7 +408,7 @@
      * @param string   $parameterName
      * @param iterable $value
      *
-     * @return iterable
+     * @return Generator
      */
     private function parseOperationsAndArguments(string $parameterName, iterable $value): iterable
     {

Spacing +3 added lines, -3 removed lines

@@ -245,7 +245,7 @@ 
     public function getMappedIncludes(): iterable
     {
         $fromScheme        = $this->getRootScheme();
-        $getMappedRelLinks = function (iterable $links) use ($fromScheme) : iterable {
+        $getMappedRelLinks = function(iterable $links) use ($fromScheme) : iterable {
             foreach ($links as $link) {
                 assert(is_string($link));
                 $fromSchemaClass = get_class($fromScheme);
@@ -289,14 +289,14 @@ 
         //     ['rel2_name1', ],
         //     ['rel3_name1', 'rel3_name2', 'rel3_name3', ],
         // ]
-        $includeAsModelNames = function (iterable $relationships): iterable {
+        $includeAsModelNames = function(iterable $relationships): iterable {
             foreach ($relationships as $relationship) {
                 assert($relationship instanceof RelationshipInterface);
                 yield $relationship->getNameInModel();
             }
         };
         $mappedIncludes      = $this->getMappedIncludes();
-        $getIncludes         = function () use ($mappedIncludes, $includeAsModelNames) : iterable {
+        $getIncludes         = function() use ($mappedIncludes, $includeAsModelNames) : iterable {
             foreach ($mappedIncludes as $relationships) {
                 yield $includeAsModelNames($relationships);
             }

src/Http/Query/QueryParser.php

Doc Comments +1 added lines, -1 removed lines

@@ -264,7 +264,7 @@
      * @param string $parameterName
      * @param array  $value
      *
-     * @return iterable
+     * @return Generator
      */
     private function parseOperationsAndArguments(string $parameterName, array $value): iterable
     {

Spacing +3 added lines, -3 removed lines

@@ -92,7 +92,7 @@ 
     {
         // debug check all fields are strings
         assert(
-            (function () use ($fields) {
+            (function() use ($fields) {
                 $allAreStrings = !empty($fields);
                 foreach ($fields as $field) {
                     $allAreStrings = $allAreStrings === true && is_string($field) === true && empty($field) === false;
@@ -134,7 +134,7 @@ 
     {
         // debug check all fields are strings
         assert(
-            (function () use ($fields) {
+            (function() use ($fields) {
                 $allAreStrings = !empty($fields);
                 foreach ($fields as $field) {
                     $allAreStrings = $allAreStrings === true && is_string($field) === true && empty($field) === false;
@@ -176,7 +176,7 @@ 
     {
         // debug check all fields are strings
         assert(
-            (function () use ($paths) {
+            (function() use ($paths) {
                 $allAreStrings = !empty($paths);
                 foreach ($paths as $path) {
                     $allAreStrings = $allAreStrings === true && is_string($path) === true && empty($path) === false;

src/Adapters/ModelQueryBuilder.php

Doc Comments +8 added lines, -8 removed lines

@@ -128,7 +128,7 @@ 
     }
 
     /**
-     * @param iterable $attributes
+     * @param Generator $attributes
      *
      * @return self
      *
@@ -148,7 +148,7 @@ 
     }
 
     /**
-     * @param iterable $attributes
+     * @param Generator $attributes
      *
      * @return self
      *
@@ -169,7 +169,7 @@ 
      * @param string   $modelClass
      * @param iterable $attributes
      *
-     * @return iterable
+     * @return Generator
      *
      * @SuppressWarnings(PHPMD.StaticAccess)
      */
@@ -249,7 +249,7 @@ 
     }
 
     /**
-     * @param iterable $filters
+     * @param \Limoncello\Flute\Api\iterable|null $filters
      *
      * @return self
      */
@@ -259,7 +259,7 @@ 
     }
 
     /**
-     * @param iterable $filters
+     * @param \Limoncello\Flute\Api\iterable|null $filters
      *
      * @return self
      */
@@ -269,7 +269,7 @@ 
     }
 
     /**
-     * @param iterable $filters
+     * @param \Limoncello\Flute\Api\iterable|null $filters
      *
      * @return self
      */
@@ -314,7 +314,7 @@ 
 
     /**
      * @param string        $relationshipName
-     * @param iterable      $relationshipFilters
+     * @param \Limoncello\Flute\Api\iterable|null      $relationshipFilters
      * @param iterable|null $relationshipSorts
      *
      * @return self
@@ -335,7 +335,7 @@ 
     }
 
     /**
-     * @param iterable $sortParameters
+     * @param \Limoncello\Flute\Api\iterable|null $sortParameters
      *
      * @return self
      */

Spacing +13 added lines, -13 removed lines

@@ -320,7 +320,7 @@ 
         string $relationshipName,
         iterable $relationshipFilters,
         ?iterable $relationshipSorts
-    ): self {
+    ) : self {
         $joinWith = $this->expr()->andX();
 
         return $this->addRelationshipFiltersAndSorts(
@@ -354,7 +354,7 @@ 
         string $relationshipName,
         iterable $relationshipFilters,
         ?iterable $relationshipSorts
-    ): self {
+    ) : self {
         $joinWith = $this->expr()->orX();
 
         return $this->addRelationshipFiltersAndSorts(
@@ -415,7 +415,7 @@ 
         CompositeExpression $filterLink,
         iterable $relationshipFilters,
         ?iterable $relationshipSorts
-    ): self {
+    ) : self {
         $relationshipType = $this->getModelSchemes()->getRelationshipType($this->getModelClass(), $relationshipName);
         switch ($relationshipType) {
             case RelationshipTypes::BELONGS_TO:
@@ -488,7 +488,7 @@ 
         CompositeExpression $filterLink,
         iterable $relationshipFilters,
         ?iterable $relationshipSorts
-    ): self {
+    ) : self {
         $foreignKey = $this->getModelSchemes()->getForeignKey($this->getModelClass(), $relationshipName);
         list($onePrimaryKey, $oneTable) =
             $this->getModelSchemes()->getReversePrimaryKey($this->getModelClass(), $relationshipName);
@@ -522,7 +522,7 @@ 
         CompositeExpression $filterLink,
         iterable $relationshipFilters,
         ?iterable $relationshipSorts
-    ): self {
+    ) : self {
         $primaryKey = $this->getModelSchemes()->getPrimaryKey($this->getModelClass());
         list($manyForeignKey, $manyTable) =
             $this->getModelSchemes()->getReverseForeignKey($this->getModelClass(), $relationshipName);
@@ -556,7 +556,7 @@ 
         CompositeExpression $targetFilterLink,
         iterable $relationshipFilters,
         ?iterable $relationshipSorts
-    ): self {
+    ) : self {
         $primaryKey = $this->getModelSchemes()->getPrimaryKey($this->getModelClass());
         list ($intermediateTable, $intermediatePk, $intermediateFk) =
             $this->getModelSchemes()->getBelongsToManyRelationship($this->getModelClass(), $relationshipName);
@@ -606,9 +606,9 @@ 
         ?CompositeExpression $targetFilterLink,
         ?iterable $targetFilterParams,
         ?iterable $relationshipSorts
-    ): string {
+    ) : string {
         $targetAlias   = $this->createAlias($targetTable);
-        $joinCondition = $this->buildColumnName($fromAlias, $fromColumn) . '=' .
+        $joinCondition = $this->buildColumnName($fromAlias, $fromColumn).'='.
             $this->buildColumnName($targetAlias, $targetColumn);
 
         $this->innerJoin(
@@ -667,7 +667,7 @@ 
         ?CompositeExpression $targetFilterLink,
         ?iterable $targetFilterParams,
         ?iterable $targetSortParams
-    ): string {
+    ) : string {
         $intNoSorting = null;
         $intAlias     = $this->innerJoinOneTable(
             $fromAlias,
@@ -678,7 +678,7 @@ 
             $intFilterParams,
             $intNoSorting
         );
-        $targetAlias  = $this->innerJoinOneTable(
+        $targetAlias = $this->innerJoinOneTable(
             $intAlias,
             $intToTargetColumn,
             $targetTable,
@@ -698,7 +698,7 @@ 
      */
     private function createAlias(string $tableName): string
     {
-        $alias                          = $tableName . (++$this->aliasIdCounter);
+        $alias                          = $tableName.(++$this->aliasIdCounter);
         $this->knownAliases[$tableName] = $alias;
 
         return $alias;
@@ -876,7 +876,7 @@ 
         if ($this->dtToDbConverter === null) {
             $type     = Type::getType(DateTimeType::DATETIME);
             $platform = $this->getConnection()->getDatabasePlatform();
-            $this->dtToDbConverter = function (DateTimeInterface $dateTime) use ($type, $platform) : string {
+            $this->dtToDbConverter = function(DateTimeInterface $dateTime) use ($type, $platform) : string {
                 return $type->convertToDatabaseValue($dateTime, $platform);
             };
         }
@@ -902,7 +902,7 @@ 
         } else {
             assert(
                 $value !== null,
-                'It seems you are trying to use `null` with =, >, <, or etc operator. ' .
+                'It seems you are trying to use `null` with =, >, <, or etc operator. '.
                 'Use `is null` or `not null` instead.'
             );
             assert(is_string($value), "Only strings, booleans and integers are supported.");

src/Contracts/Api/CrudInterface.php

Spacing +1 added lines, -1 removed lines

@@ -121,7 +121,7 @@
      *
      * @return array|null
      */
-    public function fetchRow(QueryBuilder $builder, string $modelClass): ?array;
+    public function fetchRow(QueryBuilder $builder, string $modelClass): ? array;
 
     /**
      * @param QueryBuilder $builder

Doc Comments +3 added lines, -3 removed lines

@@ -108,7 +108,7 @@ 
     public function fetchResources(QueryBuilder $builder, string $modelClass): PaginatedDataInterface;
 
     /**
-     * @param QueryBuilder|null $builder
+     * @param QueryBuilder $builder
      * @param string|null       $modelClass
      *
      * @return mixed|null
@@ -128,7 +128,7 @@ 
      * @param string       $modelClass
      * @param string       $columnName
      *
-     * @return iterable
+     * @return \Generator
      */
     public function fetchColumn(QueryBuilder $builder, string $modelClass, string $columnName): iterable;
 
@@ -211,7 +211,7 @@ 
     ): array;
 
     /**
-     * @param int|string    $index
+     * @param string    $index
      * @param string        $name
      * @param iterable|null $relationshipFilters
      * @param iterable|null $relationshipSorts

src/Http/BaseController.php

Spacing +4 added lines, -4 removed lines

@@ -240,7 +240,7 @@ 
      */
     protected static function readJsonFromRequest(ContainerInterface $container, ServerRequestInterface $request): array
     {
-        $body = (string)$request->getBody();
+        $body = (string) $request->getBody();
         if (empty($body) === true || ($json = json_decode($body, true)) === null) {
             /** @var FactoryInterface $factory */
             $factory = $container->get(FactoryInterface::class);
@@ -383,7 +383,7 @@ 
     {
         assert(
             empty(static::ON_CREATE_VALIDATION_RULES_SET_CLASS) === false,
-            'Validation rules set should be defined for class ' . static::class . '.'
+            'Validation rules set should be defined for class '.static::class.'.'
         );
 
         return static::createJsonApiValidator($container, static::ON_CREATE_VALIDATION_RULES_SET_CLASS);
@@ -398,7 +398,7 @@ 
     {
         assert(
             empty(static::ON_UPDATE_VALIDATION_RULES_SET_CLASS) === false,
-            'Validation rules set should be defined for class ' . static::class . '.'
+            'Validation rules set should be defined for class '.static::class.'.'
         );
 
         return static::createJsonApiValidator($container, static::ON_UPDATE_VALIDATION_RULES_SET_CLASS);
@@ -606,7 +606,7 @@ 
         ContainerInterface $container,
         ServerRequestInterface $request
     ): array {
-        $jsonData  = static::normalizeIndexValueOnUpdate(
+        $jsonData = static::normalizeIndexValueOnUpdate(
             $routeParams,
             $container,
             static::readJsonFromRequest($container, $request)

src/Http/Query/QueryValidator.php

Doc Comments +2 added lines, -2 removed lines

@@ -270,7 +270,7 @@ 
      * @param string   $name
      * @param iterable $operationsAndArgs
      *
-     * @return iterable
+     * @return \Generator
      */
     private function getValidatedOperationsAndArguments(
         int $blockIndex,
@@ -287,7 +287,7 @@ 
      * @param string   $name
      * @param iterable $arguments
      *
-     * @return iterable
+     * @return \Generator
      */
     private function getValidatedArguments(int $blockIndex, string $name, iterable $arguments): iterable
     {

Spacing +1 added lines, -1 removed lines

@@ -378,7 +378,7 @@
     /**
      * @return int[]|null
      */
-    private function getAttributeRules(): ?array
+    private function getAttributeRules(): ? array
     {
         return $this->attributeRules;
     }

src/Validation/JsonApi/ValidatorWrapper.php

Spacing +1 added lines, -1 removed lines

@@ -162,7 +162,7 @@
     protected function setWrapperErrors(array $wrapperErrors): self
     {
         if (empty($wrapperErrors) === false) {
-            assert(call_user_func(function () use ($wrapperErrors) : bool {
+            assert(call_user_func(function() use ($wrapperErrors) : bool {
                 $allAreErrors = true;
 
                 foreach ($wrapperErrors as $error) {

src/Validation/JsonApi/Rules/ExistInDbTableMultipleWithDoctrine.php

Spacing +1 added lines, -2 removed lines

@@ -105,8 +105,7 @@
         }
 
         $reply = $count > 0 ?
-            BlockReplies::createSuccessReply($values) :
-            BlockReplies::createErrorReply($context, $values, ErrorCodes::EXIST_IN_DATABASE_MULTIPLE);
+            BlockReplies::createSuccessReply($values) : BlockReplies::createErrorReply($context, $values, ErrorCodes::EXIST_IN_DATABASE_MULTIPLE);
 
         return $reply;
     }

src/Validation/JsonApi/Rules/ToManyRelationshipTypeChecker.php

Spacing +1 added lines, -2 removed lines

@@ -86,8 +86,7 @@
         }
 
         $reply = $foundInvalidType === null ?
-            BlockReplies::createSuccessReply($indexes) :
-            BlockReplies::createErrorReply($context, $foundInvalidType, ErrorCodes::INVALID_RELATIONSHIP_TYPE);
+            BlockReplies::createSuccessReply($indexes) : BlockReplies::createErrorReply($context, $foundInvalidType, ErrorCodes::INVALID_RELATIONSHIP_TYPE);
 
         return $reply;
     }