Completed
Push — master ( cff73b...5571b3 )
by Neomerx
11:32
created
src/Api/Crud.php 1 patch
Spacing   +18 added lines, -19 removed lines patch added patch discarded remove patch
@@ -687,7 +687,7 @@  discard block
 block discarded – undo
687 687
             $classAtPath = new ArrayObject();
688 688
             $idsAtPath   = new ArrayObject();
689 689
 
690
-            $registerModelAtRoot = function ($model) use ($modelStorage, $modelsAtPath, $idsAtPath): void {
690
+            $registerModelAtRoot = function($model) use ($modelStorage, $modelsAtPath, $idsAtPath): void {
691 691
                 self::registerModelAtPath(
692 692
                     $model,
693 693
                     static::ROOT_PATH,
@@ -775,7 +775,7 @@  discard block
 block discarded – undo
775 775
                 assert(is_string($pathPiece));
776 776
                 $parent                    = $tmpPath;
777 777
                 $tmpPath                   = empty($tmpPath) === true ?
778
-                    $pathPiece : $tmpPath . static::PATH_SEPARATOR . $pathPiece;
778
+                    $pathPiece : $tmpPath.static::PATH_SEPARATOR.$pathPiece;
779 779
                 $normalizedPaths[$tmpPath] = [$parent, $pathPiece];
780 780
                 $pathsDepths[$parent]      = $parentDepth++;
781 781
             }
@@ -930,7 +930,7 @@  discard block
 block discarded – undo
930 930
             $countBuilder = $this->getConnection()->createQueryBuilder();
931 931
             $countBuilder->setParameters($builder->getParameters());
932 932
 
933
-            $countBuilder->select('COUNT(*)')->from('(' . $builder->getSQL() . ') AS RESULT');
933
+            $countBuilder->select('COUNT(*)')->from('('.$builder->getSQL().') AS RESULT');
934 934
 
935 935
             $builder = $countBuilder;
936 936
         }
@@ -960,7 +960,7 @@  discard block
 block discarded – undo
960 960
     ): ModelQueryBuilder {
961 961
         assert(
962 962
             $this->getModelSchemes()->hasRelationship($this->getModelClass(), $relationshipName),
963
-            "Relationship `$relationshipName` do not exist in model `" . $this->getModelClass() . '`'
963
+            "Relationship `$relationshipName` do not exist in model `".$this->getModelClass().'`'
964 964
         );
965 965
 
966 966
         // as we read data from a relationship our main table and model would be the table/model in the relationship
@@ -979,8 +979,7 @@  discard block
 block discarded – undo
979 979
             $filters = $this->getFilters();
980 980
             $sorts   = $this->getSorts();
981 981
             $this->areFiltersWithAnd() ?
982
-                $builder->addRelationshipFiltersAndSortsWithAnd($reverseRelName, $filters, $sorts) :
983
-                $builder->addRelationshipFiltersAndSortsWithOr($reverseRelName, $filters, $sorts);
982
+                $builder->addRelationshipFiltersAndSortsWithAnd($reverseRelName, $filters, $sorts) : $builder->addRelationshipFiltersAndSortsWithOr($reverseRelName, $filters, $sorts);
984 983
         }
985 984
         // ... and the input filters to actual data we select
986 985
         if ($relationshipFilters !== null) {
@@ -1008,7 +1007,7 @@  discard block
 block discarded – undo
1008 1007
     ) {
1009 1008
         assert(
1010 1009
             $this->getModelSchemes()->hasRelationship($this->getModelClass(), $name),
1011
-            "Relationship `$name` do not exist in model `" . $this->getModelClass() . '`'
1010
+            "Relationship `$name` do not exist in model `".$this->getModelClass().'`'
1012 1011
         );
1013 1012
 
1014 1013
         // depending on the relationship type we expect the result to be either single resource or a collection
@@ -1035,7 +1034,7 @@  discard block
 block discarded – undo
1035 1034
     ): array {
1036 1035
         assert(
1037 1036
             $this->getModelSchemes()->hasRelationship($this->getModelClass(), $name),
1038
-            "Relationship `$name` do not exist in model `" . $this->getModelClass() . '`'
1037
+            "Relationship `$name` do not exist in model `".$this->getModelClass().'`'
1039 1038
         );
1040 1039
 
1041 1040
         // depending on the relationship type we expect the result to be either single resource or a collection
@@ -1109,7 +1108,7 @@  discard block
 block discarded – undo
1109 1108
 
1110 1109
         $this->clearFetchParameters();
1111 1110
 
1112
-        return (int)$deleted;
1111
+        return (int) $deleted;
1113 1112
     }
1114 1113
 
