limoncello-php-dist / flute

src/Validation/JsonApi/Execution/JsonApiQueryRulesSerializer.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     private $serializedRules = [];
 
     /** Index key */
-    protected const IDENTITY_RULE= 0;
+    protected const IDENTITY_RULE = 0;
 
     /** Index key */
     protected const FILTER_RULES = self::IDENTITY_RULE + 1;

src/Http/Traits/DefaultControllerMethodsTrait.php

Spacing +3 added lines, -3 removed lines

@@ -649,7 +649,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
         );
 
@@ -904,7 +904,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.'`.');
     }
 
     /**
@@ -917,7 +917,7 @@ 
     {
         assert(
             array_key_exists($interface, class_implements($class)) === true,
-            "Class `$class` should implement `" . $interface . '` interface.'
+            "Class `$class` should implement `".$interface.'` interface.'
         );
     }
 }

Doc Comments +1 added lines, -1 removed lines

@@ -279,7 +279,7 @@
      * @param string                     $messagesNamespace
      * @param string                     $errorMessage
      *
-     * @return ResponseInterface
+     * @return string
      */
     protected static function defaultCreate(
         string $requestBody,

src/Validation/JsonApi/DataParser.php

Spacing +5 added lines, -6 removed lines

@@ -242,12 +242,11 @@ 
             $this->executeStarts($this->getSerializer()::readRuleStartIndexes($ruleIndexes));
             $ruleIndex = $this->getSerializer()::readRuleIndex($ruleIndexes);
             $isFoundInToOne === true ?
-                $this->validateAsToOneRelationship($ruleIndex, $name, $jsonData) :
-                $this->validateAsToManyRelationship($ruleIndex, $name, $jsonData);
+                $this->validateAsToOneRelationship($ruleIndex, $name, $jsonData) : $this->validateAsToManyRelationship($ruleIndex, $name, $jsonData);
             $this->executeEnds($this->getSerializer()::readRuleEndIndexes($ruleIndexes));
 
             if (count($this->getErrorAggregator()) > 0) {
-                $status  = JsonApiResponse::HTTP_CONFLICT;
+                $status = JsonApiResponse::HTTP_CONFLICT;
                 foreach ($this->getErrorAggregator()->get() as $error) {
                     $this->getJsonApiErrorCollection()->addValidationRelationshipError($error, $status);
                     $this->addErrorStatus($status);
@@ -364,7 +363,7 @@ 
 
         if (count($this->getErrorAggregator()) > 0) {
             $title = $this->formatMessage(ErrorCodes::INVALID_VALUE);
-            $status  = JsonApiResponse::HTTP_CONFLICT;
+            $status = JsonApiResponse::HTTP_CONFLICT;
             foreach ($this->getErrorAggregator()->get() as $error) {
                 $details = $this->getMessage($error);
                 $this->getJsonApiErrorCollection()->addDataTypeError($title, $details, $status);
@@ -459,7 +458,7 @@ 
         $this->executeEnds($ends);
 
         if (count($this->getErrorAggregator()) > 0) {
-            $status  = JsonApiResponse::HTTP_UNPROCESSABLE_ENTITY;
+            $status = JsonApiResponse::HTTP_UNPROCESSABLE_ENTITY;
             foreach ($this->getErrorAggregator()->get() as $error) {
                 $this->getJsonApiErrorCollection()->addValidationAttributeError($error, $status);
             }
@@ -528,7 +527,7 @@ 
         $this->executeEnds($ends);
 
         if (count($this->getErrorAggregator()) > 0) {
-            $status  = JsonApiResponse::HTTP_CONFLICT;
+            $status = JsonApiResponse::HTTP_CONFLICT;
             foreach ($this->getErrorAggregator()->get() as $error) {
                 $this->getJsonApiErrorCollection()->addValidationRelationshipError($error, $status);
             }

src/Schema/Schema.php

Spacing +2 added lines, -2 removed lines

@@ -212,12 +212,12 @@
             return [static::DATA => $data->getData()];
         }
 
-        $buildUrl = function ($offset) use ($data, $uri) {
+        $buildUrl = function($offset) use ($data, $uri) {
             $paramsWithPaging = [
                 JsonApiQueryParserInterface::PARAM_PAGING_OFFSET => $offset,
                 JsonApiQueryParserInterface::PARAM_PAGING_LIMIT  => $data->getLimit(),
             ];
-            $fullUrl          = $uri . '?' . http_build_query($paramsWithPaging);
+            $fullUrl = $uri.'?'.http_build_query($paramsWithPaging);
 
             return $fullUrl;
         };

src/Http/Traits/FluteRoutesTrait.php

Spacing +20 added lines, -20 removed lines

@@ -48,11 +48,11 @@ 
         assert(class_exists($controllerClass) === true);
 
         $groupPrefix = $group->getUriPrefix();
-        $indexSlug   = '/{' . JCI::ROUTE_KEY_INDEX . '}';
-        $params      = function (string $method) use ($groupPrefix, $resourceName): array {
+        $indexSlug   = '/{'.JCI::ROUTE_KEY_INDEX.'}';
+        $params      = function(string $method) use ($groupPrefix, $resourceName): array {
             return [RouteInterface::PARAM_NAME => static::routeName($groupPrefix, $resourceName, $method)];
         };
-        $handler     = function (string $method) use ($controllerClass): array {
+        $handler     = function(string $method) use ($controllerClass): array {
             return [$controllerClass, $method];
         };
 
@@ -65,13 +65,13 @@ 
             $group->post($resourceName, $handler(JCI::METHOD_CREATE), $params(JCI::METHOD_CREATE));
         }
         if (in_array(ControllerReadInterface::class, $classInterfaces) === true) {
-            $group->get($resourceName . $indexSlug, $handler(JCI::METHOD_READ), $params(JCI::METHOD_READ));
+            $group->get($resourceName.$indexSlug, $handler(JCI::METHOD_READ), $params(JCI::METHOD_READ));
         }
         if (in_array(ControllerUpdateInterface::class, $classInterfaces) === true) {
-            $group->patch($resourceName . $indexSlug, $handler(JCI::METHOD_UPDATE), $params(JCI::METHOD_UPDATE));
+            $group->patch($resourceName.$indexSlug, $handler(JCI::METHOD_UPDATE), $params(JCI::METHOD_UPDATE));
         }
         if (in_array(ControllerDeleteInterface::class, $classInterfaces) === true) {
-            $group->delete($resourceName . $indexSlug, $handler(JCI::METHOD_DELETE), $params(JCI::METHOD_DELETE));
+            $group->delete($resourceName.$indexSlug, $handler(JCI::METHOD_DELETE), $params(JCI::METHOD_DELETE));
         }
 
         return $group;
@@ -97,11 +97,11 @@ 
         }
 
         $groupPrefix = $group->getUriPrefix();
-        $slugged     = $subUri . '/{' . FCI::ROUTE_KEY_INDEX . '}';
-        $params      = function (string $method) use ($groupPrefix, $subUri) : array {
+        $slugged     = $subUri.'/{'.FCI::ROUTE_KEY_INDEX.'}';
+        $params      = function(string $method) use ($groupPrefix, $subUri) : array {
             return [RouteInterface::PARAM_NAME => static::routeName($groupPrefix, $subUri, $method)];
         };
-        $handler     = function (string $method) use ($controllerClass): array {
+        $handler     = function(string $method) use ($controllerClass): array {
             return [$controllerClass, $method];
         };
 
@@ -112,10 +112,10 @@ 
             $group->get($subUri, $handler(FCI::METHOD_INDEX), $params(FCI::METHOD_INDEX));
         }
         if (in_array(ControllerInstanceInterface::class, $classInterfaces) === true) {
-            $group->get($subUri . $createSubUrl, $handler(FCI::METHOD_INSTANCE), $params(FCI::METHOD_INSTANCE));
+            $group->get($subUri.$createSubUrl, $handler(FCI::METHOD_INSTANCE), $params(FCI::METHOD_INSTANCE));
         }
         if (in_array(ControllerCreateInterface::class, $classInterfaces) === true) {
-            $group->post($subUri . $createSubUrl, $handler(FCI::METHOD_CREATE), $params(FCI::METHOD_CREATE));
+            $group->post($subUri.$createSubUrl, $handler(FCI::METHOD_CREATE), $params(FCI::METHOD_CREATE));
         }
         if (in_array(ControllerReadInterface::class, $classInterfaces) === true) {
             $group->get($slugged, $handler(FCI::METHOD_READ), $params(FCI::METHOD_READ));
@@ -124,7 +124,7 @@ 
             $group->post($slugged, $handler(FCI::METHOD_UPDATE), $params(FCI::METHOD_UPDATE));
         }
         if (in_array(ControllerDeleteInterface::class, $classInterfaces) === true) {
-            $deleteUri = $slugged . '/' . FCI::METHOD_DELETE;
+            $deleteUri = $slugged.'/'.FCI::METHOD_DELETE;
             $group->post($deleteUri, $handler(FCI::METHOD_DELETE), $params(FCI::METHOD_DELETE));
         }
 
@@ -147,14 +147,14 @@ 
         string $controllerClass,
         string $selfGetMethod
     ): GroupInterface {
-        $resourceIdUri = $resourceName . '/{' . JCI::ROUTE_KEY_INDEX . '}/';
-        $selfUri       = $resourceIdUri . DocumentInterface::KEYWORD_RELATIONSHIPS . '/' . $relationshipName;
+        $resourceIdUri = $resourceName.'/{'.JCI::ROUTE_KEY_INDEX.'}/';
+        $selfUri       = $resourceIdUri.DocumentInterface::KEYWORD_RELATIONSHIPS.'/'.$relationshipName;
 
         return $group
             // `self`
             ->get($selfUri, [$controllerClass, $selfGetMethod])
             // `related`
-            ->get($resourceIdUri . $relationshipName, [$controllerClass, $selfGetMethod]);
+            ->get($resourceIdUri.$relationshipName, [$controllerClass, $selfGetMethod]);
     }
 
     /**
@@ -173,8 +173,8 @@ 
         string $controllerClass,
         string $addMethod
     ): GroupInterface {
-        $url = $resourceName . '/{' . JCI::ROUTE_KEY_INDEX . '}/' .
-            DocumentInterface::KEYWORD_RELATIONSHIPS . '/' . $relationshipName;
+        $url = $resourceName.'/{'.JCI::ROUTE_KEY_INDEX.'}/'.
+            DocumentInterface::KEYWORD_RELATIONSHIPS.'/'.$relationshipName;
 
         return $group->post($url, [$controllerClass, $addMethod]);
     }
@@ -195,8 +195,8 @@ 
         string $controllerClass,
         string $deleteMethod
     ): GroupInterface {
-        $url = $resourceName . '/{' . JCI::ROUTE_KEY_INDEX . '}/' .
-            DocumentInterface::KEYWORD_RELATIONSHIPS . '/' . $relationshipName;
+        $url = $resourceName.'/{'.JCI::ROUTE_KEY_INDEX.'}/'.
+            DocumentInterface::KEYWORD_RELATIONSHIPS.'/'.$relationshipName;
 
         return $group->delete($url, [$controllerClass, $deleteMethod]);
     }
@@ -222,6 +222,6 @@ 
             $subUri = substr($subUri, 0, -1);
         }
 
-        return $prefix . $subUri . '::' . $method;
+        return $prefix.$subUri.'::'.$method;
     }
 }

src/Encoder/Encoder.php

Spacing +2 added lines, -2 removed lines

@@ -125,7 +125,7 @@ 
 
         parse_str($this->getOriginalUri()->getQuery(), $queryParams);
 
-        return function ($offset) use ($pageSize, $queryParams) {
+        return function($offset) use ($pageSize, $queryParams) {
             $paramsWithPaging = array_merge($queryParams, [
                 BaseQueryParserInterface::PARAM_PAGE => [
                     JsonApiQueryParserInterface::PARAM_PAGING_OFFSET => $offset,
@@ -133,7 +133,7 @@ 
                 ],
             ]);
             $newUri           = $this->getOriginalUri()->withQuery(http_build_query($paramsWithPaging));
-            $fullUrl          = (string)$newUri;
+            $fullUrl          = (string) $newUri;
             $link             = $this->getFactory()->createLink($fullUrl, null, true);
 
             return $link;

src/Adapters/ModelQueryBuilder.php

Doc Comments +7 added lines, -7 removed lines

@@ -181,7 +181,7 @@ 
     }
 
     /**
-     * @param iterable $attributes
+     * @param \Generator $attributes
      *
      * @return self
      *
@@ -203,7 +203,7 @@ 
     }
 
     /**
-     * @param iterable $attributes
+     * @param \Generator $attributes
      *
      * @return self
      *
@@ -226,7 +226,7 @@ 
      * @param string   $modelClass
      * @param iterable $attributes
      *
-     * @return iterable
+     * @return \Generator
      *
      * @SuppressWarnings(PHPMD.StaticAccess)
      *
@@ -357,7 +357,7 @@ 
     }
 
     /**
-     * @param iterable $filters
+     * @param \Limoncello\Flute\Api\iterable|null $filters
      *
      * @return self
      *
@@ -373,7 +373,7 @@ 
     }
 
     /**
-     * @param iterable $filters
+     * @param \Limoncello\Flute\Api\iterable|null $filters
      *
      * @return self
      *
@@ -389,7 +389,7 @@ 
     }
 
     /**
-     * @param iterable $filters
+     * @param \Limoncello\Flute\Api\iterable|null $filters
      *
      * @return self
      *
@@ -438,7 +438,7 @@ 
     }
 
     /**
-     * @param iterable $sortParameters
+     * @param \Limoncello\Flute\Api\iterable|null $sortParameters
      *
      * @return self
      */

Spacing +10 added lines, -10 removed lines

@@ -48,7 +48,7 @@ 
     /**
      * Condition joining method.
      */
-    public const OR = self::AND + 1;
+    public const OR = self:: AND + 1;
 
     /**
      * @var string
@@ -419,16 +419,16 @@ 
         string $relationshipName,
         ?iterable $relationshipFilters,
         ?iterable $relationshipSorts,
-        int $joinIndividuals = self::AND,
-        int $joinRelationship = self::AND
+        int $joinIndividuals = self:: AND ,
+        int $joinRelationship = self:: AND
     ): self {
         $targetAlias = $this->createRelationshipAlias($relationshipName);
 
         if ($relationshipFilters !== null) {
-            $addWith = $joinIndividuals === self::AND ? $this->expr()->andX() : $this->expr()->orX();
+            $addWith = $joinIndividuals === self:: AND ? $this->expr()->andX() : $this->expr()->orX();
             $this->applyFilters($addWith, $targetAlias, $relationshipFilters);
             if ($addWith->count() > 0) {
-                $joinRelationship === self::AND ? $this->andWhere($addWith) : $this->orWhere($addWith);
+                $joinRelationship === self:: AND ? $this->andWhere($addWith) : $this->orWhere($addWith);
             }
         }
 
@@ -551,7 +551,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));
@@ -586,7 +586,7 @@ 
      */
     public function buildColumnName(string $table, string $column): string
     {
-        return $this->quoteTableName($table) . '.' . $this->quoteColumnName($column);
+        return $this->quoteTableName($table).'.'.$this->quoteColumnName($column);
     }
 
     /**
@@ -628,7 +628,7 @@ 
      */
     public function createAlias(string $tableName): string
     {
-        $alias                          = $tableName . (++$this->aliasIdCounter);
+        $alias                          = $tableName.(++$this->aliasIdCounter);
         $this->knownAliases[$tableName] = $alias;
 
         return $alias;
@@ -682,7 +682,7 @@ 
         string $targetColumn
     ): string {
         $targetAlias   = $this->createAlias($targetTable);
-        $joinCondition = $this->buildColumnName($fromAlias, $fromColumn) . '=' .
+        $joinCondition = $this->buildColumnName($fromAlias, $fromColumn).'='.
             $this->buildColumnName($targetAlias, $targetColumn);
 
         $this->innerJoin(
@@ -880,7 +880,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.");