limoncello-php-dist / flute

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.");

src/Api/Crud.php

Spacing +23 added lines, -23 removed lines

@@ -208,7 +208,7 @@ 
      */
     public function withIndexesFilter(array $indexes): CrudInterface
     {
-        assert(call_user_func(function () use ($indexes) {
+        assert(call_user_func(function() use ($indexes) {
             $allOk = true;
 
             foreach ($indexes as $index) {
@@ -760,7 +760,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,
@@ -848,7 +848,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++;
             }
@@ -1017,7 +1017,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
@@ -1035,7 +1035,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
@@ -1066,7 +1066,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
@@ -1095,7 +1095,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
@@ -1171,7 +1171,7 @@ 
 
         $this->clearFetchParameters();
 
-        return (int)$deleted;
+        return (int) $deleted;
     }
 
     /**
@@ -1187,7 +1187,7 @@ 
 
         $this->clearFetchParameters();
 
-        return (int)$deleted > 0;
+        return (int) $deleted > 0;
     }
 
     /**
@@ -1212,7 +1212,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
@@ -1253,12 +1253,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']);
@@ -1274,7 +1274,7 @@ 
                 )->execute();
 
                 // add new ones
-                $updated   += $this->addInToManyRelationship(
+                $updated += $this->addInToManyRelationship(
                     $connection,
                     $index,
                     $relationshipName,
@@ -1284,7 +1284,7 @@ 
             }
         });
 
-        return (int)$updated;
+        return (int) $updated;
     }
 
     /**
@@ -1299,9 +1299,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;
 
@@ -1323,9 +1323,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;
 
@@ -1493,7 +1493,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;
     }
@@ -1525,7 +1525,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.
@@ -1772,7 +1772,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,
@@ -1784,7 +1784,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) =
@@ -1839,7 +1839,7 @@ 
                             [$pkName => [FilterParameterInterface::OPERATION_EQUALS => [$parent->{$pkName}]]],
                             []
                         );
-                        $children      = $this->fetchPaginatedResourcesWithoutRelationships(
+                        $children = $this->fetchPaginatedResourcesWithoutRelationships(
                             $clonedBuilder,
                             $clonedBuilder->getModelClass()
                         );