limoncello-php-dist / flute

src/Contracts/Http/Query/ParametersMapperInterface.php

Doc Comments +3 added lines, -3 removed lines

@@ -52,17 +52,17 @@
     public function withIncludes(iterable $includes): self;
 
     /**
-     * @return iterable
+     * @return \Generator
      */
     public function getMappedFilters(): iterable;
 
     /**
-     * @return iterable
+     * @return \Generator
      */
     public function getMappedSorts(): iterable;
 
     /**
-     * @return iterable
+     * @return \Generator
      */
     public function getMappedIncludes(): iterable;
 

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php declare (strict_types = 1);
+<?php declare(strict_types=1);
 
 namespace Limoncello\Flute\Contracts\Http\Query;
 

src/Api/Crud.php

Doc Comments +7 added lines, -7 removed lines

@@ -756,9 +756,9 @@ 
     }
 
     /**
-     * @param iterable $paths (string[])
+     * @param iterable|null $paths (string[])
      *
-     * @return iterable
+     * @return Generator
      */
     private static function getPaths(iterable $paths): iterable
     {
@@ -1399,7 +1399,7 @@ 
      * @param string       $modelClass
      * @param string       $keyColumnName
      *
-     * @return iterable
+     * @return Generator
      *
      * @throws DBALException
      *
@@ -1503,7 +1503,7 @@ 
      * @param null|string $index
      * @param iterable    $attributes
      *
-     * @return iterable
+     * @return Generator
      */
     protected function filterAttributesOnCreate(?string $index, iterable $attributes): iterable
     {
@@ -1523,7 +1523,7 @@ 
     /**
      * @param iterable $attributes
      *
-     * @return iterable
+     * @return Generator
      */
     protected function filterAttributesOnUpdate(iterable $attributes): iterable
     {
@@ -1722,10 +1722,10 @@ 
 
     /**
      * @param iterable         $attributes
-     * @param array            $typeNames
+     * @param Type[]            $typeNames
      * @param AbstractPlatform $platform
      *
-     * @return iterable
+     * @return Generator
      *
      * @throws DBALException
      */

Spacing +25 added lines, -25 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare (strict_types = 1);
+<?php declare(strict_types=1);
 
 namespace Limoncello\Flute\Api;
 
@@ -209,7 +209,7 @@ 
             throw new InvalidArgumentException($this->getMessage(Messages::MSG_ERR_INVALID_ARGUMENT));
         }
 
-        assert(call_user_func(function () use ($indexes) {
+        assert(call_user_func(function() use ($indexes) {
             $allOk = true;
 
             foreach ($indexes as $index) {
@@ -763,7 +763,7 @@ 
             $classAtPath = new ArrayObject();
             $idsAtPath   = new ArrayObject();
 
-            $registerModelAtRoot = function ($model) use ($modelStorage, $modelsAtPath, $idsAtPath): void {
+            $registerModelAtRoot = function($model) use ($modelStorage, $modelsAtPath, $idsAtPath): void {
                 self::registerModelAtPath(
                     $model,
                     static::ROOT_PATH,
@@ -851,7 +851,7 @@ 
                 assert(is_string($pathPiece));
                 $parent                    = $tmpPath;
                 $tmpPath                   = empty($tmpPath) === true ?
-                    $pathPiece : $tmpPath . static::PATH_SEPARATOR . $pathPiece;
+                    $pathPiece : $tmpPath.static::PATH_SEPARATOR.$pathPiece;
                 $normalizedPaths[$tmpPath] = [$parent, $pathPiece];
                 $pathsDepths[$parent]      = $parentDepth++;
             }
@@ -999,7 +999,7 @@ 
             $this->createCountBuilderFromBuilder($this->createIndexModelBuilder())
         )->execute()->fetchColumn();
 
-        return $result === false ? null : (int)$result;
+        return $result === false ? null : (int) $result;
     }
 
     /**
@@ -1020,7 +1020,7 @@ 
     ): ModelQueryBuilder {
         assert(
             $this->getModelSchemas()->hasRelationship($this->getModelClass(), $relationshipName),
-            "Relationship `$relationshipName` do not exist in model `" . $this->getModelClass() . '`'
+            "Relationship `$relationshipName` do not exist in model `".$this->getModelClass().'`'
         );
 
         // as we read data from a relationship our main table and model would be the table/model in the relationship
@@ -1038,7 +1038,7 @@ 
         if ($this->hasFilters() === true) {
             $filters = $this->getFilters();
             $sorts   = $this->getSorts();
-            $joinCondition = $this->areFiltersWithAnd() === true ? ModelQueryBuilder::AND : ModelQueryBuilder::OR;
+            $joinCondition = $this->areFiltersWithAnd() === true ? ModelQueryBuilder:: AND : ModelQueryBuilder:: OR ;
             $builder->addRelationshipFiltersAndSorts($reverseRelName, $filters, $sorts, $joinCondition);
         }
         // ... and the input filters to actual data we select
@@ -1069,7 +1069,7 @@ 
     ) {
         assert(
             $this->getModelSchemas()->hasRelationship($this->getModelClass(), $name),
-            "Relationship `$name` do not exist in model `" . $this->getModelClass() . '`'
+            "Relationship `$name` do not exist in model `".$this->getModelClass().'`'
         );
 
         // depending on the relationship type we expect the result to be either single resource or a collection
@@ -1098,7 +1098,7 @@ 
     ): array {
         assert(
             $this->getModelSchemas()->hasRelationship($this->getModelClass(), $name),
-            "Relationship `$name` do not exist in model `" . $this->getModelClass() . '`'
+            "Relationship `$name` do not exist in model `".$this->getModelClass().'`'
         );
 
         // depending on the relationship type we expect the result to be either single resource or a collection
@@ -1167,7 +1167,7 @@ 
 
         $this->clearFetchParameters();
 
-        return (int)$deleted;
+        return (int) $deleted;
     }
 
     /**
@@ -1183,7 +1183,7 @@ 
 
         $this->clearFetchParameters();
 
-        return (int)$deleted > 0;
+        return (int) $deleted > 0;
     }
 
     /**
@@ -1204,7 +1204,7 @@ 
 
         $this->clearBuilderParameters()->clearFetchParameters();
 
-        $this->inTransaction(function () use ($saveMain, $toMany, &$index) {
+        $this->inTransaction(function() use ($saveMain, $toMany, &$index) {
             $saveMain->execute();
 
             // if no index given will use last insert ID as index
@@ -1241,12 +1241,12 @@ 
             ->createBuilder($this->getModelClass())
             ->updateModels($allowedChanges)
             ->addFiltersWithAndToTable($filters);
-        $saveMain       = $this->builderSaveResourceOnUpdate($saveMain);
+        $saveMain = $this->builderSaveResourceOnUpdate($saveMain);
         $saveMain->getSQL(); // prepare
 
         $this->clearBuilderParameters()->clearFetchParameters();
 
-        $this->inTransaction(function () use ($saveMain, $toMany, $index, &$updated) {
+        $this->inTransaction(function() use ($saveMain, $toMany, $index, &$updated) {
             $updated = $saveMain->execute();
 
             $builderHook = Closure::fromCallable([$this, 'builderSaveRelationshipOnUpdate']);
@@ -1262,7 +1262,7 @@ 
                 )->execute();
 
                 // add new ones
-                $updated   += $this->addInToManyRelationship(
+                $updated += $this->addInToManyRelationship(
                     $connection,
                     $index,
                     $relationshipName,
@@ -1272,7 +1272,7 @@ 
             }
         });
 
-        return (int)$updated;
+        return (int) $updated;
     }
 
     /**
@@ -1289,9 +1289,9 @@ 
     public function createInBelongsToManyRelationship(string $parentId, string $name, iterable $childIds): int
     {
         // Check that relationship is `BelongsToMany`
-        assert(call_user_func(function () use ($name): bool {
+        assert(call_user_func(function() use ($name): bool {
             $relType = $this->getModelSchemas()->getRelationshipType($this->getModelClass(), $name);
-            $errMsg  = "Relationship `$name` of class `" . $this->getModelClass() .
+            $errMsg  = "Relationship `$name` of class `".$this->getModelClass().
                 '` either is not `belongsToMany` or do not exist in the class.';
             $isOk = $relType === RelationshipTypes::BELONGS_TO_MANY;
 
@@ -1313,9 +1313,9 @@ 
     public function removeInBelongsToManyRelationship(string $parentId, string $name, iterable $childIds): int
     {
         // Check that relationship is `BelongsToMany`
-        assert(call_user_func(function () use ($name): bool {
+        assert(call_user_func(function() use ($name): bool {
             $relType = $this->getModelSchemas()->getRelationshipType($this->getModelClass(), $name);
-            $errMsg  = "Relationship `$name` of class `" . $this->getModelClass() .
+            $errMsg  = "Relationship `$name` of class `".$this->getModelClass().
                 '` either is not `belongsToMany` or do not exist in the class.';
             $isOk = $relType === RelationshipTypes::BELONGS_TO_MANY;
 
@@ -1483,7 +1483,7 @@ 
     {
         $countBuilder = $this->createBuilder($this->getModelClass());
         $countBuilder->setParameters($builder->getParameters());
-        $countBuilder->select('COUNT(*)')->from('(' . $builder->getSQL() . ') AS RESULT');
+        $countBuilder->select('COUNT(*)')->from('('.$builder->getSQL().') AS RESULT');
 
         return $countBuilder;
     }
@@ -1517,7 +1517,7 @@ 
 
         foreach ($secondaryIdentities as $secondaryId) {
             try {
-                $inserted += (int)$saveToMany->setParameter($secondaryIdBindName, $secondaryId)->execute();
+                $inserted += (int) $saveToMany->setParameter($secondaryIdBindName, $secondaryId)->execute();
             } /** @noinspection PhpRedundantCatchClauseInspection */ catch (UcvException $exception) {
                 // Spec: If all of the specified resources can be added to, or are already present in,
                 // the relationship then the server MUST return a successful response.
@@ -1764,7 +1764,7 @@ 
 
         $pkName = $this->getModelSchemas()->getPrimaryKey($parentClass);
 
-        $registerModelAtPath = function ($model, string $path) use ($deDup, $modelsAtPath, $idsAtPath) {
+        $registerModelAtPath = function($model, string $path) use ($deDup, $modelsAtPath, $idsAtPath) {
             return self::registerModelAtPath(
                 $model,
                 $path,
@@ -1776,7 +1776,7 @@ 
         };
 
         foreach ($childRelationships as $name) {
-            $childrenPath = $parentsPath !== static::ROOT_PATH ? $parentsPath . static::PATH_SEPARATOR . $name : $name;
+            $childrenPath = $parentsPath !== static::ROOT_PATH ? $parentsPath.static::PATH_SEPARATOR.$name : $name;
 
             $relationshipType = $this->getModelSchemas()->getRelationshipType($parentClass, $name);
             list ($targetModelClass, $reverseRelName) =
@@ -1831,7 +1831,7 @@ 
                             [$pkName => [FilterParameterInterface::OPERATION_EQUALS => [$parent->{$pkName}]]],
                             []
                         );
-                        $children      = $this->fetchPaginatedResourcesWithoutRelationships(
+                        $children = $this->fetchPaginatedResourcesWithoutRelationships(
                             $clonedBuilder,
                             $clonedBuilder->getModelClass()
                         );

src/Http/Query/ParametersMapper.php

Doc Comments +1 added lines, -1 removed lines

@@ -409,7 +409,7 @@
      * @param string   $parameterName
      * @param iterable $value
      *
-     * @return iterable
+     * @return Generator
      *
      * @SuppressWarnings(PHPMD.CyclomaticComplexity)
      */

Spacing +4 added lines, -4 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare (strict_types = 1);
+<?php declare(strict_types=1);
 
 namespace Limoncello\Flute\Http\Query;
 
@@ -249,7 +249,7 @@ 
     public function getMappedIncludes(): iterable
     {
         $fromSchema        = $this->getRootSchema();
-        $getMappedRelLinks = function (iterable $links) use ($fromSchema) : iterable {
+        $getMappedRelLinks = function(iterable $links) use ($fromSchema) : iterable {
             foreach ($links as $link) {
                 assert(is_string($link));
                 $fromSchemaClass = get_class($fromSchema);
@@ -299,14 +299,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/Validation/JsonApi/QueryParser.php

Doc Comments +12 added lines, -12 removed lines

@@ -379,7 +379,7 @@ 
     }
 
     /**
-     * @return iterable
+     * @return Generator
      *
      * @SuppressWarnings(PHPMD.ElseExpression)
      */
@@ -421,9 +421,9 @@ 
     }
 
     /**
-     * @param iterable $fieldsFromParent
+     * @param Generator $fieldsFromParent
      *
-     * @return iterable
+     * @return Generator
      *
      * @SuppressWarnings(PHPMD.StaticAccess)
      * @SuppressWarnings(PHPMD.ElseExpression)
@@ -456,9 +456,9 @@ 
     }
 
     /**
-     * @param iterable $sortsFromParent
+     * @param Generator $sortsFromParent
      *
-     * @return iterable
+     * @return Generator
      *
      * @SuppressWarnings(PHPMD.StaticAccess)
      * @SuppressWarnings(PHPMD.ElseExpression)
@@ -488,9 +488,9 @@ 
     }
 
     /**
-     * @param iterable $includesFromParent
+     * @param Generator $includesFromParent
      *
-     * @return iterable
+     * @return Generator
      *
      * @SuppressWarnings(PHPMD.StaticAccess)
      * @SuppressWarnings(PHPMD.ElseExpression)
@@ -520,7 +520,7 @@ 
     }
 
     /**
-     * @param iterable $iterable
+     * @param Generator $iterable
      *
      * @return array
      */
@@ -683,7 +683,7 @@ 
      * @param int      $ruleIndex
      * @param iterable $values
      *
-     * @return iterable
+     * @return Generator
      */
     private function validateValues(int $ruleIndex, iterable $values): iterable
     {
@@ -698,9 +698,9 @@ 
 
     /**
      * @param int      $ruleIndex
-     * @param iterable $opsAndArgs
+     * @param Generator $opsAndArgs
      *
-     * @return iterable
+     * @return Generator
      */
     private function validateFilterArguments(int $ruleIndex, iterable $opsAndArgs): iterable
     {
@@ -858,7 +858,7 @@ 
      * @param string $parameterName
      * @param array  $value
      *
-     * @return iterable
+     * @return Generator
      *
      * @SuppressWarnings(PHPMD.ElseExpression)
      */

Spacing +4 added lines, -4 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare (strict_types = 1);
+<?php declare(strict_types=1);
 
 namespace Limoncello\Flute\Validation\JsonApi;
 
@@ -189,7 +189,7 @@ 
     ) {
         assert(
             in_array(JsonApiQueryRulesSerializerInterface::class, class_implements($serializerClass)),
-            "`$serializerClass` should implement interface `" . JsonApiQueryRulesSerializerInterface::class . '`.'
+            "`$serializerClass` should implement interface `".JsonApiQueryRulesSerializerInterface::class.'`.'
         );
 
         $parameters = [];
@@ -691,7 +691,7 @@ 
     {
         // no validation rule means we should accept any input value
         if ($ruleIndexes === null) {
-            return is_numeric($value) === true ? (int)$value : 0;
+            return is_numeric($value) === true ? (int) $value : 0;
         }
 
         $ruleIndex = $this->serializerClass::readRuleMainIndex($ruleIndexes);
@@ -702,7 +702,7 @@ 
 
         $validatedValue = $this->readSingleCapturedValue();
 
-        return (int)$validatedValue;
+        return (int) $validatedValue;
     }
 
     /**

src/Contracts/Api/CrudInterface.php

Doc Comments +2 added lines, -2 removed lines

@@ -120,7 +120,7 @@ 
     public function fetchResources(QueryBuilder $builder, string $modelClass): PaginatedDataInterface;
 
     /**
-     * @param QueryBuilder|null $builder
+     * @param QueryBuilder $builder
      * @param string|null       $modelClass
      *
      * @return mixed|null
@@ -140,7 +140,7 @@ 
      * @param string       $modelClass
      * @param string       $columnName
      *
-     * @return iterable
+     * @return \Generator
      */
     public function fetchColumn(QueryBuilder $builder, string $modelClass, string $columnName): iterable;
 

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php declare (strict_types = 1);
+<?php declare(strict_types=1);
 
 namespace Limoncello\Flute\Contracts\Api;
 

src/Contracts/Validation/JsonApiDataRulesSerializerInterface.php

Doc Comments +1 added lines, -1 removed lines

@@ -161,7 +161,7 @@
     public static function readRulesEndIndexes(array $arrayRuleIndexes): array;
 
     /**
-     * @param array  $arrayRuleIndexes
+     * @param integer[]  $arrayRuleIndexes
      * @param string $name
      *
      * @return array

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?php declare (strict_types = 1);
+<?php declare(strict_types=1);
 
 namespace Limoncello\Flute\Contracts\Validation;
 

src/Adapters/ModelQueryBuilder.php

Doc Comments +7 added lines, -7 removed lines

@@ -217,7 +217,7 @@ 
     }
 
     /**
-     * @param iterable $attributes
+     * @param \Generator $attributes
      *
      * @return self
      *
@@ -239,7 +239,7 @@ 
     }
 
     /**
-     * @param iterable $attributes
+     * @param \Generator $attributes
      *
      * @return self
      *
@@ -262,7 +262,7 @@ 
      * @param string   $modelClass
      * @param iterable $attributes
      *
-     * @return iterable
+     * @return \Generator
      *
      * @SuppressWarnings(PHPMD.StaticAccess)
      *
@@ -397,7 +397,7 @@ 
     }
 
     /**
-     * @param iterable $filters
+     * @param \Limoncello\Flute\Api\iterable|null $filters
      *
      * @return self
      *
@@ -413,7 +413,7 @@ 
     }
 
     /**
-     * @param iterable $filters
+     * @param \Limoncello\Flute\Api\iterable|null $filters
      *
      * @return self
      *
@@ -429,7 +429,7 @@ 
     }
 
     /**
-     * @param iterable $filters
+     * @param \Limoncello\Flute\Api\iterable|null $filters
      *
      * @return self
      *
@@ -523,7 +523,7 @@ 
     }
 
     /**
-     * @param iterable $sortParameters
+     * @param \Limoncello\Flute\Api\iterable|null $sortParameters
      *
      * @return self
      *

Spacing +12 added lines, -12 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare (strict_types = 1);
+<?php declare(strict_types=1);
 
 namespace Limoncello\Flute\Adapters;
 
@@ -50,7 +50,7 @@ 
     /**
      * Condition joining method.
      */
-    public const OR = self::AND + 1;
+    public const OR = self:: AND + 1;
 
     /**
      * @var string
@@ -465,8 +465,8 @@ 
         string $relationshipName,
         ?iterable $relationshipFilters,
         ?iterable $relationshipSorts,
-        int $joinIndividuals = self::AND,
-        int $joinRelationship = self::AND
+        int $joinIndividuals = self:: AND ,
+        int $joinRelationship = self:: AND
     ): self {
         $targetAlias = null;
 
@@ -478,7 +478,7 @@ 
             $reversePk = $isBelongsTo === true ?
                 $this->getModelSchemas()->getReversePrimaryKey($this->getModelClass(), $relationshipName)[0] : null;
 
-            $addWith = $joinIndividuals === self::AND ? $this->expr()->andX() : $this->expr()->orX();
+            $addWith = $joinIndividuals === self:: AND ? $this->expr()->andX() : $this->expr()->orX();
 
             foreach ($relationshipFilters as $columnName => $operationsWithArgs) {
                 if ($columnName === $reversePk) {
@@ -498,14 +498,14 @@ 
                 foreach ($operationsWithArgs as $operation => $arguments) {
                     assert(
                         is_iterable($arguments) === true || is_array($arguments) === true,
-                        "Operation arguments are missing for `$columnName` column. " .
+                        "Operation arguments are missing for `$columnName` column. ".
                         'Use an empty array as an empty argument list.'
                     );
                     $addWith->add($this->createFilterExpression($fullColumnName, $operation, $arguments));
                 }
 
                 if ($addWith->count() > 0) {
-                    $joinRelationship === self::AND ? $this->andWhere($addWith) : $this->orWhere($addWith);
+                    $joinRelationship === self:: AND ? $this->andWhere($addWith) : $this->orWhere($addWith);
                 }
             }
         }
@@ -646,7 +646,7 @@ 
             foreach ($operationsWithArgs as $operation => $arguments) {
                 assert(
                     is_iterable($arguments) === true || is_array($arguments) === true,
-                    "Operation arguments are missing for `$columnName` column. " .
+                    "Operation arguments are missing for `$columnName` column. ".
                     'Use an empty array as an empty argument list.'
                 );
                 $expression->add($this->createFilterExpression($fullColumnName, $operation, $arguments));
@@ -699,7 +699,7 @@ 
     {
         $platform = $this->getConnection()->getDatabasePlatform();
 
-        return $platform->quoteSingleIdentifier($tableOrAlias) . '.' . $platform->quoteSingleIdentifier($column);
+        return $platform->quoteSingleIdentifier($tableOrAlias).'.'.$platform->quoteSingleIdentifier($column);
     }
 
     /**
@@ -741,7 +741,7 @@ 
      */
     public function createAlias(string $tableName): string
     {
-        $alias                          = $tableName . (++$this->aliasIdCounter);
+        $alias                          = $tableName.(++$this->aliasIdCounter);
         $this->knownAliases[$tableName] = $alias;
 
         return $alias;
@@ -764,7 +764,7 @@ 
         string $targetColumn
     ): string {
         $targetAlias   = $this->createAlias($targetTable);
-        $joinCondition = $this->quoteDoubleIdentifier($fromAlias, $fromColumn) . '=' .
+        $joinCondition = $this->quoteDoubleIdentifier($fromAlias, $fromColumn).'='.
             $this->quoteDoubleIdentifier($targetAlias, $targetColumn);
 
         $this->innerJoin(
@@ -977,7 +977,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/Http/Traits/DefaultControllerMethodsTrait.php

Doc Comments +1 added lines, -1 removed lines

@@ -251,7 +251,7 @@
      * @param FactoryInterface           $errorFactory
      * @param FormatterFactoryInterface  $formatterFactory
      *
-     * @return ResponseInterface
+     * @return string
      */
     protected static function defaultCreate(
         string $requestBody,

Spacing +13 added lines, -13 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare (strict_types = 1);
+<?php declare(strict_types=1);
 
 namespace Limoncello\Flute\Http\Traits;
 
@@ -110,7 +110,7 @@ 
         EncoderInterface $encoder
     ): ResponseInterface {
         $queryParser->parse($index, $queryParams);
-        $validatedIndex = (string)$queryParser->getIdentity();
+        $validatedIndex = (string) $queryParser->getIdentity();
 
         $model = $mapper->applyQueryParameters($queryParser, $crud)->read($validatedIndex);
         assert(!($model instanceof PaginatedDataInterface));
@@ -280,7 +280,7 @@ 
             $title     = $formatter->formatMessage(Messages::MSG_ERR_CANNOT_CREATE_NON_UNIQUE_RESOURCE);
             $details   = null;
             $errorCode = JsonApiResponse::HTTP_CONFLICT;
-            $errors->addDataError($title, $details, (string)$errorCode);
+            $errors->addDataError($title, $details, (string) $errorCode);
 
             throw new JsonApiException($errors);
         }
@@ -409,7 +409,7 @@ 
         list ($index, $attributes, $toMany) = static::mapSchemaDataToModelData($captures, $schemaClass, $schemaInfo);
 
         try {
-            $updated = $crud->update((string)$index, $attributes, $toMany);
+            $updated = $crud->update((string) $index, $attributes, $toMany);
         } /** @noinspection PhpRedundantCatchClauseInspection */ catch (UniqueConstraintViolationException $exception) {
             $errors    = $errorFactory->createErrorCollection();
             $formatter = $formatterFactory->createFormatter(Messages::NAMESPACE_NAME);
@@ -417,7 +417,7 @@ 
                 ->formatMessage(Messages::MSG_ERR_CANNOT_UPDATE_WITH_UNIQUE_CONSTRAINT_VIOLATION);
             $details   = null;
             $errorCode = JsonApiResponse::HTTP_CONFLICT;
-            $errors->addDataError($title, $details, (string)$errorCode);
+            $errors->addDataError($title, $details, (string) $errorCode);
 
             throw new JsonApiException($errors);
         }
@@ -470,7 +470,7 @@ 
         CrudInterface $crud,
         EncoderInterface $encoder
     ): ResponseInterface {
-        $validatedIndex = (string)$queryParser->parse($index)->getIdentity();
+        $validatedIndex = (string) $queryParser->parse($index)->getIdentity();
         $crud->remove($validatedIndex);
 
         $responses = static::defaultCreateResponses($requestUri, $encoder);
@@ -523,7 +523,7 @@ 
         $captures = $dataValidator->assertRelationship($index, $jsonRelName, $jsonData)->getCaptures();
         $relIds   = $captures[$jsonRelName];
 
-        $validatedIndex = (string)$queryParser->parse($index)->getIdentity();
+        $validatedIndex = (string) $queryParser->parse($index)->getIdentity();
         $parentCrud->createInBelongsToManyRelationship($validatedIndex, $modelRelName, $relIds);
 
         $responses = static::defaultCreateResponses($requestUri, $encoder);
@@ -576,7 +576,7 @@ 
         $captures = $dataValidator->assertRelationship($index, $jsonRelName, $jsonData)->getCaptures();
         $relIds   = $captures[$jsonRelName];
 
-        $validatedIndex = (string)$queryParser->parse($index)->getIdentity();
+        $validatedIndex = (string) $queryParser->parse($index)->getIdentity();
         $parentCrud->removeInBelongsToManyRelationship($validatedIndex, $modelRelName, $relIds);
 
         $responses = static::defaultCreateResponses($requestUri, $encoder);
@@ -624,7 +624,7 @@ 
         $modelClass = $schemaClass::MODEL;
         assert($schemaInfo->hasRelationship($modelClass, $modelRelName));
         assert(
-            ($type =$schemaInfo->getRelationshipType($modelClass, $modelRelName)) === RelationshipTypes::BELONGS_TO ||
+            ($type = $schemaInfo->getRelationshipType($modelClass, $modelRelName)) === RelationshipTypes::BELONGS_TO ||
             $type === RelationshipTypes::BELONGS_TO_MANY
         );
 
@@ -633,7 +633,7 @@ 
 
         // If we are here then we have something in 'data' section.
 
-        $validatedIndex = (string)$queryParser->parse($index)->getIdentity();
+        $validatedIndex = (string) $queryParser->parse($index)->getIdentity();
         list (, $attributes, $toMany) = static::mapSchemaDataToModelData($captures, $schemaClass, $schemaInfo);
 
         $updated = $crud->update($validatedIndex, $attributes, $toMany);
@@ -875,7 +875,7 @@ 
     ): string {
         $schema    = $jsonSchemas->getSchema($model);
         $selfLink  = $schema->getSelfLink($model);
-        $urlPrefix = (string)$requestUri->withPath('')->withQuery('')->withFragment('');
+        $urlPrefix = (string) $requestUri->withPath('')->withQuery('')->withFragment('');
         $fullUrl   = $selfLink->getStringRepresentation($urlPrefix);
 
         return $fullUrl;
@@ -888,7 +888,7 @@ 
      */
     private static function assertClassValueDefined(?string $value): void
     {
-        assert(empty($value) === false, 'Value should be defined in `' . static::class . '`.');
+        assert(empty($value) === false, 'Value should be defined in `'.static::class.'`.');
     }
 
     /**
@@ -901,7 +901,7 @@ 
     {
         assert(
             array_key_exists($interface, class_implements($class)) === true,
-            "Class `$class` should implement `" . $interface . '` interface.'
+            "Class `$class` should implement `".$interface.'` interface.'
         );
     }
 }

src/Http/ThrowableHandlers/FluteThrowableHandler.php

Spacing +3 added lines, -3 removed lines

@@ -1,4 +1,4 @@ 
-<?php declare (strict_types = 1);
+<?php declare(strict_types=1);
 
 namespace Limoncello\Flute\Http\ThrowableHandlers;
 
@@ -132,9 +132,9 @@ 
             $details  = null;
             if ($this->isDebug === true) {
                 $message = $throwable->getMessage();
-                $details = (string)$throwable;
+                $details = (string) $throwable;
             }
-            $errors->add(new Error(null, null, null, (string)$httpCode, null, $message, $details));
+            $errors->add(new Error(null, null, null, (string) $httpCode, null, $message, $details));
         }
 
         // encode the error and send to client