Passed
Pull Request — 5.3 (#279)
by
unknown
08:45
created
src/Utils/DirectForeignKeyMethodDescriptor.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -70,9 +70,9 @@  discard block
 block discarded – undo
70 70
     public function getName(): string
71 71
     {
72 72
         if (!$this->useAlternateName) {
73
-            return 'get' . $this->getPropertyName();
73
+            return 'get'.$this->getPropertyName();
74 74
         } else {
75
-            $methodName = 'get' . $this->getPropertyName() . 'By';
75
+            $methodName = 'get'.$this->getPropertyName().'By';
76 76
 
77 77
             $camelizedColumns = array_map([TDBMDaoGenerator::class, 'toCamelCase'], $this->foreignKey->getUnquotedLocalColumns());
78 78
 
@@ -137,23 +137,23 @@  discard block
 block discarded – undo
137 137
         $getter = new MethodGenerator($this->getName());
138 138
 
139 139
         if ($this->hasLocalUniqueIndex()) {
140
-            $classType = '\\' . $this->beanNamespace . '\\' . $beanClass;
140
+            $classType = '\\'.$this->beanNamespace.'\\'.$beanClass;
141 141
             $getterDocBlock = new DocBlockGenerator(sprintf('Returns the %s pointing to this bean via the %s column.', $beanClass, implode(', ', $this->foreignKey->getUnquotedLocalColumns())));
142
-            $getterDocBlock->setTag([new ReturnTag([$classType . '|null'])]);
142
+            $getterDocBlock->setTag([new ReturnTag([$classType.'|null'])]);
143 143
             $getterDocBlock->setWordWrap(false);
144 144
             $getter->setDocBlock($getterDocBlock);
145
-            $getter->setReturnType('?' . $classType);
145
+            $getter->setReturnType('?'.$classType);
146 146
 
147 147
             $code = sprintf(
148 148
                 'return $this->retrieveManyToOneRelationshipsStorage(%s, %s, %s, null, %s)->first();',
149 149
                 var_export($this->foreignKey->getLocalTableName(), true),
150 150
                 var_export($tdbmFk->getCacheKey(), true),
151 151
                 $this->getFilters($this->foreignKey),
152
-                '\\' . $this->resultIteratorNamespace . '\\' . $this->getResultIteratorClassName() .'::class'
152
+                '\\'.$this->resultIteratorNamespace.'\\'.$this->getResultIteratorClassName().'::class'
153 153
             );
154 154
         } else {
155 155
             $getterDocBlock = new DocBlockGenerator(sprintf('Returns the list of %s pointing to this bean via the %s column.', $beanClass, implode(', ', $this->foreignKey->getUnquotedLocalColumns())));
156
-            $getterDocBlock->setTag(new ReturnTag([$beanClass . '[]', '\\' . AlterableResultIterator::class]));
156
+            $getterDocBlock->setTag(new ReturnTag([$beanClass.'[]', '\\'.AlterableResultIterator::class]));
157 157
             $getterDocBlock->setWordWrap(false);
158 158
             $getter->setDocBlock($getterDocBlock);
159 159
             $getter->setReturnType(AlterableResultIterator::class);
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
                 var_export($this->foreignKey->getLocalTableName(), true),
164 164
                 var_export($tdbmFk->getCacheKey(), true),
165 165
                 $this->getFilters($this->foreignKey),
166
-                '\\' . $this->resultIteratorNamespace . '\\' . $this->getResultIteratorClassName() .'::class'
166
+                '\\'.$this->resultIteratorNamespace.'\\'.$this->getResultIteratorClassName().'::class'
167 167
             );
168 168
         }
169 169
 
@@ -173,7 +173,7 @@  discard block
 block discarded – undo
173 173
             $getter->setVisibility(AbstractMemberGenerator::VISIBILITY_PROTECTED);
174 174
         }
175 175
 
176
-        return [ $getter ];
176
+        return [$getter];
177 177
     }
178 178
 
