Failed Conditions
Pull Request — master (#6749)
by Konstantin
21:03 queued 11:29
created
tests/Doctrine/Tests/ORM/Functional/DetachedEntityTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -173,7 +173,7 @@  discard block
 block discarded – undo
173 173
         $this->_em->flush();
174 174
         $this->_em->detach($user);
175 175
 
176
-        $dql = 'SELECT u FROM ' . CmsUser::class . ' u WHERE u.id = ?1';
176
+        $dql = 'SELECT u FROM '.CmsUser::class.' u WHERE u.id = ?1';
177 177
         $query = $this->_em->createQuery($dql);
178 178
         $query->setParameter(1, $user);
179 179
 
@@ -216,7 +216,7 @@  discard block
 block discarded – undo
216 216
 
217 217
         $this->_em->detach($article);
218 218
 
219
-        $sql = 'UPDATE cms_articles SET version = version + 1 WHERE id = ' . $article->id;
219
+        $sql = 'UPDATE cms_articles SET version = version + 1 WHERE id = '.$article->id;
220 220
         $this->_em->getConnection()->executeUpdate($sql);
221 221
 
222 222
         $this->expectException(OptimisticLockException::class);
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
@@ -265,8 +265,8 @@
 block discarded – undo
265 265
         $this->_em->clear();
266 266
 
267 267
         $newUser = $this->_em->createQuery('SELECT u, g FROM Doctrine\Tests\Models\CMS\CmsUser u LEFT JOIN u.groups g WHERE u.id = ?1')
268
-                             ->setParameter(1, $user->getId())
269
-                             ->getSingleResult();
268
+                                ->setParameter(1, $user->getId())
269
+                                ->getSingleResult();
270 270
         $this->assertEquals(0, count($newUser->groups));
271 271
         $this->assertInternalType('array', $newUser->groups->getMapping());
272 272
 
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -290,9 +290,9 @@  discard block
 block discarded – undo
290 290
         $user->username = 'gblanco';
291 291
         $user->status = 'developer';
292 292
 
293
-        for ($i=0; $i < $groupCount; ++$i) {
293
+        for ($i = 0; $i < $groupCount; ++$i) {
294 294
             $group = new CmsGroup;
295
-            $group->name = 'Developers_' . $i;
295
+            $group->name = 'Developers_'.$i;
296 296
             $user->addGroup($group);
297 297
         }
298 298
 
@@ -406,7 +406,7 @@  discard block
 block discarded – undo
406 406
             $user
407 407
                 ->getGroups()
408 408
                 ->matching($criteria)
409
-                ->map(function (CmsGroup $group) {
409
+                ->map(function(CmsGroup $group) {
410 410
                     return $group->getName();
411 411
                 })
412 412
                 ->toArray()
@@ -450,7 +450,7 @@  discard block
 block discarded – undo
450 450
             $user
451 451
                 ->getTags()
452 452
                 ->matching($criteria)
453
-                ->map(function (CmsTag $tag) {
453
+                ->map(function(CmsTag $tag) {
454 454
                     return $tag->getName();
455 455
                 })
456 456
                 ->toArray()
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/MergeProxiesTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -237,10 +237,10 @@
 block discarded – undo
237 237
     {
238 238
         $config = new Configuration();
239 239
 
240
-        $config->setProxyDir(realpath(__DIR__ . '/../../Proxies'));
240
+        $config->setProxyDir(realpath(__DIR__.'/../../Proxies'));
241 241
         $config->setProxyNamespace('Doctrine\Tests\Proxies');
242 242
         $config->setMetadataDriverImpl($config->newDefaultAnnotationDriver(
243
-            [realpath(__DIR__ . '/../../Models/Cache')],
243
+            [realpath(__DIR__.'/../../Models/Cache')],
244 244
             true
245 245
         ));
246 246
         $config->setSQLLogger($logger);
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/CascadeRemoveOrderTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -152,10 +152,10 @@
 block discarded – undo
152 152
      */
153 153
     private $ownerO;
154 154
 
155
-    public function __construct(CascadeRemoveOrderEntityO $eO, $position=1)
155
+    public function __construct(CascadeRemoveOrderEntityO $eO, $position = 1)
156 156
     {
157 157
         $this->position = $position;
158
-        $this->ownerO= $eO;
158
+        $this->ownerO = $eO;
159 159
         $this->ownerO->addOneToManyG($this);
160 160
     }
161 161
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/SecondLevelCacheManyToOneTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -214,7 +214,7 @@
 block discarded – undo
214 214
         $this->assertNull($this->cache->getEntityCacheRegion(ComplexAction::class));
215 215
         $this->assertInstanceOf(Region::class, $this->cache->getEntityCacheRegion(Token::class));
216 216
 
217
-        $token  = new Token('token-hash');
217
+        $token = new Token('token-hash');
218 218
 
219 219
         $action1 = new Action('login');
220 220
         $action2 = new Action('logout');
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/SingleTableInheritanceTest.php 2 patches
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -411,9 +411,9 @@
 block discarded – undo
411 411
 
412 412
         $dql = 'SELECT f FROM Doctrine\Tests\Models\Company\CompanyFixContract f WHERE f.id = ?1';
413 413
         $contract = $this->_em->createQuery($dql)
414
-                              ->setFetchMode(CompanyFixContract::class, 'salesPerson', ClassMetadata::FETCH_EAGER)
415
-                              ->setParameter(1, $this->fix->getId())
416
-                              ->getSingleResult();
414
+                                ->setFetchMode(CompanyFixContract::class, 'salesPerson', ClassMetadata::FETCH_EAGER)
415
+                                ->setParameter(1, $this->fix->getId())
416
+                                ->getSingleResult();
417 417
 
418 418
         $this->assertNotInstanceOf(Proxy::class, $contract->getSalesPerson());
419 419
     }
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -330,19 +330,19 @@
 block discarded – undo
330 330
 
331 331
         $repos = $this->_em->getRepository(CompanyContract::class);
332 332
         $contracts = $repos->findBy(['salesPerson' => $this->salesPerson->getId()]);
333
-        $this->assertEquals(3, count($contracts), "There should be 3 entities related to " . $this->salesPerson->getId() . " for 'Doctrine\Tests\Models\Company\CompanyContract'");
333
+        $this->assertEquals(3, count($contracts), "There should be 3 entities related to ".$this->salesPerson->getId()." for 'Doctrine\Tests\Models\Company\CompanyContract'");
334 334
 
335 335
         $repos = $this->_em->getRepository(CompanyFixContract::class);
336 336
         $contracts = $repos->findBy(['salesPerson' => $this->salesPerson->getId()]);
337
-        $this->assertEquals(1, count($contracts), "There should be 1 entities related to " . $this->salesPerson->getId() . " for 'Doctrine\Tests\Models\Company\CompanyFixContract'");
337
+        $this->assertEquals(1, count($contracts), "There should be 1 entities related to ".$this->salesPerson->getId()." for 'Doctrine\Tests\Models\Company\CompanyFixContract'");
338 338
 
339 339
         $repos = $this->_em->getRepository(CompanyFlexContract::class);
340 340
         $contracts = $repos->findBy(['salesPerson' => $this->salesPerson->getId()]);
341
-        $this->assertEquals(2, count($contracts), "There should be 2 entities related to " . $this->salesPerson->getId() . " for 'Doctrine\Tests\Models\Company\CompanyFlexContract'");
341
+        $this->assertEquals(2, count($contracts), "There should be 2 entities related to ".$this->salesPerson->getId()." for 'Doctrine\Tests\Models\Company\CompanyFlexContract'");
342 342
 
343 343
         $repos = $this->_em->getRepository(CompanyFlexUltraContract::class);
344 344
         $contracts = $repos->findBy(['salesPerson' => $this->salesPerson->getId()]);
345
-        $this->assertEquals(1, count($contracts), "There should be 1 entities related to " . $this->salesPerson->getId() . " for 'Doctrine\Tests\Models\Company\CompanyFlexUltraContract'");
345
+        $this->assertEquals(1, count($contracts), "There should be 1 entities related to ".$this->salesPerson->getId()." for 'Doctrine\Tests\Models\Company\CompanyFlexUltraContract'");
346 346
     }
347 347
 
348 348
     /**
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/DefaultValuesTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
      */
111 111
     public $address;
112 112
 
113
-    public function getId() {return $this->id;}
113
+    public function getId() {return $this->id; }
114 114
 }
115 115
 
116 116
 /**
@@ -152,5 +152,5 @@  discard block
 block discarded – undo
152 152
      */
153 153
     public $user;
154 154
 
155
-    public function getUser() {return $this->user;}
155
+    public function getUser() {return $this->user; }
156 156
 }
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/SchemaValidatorTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
      */
35 35
     private function registerType(string $className)
36 36
     {
37
-        $type = constant($className . '::NAME');
37
+        $type = constant($className.'::NAME');
38 38
 
39 39
         if (DBALType::hasType($type)) {
40 40
             DBALType::overrideType($type, $className);
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
         foreach ($classes as $class) {
71 71
             $ce = $validator->validateClass($class);
72 72
 
73
-            $this->assertEmpty($ce, "Invalid Modelset: " . $modelSet . " class " . $class->name . ": ". implode("\n", $ce));
73
+            $this->assertEmpty($ce, "Invalid Modelset: ".$modelSet." class ".$class->name.": ".implode("\n", $ce));
74 74
         }
75 75
     }
76 76
 }
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/PostLoadEventTest.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -232,8 +232,8 @@  discard block
 block discarded – undo
232 232
         $eventManager->addEventListener([Events::postLoad], $listener);
233 233
 
234 234
         $this->_em->find(CmsUser::class, $this->userId);
235
-        $this->assertSame(1, $listener->countHandledEvents(CmsUser::class), CmsUser::class . ' should be handled once!');
236
-        $this->assertSame(1, $listener->countHandledEvents(CmsEmail::class), CmsEmail::class . ' should be handled once!');
235
+        $this->assertSame(1, $listener->countHandledEvents(CmsUser::class), CmsUser::class.' should be handled once!');
236
+        $this->assertSame(1, $listener->countHandledEvents(CmsEmail::class), CmsEmail::class.' should be handled once!');
237 237
     }
238 238
 
239 239
     private function loadFixture()
@@ -309,7 +309,7 @@  discard block
 block discarded – undo
309 309
     {
310 310
         $object = $event->getObject();
311 311
         $class = ClassUtils::getClass($object);
312
-        if (!isset($this->firedByClasses[$class])) {
312
+        if ( ! isset($this->firedByClasses[$class])) {
313 313
             $this->firedByClasses[$class] = 1;
314 314
         } else {
315 315
             $this->firedByClasses[$class]++;
Please login to merge, or discard this patch.