Completed
Push — develop ( f8f2bd...c1ef57 )
by Neomerx
11:00 queued 09:15
created
src/Validation/JsonApi/Rules/ExistInDbTableMultipleWithDoctrineRule.php 1 patch
Spacing   +2 added lines, -3 removed lines patch added patch discarded remove patch
@@ -83,13 +83,12 @@
 block discarded – undo
83 83
                 ->where($builder->expr()->in($primaryName, $placeholders))
84 84
                 ->execute();
85 85
 
86
-            $count  = (int)$statement->fetchColumn();
86
+            $count  = (int) $statement->fetchColumn();
87 87
             $result = $count === count($values);
88 88
         }
89 89
 
90 90
         $reply = $result === true ?
91
-            static::createSuccessReply($values) :
92
-            static::createErrorReply($context, $values, ErrorCodes::EXIST_IN_DATABASE_MULTIPLE);
91
+            static::createSuccessReply($values) : static::createErrorReply($context, $values, ErrorCodes::EXIST_IN_DATABASE_MULTIPLE);
93 92
 
94 93
         return $reply;
95 94
     }
Please login to merge, or discard this patch.
src/Validation/JsonApi/DataParser.php 1 patch
Spacing   +5 added lines, -6 removed lines patch added patch discarded remove patch
@@ -242,12 +242,11 @@  discard block
 block discarded – undo
242 242
             $this->executeStarts($this->getSerializer()::readRuleStartIndexes($ruleIndexes));
243 243
             $ruleIndex = $this->getSerializer()::readRuleIndex($ruleIndexes);
244 244
             $isFoundInToOne === true ?
245
-                $this->validateAsToOneRelationship($ruleIndex, $name, $jsonData) :
246
-                $this->validateAsToManyRelationship($ruleIndex, $name, $jsonData);
245
+                $this->validateAsToOneRelationship($ruleIndex, $name, $jsonData) : $this->validateAsToManyRelationship($ruleIndex, $name, $jsonData);
247 246
             $this->executeEnds($this->getSerializer()::readRuleEndIndexes($ruleIndexes));
248 247
 