179 179
     private function getFilters(ForeignKeyConstraint $fk): string
@@ -245,7 +245,7 @@  discard block
 block discarded – undo
245 245
         /** @var Annotation\JsonFormat|null $jsonFormat */
246 246
         $jsonFormat = $this->findAnnotation(Annotation\JsonFormat::class);
247 247
         if ($jsonFormat !== null) {
248
-            $method = $jsonFormat->method ?? 'get' . ucfirst($jsonFormat->property);
248
+            $method = $jsonFormat->method ?? 'get'.ucfirst($jsonFormat->property);
249 249
             $format = "$method()";
250 250
         } else {
251 251
             $stopRecursion = $this->findAnnotation(Annotation\JsonRecursive::class) ? '' : 'true';
@@ -254,7 +254,7 @@  discard block
 block discarded – undo
254 254
         $isIncluded = $this->findAnnotation(Annotation\JsonInclude::class) !== null;
255 255
         $index = $jsonCollection->key ?: lcfirst($this->getPropertyName());
256 256
         $class = $this->getBeanClassName();
257
-        $variableName = '$' . TDBMDaoGenerator::toVariableName($class);
257
+        $variableName = '$'.TDBMDaoGenerator::toVariableName($class);
258 258
         $getter = $this->getName();
259 259
         if ($this->hasLocalUniqueIndex()) {
260 260
             $code = "\$array['$index'] = (\$object = \$this->$getter()) ? \$object->$format : null;";
Please login to merge, or discard this patch.
src/Utils/ScalarBeanPropertyDescriptor.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
                 'time',
164 164
                 'time_immutable',
165 165
             ], true)) {
166
-                if ($default !== null && in_array(strtoupper($default), ['CURRENT_TIMESTAMP' /* MySQL */, 'NOW()' /* PostgreSQL */, 'SYSDATE' /* Oracle */ , 'CURRENT_TIMESTAMP()' /* MariaDB 10.3 */], true)) {
166
+                if ($default !== null && in_array(strtoupper($default), ['CURRENT_TIMESTAMP' /* MySQL */, 'NOW()' /* PostgreSQL */, 'SYSDATE' /* Oracle */, 'CURRENT_TIMESTAMP()' /* MariaDB 10.3 */], true)) {
167 167
                     $defaultCode = 'new \DateTimeImmutable()';
168 168
                 } else {
169 169
                     throw new TDBMException('Unable to set default value for date in "'.$this->table->getName().'.'.$this->column->getName().'". Database passed this default value: "'.$default.'"');
@@ -187,7 +187,7 @@  discard block
 block discarded – undo
187 187
             case 'v4':
188 188
                 return 'Uuid::uuid4()->toString()';
189 189
             default:
190
-                throw new TDBMException('@UUID annotation accepts either "v1" or "v4" parameter. Unexpected parameter: ' . $comment);
190
+                throw new TDBMException('@UUID annotation accepts either "v1" or "v4" parameter. Unexpected parameter: '.$comment);
191 191
         }
192 192
     }
193 193
 
@@ -235,7 +235,7 @@  discard block
 block discarded – undo
235 235
             $resourceTypeCheck = sprintf($resourceTypeCheck, $checkNullable, $variableName, $variableName);
236 236
         }
237 237
 
238
-        $types = [ $normalizedType ];
238
+        $types = [$normalizedType];
239 239
         if ($isNullable) {
240 240
             $types[] = 'null';
241 241
         }
@@ -338,11 +338,11 @@  discard block
 block discarded – undo
338 338
                             break;
339 339
                         }
340 340
                     }
