Completed
Pull Request — master (#7921)
by Guilherme
14:09
created
tests/Doctrine/Tests/ORM/Functional/ClassTableInheritanceTest.php 2 patches
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -301,7 +301,7 @@  discard block
 block discarded – undo
301 301
     public function testBulkUpdateIssueDDC368() : void
302 302
     {
303 303
         $this->em->createQuery('UPDATE ' . CompanyEmployee::class . ' AS p SET p.salary = 1')
304
-                  ->execute();
304
+                    ->execute();
305 305
 
306 306
         $result = $this->em->createQuery('SELECT count(p.id) FROM ' . CompanyEmployee::class . ' p WHERE p.salary = 1')
307 307
                             ->getResult();
@@ -315,9 +315,9 @@  discard block
 block discarded – undo
315 315
     public function testBulkUpdateNonScalarParameterDDC1341() : void
316 316
     {
317 317
         $this->em->createQuery('UPDATE ' . CompanyEmployee::class . ' AS p SET p.startDate = ?0 WHERE p.department = ?1')
318
-                  ->setParameter(0, new DateTime())
319
-                  ->setParameter(1, 'IT')
320
-                  ->execute();
318
+                    ->setParameter(0, new DateTime())
319
+                    ->setParameter(1, 'IT')
320
+                    ->execute();
321 321
 
322 322
         self::addToAssertionCount(1);
323 323
     }
Please login to merge, or discard this patch.
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
 
51 51
         $this->em->clear();
52 52
 
53
-        $query = $this->em->createQuery('select p from ' . CompanyPerson::class . ' p order by p.name desc');
53
+        $query = $this->em->createQuery('select p from '.CompanyPerson::class.' p order by p.name desc');
54 54
 
55 55
         $entities = $query->getResult();
56 56
 
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
 
66 66
         $this->em->clear();
67 67
 
68
-        $query = $this->em->createQuery('select p from ' . CompanyEmployee::class . ' p');
68
+        $query = $this->em->createQuery('select p from '.CompanyEmployee::class.' p');
69 69
 
70 70
         $entities = $query->getResult();
71 71
 
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
 
85 85
         $this->em->clear();
86 86
 
87
-        $query = $this->em->createQuery('update ' . CompanyEmployee::class . " p set p.name = ?1, p.department = ?2 where p.name='Guilherme Blanco' and p.salary = ?3");
87
+        $query = $this->em->createQuery('update '.CompanyEmployee::class." p set p.name = ?1, p.department = ?2 where p.name='Guilherme Blanco' and p.salary = ?3");
88 88
 
89 89
         $query->setParameter(1, 'NewName', 'string');
90 90
         $query->setParameter(2, 'NewDepartment');
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
 
96 96
         self::assertEquals(1, $numUpdated);
97 97
 
98
-        $query = $this->em->createQuery('delete from ' . CompanyPerson::class . ' p');
98
+        $query = $this->em->createQuery('delete from '.CompanyPerson::class.' p');
99 99
 
100 100
         $numDeleted = $query->execute();
101 101
 
@@ -170,7 +170,7 @@  discard block
 block discarded – undo
170 170
         $this->em->flush();
171 171
         $this->em->clear();
172 172
 
173
-        $query = $this->em->createQuery('select p, s from ' . CompanyPerson::class . ' p join p.spouse s where p.name=\'Mary Smith\'');
173
+        $query = $this->em->createQuery('select p, s from '.CompanyPerson::class.' p join p.spouse s where p.name=\'Mary Smith\'');
174 174
 
175 175
         $result = $query->getResult();
176 176
 
@@ -202,7 +202,7 @@  discard block
 block discarded – undo
202 202
 
203 203
         $this->em->clear();
204 204
 
205
-        $query = $this->em->createQuery('select p, f from ' . CompanyPerson::class . ' p join p.friends f where p.name=?1');
205
+        $query = $this->em->createQuery('select p, f from '.CompanyPerson::class.' p join p.friends f where p.name=?1');
206 206
 
207 207
         $query->setParameter(1, 'Roman');
208 208
 
@@ -268,7 +268,7 @@  discard block
 block discarded – undo
268 268
         $this->em->flush();
269 269
         $this->em->clear();
270 270
 
271
-        $q = $this->em->createQuery('select a from ' . CompanyEvent::class . ' a where a.id = ?1');
271
+        $q = $this->em->createQuery('select a from '.CompanyEvent::class.' a where a.id = ?1');
272 272
 
273 273
         $q->setParameter(1, $event1->getId());
274 274
 
@@ -279,7 +279,7 @@  discard block
 block discarded – undo
279 279
 
280 280
         $this->em->clear();
281 281
 
282
-        $q = $this->em->createQuery('select a from ' . CompanyOrganization::class . ' a where a.id = ?1');
282
+        $q = $this->em->createQuery('select a from '.CompanyOrganization::class.' a where a.id = ?1');
283 283
 
284 284
         $q->setParameter(1, $org->getId());
285 285
 
@@ -300,10 +300,10 @@  discard block
 block discarded – undo
300 300
      */
301 301
     public function testBulkUpdateIssueDDC368() : void
302 302
     {
303
-        $this->em->createQuery('UPDATE ' . CompanyEmployee::class . ' AS p SET p.salary = 1')
303
+        $this->em->createQuery('UPDATE '.CompanyEmployee::class.' AS p SET p.salary = 1')
304 304
                   ->execute();
305 305
 
306
-        $result = $this->em->createQuery('SELECT count(p.id) FROM ' . CompanyEmployee::class . ' p WHERE p.salary = 1')
306
+        $result = $this->em->createQuery('SELECT count(p.id) FROM '.CompanyEmployee::class.' p WHERE p.salary = 1')
307 307
                             ->getResult();
308 308
 
309 309
         self::assertGreaterThan(0, $result);
@@ -314,7 +314,7 @@  discard block
 block discarded – undo
314 314
      */
315 315
     public function testBulkUpdateNonScalarParameterDDC1341() : void
316 316
     {
317
-        $this->em->createQuery('UPDATE ' . CompanyEmployee::class . ' AS p SET p.startDate = ?0 WHERE p.department = ?1')
317
+        $this->em->createQuery('UPDATE '.CompanyEmployee::class.' AS p SET p.startDate = ?0 WHERE p.department = ?1')
318 318
                   ->setParameter(0, new DateTime())
319 319
                   ->setParameter(1, 'IT')
320 320
                   ->execute();
@@ -372,7 +372,7 @@  discard block
 block discarded – undo
372 372
         $this->em->flush();
373 373
         $this->em->clear();
374 374
 
375
-        $dqlManager = $this->em->createQuery('SELECT m FROM ' . CompanyManager::class . ' m WHERE m.spouse = ?1')
375
+        $dqlManager = $this->em->createQuery('SELECT m FROM '.CompanyManager::class.' m WHERE m.spouse = ?1')
376 376
                                 ->setParameter(1, $person->getId())
377 377
                                 ->getSingleResult();
378 378
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/SingleTableInheritanceTest.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -233,7 +233,7 @@  discard block
 block discarded – undo
233 233
 
234 234
         $contracts = $this->em->createQuery('SELECT c FROM Doctrine\Tests\Models\Company\CompanyContract c ORDER BY c.id')->getScalarResult();
235 235
 
236
-        $discrValues = array_map(static function ($a) {
236
+        $discrValues = array_map(static function($a) {
237 237
             return $a['c_discr'];
238 238
         }, $contracts);
239 239
 
@@ -335,19 +335,19 @@  discard block
 block discarded – undo
335 335
 
336 336
         $repos     = $this->em->getRepository(CompanyContract::class);
337 337
         $contracts = $repos->findBy(['salesPerson' => $this->salesPerson->getId()]);
338
-        self::assertCount(3, $contracts, 'There should be 3 entities related to ' . $this->salesPerson->getId() . " for 'Doctrine\Tests\Models\Company\CompanyContract'");
338
+        self::assertCount(3, $contracts, 'There should be 3 entities related to '.$this->salesPerson->getId()." for 'Doctrine\Tests\Models\Company\CompanyContract'");
339 339
 
340 340
         $repos     = $this->em->getRepository(CompanyFixContract::class);
341 341
         $contracts = $repos->findBy(['salesPerson' => $this->salesPerson->getId()]);
342
-        self::assertCount(1, $contracts, 'There should be 1 entities related to ' . $this->salesPerson->getId() . " for 'Doctrine\Tests\Models\Company\CompanyFixContract'");
342
+        self::assertCount(1, $contracts, 'There should be 1 entities related to '.$this->salesPerson->getId()." for 'Doctrine\Tests\Models\Company\CompanyFixContract'");
343 343
 
344 344
         $repos     = $this->em->getRepository(CompanyFlexContract::class);
345 345
         $contracts = $repos->findBy(['salesPerson' => $this->salesPerson->getId()]);
346
-        self::assertCount(2, $contracts, 'There should be 2 entities related to ' . $this->salesPerson->getId() . " for 'Doctrine\Tests\Models\Company\CompanyFlexContract'");
346
+        self::assertCount(2, $contracts, 'There should be 2 entities related to '.$this->salesPerson->getId()." for 'Doctrine\Tests\Models\Company\CompanyFlexContract'");
347 347
 
348 348
         $repos     = $this->em->getRepository(CompanyFlexUltraContract::class);
349 349
         $contracts = $repos->findBy(['salesPerson' => $this->salesPerson->getId()]);
350
-        self::assertCount(1, $contracts, 'There should be 1 entities related to ' . $this->salesPerson->getId() . " for 'Doctrine\Tests\Models\Company\CompanyFlexUltraContract'");
350
+        self::assertCount(1, $contracts, 'There should be 1 entities related to '.$this->salesPerson->getId()." for 'Doctrine\Tests\Models\Company\CompanyFlexUltraContract'");
351 351
     }
352 352
 
353 353
     /**
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/Ticket/DDC2602Test.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
     {
138 138
         $entity = $event->getEntity();
139 139
 
140
-        if (! ($entity instanceof DDC2602Biography)) {
140
+        if ( ! ($entity instanceof DDC2602Biography)) {
141 141
             return;
142 142
         }
143 143
 
@@ -158,7 +158,7 @@  discard block
 block discarded – undo
158 158
             $fieldSelection = new DDC2602FieldSelection();
159 159
 
160 160
             $fieldSelection->field      = $field;
161
-            $fieldSelection->choiceList = $field->choiceList->filter(static function ($choice) use ($choiceList) {
161
+            $fieldSelection->choiceList = $field->choiceList->filter(static function($choice) use ($choiceList) {
162 162
                 return in_array($choice->id, $choiceList, true);
163 163
             });
164 164
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/Ticket/DDC522Test.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -51,7 +51,7 @@
 block discarded – undo
51 51
         $this->em->clear();
52 52
 
53 53
         $cart = $this->em
54
-            ->createQuery('select ca, c from ' . DDC522Cart::class . ' ca join ca.customer c')
54
+            ->createQuery('select ca, c from '.DDC522Cart::class.' ca join ca.customer c')
55 55
             ->getSingleResult();
56 56
 
57 57
         self::assertInstanceOf(DDC522Cart::class, $cart);
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/Ticket/DDC2084Test.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -19,8 +19,8 @@  discard block
 block discarded – undo
19 19
         try {
20 20
             $this->schemaTool->createSchema(
21 21
                 [
22
-                    $this->em->getClassMetadata(__NAMESPACE__ . '\DDC2084\MyEntity1'),
23
-                    $this->em->getClassMetadata(__NAMESPACE__ . '\DDC2084\MyEntity2'),
22
+                    $this->em->getClassMetadata(__NAMESPACE__.'\DDC2084\MyEntity1'),
23
+                    $this->em->getClassMetadata(__NAMESPACE__.'\DDC2084\MyEntity2'),
24 24
                 ]
25 25
             );
26 26
         } catch (Exception $exc) {
@@ -47,10 +47,10 @@  discard block
 block discarded – undo
47 47
     {
48 48
         $e1 = $this->loadFixture();
49 49
         $e2 = $e1->getMyEntity2();
50
-        $e  = $this->em->find(__NAMESPACE__ . '\DDC2084\MyEntity1', $e2);
50
+        $e  = $this->em->find(__NAMESPACE__.'\DDC2084\MyEntity1', $e2);
51 51
 
52
-        self::assertInstanceOf(__NAMESPACE__ . '\DDC2084\MyEntity1', $e);
53
-        self::assertInstanceOf(__NAMESPACE__ . '\DDC2084\MyEntity2', $e->getMyEntity2());
52
+        self::assertInstanceOf(__NAMESPACE__.'\DDC2084\MyEntity1', $e);
53
+        self::assertInstanceOf(__NAMESPACE__.'\DDC2084\MyEntity2', $e->getMyEntity2());
54 54
         self::assertEquals('Foo', $e->getMyEntity2()->getValue());
55 55
     }
56 56
 
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
      */
61 61
     public function testinvalidIdentifierBindingEntityException() : void
62 62
     {
63
-        $this->em->find(__NAMESPACE__ . '\DDC2084\MyEntity1', new DDC2084\MyEntity2('Foo'));
63
+        $this->em->find(__NAMESPACE__.'\DDC2084\MyEntity1', new DDC2084\MyEntity2('Foo'));
64 64
     }
65 65
 }
66 66
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/Ticket/GH6141Test.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
      */
81 81
     public function convertToDatabaseValue($value, AbstractPlatform $platform)
82 82
     {
83
-        if (! $value instanceof GH6141People) {
83
+        if ( ! $value instanceof GH6141People) {
84 84
             $value = GH6141People::get($value);
85 85
         }
86 86
 
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
      */
122 122
     public static function get($value)
123 123
     {
124
-        if (! self::isValid($value)) {
124
+        if ( ! self::isValid($value)) {
125 125
             throw new InvalidArgumentException();
126 126
         }
127 127
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/Ticket/DDC3123Test.php 2 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -33,8 +33,8 @@
 block discarded – undo
33 33
         $uow->scheduleExtraUpdate($user, ['name' => 'changed name']);
34 34
 
35 35
         $listener = $this->getMockBuilder(stdClass::class)
36
-                         ->setMethods([Events::postFlush])
37
-                         ->getMock();
36
+                            ->setMethods([Events::postFlush])
37
+                            ->getMock();
38 38
 
39 39
         $listener
40 40
             ->expects($this->once())
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@
 block discarded – undo
38 38
         $listener
39 39
             ->expects($this->once())
40 40
             ->method(Events::postFlush)
41
-            ->will($this->returnCallback(static function () use ($uow) {
41
+            ->will($this->returnCallback(static function() use ($uow) {
42 42
                 self::assertAttributeEmpty('extraUpdates', $uow, 'ExtraUpdates are reset before postFlush');
43 43
             }));
44 44
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/Ticket/DDC1655Test.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
                 ]
31 31
             );
32 32
         } catch (Exception $e) {
33
-            $this->fail($e->getMessage() . PHP_EOL . $e->getTraceAsString());
33
+            $this->fail($e->getMessage().PHP_EOL.$e->getTraceAsString());
34 34
         }
35 35
     }
36 36
 
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
         $conn = static::$sharedConn;
40 40
 
41 41
         // In case test is skipped, tearDown is called, but no setup may have run
42
-        if (! $conn) {
42
+        if ( ! $conn) {
43 43
             return;
44 44
         }
45 45
 
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
 
82 82
         $baz = $this->em->find(get_class($baz), $baz->id);
83 83
         foreach ($baz->foos as $foo) {
84
-            self::assertEquals(1, $foo->loaded, 'should have loaded callback counter incremented for ' . get_class($foo));
84
+            self::assertEquals(1, $foo->loaded, 'should have loaded callback counter incremented for '.get_class($foo));
85 85
         }
86 86
     }
87 87
 
@@ -105,7 +105,7 @@  discard block
 block discarded – undo
105 105
         self::assertEquals(1, $bar->loaded);
106 106
         self::assertEquals(1, $bar->subLoaded);
107 107
 
108
-        $dql = 'SELECT b FROM ' . __NAMESPACE__ . '\DDC1655Bar b WHERE b.id = ?1';
108
+        $dql = 'SELECT b FROM '.__NAMESPACE__.'\DDC1655Bar b WHERE b.id = ?1';
109 109
         $bar = $this->em->createQuery($dql)->setParameter(1, $bar->id)->getSingleResult();
110 110
 
111 111
         self::assertEquals(1, $bar->loaded);
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/SchemaTool/DBAL483Test.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@
 block discarded – undo
32 32
 
33 33
         $updateSql = $this->schemaTool->getUpdateSchemaSql([$class]);
34 34
 
35
-        $updateSql = array_filter($updateSql, static function ($sql) {
35
+        $updateSql = array_filter($updateSql, static function($sql) {
36 36
             return strpos($sql, 'DBAL483') !== false;
37 37
         });
38 38
 
Please login to merge, or discard this patch.