Failed Conditions
Pull Request — develop (#6599)
by Michael
131:05 queued 63:47
created
tests/Doctrine/Tests/ORM/Functional/SecondLevelCacheOneToManyTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Doctrine\Tests\ORM\Functional;
6 6
 
@@ -396,7 +396,7 @@  discard block
 block discarded – undo
396 396
 
397 397
         $l1 = new Login('session1');
398 398
         $l2 = new Login('session2');
399
-        $token  = new Token('token-hash');
399
+        $token = new Token('token-hash');
400 400
         $token->addLogin($l1);
401 401
         $token->addLogin($l2);
402 402
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/DatabaseDriverTest.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Doctrine\Tests\ORM\Functional;
6 6
 
@@ -31,7 +31,7 @@  discard block
 block discarded – undo
31 31
      */
32 32
     public function testIssue2059()
33 33
     {
34
-        if (!$this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
34
+        if ( ! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
35 35
             $this->markTestSkipped('Platform does not support foreign keys.');
36 36
         }
37 37
 
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
 
54 54
     public function testLoadMetadataFromDatabase()
55 55
     {
56
-        if (!$this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
56
+        if ( ! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
57 57
             $this->markTestSkipped('Platform does not support foreign keys.');
58 58
         }
59 59
 
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
 
92 92
     public function testLoadMetadataWithForeignKeyFromDatabase()
93 93
     {
94
-        if (!$this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
94
+        if ( ! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
95 95
             $this->markTestSkipped('Platform does not support foreign keys.');
96 96
         }
97 97
 
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
 
125 125
     public function testDetectManyToManyTables()
126 126
     {
127
-        if (!$this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
127
+        if ( ! $this->em->getConnection()->getDatabasePlatform()->supportsForeignKeyConstraints()) {
128 128
             $this->markTestSkipped('Platform does not support foreign keys.');
129 129
         }
130 130
 
@@ -179,7 +179,7 @@  discard block
 block discarded – undo
179 179
 
180 180
         $table->addColumn('column_index1', 'string');
181 181
         $table->addColumn('column_index2', 'string');
182
-        $table->addIndex(['column_index1','column_index2'], 'index1');
182
+        $table->addIndex(['column_index1', 'column_index2'], 'index1');
183 183
 
184 184
         $table->addColumn('column_unique_index1', 'string');
185 185
         $table->addColumn('column_unique_index2', 'string');
@@ -245,7 +245,7 @@  discard block
 block discarded – undo
245 245
 
246 246
         self::assertTrue( ! empty($indexes['index1']['columns']));
247 247
         self::assertEquals(
248
-            ['column_index1','column_index2'],
248
+            ['column_index1', 'column_index2'],
249 249
             $indexes['index1']['columns']
250 250
         );
251 251
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/ManyToManyBasicAssociationTest.php 2 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -268,8 +268,8 @@
 block discarded – undo
268 268
         $this->em->clear();
269 269
 
270 270
         $newUser = $this->em->createQuery('SELECT u, g FROM Doctrine\Tests\Models\CMS\CmsUser u LEFT JOIN u.groups g WHERE u.id = ?1')
271
-                             ->setParameter(1, $user->getId())
272
-                             ->getSingleResult();
271
+                                ->setParameter(1, $user->getId())
272
+                                ->getSingleResult();
273 273
         self::assertEquals(0, count($newUser->groups));
274 274
         self::assertInstanceOf(ManyToManyAssociationMetadata::class, $newUser->groups->getMapping());
275 275
 
Please login to merge, or discard this patch.
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Doctrine\Tests\ORM\Functional;
6 6
 
@@ -293,9 +293,9 @@  discard block
 block discarded – undo
293 293
         $user->username = 'gblanco';
294 294
         $user->status = 'developer';
295 295
 
296
-        for ($i=0; $i < $groupCount; ++$i) {
296
+        for ($i = 0; $i < $groupCount; ++$i) {
297 297
             $group = new CmsGroup;
298
-            $group->name = 'Developers_' . $i;
298
+            $group->name = 'Developers_'.$i;
299 299
             $user->addGroup($group);
300 300
         }
301 301
 
@@ -418,7 +418,7 @@  discard block
 block discarded – undo
418 418
             $user
419 419
                 ->getGroups()
420 420
                 ->matching($criteria)
421
-                ->map(function (CmsGroup $group) {
421
+                ->map(function(CmsGroup $group) {
422 422
                     return $group->getName();
423 423
                 })
424 424
                 ->toArray()
@@ -462,7 +462,7 @@  discard block
 block discarded – undo
462 462
             $user
463 463
                 ->getTags()
464 464
                 ->matching($criteria)
465
-                ->map(function (CmsTag $tag) {
465
+                ->map(function(CmsTag $tag) {
466 466
                     return $tag->getName();
467 467
                 })
468 468
                 ->toArray()
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/PostLoadEventTest.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -1,5 +1,5 @@  discard block
 block discarded – undo
1 1
 <?php
2
-declare(strict_types=1);
2
+declare(strict_types = 1);
3 3
 
4 4
 namespace Doctrine\Tests\ORM\Functional;
5 5
 
@@ -234,8 +234,8 @@  discard block
 block discarded – undo
234 234
         $eventManager->addEventListener([Events::postLoad], $listener);
235 235
 
236 236
         $this->em->find(CmsUser::class, $this->userId);
237
-        self::assertSame(1, $listener->countHandledEvents(CmsUser::class), CmsUser::class . ' should be handled once!');
238
-        self::assertSame(1, $listener->countHandledEvents(CmsEmail::class), CmsEmail::class . ' should be handled once!');
237
+        self::assertSame(1, $listener->countHandledEvents(CmsUser::class), CmsUser::class.' should be handled once!');
238
+        self::assertSame(1, $listener->countHandledEvents(CmsEmail::class), CmsEmail::class.' should be handled once!');
239 239
     }
240 240
 
241 241
     private function loadFixture()
@@ -311,7 +311,7 @@  discard block
 block discarded – undo
311 311
     {
312 312
         $object = $event->getObject();
313 313
         $class = ClassUtils::getClass($object);
314
-        if (!isset($this->firedByClasses[$class])) {
314
+        if ( ! isset($this->firedByClasses[$class])) {
315 315
             $this->firedByClasses[$class] = 1;
316 316
         } else {
317 317
             $this->firedByClasses[$class]++;
Please login to merge, or discard this patch.
Tests/ORM/Functional/ValueConversionType/ManyToManyExtraLazyTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Doctrine\Tests\ORM\Functional\ValueConversionType;
6 6
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/CustomFunctionsTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Doctrine\Tests\ORM\Functional;
6 6
 
@@ -13,7 +13,7 @@  discard block
 block discarded – undo
13 13
 use Doctrine\Tests\Models\CMS\CmsUser;
14 14
 use Doctrine\Tests\OrmFunctionalTestCase;
15 15
 
16
-require_once __DIR__ . '/../../TestInit.php';
16
+require_once __DIR__.'/../../TestInit.php';
17 17
 
18 18
 class CustomFunctionsTest extends OrmFunctionalTestCase
19 19
 {
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/SecondLevelCacheQueryCacheTest.php 1 patch
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Doctrine\Tests\ORM\Functional;
6 6
 
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
 
52 52
         $this->em->clear();
53 53
 
54
-        $result2  = $this->em->createQuery($dql)
54
+        $result2 = $this->em->createQuery($dql)
55 55
             ->setCacheable(true)
56 56
             ->getResult();
57 57
 
@@ -254,7 +254,7 @@  discard block
 block discarded – undo
254 254
 
255 255
         $this->em->clear();
256 256
 
257
-        $result2  = $this->em->createQuery($dql)
257
+        $result2 = $this->em->createQuery($dql)
258 258
             ->setCacheable(true)
259 259
             ->getResult();
260 260
 
@@ -346,7 +346,7 @@  discard block
 block discarded – undo
346 346
 
347 347
         $this->em->clear();
348 348
 
349
-        $result2  = $this->em->createQuery($dql)
349
+        $result2 = $this->em->createQuery($dql)
350 350
             ->setCacheable(true)
351 351
             ->getResult();
352 352
 
@@ -394,7 +394,7 @@  discard block
 block discarded – undo
394 394
 
395 395
         $this->em->clear();
396 396
 
397
-        $result2  = $this->em->createQuery($dql)->setCacheable(true)
397
+        $result2 = $this->em->createQuery($dql)->setCacheable(true)
398 398
                 ->setParameter('name', $name)
399 399
                 ->getResult();
400 400
 
@@ -422,7 +422,7 @@  discard block
 block discarded – undo
422 422
         $result1    = $this->em->createQuery($dql)->setCacheable(true)->getResult();
423 423
 
424 424
         self::assertCount(2, $result1);
425
-        self::assertEquals($queryCount + 1 , $this->getCurrentQueryCount());
425
+        self::assertEquals($queryCount + 1, $this->getCurrentQueryCount());
426 426
         self::assertEquals($this->countries[0]->getId(), $result1[0]->getId());
427 427
         self::assertEquals($this->countries[1]->getId(), $result1[1]->getId());
428 428
         self::assertEquals($this->countries[0]->getName(), $result1[0]->getName());
@@ -438,11 +438,11 @@  discard block
 block discarded – undo
438 438
 
439 439
         $this->em->clear();
440 440
 
441
-        $result2  = $this->em->createQuery($dql)
441
+        $result2 = $this->em->createQuery($dql)
442 442
             ->setCacheable(true)
443 443
             ->getResult();
444 444
 
445
-        self::assertEquals($queryCount + 2 , $this->getCurrentQueryCount());
445
+        self::assertEquals($queryCount + 2, $this->getCurrentQueryCount());
446 446
         self::assertCount(2, $result2);
447 447
 
448 448
         self::assertEquals(5, $this->secondLevelCacheLogger->getPutCount());
@@ -459,7 +459,7 @@  discard block
 block discarded – undo
459 459
         self::assertEquals($result1[0]->getName(), $result2[0]->getName());
460 460
         self::assertEquals($result1[1]->getName(), $result2[1]->getName());
461 461
 
462
-        self::assertEquals($queryCount + 2 , $this->getCurrentQueryCount());
462
+        self::assertEquals($queryCount + 2, $this->getCurrentQueryCount());
463 463
     }
464 464
 
465 465
     public function testBasicQueryFetchJoinsOneToMany()
@@ -500,7 +500,7 @@  discard block
 block discarded – undo
500 500
 
501 501
         $this->em->clear();
502 502
 
503
-        $result2  = $this->em->createQuery($dql)
503
+        $result2 = $this->em->createQuery($dql)
504 504
                 ->setCacheable(true)
505 505
                 ->getResult();
506 506
 
@@ -566,7 +566,7 @@  discard block
 block discarded – undo
566 566
         $this->em->clear();
567 567
         $this->secondLevelCacheLogger->clearStats();
568 568
 
569
-        $result2  = $this->em->createQuery($dql)
569
+        $result2 = $this->em->createQuery($dql)
570 570
                 ->setCacheable(true)
571 571
                 ->getResult();
572 572
 
@@ -626,7 +626,7 @@  discard block
 block discarded – undo
626 626
 
627 627
         $this->cache->evictEntityRegion(State::class);
628 628
 
629
-        $result2  = $this->em->createQuery($dql)
629
+        $result2 = $this->em->createQuery($dql)
630 630
                 ->setCacheable(true)
631 631
                 ->getResult();
632 632
 
@@ -669,7 +669,7 @@  discard block
 block discarded – undo
669 669
 
670 670
         $this->cache->evictEntityRegion(City::class);
671 671
 
672
-        $result2  = $this->em->createQuery($dql)
672
+        $result2 = $this->em->createQuery($dql)
673 673
                 ->setCacheable(true)
674 674
                 ->getResult();
675 675
 
@@ -720,7 +720,7 @@  discard block
 block discarded – undo
720 720
 
721 721
         $this->em->clear();
722 722
 
723
-        $result2  = $this->em->createNativeQuery($sql, $rsm)
723
+        $result2 = $this->em->createNativeQuery($sql, $rsm)
724 724
             ->setCacheable(true)
725 725
             ->getResult();
726 726
 
@@ -775,7 +775,7 @@  discard block
 block discarded – undo
775 775
 
776 776
         $this->em->clear();
777 777
 
778
-        $result2  = $this->em->createQuery($dql)
778
+        $result2 = $this->em->createQuery($dql)
779 779
             ->setCacheable(true)
780 780
             ->setFirstResult(2)
781 781
             ->setMaxResults(1)
@@ -787,7 +787,7 @@  discard block
 block discarded – undo
787 787
 
788 788
         $this->em->clear();
789 789
 
790
-        $result3  = $this->em->createQuery($dql)
790
+        $result3 = $this->em->createQuery($dql)
791 791
             ->setCacheable(true)
792 792
             ->getResult();
793 793
 
@@ -804,7 +804,7 @@  discard block
 block discarded – undo
804 804
         $this->secondLevelCacheLogger->clearStats();
805 805
         $this->em->clear();
806 806
 
807
-        $getHash = function(AbstractQuery $query){
807
+        $getHash = function(AbstractQuery $query) {
808 808
             $method = new \ReflectionMethod($query, 'getHash');
809 809
             $method->setAccessible(true);
810 810
 
@@ -837,7 +837,7 @@  discard block
 block discarded – undo
837 837
             ->getRegion()
838 838
             ->put($key, $entry);
839 839
 
840
-        $result2  = $this->em->createQuery($dql)
840
+        $result2 = $this->em->createQuery($dql)
841 841
             ->setCacheable(true)
842 842
             ->setLifetime(3600)
843 843
             ->getResult();
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/SchemaValidatorTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Doctrine\Tests\ORM\Functional;
6 6
 
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
      */
37 37
     private function registerType(string $className)
38 38
     {
39
-        $type = constant($className . '::NAME');
39
+        $type = constant($className.'::NAME');
40 40
 
41 41
         if (DBALType::hasType($type)) {
42 42
             DBALType::overrideType($type, $className);
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
         foreach ($classes as $class) {
73 73
             $ce = $validator->validateClass($class);
74 74
 
75
-            self::assertEmpty($ce, "Invalid Modelset: " . $modelSet . " class " . $class->getClassName() . ": ". implode("\n", $ce));
75
+            self::assertEmpty($ce, "Invalid Modelset: ".$modelSet." class ".$class->getClassName().": ".implode("\n", $ce));
76 76
         }
77 77
     }
78 78
 }
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/ReferenceProxyTest.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -1,6 +1,6 @@  discard block
 block discarded – undo
1 1
 <?php
2 2
 
3
-declare(strict_types=1);
3
+declare(strict_types = 1);
4 4
 
5 5
 namespace Doctrine\Tests\ORM\Functional;
6 6
 
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
         parent::setUp();
43 43
 
44 44
         $namespace = 'Doctrine\Tests\Proxies';
45
-        $directory = __DIR__ . '/../../Proxies';
45
+        $directory = __DIR__.'/../../Proxies';
46 46
 
47 47
         $this->resolver = new DefaultProxyResolver($namespace, $directory);
48 48
 
@@ -96,7 +96,7 @@  discard block
 block discarded – undo
96 96
     {
97 97
         $id = $this->createProduct();
98 98
 
99
-        $entity = $this->em->getReference(ECommerceProduct::class , $id);
99
+        $entity = $this->em->getReference(ECommerceProduct::class, $id);
100 100
         $class = $this->em->getClassMetadata(get_class($entity));
101 101
 
102 102
         self::assertEquals(ECommerceProduct::class, $class->getClassName());
@@ -109,8 +109,8 @@  discard block
 block discarded – undo
109 109
     {
110 110
         $id = $this->createProduct();
111 111
 
112
-        $entity = $this->em->getReference(ECommerceProduct::class , $id);
113
-        $entity2 = $this->em->find(ECommerceProduct::class , $id);
112
+        $entity = $this->em->getReference(ECommerceProduct::class, $id);
113
+        $entity2 = $this->em->find(ECommerceProduct::class, $id);
114 114
 
115 115
         self::assertSame($entity, $entity2);
116 116
         self::assertEquals('Doctrine Cookbook', $entity2->getName());
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
         $id = $this->createProduct();
125 125
 
126 126
         /* @var $entity ECommerceProduct */
127
-        $entity = $this->em->getReference(ECommerceProduct::class , $id);
127
+        $entity = $this->em->getReference(ECommerceProduct::class, $id);
128 128
 
129 129
         /* @var $clone ECommerceProduct */
130 130
         $clone = clone $entity;
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
         $id = $this->createProduct();
150 150
 
151 151
         /* @var $entity ECommerceProduct */
152
-        $entity = $this->em->getReference(ECommerceProduct::class , $id);
152
+        $entity = $this->em->getReference(ECommerceProduct::class, $id);
153 153
 
154 154
         self::assertFalse($entity->__isInitialized(), "Pre-Condition: Object is unitialized proxy.");
155 155
 
@@ -166,14 +166,14 @@  discard block
 block discarded – undo
166 166
         $id = $this->createProduct();
167 167
 
168 168
         /* @var $entity ECommerceProduct */
169
-        $entity = $this->em->getReference(ECommerceProduct::class , $id);
169
+        $entity = $this->em->getReference(ECommerceProduct::class, $id);
170 170
 
171 171
         $entity->setName('Doctrine 2 Cookbook');
172 172
 
173 173
         $this->em->flush();
174 174
         $this->em->clear();
175 175
 
176
-        $entity = $this->em->getReference(ECommerceProduct::class , $id);
176
+        $entity = $this->em->getReference(ECommerceProduct::class, $id);
177 177
 
178 178
         self::assertEquals('Doctrine 2 Cookbook', $entity->getName());
179 179
     }
@@ -186,7 +186,7 @@  discard block
 block discarded – undo
186 186
         $id = $this->createProduct();
187 187
 
188 188
         /* @var $entity ECommerceProduct */
189
-        $entity = $this->em->getReference(ECommerceProduct::class , $id);
189
+        $entity = $this->em->getReference(ECommerceProduct::class, $id);
190 190
 
191 191
         self::assertFalse($entity->wakeUp);
192 192
 
@@ -200,7 +200,7 @@  discard block
 block discarded – undo
200 200
         $id = $this->createProduct();
201 201
 
202 202
         /* @var $entity ECommerceProduct */
203
-        $entity = $this->em->getReference(ECommerceProduct::class , $id);
203
+        $entity = $this->em->getReference(ECommerceProduct::class, $id);
204 204
 
205 205
         self::assertFalse($entity->__isInitialized(), "Pre-Condition: Object is unitialized proxy.");
206 206
         self::assertEquals($id, $entity->getId());
@@ -215,7 +215,7 @@  discard block
 block discarded – undo
215 215
         $id = $this->createAuction();
216 216
 
217 217
         /* @var $entity CompanyAuction */
218
-        $entity = $this->em->getReference(CompanyAuction::class , $id);
218
+        $entity = $this->em->getReference(CompanyAuction::class, $id);
219 219
 
220 220
         self::assertFalse($entity->__isInitialized(), "Pre-Condition: Object is unitialized proxy.");
221 221
         self::assertEquals($id, $entity->getId());
@@ -252,7 +252,7 @@  discard block
 block discarded – undo
252 252
         $id = $this->createProduct();
253 253
 
254 254
         /* @var $entity ECommerceProduct */
255
-        $entity = $this->em->getReference(ECommerceProduct::class , $id);
255
+        $entity = $this->em->getReference(ECommerceProduct::class, $id);
256 256
 
257 257
         self::assertFalse($entity->__isInitialized(), "Pre-Condition: Object is unitialized proxy.");
258 258
         self::assertEquals('Doctrine Cookbook', $entity->getName());
@@ -267,7 +267,7 @@  discard block
 block discarded – undo
267 267
         $id = $this->createProduct();
268 268
 
269 269
         /* @var $entity ECommerceProduct */
270
-        $entity = $this->em->getReference(ECommerceProduct::class , $id);
270
+        $entity = $this->em->getReference(ECommerceProduct::class, $id);
271 271
 
272 272
         $className = ClassUtils::getClass($entity);
273 273
 
@@ -275,7 +275,7 @@  discard block
 block discarded – undo
275 275
         self::assertFalse($entity->__isInitialized());
276 276
         self::assertEquals(ECommerceProduct::class, $className);
277 277
 
278
-        $proxyFileName = $this->resolver->resolveProxyClassPath(ECommerceProduct::class );
278
+        $proxyFileName = $this->resolver->resolveProxyClassPath(ECommerceProduct::class);
279 279
 
280 280
         self::assertTrue(file_exists($proxyFileName), "Proxy file name cannot be found generically.");
281 281
 
Please login to merge, or discard this patch.