341
-                    $args = array_map(function ($v) {
341
+                    $args = array_map(function($v) {
342 342
                         return var_export($v, true);
343 343
                     }, $args);
344
-                    $args = empty($args) ? '' : ', ' . implode(', ', $args);
345
-                    $unit = $jsonFormat->unit ? ' . ' . var_export($jsonFormat->unit, true) : '';
344
+                    $args = empty($args) ? '' : ', '.implode(', ', $args);
345
+                    $unit = $jsonFormat->unit ? ' . '.var_export($jsonFormat->unit, true) : '';
346 346
                     if ($this->column->getNotnull()) {
347 347
                         return "\$array['$index'] = number_format(\$this->$getter()$args)$unit;";
348 348
                     } else {
Please login to merge, or discard this patch.
src/AbstractTDBMObject.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -258,7 +258,7 @@  discard block
 block discarded – undo
258 258
      */
259 259
     protected function setRef(string $foreignKeyName, ?AbstractTDBMObject $bean, string $tableName, string $className, string $resultIteratorClass): void
260 260
     {
261
-        assert($bean === null || is_a($bean, $className), new TDBMInvalidArgumentException('$bean should be `null` or `' . $className . '`. `' . ($bean === null ? 'null' : get_class($bean)) . '` provided.'));
261
+        assert($bean === null || is_a($bean, $className), new TDBMInvalidArgumentException('$bean should be `null` or `'.$className.'`. `'.($bean === null ? 'null' : get_class($bean)).'` provided.'));
262 262
 
263 263
         if (!isset($this->dbRows[$tableName])) {
264 264
             $this->registerTable($tableName);
@@ -522,7 +522,7 @@  discard block
 block discarded – undo
522 522
      */
523 523
     protected function retrieveManyToOneRelationshipsStorage(string $tableName, string $foreignKeyName, array $searchFilter, ?string $orderString, string $resultIteratorClass): AlterableResultIterator
524 524
     {
525
-        assert(is_a($resultIteratorClass, ResultIterator::class, true), new TDBMInvalidArgumentException('$resultIteratorClass should be a `'. ResultIterator::class. '`. `' . $resultIteratorClass . '` provided.'));
525
+        assert(is_a($resultIteratorClass, ResultIterator::class, true), new TDBMInvalidArgumentException('$resultIteratorClass should be a `'.ResultIterator::class.'`. `'.$resultIteratorClass.'` provided.'));
526 526
         $key = $tableName.'___'.$foreignKeyName;
527 527
         $alterableResultIterator = $this->getManyToOneAlterableResultIterator($tableName, $foreignKeyName);
528 528
         if ($this->status === TDBMObjectStateEnum::STATE_DETACHED || $this->status === TDBMObjectStateEnum::STATE_NEW || (isset($this->manyToOneRelationships[$key]) && $this->manyToOneRelationships[$key]->getUnderlyingResultIterator() !== null)) {
Please login to merge, or discard this patch.
src/NoBeanFoundException.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -35,7 +35,7 @@
 block discarded – undo
35 35
      */
36 36
     public static function missPrimaryKeyRecord(string $tableName, array $primaryKeys, string $className, Exception $previous): self
37 37
     {
38
-        $primaryKeysStringified = implode(' and ', array_map(function ($key, $value) {
38
+        $primaryKeysStringified = implode(' and ', array_map(function($key, $value) {
39 39
             return "'".$key."' = ".$value;
40 40
         }, array_keys($primaryKeys), $primaryKeys));
41 41
         $exception = new self("No result found for query on table '".$tableName."' for ".$primaryKeysStringified, 0, $previous);
Please login to merge, or discard this patch.
src/InnerResultIterator.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -137,7 +137,7 @@
 block discarded – undo
137 137
         if ($this->fetchStarted && $this->tdbmService->getConnection()->getDatabasePlatform() instanceof MySqlPlatform) {
138 138
             // Optimisation: we don't need a separate "count" SQL request in MySQL.
139 139
             assert($this->statement instanceof Statement || $this->statement instanceof Result);
140
-            $this->count = (int)$this->statement->rowCount();
140
+            $this->count = (int) $this->statement->rowCount();
141 141
             return $this->count;
142 142
         }
143 143
         return $this->getRowCountViaSqlQuery();
Please login to merge, or discard this patch.