Failed Conditions
Push — master ( a3e53b...559253 )
by Guilherme
14:58
created
tests/Doctrine/Tests/ORM/Hydration/ObjectHydratorTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -1024,7 +1024,7 @@  discard block
 block discarded – undo
1024 1024
         ];
1025 1025
 
1026 1026
         $proxyInstance = (new LazyLoadingGhostFactory(new Configuration()))
1027
-            ->createProxy(ECommerceShipping::class, static function () {
1027
+            ->createProxy(ECommerceShipping::class, static function() {
1028 1028
                 self::fail('Proxy is not supposed to be lazy-loaded');
1029 1029
             });
1030 1030
 
@@ -1075,7 +1075,7 @@  discard block
 block discarded – undo
1075 1075
         ];
1076 1076
 
1077 1077
         $proxyInstance = (new LazyLoadingGhostFactory(new Configuration()))
1078
-            ->createProxy(ECommerceShipping::class, static function () {
1078
+            ->createProxy(ECommerceShipping::class, static function() {
1079 1079
                 self::fail('Proxy is not supposed to be lazy-loaded');
1080 1080
             });
1081 1081
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Mapping/XmlMappingDriverTest.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
 {
29 29
     protected function loadDriver()
30 30
     {
31
-        return new XmlDriver(__DIR__ . DIRECTORY_SEPARATOR . 'xml');
31
+        return new XmlDriver(__DIR__.DIRECTORY_SEPARATOR.'xml');
32 32
     }
33 33
 
34 34
     public function testClassTableInheritanceDiscriminatorMap() : void
@@ -172,7 +172,7 @@  discard block
 block discarded – undo
172 172
      */
173 173
     public function testValidateXmlSchema($xmlMappingFile) : void
174 174
     {
175
-        $xsdSchemaFile = __DIR__ . '/../../../../../doctrine-mapping.xsd';
175
+        $xsdSchemaFile = __DIR__.'/../../../../../doctrine-mapping.xsd';
176 176
         $dom           = new DOMDocument();
177 177
 
178 178
         $dom->load($xmlMappingFile);
@@ -182,14 +182,14 @@  discard block
 block discarded – undo
182 182
 
183 183
     public static function dataValidSchema()
184 184
     {
185
-        $list    = glob(__DIR__ . '/xml/*.xml');
185
+        $list    = glob(__DIR__.'/xml/*.xml');
186 186
         $invalid = ['Doctrine.Tests.Models.DDC889.DDC889Class.dcm'];
187 187
 
188
-        $list = array_filter($list, static function ($item) use ($invalid) {
188
+        $list = array_filter($list, static function($item) use ($invalid) {
189 189
             return ! in_array(pathinfo($item, PATHINFO_FILENAME), $invalid, true);
190 190
         });
191 191
 
192
-        return array_map(static function ($item) {
192
+        return array_map(static function($item) {
193 193
             return [$item];
194 194
         }, $list);
195 195
     }
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Query/SelectSqlGenerationTest.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
 
66 66
             $query->free();
67 67
         } catch (Exception $e) {
68
-            $this->fail($e->getMessage() . "\n" . $e->getTraceAsString());
68
+            $this->fail($e->getMessage()."\n".$e->getTraceAsString());
69 69
         }
70 70
 
71 71
         self::assertEquals($sqlToBeConfirmed, $sqlGenerated);
@@ -1625,7 +1625,7 @@  discard block
 block discarded – undo
1625 1625
     public function testAliasDoesNotExceedPlatformDefinedLength() : void
1626 1626
     {
1627 1627
         $this->assertSqlGeneration(
1628
-            'SELECT m FROM ' . __NAMESPACE__ . '\\DDC1384Model m',
1628
+            'SELECT m FROM '.__NAMESPACE__.'\\DDC1384Model m',
1629 1629
             'SELECT t0."aVeryLongIdentifierThatShouldBeShortenedByTheSQLWalker_fooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo" AS c0 FROM "DDC1384Model" t0'
1630 1630
         );
1631 1631
     }
@@ -1658,12 +1658,12 @@  discard block
 block discarded – undo
1658 1658
     public function testSelectWithArithmeticExpressionBeforeField() : void
1659 1659
     {
1660 1660
         $this->assertSqlGeneration(
1661
-            'SELECT - e.value AS value, e.id FROM ' . __NAMESPACE__ . '\DDC1474Entity e',
1661
+            'SELECT - e.value AS value, e.id FROM '.__NAMESPACE__.'\DDC1474Entity e',
1662 1662
             'SELECT -t0."value" AS c0, t0."id" AS c1 FROM "DDC1474Entity" t0'
1663 1663
         );
1664 1664
 
1665 1665
         $this->assertSqlGeneration(
1666
-            'SELECT e.id, + e.value AS value FROM ' . __NAMESPACE__ . '\DDC1474Entity e',
1666
+            'SELECT e.id, + e.value AS value FROM '.__NAMESPACE__.'\DDC1474Entity e',
1667 1667
             'SELECT t0."id" AS c0, +t0."value" AS c1 FROM "DDC1474Entity" t0'
1668 1668
         );
1669 1669
     }
@@ -2340,8 +2340,8 @@  discard block
 block discarded – undo
2340 2340
     public function testHavingRegressionUsingVariableWithMathOperatorsExpression($operator) : void
2341 2341
     {
2342 2342
         $this->assertSqlGeneration(
2343
-            'SELECT COUNT(u.name) AS countName FROM Doctrine\Tests\Models\CMS\CmsUser u HAVING 1 ' . $operator . ' countName > 0',
2344
-            'SELECT COUNT(t0."name") AS c0 FROM "cms_users" t0 HAVING 1 ' . $operator . ' c0 > 0'
2343
+            'SELECT COUNT(u.name) AS countName FROM Doctrine\Tests\Models\CMS\CmsUser u HAVING 1 '.$operator.' countName > 0',
2344
+            'SELECT COUNT(t0."name") AS c0 FROM "cms_users" t0 HAVING 1 '.$operator.' c0 > 0'
2345 2345
         );
2346 2346
     }
2347 2347
 
@@ -2360,7 +2360,7 @@  discard block
 block discarded – undo
2360 2360
 
2361 2361
     public function getSql(SqlWalker $sqlWalker)
2362 2362
     {
2363
-        return 'ABS(' . $sqlWalker->walkSimpleArithmeticExpression($this->simpleArithmeticExpression) . ')';
2363
+        return 'ABS('.$sqlWalker->walkSimpleArithmeticExpression($this->simpleArithmeticExpression).')';
2364 2364
     }
2365 2365
 
2366 2366
     public function parse(Parser $parser)
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Query/CustomTreeWalkersTest.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -45,7 +45,7 @@  discard block
 block discarded – undo
45 45
         try {
46 46
             $sqlGenerated = $this->generateSql($dqlToBeTested, $treeWalkers, $outputWalker);
47 47
         } catch (Exception $e) {
48
-            $this->fail($e->getMessage() . ' at "' . $e->getFile() . '" on line ' . $e->getLine());
48
+            $this->fail($e->getMessage().' at "'.$e->getFile().'" on line '.$e->getLine());
49 49
         }
50 50
 
51 51
         self::assertEquals($sqlToBeConfirmed, $sqlGenerated);
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
145 145
             $condExpr = $whereClause->conditionalExpression;
146 146
 
147 147
             // Since Phase 1 AST optimizations were included, we need to re-add the ConditionalExpression
148
-            if (! ($condExpr instanceof Query\AST\ConditionalExpression)) {
148
+            if ( ! ($condExpr instanceof Query\AST\ConditionalExpression)) {
149 149
                 $condExpr = new Query\AST\ConditionalExpression([$condExpr]);
150 150
 
151 151
                 $whereClause->conditionalExpression = $condExpr;
@@ -168,7 +168,7 @@  discard block
 block discarded – undo
168 168
                 $singleTerm = $selectStatement->whereClause->conditionalExpression->conditionalTerms[0];
169 169
 
170 170
                 // Since Phase 1 AST optimizations were included, we need to re-add the ConditionalExpression
171
-                if (! ($singleTerm instanceof Query\AST\ConditionalTerm)) {
171
+                if ( ! ($singleTerm instanceof Query\AST\ConditionalTerm)) {
172 172
                     $singleTerm = new Query\AST\ConditionalTerm([$singleTerm]);
173 173
 
174 174
                     $selectStatement->whereClause->conditionalExpression->conditionalTerms[0] = $singleTerm;
@@ -206,9 +206,9 @@  discard block
 block discarded – undo
206 206
     {
207 207
         $rangeVariableDecl       = $identificationVariableDecl->rangeVariableDeclaration;
208 208
         $joinAssocPathExpression = new Query\AST\JoinAssociationPathExpression($rangeVariableDecl->aliasIdentificationVariable, 'address');
209
-        $joinAssocDeclaration    = new Query\AST\JoinAssociationDeclaration($joinAssocPathExpression, $rangeVariableDecl->aliasIdentificationVariable . 'a', null);
209
+        $joinAssocDeclaration    = new Query\AST\JoinAssociationDeclaration($joinAssocPathExpression, $rangeVariableDecl->aliasIdentificationVariable.'a', null);
210 210
         $join                    = new Query\AST\Join(Query\AST\Join::JOIN_TYPE_LEFT, $joinAssocDeclaration);
211
-        $selectExpression        = new Query\AST\SelectExpression($rangeVariableDecl->aliasIdentificationVariable . 'a', null, false);
211
+        $selectExpression        = new Query\AST\SelectExpression($rangeVariableDecl->aliasIdentificationVariable.'a', null, false);
212 212
 
213 213
         $identificationVariableDecl->joins[]                = $join;
214 214
         $selectStatement->selectClause->selectExpressions[] = $selectExpression;
@@ -218,7 +218,7 @@  discard block
 block discarded – undo
218 218
         $addressMetadata = $entityManager->getClassMetadata(CmsAddress::class);
219 219
 
220 220
         $this->setQueryComponent(
221
-            $rangeVariableDecl->aliasIdentificationVariable . 'a',
221
+            $rangeVariableDecl->aliasIdentificationVariable.'a',
222 222
             [
223 223
                 'metadata'     => $addressMetadata,
224 224
                 'parent'       => $rangeVariableDecl->aliasIdentificationVariable,
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Cache/FileLockRegionTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -59,7 +59,7 @@  discard block
 block discarded – undo
59 59
 
60 60
     protected function createRegion()
61 61
     {
62
-        $this->directory = sys_get_temp_dir() . '/doctrine_lock_' . uniqid();
62
+        $this->directory = sys_get_temp_dir().'/doctrine_lock_'.uniqid();
63 63
 
64 64
         $region = new DefaultRegion('concurren_region_test', $this->cache);
65 65
 
@@ -262,7 +262,7 @@  discard block
 block discarded – undo
262 262
         $reflectionDirectory->setAccessible(true);
263 263
         $reflectionDirectory->setValue($region, str_repeat('a', 10000));
264 264
 
265
-        set_error_handler(static function () {
265
+        set_error_handler(static function() {
266 266
         }, E_WARNING);
267 267
         self::assertTrue($region->evictAll());
268 268
         restore_error_handler();
@@ -275,7 +275,7 @@  discard block
 block discarded – undo
275 275
     {
276 276
         $path = $path ?: $this->directory;
277 277
 
278
-        if (! is_dir($path)) {
278
+        if ( ! is_dir($path)) {
279 279
             return;
280 280
         }
281 281
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/PersistentCollectionTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -173,7 +173,7 @@  discard block
 block discarded – undo
173 173
             ->expects(self::once())
174 174
             ->method('loadCollection')
175 175
             ->with($this->collection)
176
-            ->willReturnCallback(static function (PersistentCollection $persistentCollection) use ($persistedElement) : void {
176
+            ->willReturnCallback(static function(PersistentCollection $persistentCollection) use ($persistedElement) : void {
177 177
                 $persistentCollection->unwrap()->add($persistedElement);
178 178
             });
179 179
 
@@ -210,7 +210,7 @@  discard block
 block discarded – undo
210 210
             ->expects(self::once())
211 211
             ->method('loadCollection')
212 212
             ->with($this->collection)
213
-            ->willReturnCallback(static function (PersistentCollection $persistentCollection) use (
213
+            ->willReturnCallback(static function(PersistentCollection $persistentCollection) use (
214 214
                 $persistedElement,
215 215
                 $newElementThatIsAlsoPersisted
216 216
             ) : void {
@@ -252,7 +252,7 @@  discard block
 block discarded – undo
252 252
             ->expects(self::once())
253 253
             ->method('loadCollection')
254 254
             ->with($this->collection)
255
-            ->willReturnCallback(static function (PersistentCollection $persistentCollection) use (
255
+            ->willReturnCallback(static function(PersistentCollection $persistentCollection) use (
256 256
                 $persistedElement,
257 257
                 $newElementThatIsAlsoPersisted
258 258
             ) : void {
Please login to merge, or discard this patch.
tests/Doctrine/Tests/Models/CMS/CmsComment.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -35,6 +35,6 @@
 block discarded – undo
35 35
 
36 36
     public function __toString()
37 37
     {
38
-        return self::class . '[id=' . $this->id . ']';
38
+        return self::class.'[id='.$this->id.']';
39 39
     }
40 40
 }
Please login to merge, or discard this patch.
tests/Doctrine/Tests/TestInit.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -15,20 +15,20 @@
 block discarded – undo
15 15
 error_reporting(E_ALL | E_STRICT);
16 16
 date_default_timezone_set('UTC');
17 17
 
18
-if (file_exists(__DIR__ . '/../../../vendor/autoload.php')) {
18
+if (file_exists(__DIR__.'/../../../vendor/autoload.php')) {
19 19
     // dependencies were installed via composer - this is the main project
20
-    require __DIR__ . '/../../../vendor/autoload.php';
21
-} elseif (file_exists(__DIR__ . '/../../../../../autoload.php')) {
20
+    require __DIR__.'/../../../vendor/autoload.php';
21
+} elseif (file_exists(__DIR__.'/../../../../../autoload.php')) {
22 22
     // installed as a dependency in `vendor`
23
-    require __DIR__ . '/../../../../../autoload.php';
23
+    require __DIR__.'/../../../../../autoload.php';
24 24
 } else {
25 25
     throw new Exception('Can\'t find autoload.php. Did you install dependencies via composer?');
26 26
 }
27 27
 
28
-if (! file_exists(__DIR__ . '/Proxies') && ! mkdir(__DIR__ . '/Proxies')) {
29
-    throw new Exception('Could not create ' . __DIR__ . '/Proxies Folder.');
28
+if ( ! file_exists(__DIR__.'/Proxies') && ! mkdir(__DIR__.'/Proxies')) {
29
+    throw new Exception('Could not create '.__DIR__.'/Proxies Folder.');
30 30
 }
31 31
 
32
-if (! file_exists(__DIR__ . '/ORM/Proxy/generated') && ! mkdir(__DIR__ . '/ORM/Proxy/generated')) {
33
-    throw new Exception('Could not create ' . __DIR__ . '/ORM/Proxy/generated Folder.');
32
+if ( ! file_exists(__DIR__.'/ORM/Proxy/generated') && ! mkdir(__DIR__.'/ORM/Proxy/generated')) {
33
+    throw new Exception('Could not create '.__DIR__.'/ORM/Proxy/generated Folder.');
34 34
 }
Please login to merge, or discard this patch.
lib/Doctrine/ORM/Persisters/Entity/BasicEntityPersister.php 2 patches
Indentation   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -305,8 +305,8 @@  discard block
 block discarded – undo
305 305
 
306 306
         // FIXME: Order with composite keys might not be correct
307 307
         $sql = 'SELECT ' . $columnName
308
-             . ' FROM ' . $tableName
309
-             . ' WHERE ' . implode(' = ? AND ', $identifier) . ' = ?';
308
+                . ' FROM ' . $tableName
309
+                . ' WHERE ' . implode(' = ? AND ', $identifier) . ' = ?';
310 310
 
311 311
         $flattenedId = $this->em->getIdentifierFlattener()->flattenIdentifier($versionedClass, $id);
312 312
         $versionType = $versionProperty->getType();
@@ -519,8 +519,8 @@  discard block
 block discarded – undo
519 519
         }
520 520
 
521 521
         $sql = 'UPDATE ' . $quotedTableName
522
-             . ' SET ' . implode(', ', $set)
523
-             . ' WHERE ' . implode(' = ? AND ', $where) . ' = ?';
522
+                . ' SET ' . implode(', ', $set)
523
+                . ' WHERE ' . implode(' = ? AND ', $where) . ' = ?';
524 524
 
525 525
         $result = $this->conn->executeUpdate($sql, $params, $types);
526 526
 
@@ -1641,9 +1641,9 @@  discard block
 block discarded – undo
1641 1641
         $lock  = $this->getLockTablesSql($lockMode);
1642 1642
         $where = ($conditionSql ? ' WHERE ' . $conditionSql : '') . ' ';
1643 1643
         $sql   = 'SELECT 1 '
1644
-             . $lock
1645
-             . $where
1646
-             . $lockSql;
1644
+                . $lock
1645
+                . $where
1646
+                . $lockSql;
1647 1647
 
1648 1648
         [$params, $types] = $this->expandParameters($criteria);
1649 1649
 
@@ -2112,8 +2112,8 @@  discard block
 block discarded – undo
2112 2112
         $alias = $this->getSQLTableAlias($this->class->getTableName());
2113 2113
 
2114 2114
         $sql = 'SELECT 1 '
2115
-             . $this->getLockTablesSql(null)
2116
-             . ' WHERE ' . $this->getSelectConditionSQL($criteria);
2115
+                . $this->getLockTablesSql(null)
2116
+                . ' WHERE ' . $this->getSelectConditionSQL($criteria);
2117 2117
 
2118 2118
         [$params, $types] = $this->expandParameters($criteria);
2119 2119
 
Please login to merge, or discard this patch.
Spacing   +72 added lines, -72 removed lines patch added patch discarded remove patch
@@ -294,16 +294,16 @@  discard block
 block discarded – undo
294 294
         $tableName      = $versionedClass->table->getQuotedQualifiedName($this->platform);
295 295
         $columnName     = $this->platform->quoteIdentifier($versionProperty->getColumnName());
296 296
         $identifier     = array_map(
297
-            function ($columnName) {
297
+            function($columnName) {
298 298
                 return $this->platform->quoteIdentifier($columnName);
299 299
             },
300 300
             array_keys($versionedClass->getIdentifierColumns($this->em))
301 301
         );
302 302
 
303 303
         // FIXME: Order with composite keys might not be correct
304
-        $sql = 'SELECT ' . $columnName
305
-             . ' FROM ' . $tableName
306
-             . ' WHERE ' . implode(' = ? AND ', $identifier) . ' = ?';
304
+        $sql = 'SELECT '.$columnName
305
+             . ' FROM '.$tableName
306
+             . ' WHERE '.implode(' = ? AND ', $identifier).' = ?';
307 307
 
308 308
         $flattenedId = $this->em->getIdentifierFlattener()->flattenIdentifier($versionedClass, $id);
309 309
         $versionType = $versionProperty->getType();
@@ -341,13 +341,13 @@  discard block
 block discarded – undo
341 341
         $tableName  = $this->class->getTableName();
342 342
         $updateData = $this->prepareUpdateData($entity);
343 343
 
344
-        if (! isset($updateData[$tableName])) {
344
+        if ( ! isset($updateData[$tableName])) {
345 345
             return;
346 346
         }
347 347
 
348 348
         $data = $updateData[$tableName];
349 349
 
350
-        if (! $data) {
350
+        if ( ! $data) {
351 351
             return;
352 352
         }
353 353
 
@@ -399,7 +399,7 @@  discard block
 block discarded – undo
399 399
 
400 400
             if ($value !== null) {
401 401
                 // @todo guilhermeblanco Make sure we do not have flat association values.
402
-                if (! is_array($value)) {
402
+                if ( ! is_array($value)) {
403 403
                     $value = [$targetClass->identifier[0] => $value];
404 404
                 }
405 405
 
@@ -495,18 +495,18 @@  discard block
 block discarded – undo
495 495
                 case Type::SMALLINT:
496 496
                 case Type::INTEGER:
497 497
                 case Type::BIGINT:
498
-                    $set[] = $versionColumnName . ' = ' . $versionColumnName . ' + 1';
498
+                    $set[] = $versionColumnName.' = '.$versionColumnName.' + 1';
499 499
                     break;
500 500
 
501 501
                 case Type::DATETIME:
502
-                    $set[] = $versionColumnName . ' = CURRENT_TIMESTAMP';
502
+                    $set[] = $versionColumnName.' = CURRENT_TIMESTAMP';
503 503
                     break;
504 504
             }
505 505
         }
506 506
 
507
-        $sql = 'UPDATE ' . $quotedTableName
508
-             . ' SET ' . implode(', ', $set)
509
-             . ' WHERE ' . implode(' = ? AND ', $where) . ' = ?';
507
+        $sql = 'UPDATE '.$quotedTableName
508
+             . ' SET '.implode(', ', $set)
509
+             . ' WHERE '.implode(' = ? AND ', $where).' = ?';
510 510
 
511 511
         $result = $this->conn->executeUpdate($sql, $params, $types);
512 512
 
@@ -523,7 +523,7 @@  discard block
 block discarded – undo
523 523
     protected function deleteJoinTableRecords($identifier)
524 524
     {
525 525
         foreach ($this->class->getPropertiesIterator() as $association) {
526
-            if (! ($association instanceof ManyToManyAssociationMetadata)) {
526
+            if ( ! ($association instanceof ManyToManyAssociationMetadata)) {
527 527
                 continue;
528 528
             }
529 529
 
@@ -535,7 +535,7 @@  discard block
 block discarded – undo
535 535
             $otherKeys         = [];
536 536
             $keys              = [];
537 537
 
538
-            if (! $owningAssociation->isOwningSide()) {
538
+            if ( ! $owningAssociation->isOwningSide()) {
539 539
                 $class             = $this->em->getClassMetadata($association->getTargetEntity());
540 540
                 $owningAssociation = $class->getProperty($association->getMappedBy());
541 541
             }
@@ -645,7 +645,7 @@  discard block
 block discarded – undo
645 645
             }
646 646
 
647 647
             // Only owning side of x-1 associations can have a FK column.
648
-            if (! $property instanceof ToOneAssociationMetadata || ! $property->isOwningSide()) {
648
+            if ( ! $property instanceof ToOneAssociationMetadata || ! $property->isOwningSide()) {
649 649
                 continue;
650 650
             }
651 651
 
@@ -688,7 +688,7 @@  discard block
 block discarded – undo
688 688
         $propertyName = $this->class->fieldNames[$columnName];
689 689
         $property     = $this->class->getProperty($propertyName);
690 690
 
691
-        if (! $property) {
691
+        if ( ! $property) {
692 692
             return null;
693 693
         }
694 694
 
@@ -808,7 +808,7 @@  discard block
 block discarded – undo
808 808
             $sourceKeyColumn = $joinColumn->getReferencedColumnName();
809 809
             $targetKeyColumn = $joinColumn->getColumnName();
810 810
 
811
-            if (! isset($sourceClass->fieldNames[$sourceKeyColumn])) {
811
+            if ( ! isset($sourceClass->fieldNames[$sourceKeyColumn])) {
812 812
                 throw MappingException::joinColumnMustPointToMappedField(
813 813
                     $sourceClass->getClassName(),
814 814
                     $sourceKeyColumn
@@ -1035,7 +1035,7 @@  discard block
 block discarded – undo
1035 1035
         $criteria    = [];
1036 1036
         $parameters  = [];
1037 1037
 
1038
-        if (! $association->isOwningSide()) {
1038
+        if ( ! $association->isOwningSide()) {
1039 1039
             $class       = $this->em->getClassMetadata($association->getTargetEntity());
1040 1040
             $owningAssoc = $class->getProperty($association->getMappedBy());
1041 1041
         }
@@ -1063,7 +1063,7 @@  discard block
 block discarded – undo
1063 1063
                 $value = $value[$targetClass->identifier[0]];
1064 1064
             }
1065 1065
 
1066
-            $criteria[$joinTableName . '.' . $quotedColumnName] = $value;
1066
+            $criteria[$joinTableName.'.'.$quotedColumnName] = $value;
1067 1067
             $parameters[]                                       = [
1068 1068
                 'value' => $value,
1069 1069
                 'field' => $fieldName,
@@ -1113,11 +1113,11 @@  discard block
 block discarded – undo
1113 1113
 
1114 1114
         switch ($lockMode) {
1115 1115
             case LockMode::PESSIMISTIC_READ:
1116
-                $lockSql = ' ' . $this->platform->getReadLockSQL();
1116
+                $lockSql = ' '.$this->platform->getReadLockSQL();
1117 1117
                 break;
1118 1118
 
1119 1119
             case LockMode::PESSIMISTIC_WRITE:
1120
-                $lockSql = ' ' . $this->platform->getWriteLockSQL();
1120
+                $lockSql = ' '.$this->platform->getWriteLockSQL();
1121 1121
                 break;
1122 1122
         }
1123 1123
 
@@ -1128,14 +1128,14 @@  discard block
 block discarded – undo
1128 1128
 
1129 1129
         if ($filterSql !== '') {
1130 1130
             $conditionSql = $conditionSql
1131
-                ? $conditionSql . ' AND ' . $filterSql
1131
+                ? $conditionSql.' AND '.$filterSql
1132 1132
                 : $filterSql;
1133 1133
         }
1134 1134
 
1135
-        $select = 'SELECT ' . $columnList;
1136
-        $from   = ' FROM ' . $tableName . ' ' . $tableAlias;
1137
-        $join   = $this->currentPersisterContext->selectJoinSql . $joinSql;
1138
-        $where  = ($conditionSql ? ' WHERE ' . $conditionSql : '');
1135
+        $select = 'SELECT '.$columnList;
1136
+        $from   = ' FROM '.$tableName.' '.$tableAlias;
1137
+        $join   = $this->currentPersisterContext->selectJoinSql.$joinSql;
1138
+        $where  = ($conditionSql ? ' WHERE '.$conditionSql : '');
1139 1139
         $lock   = $this->platform->appendLockHint($from, $lockMode);
1140 1140
         $query  = $select
1141 1141
             . $lock
@@ -1143,7 +1143,7 @@  discard block
 block discarded – undo
1143 1143
             . $where
1144 1144
             . $orderBySql;
1145 1145
 
1146
-        return $this->platform->modifyLimitQuery($query, $limit, $offset ?? 0) . $lockSql;
1146
+        return $this->platform->modifyLimitQuery($query, $limit, $offset ?? 0).$lockSql;
1147 1147
     }
1148 1148
 
1149 1149
     /**
@@ -1162,13 +1162,13 @@  discard block
 block discarded – undo
1162 1162
 
1163 1163
         if ($filterSql !== '') {
1164 1164
             $conditionSql = $conditionSql
1165
-                ? $conditionSql . ' AND ' . $filterSql
1165
+                ? $conditionSql.' AND '.$filterSql
1166 1166
                 : $filterSql;
1167 1167
         }
1168 1168
 
1169 1169
         return 'SELECT COUNT(*) '
1170
-            . 'FROM ' . $tableName . ' ' . $tableAlias
1171
-            . (empty($conditionSql) ? '' : ' WHERE ' . $conditionSql);
1170
+            . 'FROM '.$tableName.' '.$tableAlias
1171
+            . (empty($conditionSql) ? '' : ' WHERE '.$conditionSql);
1172 1172
     }
1173 1173
 
1174 1174
     /**
@@ -1183,7 +1183,7 @@  discard block
 block discarded – undo
1183 1183
      */
1184 1184
     final protected function getOrderBySQL(array $orderBy, $baseTableAlias)
1185 1185
     {
1186
-        if (! $orderBy) {
1186
+        if ( ! $orderBy) {
1187 1187
             return '';
1188 1188
         }
1189 1189
 
@@ -1192,7 +1192,7 @@  discard block
 block discarded – undo
1192 1192
         foreach ($orderBy as $fieldName => $orientation) {
1193 1193
             $orientation = strtoupper(trim($orientation));
1194 1194
 
1195
-            if (! in_array($orientation, ['ASC', 'DESC'], true)) {
1195
+            if ( ! in_array($orientation, ['ASC', 'DESC'], true)) {
1196 1196
                 throw InvalidOrientation::fromClassNameAndField($this->class->getClassName(), $fieldName);
1197 1197
             }
1198 1198
 
@@ -1202,11 +1202,11 @@  discard block
 block discarded – undo
1202 1202
                 $tableAlias = $this->getSQLTableAlias($property->getTableName());
1203 1203
                 $columnName = $this->platform->quoteIdentifier($property->getColumnName());
1204 1204
 
1205
-                $orderByList[] = $tableAlias . '.' . $columnName . ' ' . $orientation;
1205
+                $orderByList[] = $tableAlias.'.'.$columnName.' '.$orientation;
1206 1206
 
1207 1207
                 continue;
1208 1208
             } elseif ($property instanceof AssociationMetadata) {
1209
-                if (! $property->isOwningSide()) {
1209
+                if ( ! $property->isOwningSide()) {
1210 1210
                     throw InvalidFindByCall::fromInverseSideUsage(
1211 1211
                         $this->class->getClassName(),
1212 1212
                         $fieldName
@@ -1222,7 +1222,7 @@  discard block
 block discarded – undo
1222 1222
                     /** @var JoinColumnMetadata $joinColumn */
1223 1223
                     $quotedColumnName = $this->platform->quoteIdentifier($joinColumn->getColumnName());
1224 1224
 
1225
-                    $orderByList[] = $tableAlias . '.' . $quotedColumnName . ' ' . $orientation;
1225
+                    $orderByList[] = $tableAlias.'.'.$quotedColumnName.' '.$orientation;
1226 1226
                 }
1227 1227
 
1228 1228
                 continue;
@@ -1231,7 +1231,7 @@  discard block
 block discarded – undo
1231 1231
             throw UnrecognizedField::byName($fieldName);
1232 1232
         }
1233 1233
 
1234
-        return ' ORDER BY ' . implode(', ', $orderByList);
1234
+        return ' ORDER BY '.implode(', ', $orderByList);
1235 1235
     }
1236 1236
 
1237 1237
     /**
@@ -1273,7 +1273,7 @@  discard block
 block discarded – undo
1273 1273
                     $isAssocToOneInverseSide = $property instanceof ToOneAssociationMetadata && ! $property->isOwningSide();
1274 1274
                     $isAssocFromOneEager     = ! $property instanceof ManyToManyAssociationMetadata && $property->getFetchMode() === FetchMode::EAGER;
1275 1275
 
1276
-                    if (! ($isAssocFromOneEager || $isAssocToOneInverseSide)) {
1276
+                    if ( ! ($isAssocFromOneEager || $isAssocToOneInverseSide)) {
1277 1277
                         break;
1278 1278
                     }
1279 1279
 
@@ -1288,7 +1288,7 @@  discard block
 block discarded – undo
1288 1288
                         break; // now this is why you shouldn't use inheritance
1289 1289
                     }
1290 1290
 
1291
-                    $assocAlias = 'e' . ($eagerAliasCounter++);
1291
+                    $assocAlias = 'e'.($eagerAliasCounter++);
1292 1292
 
1293 1293
                     $this->currentPersisterContext->rsm->addJoinedEntityResult($targetEntity, $assocAlias, 'r', $fieldName);
1294 1294
 
@@ -1316,14 +1316,14 @@  discard block
 block discarded – undo
1316 1316
                         $this->currentPersisterContext->rsm->addIndexBy($assocAlias, $property->getIndexedBy());
1317 1317
                     }
1318 1318
 
1319
-                    if (! $property->isOwningSide()) {
1319
+                    if ( ! $property->isOwningSide()) {
1320 1320
                         $owningAssociation = $eagerEntity->getProperty($property->getMappedBy());
1321 1321
                     }
1322 1322
 
1323 1323
                     $joinTableAlias = $this->getSQLTableAlias($eagerEntity->getTableName(), $assocAlias);
1324 1324
                     $joinTableName  = $eagerEntity->table->getQuotedQualifiedName($this->platform);
1325 1325
 
1326
-                    $this->currentPersisterContext->selectJoinSql .= ' ' . $this->getJoinSQLForAssociation($property);
1326
+                    $this->currentPersisterContext->selectJoinSql .= ' '.$this->getJoinSQLForAssociation($property);
1327 1327
 
1328 1328
                     $sourceClass      = $this->em->getClassMetadata($owningAssociation->getSourceEntity());
1329 1329
                     $targetClass      = $this->em->getClassMetadata($owningAssociation->getTargetEntity());
@@ -1347,7 +1347,7 @@  discard block
 block discarded – undo
1347 1347
                         $joinCondition[] = $filterSql;
1348 1348
                     }
1349 1349
 
1350
-                    $this->currentPersisterContext->selectJoinSql .= ' ' . $joinTableName . ' ' . $joinTableAlias . ' ON ';
1350
+                    $this->currentPersisterContext->selectJoinSql .= ' '.$joinTableName.' '.$joinTableAlias.' ON ';
1351 1351
                     $this->currentPersisterContext->selectJoinSql .= implode(' AND ', $joinCondition);
1352 1352
 
1353 1353
                     break;
@@ -1369,7 +1369,7 @@  discard block
 block discarded – undo
1369 1369
      */
1370 1370
     protected function getSelectColumnAssociationSQL($field, AssociationMetadata $association, ClassMetadata $class, $alias = 'r')
1371 1371
     {
1372
-        if (! ($association->isOwningSide() && $association instanceof ToOneAssociationMetadata)) {
1372
+        if ( ! ($association->isOwningSide() && $association instanceof ToOneAssociationMetadata)) {
1373 1373
             return '';
1374 1374
         }
1375 1375
 
@@ -1409,7 +1409,7 @@  discard block
 block discarded – undo
1409 1409
         $owningAssociation = $association;
1410 1410
         $sourceTableAlias  = $this->getSQLTableAlias($this->class->getTableName());
1411 1411
 
1412
-        if (! $association->isOwningSide()) {
1412
+        if ( ! $association->isOwningSide()) {
1413 1413
             $targetEntity      = $this->em->getClassMetadata($association->getTargetEntity());
1414 1414
             $owningAssociation = $targetEntity->getProperty($association->getMappedBy());
1415 1415
         }
@@ -1430,7 +1430,7 @@  discard block
 block discarded – undo
1430 1430
             );
1431 1431
         }
1432 1432
 
1433
-        return ' INNER JOIN ' . $joinTableName . ' ON ' . implode(' AND ', $conditions);
1433
+        return ' INNER JOIN '.$joinTableName.' ON '.implode(' AND ', $conditions);
1434 1434
     }
1435 1435
 
1436 1436
     /**
@@ -1494,7 +1494,7 @@  discard block
 block discarded – undo
1494 1494
                     break;
1495 1495
 
1496 1496
                 case $property instanceof LocalColumnMetadata:
1497
-                    if ((! $property->hasValueGenerator() || $property->getValueGenerator()->getType() !== GeneratorType::IDENTITY)
1497
+                    if (( ! $property->hasValueGenerator() || $property->getValueGenerator()->getType() !== GeneratorType::IDENTITY)
1498 1498
                         || $this->class->identifier[0] !== $name
1499 1499
                     ) {
1500 1500
                         $columnName = $property->getColumnName();
@@ -1552,7 +1552,7 @@  discard block
 block discarded – undo
1552 1552
 
1553 1553
         $this->currentPersisterContext->rsm->addFieldResult($alias, $columnAlias, $field, $class->getClassName());
1554 1554
 
1555
-        return $property->getType()->convertToPHPValueSQL($sql, $this->platform) . ' AS ' . $columnAlias;
1555
+        return $property->getType()->convertToPHPValueSQL($sql, $this->platform).' AS '.$columnAlias;
1556 1556
     }
1557 1557
 
1558 1558
     /**
@@ -1566,14 +1566,14 @@  discard block
 block discarded – undo
1566 1566
     protected function getSQLTableAlias($tableName, $assocName = '')
1567 1567
     {
1568 1568
         if ($tableName) {
1569
-            $tableName .= '#' . $assocName;
1569
+            $tableName .= '#'.$assocName;
1570 1570
         }
1571 1571
 
1572 1572
         if (isset($this->currentPersisterContext->sqlTableAliases[$tableName])) {
1573 1573
             return $this->currentPersisterContext->sqlTableAliases[$tableName];
1574 1574
         }
1575 1575
 
1576
-        $tableAlias = 't' . $this->currentPersisterContext->sqlAliasCounter++;
1576
+        $tableAlias = 't'.$this->currentPersisterContext->sqlAliasCounter++;
1577 1577
 
1578 1578
         $this->currentPersisterContext->sqlTableAliases[$tableName] = $tableAlias;
1579 1579
 
@@ -1599,7 +1599,7 @@  discard block
 block discarded – undo
1599 1599
         }
1600 1600
 
1601 1601
         $lock  = $this->getLockTablesSql($lockMode);
1602
-        $where = ($conditionSql ? ' WHERE ' . $conditionSql : '') . ' ';
1602
+        $where = ($conditionSql ? ' WHERE '.$conditionSql : '').' ';
1603 1603
         $sql   = 'SELECT 1 '
1604 1604
              . $lock
1605 1605
              . $where
@@ -1622,7 +1622,7 @@  discard block
 block discarded – undo
1622 1622
         $tableName = $this->class->table->getQuotedQualifiedName($this->platform);
1623 1623
 
1624 1624
         return $this->platform->appendLockHint(
1625
-            'FROM ' . $tableName . ' ' . $this->getSQLTableAlias($this->class->getTableName()),
1625
+            'FROM '.$tableName.' '.$this->getSQLTableAlias($this->class->getTableName()),
1626 1626
             $lockMode
1627 1627
         );
1628 1628
     }
@@ -1672,19 +1672,19 @@  discard block
 block discarded – undo
1672 1672
 
1673 1673
             if ($comparison !== null) {
1674 1674
                 // special case null value handling
1675
-                if (($comparison === Comparison::EQ || $comparison === Comparison::IS) && $value ===null) {
1676
-                    $selectedColumns[] = $column . ' IS NULL';
1675
+                if (($comparison === Comparison::EQ || $comparison === Comparison::IS) && $value === null) {
1676
+                    $selectedColumns[] = $column.' IS NULL';
1677 1677
 
1678 1678
                     continue;
1679 1679
                 }
1680 1680
 
1681 1681
                 if ($comparison === Comparison::NEQ && $value === null) {
1682
-                    $selectedColumns[] = $column . ' IS NOT NULL';
1682
+                    $selectedColumns[] = $column.' IS NOT NULL';
1683 1683
 
1684 1684
                     continue;
1685 1685
                 }
1686 1686
 
1687
-                $selectedColumns[] = $column . ' ' . sprintf(self::$comparisonMap[$comparison], $placeholder);
1687
+                $selectedColumns[] = $column.' '.sprintf(self::$comparisonMap[$comparison], $placeholder);
1688 1688
 
1689 1689
                 continue;
1690 1690
             }
@@ -1732,7 +1732,7 @@  discard block
 block discarded – undo
1732 1732
             $tableAlias = $this->getSQLTableAlias($property->getTableName());
1733 1733
             $columnName = $this->platform->quoteIdentifier($property->getColumnName());
1734 1734
 
1735
-            return [$tableAlias . '.' . $columnName];
1735
+            return [$tableAlias.'.'.$columnName];
1736 1736
         }
1737 1737
 
1738 1738
         if ($property instanceof AssociationMetadata) {
@@ -1741,7 +1741,7 @@  discard block
 block discarded – undo
1741 1741
 
1742 1742
             // Many-To-Many requires join table check for joinColumn
1743 1743
             if ($owningAssociation instanceof ManyToManyAssociationMetadata) {
1744
-                if (! $owningAssociation->isOwningSide()) {
1744
+                if ( ! $owningAssociation->isOwningSide()) {
1745 1745
                     $owningAssociation = $association;
1746 1746
                 }
1747 1747
 
@@ -1754,10 +1754,10 @@  discard block
 block discarded – undo
1754 1754
                 foreach ($joinColumns as $joinColumn) {
1755 1755
                     $quotedColumnName = $this->platform->quoteIdentifier($joinColumn->getColumnName());
1756 1756
 
1757
-                    $columns[] = $joinTableName . '.' . $quotedColumnName;
1757
+                    $columns[] = $joinTableName.'.'.$quotedColumnName;
1758 1758
                 }
1759 1759
             } else {
1760
-                if (! $owningAssociation->isOwningSide()) {
1760
+                if ( ! $owningAssociation->isOwningSide()) {
1761 1761
                     throw InvalidFindByCall::fromInverseSideUsage(
1762 1762
                         $this->class->getClassName(),
1763 1763
                         $field
@@ -1772,7 +1772,7 @@  discard block
 block discarded – undo
1772 1772
                 foreach ($owningAssociation->getJoinColumns() as $joinColumn) {
1773 1773
                     $quotedColumnName = $this->platform->quoteIdentifier($joinColumn->getColumnName());
1774 1774
 
1775
-                    $columns[] = $tableAlias . '.' . $quotedColumnName;
1775
+                    $columns[] = $tableAlias.'.'.$quotedColumnName;
1776 1776
                 }
1777 1777
             }
1778 1778
 
@@ -1880,7 +1880,7 @@  discard block
 block discarded – undo
1880 1880
                 $value = $value[$targetClass->identifier[0]];
1881 1881
             }
1882 1882
 
1883
-            $criteria[$tableAlias . '.' . $quotedColumnName] = $value;
1883
+            $criteria[$tableAlias.'.'.$quotedColumnName] = $value;
1884 1884
             $parameters[]                                    = [
1885 1885
                 'value' => $value,
1886 1886
                 'field' => $fieldName,
@@ -1965,7 +1965,7 @@  discard block
 block discarded – undo
1965 1965
             case $property instanceof AssociationMetadata:
1966 1966
                 $class = $this->em->getClassMetadata($property->getTargetEntity());
1967 1967
 
1968
-                if (! $property->isOwningSide()) {
1968
+                if ( ! $property->isOwningSide()) {
1969 1969
                     $property = $class->getProperty($property->getMappedBy());
1970 1970
                 }
1971 1971
 
@@ -1985,7 +1985,7 @@  discard block
 block discarded – undo
1985 1985
         }
1986 1986
 
1987 1987
         if (is_array($value)) {
1988
-            return array_map(static function ($type) {
1988
+            return array_map(static function($type) {
1989 1989
                 return $type->getBindingType() + Connection::ARRAY_PARAM_OFFSET;
1990 1990
             }, $types);
1991 1991
         }
@@ -2042,7 +2042,7 @@  discard block
 block discarded – undo
2042 2042
      */
2043 2043
     private function getIndividualValue($value)
2044 2044
     {
2045
-        if (! is_object($value) || ! $this->em->getMetadataFactory()->hasMetadataFor(StaticClassNameConverter::getClass($value))) {
2045
+        if ( ! is_object($value) || ! $this->em->getMetadataFactory()->hasMetadataFor(StaticClassNameConverter::getClass($value))) {
2046 2046
             return $value;
2047 2047
         }
2048 2048
 
@@ -2056,7 +2056,7 @@  discard block
 block discarded – undo
2056 2056
     {
2057 2057
         $criteria = $this->getIdentifier($entity);
2058 2058
 
2059
-        if (! $criteria) {
2059
+        if ( ! $criteria) {
2060 2060
             return false;
2061 2061
         }
2062 2062
 
@@ -2064,12 +2064,12 @@  discard block
 block discarded – undo
2064 2064
 
2065 2065
         $sql = 'SELECT 1 '
2066 2066
              . $this->getLockTablesSql(null)
2067
-             . ' WHERE ' . $this->getSelectConditionSQL($criteria);
2067
+             . ' WHERE '.$this->getSelectConditionSQL($criteria);
2068 2068
 
2069 2069
         [$params, $types] = $this->expandParameters($criteria);
2070 2070
 
2071 2071
         if ($extraConditions !== null) {
2072
-            $sql                             .= ' AND ' . $this->getSelectConditionCriteriaSQL($extraConditions);
2072
+            $sql                             .= ' AND '.$this->getSelectConditionCriteriaSQL($extraConditions);
2073 2073
             [$criteriaParams, $criteriaTypes] = $this->expandCriteriaParameters($extraConditions);
2074 2074
 
2075 2075
             $params = array_merge($params, $criteriaParams);
@@ -2079,7 +2079,7 @@  discard block
 block discarded – undo
2079 2079
         $filterSql = $this->generateFilterConditionSQL($this->class, $alias);
2080 2080
 
2081 2081
         if ($filterSql) {
2082
-            $sql .= ' AND ' . $filterSql;
2082
+            $sql .= ' AND '.$filterSql;
2083 2083
         }
2084 2084
 
2085 2085
         return (bool) $this->conn->fetchColumn($sql, $params, $types);
@@ -2092,13 +2092,13 @@  discard block
 block discarded – undo
2092 2092
      */
2093 2093
     protected function getJoinSQLForAssociation(AssociationMetadata $association)
2094 2094
     {
2095
-        if (! $association->isOwningSide()) {
2095
+        if ( ! $association->isOwningSide()) {
2096 2096
             return 'LEFT JOIN';
2097 2097
         }
2098 2098
 
2099 2099
         // if one of the join columns is nullable, return left join
2100 2100
         foreach ($association->getJoinColumns() as $joinColumn) {
2101
-            if (! $joinColumn->isNullable()) {
2101
+            if ( ! $joinColumn->isNullable()) {
2102 2102
                 continue;
2103 2103
             }
2104 2104
 
@@ -2115,7 +2115,7 @@  discard block
 block discarded – undo
2115 2115
      */
2116 2116
     public function getSQLColumnAlias()
2117 2117
     {
2118
-        return $this->platform->getSQLResultCasing('c' . $this->currentPersisterContext->sqlAliasCounter++);
2118
+        return $this->platform->getSQLResultCasing('c'.$this->currentPersisterContext->sqlAliasCounter++);
2119 2119
     }
2120 2120
 
2121 2121
     /**
@@ -2134,13 +2134,13 @@  discard block
 block discarded – undo
2134 2134
             $filterExpr = $filter->addFilterConstraint($targetEntity, $targetTableAlias);
2135 2135
 
2136 2136
             if ($filterExpr !== '') {
2137
-                $filterClauses[] = '(' . $filterExpr . ')';
2137
+                $filterClauses[] = '('.$filterExpr.')';
2138 2138
             }
2139 2139
         }
2140 2140
 
2141 2141
         $sql = implode(' AND ', $filterClauses);
2142 2142
 
2143
-        return $sql ? '(' . $sql . ')' : ''; // Wrap again to avoid "X or Y and FilterConditionSQL"
2143
+        return $sql ? '('.$sql.')' : ''; // Wrap again to avoid "X or Y and FilterConditionSQL"
2144 2144
     }
2145 2145
 
2146 2146
     /**
Please login to merge, or discard this patch.