249 248
             if (count($this->getErrorAggregator()) > 0) {
250
-                $status  = JsonApiResponse::HTTP_CONFLICT;
249
+                $status = JsonApiResponse::HTTP_CONFLICT;
251 250
                 foreach ($this->getErrorAggregator()->get() as $error) {
252 251
                     $this->getJsonApiErrorCollection()->addValidationRelationshipError($error, $status);
253 252
                     $this->addErrorStatus($status);
@@ -362,7 +361,7 @@  discard block
 block discarded – undo
362 361
 
363 362
         if (count($this->getErrorAggregator()) > 0) {
364 363
             $title = $this->formatMessage(ErrorCodes::INVALID_VALUE);
365
-            $status  = JsonApiResponse::HTTP_CONFLICT;
364
+            $status = JsonApiResponse::HTTP_CONFLICT;
366 365
             foreach ($this->getErrorAggregator()->get() as $error) {
367 366
                 $details = $this->getMessage($error);
368 367
                 $this->getJsonApiErrorCollection()->addDataTypeError($title, $details, $status);
@@ -453,7 +452,7 @@  discard block
 block discarded – undo
453 452
         $this->executeEnds($this->getSerializer()::readRulesEndIndexes($this->getAttributeRules()));
454 453
 
455 454
         if (count($this->getErrorAggregator()) > 0) {
456
-            $status  = JsonApiResponse::HTTP_UNPROCESSABLE_ENTITY;
455
+            $status = JsonApiResponse::HTTP_UNPROCESSABLE_ENTITY;
457 456
             foreach ($this->getErrorAggregator()->get() as $error) {
458 457
                 $this->getJsonApiErrorCollection()->addValidationAttributeError($error, $status);
459 458
             }
@@ -520,7 +519,7 @@  discard block
 block discarded – undo
520 519
         ));
521 520
 
522 521
         if (count($this->getErrorAggregator()) > 0) {
523
-            $status  = JsonApiResponse::HTTP_CONFLICT;
522
+            $status = JsonApiResponse::HTTP_CONFLICT;
524 523
             foreach ($this->getErrorAggregator()->get() as $error) {
525 524
                 $this->getJsonApiErrorCollection()->addValidationRelationshipError($error, $status);
526 525
             }
Please login to merge, or discard this patch.
src/Package/FluteContainerConfigurator.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
 
54 54
         $container[FactoryInterface::class] = $factory;
55 55
 
56
-        $container[JsonSchemasInterface::class] = function (PsrContainerInterface $container) use ($factory) {
56
+        $container[JsonSchemasInterface::class] = function(PsrContainerInterface $container) use ($factory) {
57 57
             $settings     = $container->get(SettingsProviderInterface::class)->get(FluteSettings::class);
58 58
             $modelSchemas = $container->get(ModelSchemaInfoInterface::class);
59 59
 
@@ -64,11 +64,11 @@  discard block
 block discarded – undo
64 64
             );
65 65
         };
66 66
 
67
-        $container[ParametersMapperInterface::class] = function (PsrContainerInterface $container) {
67
+        $container[ParametersMapperInterface::class] = function(PsrContainerInterface $container) {
68 68
             return new ParametersMapper($container->get(JsonSchemasInterface::class));
69 69
         };
70 70
 
71
-        $container[EncoderInterface::class] = function (PsrContainerInterface $container) use ($factory) {
71
+        $container[EncoderInterface::class] = function(PsrContainerInterface $container) use ($factory) {
72 72
             /** @var JsonSchemasInterface $jsonSchemas */
73 73
             $jsonSchemas = $container->get(JsonSchemasInterface::class);
74 74
             $settings    = $container->get(SettingsProviderInterface::class)->get(FluteSettings::class);
@@ -84,19 +84,19 @@  discard block
 block discarded – undo
84 84
             return $encoder;
85 85
         };
86 86
 
87
-        $container[RelationshipPaginationStrategyInterface::class] = function (PsrContainerInterface $container) {
87
+        $container[RelationshipPaginationStrategyInterface::class] = function(PsrContainerInterface $container) {
88 88
             $settings = $container->get(SettingsProviderInterface::class)->get(FluteSettings::class);
89 89
 
90 90
             return new BasicRelationshipPaginationStrategy($settings[FluteSettings::KEY_DEFAULT_PAGING_SIZE]);
91 91
         };
92 92
 
93
-        $container[JsonApiParserFactoryInterface::class] = function (PsrContainerInterface $container) {
93
+        $container[JsonApiParserFactoryInterface::class] = function(PsrContainerInterface $container) {
94 94
             $factory = new JsonApiParserFactory($container);
95 95
 
96 96
             return $factory;
97 97
         };
98 98
 
99
-        $container[FormValidatorFactoryInterface::class] = function (PsrContainerInterface $container) {
99
+        $container[FormValidatorFactoryInterface::class] = function(PsrContainerInterface $container) {
100 100
             $factory = new FormValidatorFactory($container);
101 101
 
102 102
             return $factory;
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
      */
115 115
     public static function configureExceptionHandler(LimoncelloContainerInterface $container)
116 116
     {
117
-        $container[ThrowableHandlerInterface::class] = function (PsrContainerInterface $container) {
117
+        $container[ThrowableHandlerInterface::class] = function(PsrContainerInterface $container) {
118 118
             /** @var CacheSettingsProviderInterface $provider */
119 119
             $provider      = $container->get(CacheSettingsProviderInterface::class);
120 120
             $appConfig     = $provider->getApplicationConfiguration();
Please login to merge, or discard this patch.
src/Encoder/Encoder.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
 
125 125
         parse_str($this->getOriginalUri()->getQuery(), $queryParams);
126 126
 
127
-        return function ($offset) use ($pageSize, $queryParams) {
127
+        return function($offset) use ($pageSize, $queryParams) {
128 128
             $paramsWithPaging = array_merge($queryParams, [
129 129
                 BaseQueryParserInterface::PARAM_PAGE => [
130 130
                     JsonApiQueryParserInterface::PARAM_PAGING_OFFSET => $offset,
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
                 ],
133 133
             ]);
134 134
             $newUri  = $this->getOriginalUri()->withQuery(http_build_query($paramsWithPaging));
135
-            $fullUrl = (string)$newUri;
135
+            $fullUrl = (string) $newUri;
136 136
             $link    = $this->getFactory()->createLink(false, $fullUrl, false);
137 137
 
138 138
             return $link;
Please login to merge, or discard this patch.
src/Http/ThrowableHandlers/FluteThrowableHandler.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -130,7 +130,7 @@
 block discarded – undo
130 130
             $details  = null;
131 131
             if ($this->isDebug === true) {
132 132
                 $message = $throwable->getMessage();
133
-                $details = (string)$throwable;
133
+                $details = (string) $throwable;
134 134
             }
135 135
             $errors->add(new Error(null, null, null, $httpCode, null, $message, $details));
136 136
         }
Please login to merge, or discard this patch.
src/Schema/Schema.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -235,14 +235,14 @@  discard block
 block discarded – undo
235 235
         // if we are here then relationship contains paginated data, so we have to add pagination links
236 236
         $offset    = $relationshipData->getOffset();
237 237
         $limit     = $relationshipData->getLimit();
238
-        $urlPrefix = $this->getRelationshipSelfSubUrl($model, $jsonRelName) . '?';
239
-        $buildLink = function (int $offset, int $limit) use ($urlPrefix) : LinkInterface {
238
+        $urlPrefix = $this->getRelationshipSelfSubUrl($model, $jsonRelName).'?';
239
+        $buildLink = function(int $offset, int $limit) use ($urlPrefix) : LinkInterface {
240 240
             $paramsWithPaging = [
241 241
                 JsonApiQueryParserInterface::PARAM_PAGING_OFFSET => $offset,
242 242
                 JsonApiQueryParserInterface::PARAM_PAGING_LIMIT  => $limit,
243 243
             ];
244 244
 
245
-            $subUrl = $urlPrefix . http_build_query($paramsWithPaging);
245
+            $subUrl = $urlPrefix.http_build_query($paramsWithPaging);
246 246
 
247 247
             return $this->getFactory()->createLink(true, $subUrl, false);
248 248
         };
@@ -341,6 +341,6 @@  discard block
 block discarded – undo
341 341
      */
342 342
     private function getRelationshipSelfSubUrl(ModelInterface $model, string $jsonRelName): string
343 343
     {
344
-        return $this->getSelfSubUrl($model) . '/relationships/' . $jsonRelName;
344
+        return $this->getSelfSubUrl($model).'/relationships/'.$jsonRelName;
345 345
     }
346 346
 }
Please login to merge, or discard this patch.
src/Adapters/ModelQueryBuilder.php 2 patches
Doc Comments   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -217,7 +217,7 @@  discard block
 block discarded – undo
217 217
     }
218 218
 
219 219
     /**
220
-     * @param iterable $attributes
220
+     * @param \Generator $attributes
221 221
      *
222 222
      * @return self
223 223
      *
@@ -239,7 +239,7 @@  discard block
 block discarded – undo
239 239
     }
240 240
 
241 241
     /**
242
-     * @param iterable $attributes
242
+     * @param \Generator $attributes
243 243
      *
244 244
      * @return self
245 245
      *
@@ -262,7 +262,7 @@  discard block
 block discarded – undo
262 262
      * @param string   $modelClass
263 263
      * @param iterable $attributes
264 264
      *
265
-     * @return iterable
265
+     * @return \Generator
266 266
      *
267 267
      * @SuppressWarnings(PHPMD.StaticAccess)
268 268
      *
@@ -397,7 +397,7 @@  discard block
 block discarded – undo
397 397
     }
398 398
 
399 399
     /**
400
-     * @param iterable $filters
400
+     * @param \Limoncello\Flute\Api\iterable|null $filters
401 401
      *
402 402
      * @return self
403 403
      *
@@ -413,7 +413,7 @@  discard block
 block discarded – undo
413 413
     }
414 414
 
415 415
     /**
416
-     * @param iterable $filters
416
+     * @param \Limoncello\Flute\Api\iterable|null $filters
417 417
      *
418 418
      * @return self
419 419
      *
@@ -429,7 +429,7 @@  discard block
 block discarded – undo
429 429
     }
430 430
 
431 431
     /**
432
-     * @param iterable $filters
432
+     * @param \Limoncello\Flute\Api\iterable|null $filters
433 433
      *
434 434
      * @return self
435 435
      *
@@ -523,7 +523,7 @@  discard block
 block discarded – undo
523 523
     }
524 524
 
525 525
     /**
526
-     * @param iterable $sortParameters
526
+     * @param \Limoncello\Flute\Api\iterable|null $sortParameters
527 527
      *
528 528
      * @return self
529 529
      *
Please login to merge, or discard this patch.
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
     /**
49 49
      * Condition joining method.
50 50
      */
51
-    public const OR = self::AND + 1;
51
+    public const OR = self:: AND + 1;
52 52
 
53 53
     /**
54 54
      * @var string
@@ -463,8 +463,8 @@  discard block
 block discarded – undo
463 463
         string $relationshipName,
464 464
         ?iterable $relationshipFilters,
465 465
         ?iterable $relationshipSorts,
466
-        int $joinIndividuals = self::AND,
467
-        int $joinRelationship = self::AND
466
+        int $joinIndividuals = self:: AND ,
467
+        int $joinRelationship = self:: AND
468 468
     ): self {
469 469
         $targetAlias = null;
470 470
 
@@ -476,7 +476,7 @@  discard block
 block discarded – undo
476 476
             $reversePk = $isBelongsTo === true ?
477 477
                 $this->getModelSchemas()->getReversePrimaryKey($this->getModelClass(), $relationshipName)[0] : null;
478 478
 
479
-            $addWith = $joinIndividuals === self::AND ? $this->expr()->andX() : $this->expr()->orX();
479
+            $addWith = $joinIndividuals === self:: AND ? $this->expr()->andX() : $this->expr()->orX();
480 480
 
481 481
             foreach ($relationshipFilters as $columnName => $operationsWithArgs) {
482 482
                 if ($columnName === $reversePk) {
@@ -496,14 +496,14 @@  discard block
 block discarded – undo
496 496
                 foreach ($operationsWithArgs as $operation => $arguments) {
497 497
                     assert(
498 498
                         is_iterable($arguments) === true || is_array($arguments) === true,
499
-                        "Operation arguments are missing for `$columnName` column. " .
499
+                        "Operation arguments are missing for `$columnName` column. ".
500 500
                         'Use an empty array as an empty argument list.'
501 501
                     );
502 502
                     $addWith->add($this->createFilterExpression($fullColumnName, $operation, $arguments));
503 503
                 }
504 504
 
505 505
                 if ($addWith->count() > 0) {
506
-                    $joinRelationship === self::AND ? $this->andWhere($addWith) : $this->orWhere($addWith);
506
+                    $joinRelationship === self:: AND ? $this->andWhere($addWith) : $this->orWhere($addWith);
507 507
                 }
508 508
             }
509 509
         }
@@ -644,7 +644,7 @@  discard block
 block discarded – undo
644 644
             foreach ($operationsWithArgs as $operation => $arguments) {
645 645
                 assert(
646 646
                     is_iterable($arguments) === true || is_array($arguments) === true,
647
-                    "Operation arguments are missing for `$columnName` column. " .
647
+                    "Operation arguments are missing for `$columnName` column. ".
648 648
                     'Use an empty array as an empty argument list.'
649 649
                 );
650 650
                 $expression->add($this->createFilterExpression($fullColumnName, $operation, $arguments));
@@ -697,7 +697,7 @@  discard block
 block discarded – undo
697 697
     {
698 698
         $platform = $this->getConnection()->getDatabasePlatform();
699 699
 
700
-        return $platform->quoteSingleIdentifier($tableOrAlias) . '.' . $platform->quoteSingleIdentifier($column);
700
+        return $platform->quoteSingleIdentifier($tableOrAlias).'.'.$platform->quoteSingleIdentifier($column);
701 701
     }
702 702
 
703 703
     /**
@@ -739,7 +739,7 @@  discard block
 block discarded – undo
739 739
      */
740 740
     public function createAlias(string $tableName): string
741 741
     {
742
-        $alias                          = $tableName . (++$this->aliasIdCounter);
742
+        $alias                          = $tableName.(++$this->aliasIdCounter);
743 743
         $this->knownAliases[$tableName] = $alias;
744 744
 
745 745
         return $alias;
@@ -762,7 +762,7 @@  discard block
 block discarded – undo
762 762
         string $targetColumn
763 763
     ): string {
764 764
         $targetAlias   = $this->createAlias($targetTable);
765
-        $joinCondition = $this->quoteDoubleIdentifier($fromAlias, $fromColumn) . '=' .
765
+        $joinCondition = $this->quoteDoubleIdentifier($fromAlias, $fromColumn).'='.
766 766
             $this->quoteDoubleIdentifier($targetAlias, $targetColumn);
767 767
 
768 768
         $this->innerJoin(
@@ -975,7 +975,7 @@  discard block
 block discarded – undo
975 975
         } else {
976 976
             assert(
977 977
                 $value !== null,
978
-                'It seems you are trying to use `null` with =, >, <, or etc operator. ' .
978
+                'It seems you are trying to use `null` with =, >, <, or etc operator. '.
979 979
                 'Use `is null` or `not null` instead.'
980 980
             );
981 981
             assert(is_string($value), "Only strings, booleans and integers are supported.");
Please login to merge, or discard this patch.