1115 1114
     /**
@@ -1123,7 +1122,7 @@  discard block
 block discarded – undo
1123 1122
 
1124 1123
         $this->clearFetchParameters();
1125 1124
 
1126
-        return (int)$deleted > 0;
1125
+        return (int) $deleted > 0;
1127 1126
     }
1128 1127
 
1129 1128
     /**
@@ -1144,7 +1143,7 @@  discard block
 block discarded – undo
1144 1143
 
1145 1144
         $this->clearBuilderParameters()->clearFetchParameters();
1146 1145
 
1147
-        $this->inTransaction(function () use ($saveMain, $toMany, &$index) {
1146
+        $this->inTransaction(function() use ($saveMain, $toMany, &$index) {
1148 1147
             $saveMain->execute();
1149 1148
 
1150 1149
             // if no index given will use last insert ID as index
@@ -1160,7 +1159,7 @@  discard block
 block discarded – undo
1160 1159
                         ->prepareCreateInToManyRelationship($relationshipName, $index, $secondaryIdBindName)
1161 1160
                 );
1162 1161
                 foreach ($secondaryIds as $secondaryId) {
1163
-                    $inserted += (int)$saveToMany->setParameter($secondaryIdBindName, $secondaryId)->execute();
1162
+                    $inserted += (int) $saveToMany->setParameter($secondaryIdBindName, $secondaryId)->execute();
1164 1163
                 }
1165 1164
             }
1166 1165
         });
@@ -1189,12 +1188,12 @@  discard block
 block discarded – undo
1189 1188
             ->createBuilder($this->getModelClass())
1190 1189
             ->updateModels($allowedChanges)
1191 1190
             ->addFiltersWithAndToTable($filters);
1192
-        $saveMain       = $this->builderSaveResourceOnUpdate($saveMain);
1191
+        $saveMain = $this->builderSaveResourceOnUpdate($saveMain);
1193 1192
         $saveMain->getSQL(); // prepare
1194 1193
 
1195 1194
         $this->clearBuilderParameters()->clearFetchParameters();
1196 1195
 
1197
-        $this->inTransaction(function () use ($saveMain, $toMany, $index, &$updated) {
1196
+        $this->inTransaction(function() use ($saveMain, $toMany, $index, &$updated) {
1198 1197
             $updated = $saveMain->execute();
1199 1198
 
1200 1199
             foreach ($toMany as $relationshipName => $secondaryIds) {
@@ -1214,12 +1213,12 @@  discard block
 block discarded – undo
1214 1213
                         ->prepareCreateInToManyRelationship($relationshipName, $index, $secondaryIdBindName)
1215 1214
                 );
1216 1215
                 foreach ($secondaryIds as $secondaryId) {
1217
-                    $updated += (int)$saveToMany->setParameter($secondaryIdBindName, $secondaryId)->execute();
1216
+                    $updated += (int) $saveToMany->setParameter($secondaryIdBindName, $secondaryId)->execute();
1218 1217
                 }
1219 1218
             }
1220 1219
         });
1221 1220
 
1222
-        return (int)$updated;
1221
+        return (int) $updated;
1223 1222
     }
1224 1223
 
1225 1224
     /**
@@ -1555,7 +1554,7 @@  discard block
 block discarded – undo
1555 1554
 
1556 1555
         $pkName = $this->getModelSchemes()->getPrimaryKey($parentClass);
1557 1556
 
1558
-        $registerModelAtPath = function ($model, string $path) use ($deDup, $modelsAtPath, $idsAtPath) {
1557
+        $registerModelAtPath = function($model, string $path) use ($deDup, $modelsAtPath, $idsAtPath) {
1559 1558
             return self::registerModelAtPath(
1560 1559
                 $model,
1561 1560
                 $path,
@@ -1567,7 +1566,7 @@  discard block
 block discarded – undo
1567 1566
         };
1568 1567
 
1569 1568
         foreach ($childRelationships as $name) {
1570
-            $childrenPath = $parentsPath !== static::ROOT_PATH ? $parentsPath . static::PATH_SEPARATOR . $name : $name;
1569
+            $childrenPath = $parentsPath !== static::ROOT_PATH ? $parentsPath.static::PATH_SEPARATOR.$name : $name;
1571 1570
 
1572 1571
             $relationshipType = $this->getModelSchemes()->getRelationshipType($parentClass, $name);
1573 1572
             list ($targetModelClass, $reverseRelName) =
@@ -1617,7 +1616,7 @@  discard block
 block discarded – undo
1617 1616
                             [$pkName => [FilterParameterInterface::OPERATION_EQUALS => [$parent->{$pkName}]]],
1618 1617
                             []
1619 1618
                         );
1620
-                        $children      = $this->fetchPaginatedResourcesWithoutRelationships(
1619
+                        $children = $this->fetchPaginatedResourcesWithoutRelationships(
1621 1620
                             $clonedBuilder,
1622 1621
                             $clonedBuilder->getModelClass()
1623 1622
                         );
Please login to merge, or discard this patch.
src/Adapters/ModelQueryBuilder.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -639,7 +639,7 @@  discard block
 block discarded – undo
639 639
         string $targetColumn
640 640
     ): string {
641 641
         $targetAlias   = $this->createAlias($targetTable);
642
-        $joinCondition = $this->buildColumnName($fromAlias, $fromColumn) . '=' .
642
+        $joinCondition = $this->buildColumnName($fromAlias, $fromColumn).'='.
643 643
             $this->buildColumnName($targetAlias, $targetColumn);
644 644
 
645 645
         $this->innerJoin(
@@ -685,7 +685,7 @@  discard block
 block discarded – undo
685 685
      */
686 686
     private function createAlias(string $tableName): string
687 687
     {
688
-        $alias                          = $tableName . (++$this->aliasIdCounter);
688
+        $alias                          = $tableName.(++$this->aliasIdCounter);
689 689
         $this->knownAliases[$tableName] = $alias;
690 690
 
691 691
         return $alias;
@@ -850,7 +850,7 @@  discard block
 block discarded – undo
850 850
         } else {
851 851
             assert(
852 852
                 $value !== null,
853
-                'It seems you are trying to use `null` with =, >, <, or etc operator. ' .
853
+                'It seems you are trying to use `null` with =, >, <, or etc operator. '.
854 854
                 'Use `is null` or `not null` instead.'
855 855
             );
856 856
             assert(is_string($value), "Only strings, booleans and integers are supported.");
Please login to merge, or discard this patch.