Passed
Pull Request — master (#7885)
by Šimon
05:58
created
tests/Doctrine/Tests/ORM/Functional/Ticket/DDC618Test.php 2 patches
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
     public function testIndexByJoin() : void
78 78
     {
79 79
         $dql    = 'SELECT A, B FROM Doctrine\Tests\ORM\Functional\Ticket\DDC618Author A ' .
80
-               'INNER JOIN A.books B INDEX BY B.title ORDER BY A.name ASC';
80
+                'INNER JOIN A.books B INDEX BY B.title ORDER BY A.name ASC';
81 81
         $result = $this->em->createQuery($dql)->getResult(Query::HYDRATE_OBJECT);
82 82
 
83 83
         self::assertCount(3, $result[0]->books); // Alice, Joe doesn't appear because he has no books.
@@ -101,14 +101,14 @@  discard block
 block discarded – undo
101 101
     public function testIndexByToOneJoinSilentlyIgnored() : void
102 102
     {
103 103
         $dql    = 'SELECT B, A FROM Doctrine\Tests\ORM\Functional\Ticket\DDC618Book B ' .
104
-               'INNER JOIN B.author A INDEX BY A.name ORDER BY A.name ASC';
104
+                'INNER JOIN B.author A INDEX BY A.name ORDER BY A.name ASC';
105 105
         $result = $this->em->createQuery($dql)->getResult(Query::HYDRATE_OBJECT);
106 106
 
107 107
         self::assertInstanceOf(DDC618Book::class, $result[0]);
108 108
         self::assertInstanceOf(DDC618Author::class, $result[0]->author);
109 109
 
110 110
         $dql    = 'SELECT B, A FROM Doctrine\Tests\ORM\Functional\Ticket\DDC618Book B ' .
111
-               'INNER JOIN B.author A INDEX BY A.name ORDER BY A.name ASC';
111
+                'INNER JOIN B.author A INDEX BY A.name ORDER BY A.name ASC';
112 112
         $result = $this->em->createQuery($dql)->getResult(Query::HYDRATE_ARRAY);
113 113
 
114 114
         self::assertEquals('Alice', $result[0]['author']['name']);
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
     public function testCombineIndexBy() : void
121 121
     {
122 122
         $dql    = 'SELECT A, B FROM Doctrine\Tests\ORM\Functional\Ticket\DDC618Author A INDEX BY A.id ' .
123
-               'INNER JOIN A.books B INDEX BY B.title ORDER BY A.name ASC';
123
+                'INNER JOIN A.books B INDEX BY B.title ORDER BY A.name ASC';
124 124
         $result = $this->em->createQuery($dql)->getResult(Query::HYDRATE_OBJECT);
125 125
 
126 126
         self::assertArrayHasKey(11, $result); // Alice
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
      */
77 77
     public function testIndexByJoin() : void
78 78
     {
79
-        $dql    = 'SELECT A, B FROM Doctrine\Tests\ORM\Functional\Ticket\DDC618Author A ' .
79
+        $dql    = 'SELECT A, B FROM Doctrine\Tests\ORM\Functional\Ticket\DDC618Author A '.
80 80
                'INNER JOIN A.books B INDEX BY B.title ORDER BY A.name ASC';
81 81
         $result = $this->em->createQuery($dql)->getResult(Query::HYDRATE_OBJECT);
82 82
 
@@ -100,14 +100,14 @@  discard block
 block discarded – undo
100 100
      */
101 101
     public function testIndexByToOneJoinSilentlyIgnored() : void
102 102
     {
103
-        $dql    = 'SELECT B, A FROM Doctrine\Tests\ORM\Functional\Ticket\DDC618Book B ' .
103
+        $dql    = 'SELECT B, A FROM Doctrine\Tests\ORM\Functional\Ticket\DDC618Book B '.
104 104
                'INNER JOIN B.author A INDEX BY A.name ORDER BY A.name ASC';
105 105
         $result = $this->em->createQuery($dql)->getResult(Query::HYDRATE_OBJECT);
106 106
 
107 107
         self::assertInstanceOf(DDC618Book::class, $result[0]);
108 108
         self::assertInstanceOf(DDC618Author::class, $result[0]->author);
109 109
 
110
-        $dql    = 'SELECT B, A FROM Doctrine\Tests\ORM\Functional\Ticket\DDC618Book B ' .
110
+        $dql    = 'SELECT B, A FROM Doctrine\Tests\ORM\Functional\Ticket\DDC618Book B '.
111 111
                'INNER JOIN B.author A INDEX BY A.name ORDER BY A.name ASC';
112 112
         $result = $this->em->createQuery($dql)->getResult(Query::HYDRATE_ARRAY);
113 113
 
@@ -119,7 +119,7 @@  discard block
 block discarded – undo
119 119
      */
120 120
     public function testCombineIndexBy() : void
121 121
     {
122
-        $dql    = 'SELECT A, B FROM Doctrine\Tests\ORM\Functional\Ticket\DDC618Author A INDEX BY A.id ' .
122
+        $dql    = 'SELECT A, B FROM Doctrine\Tests\ORM\Functional\Ticket\DDC618Author A INDEX BY A.id '.
123 123
                'INNER JOIN A.books B INDEX BY B.title ORDER BY A.name ASC';
124 124
         $result = $this->em->createQuery($dql)->getResult(Query::HYDRATE_OBJECT);
125 125
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/Ticket/DDC2692Test.php 1 patch
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -37,8 +37,8 @@
 block discarded – undo
37 37
     public function testIsListenerCalledOnlyOnceOnPreFlush() : void
38 38
     {
39 39
         $listener = $this->getMockBuilder(DDC2692Listener::class)
40
-                         ->setMethods(['preFlush'])
41
-                         ->getMock();
40
+                            ->setMethods(['preFlush'])
41
+                            ->getMock();
42 42
 
43 43
         $listener->expects($this->once())->method('preFlush');
44 44
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/QueryBuilderTest.php 2 patches
Indentation   +37 added lines, -37 removed lines patch added patch discarded remove patch
@@ -272,12 +272,12 @@  discard block
 block discarded – undo
272 272
     {
273 273
         $qb = $this->em->createQueryBuilder();
274 274
         $qb->select('u')
275
-           ->from(CmsUser::class, 'u')
276
-           ->where('u.id = :uid')
277
-           ->orWhere('u.id = :uid2')
278
-           ->andWhere('u.id = :uid3')
279
-           ->orWhere('u.name = :name1', 'u.name = :name2')
280
-           ->andWhere('u.name <> :noname');
275
+            ->from(CmsUser::class, 'u')
276
+            ->where('u.id = :uid')
277
+            ->orWhere('u.id = :uid2')
278
+            ->andWhere('u.id = :uid3')
279
+            ->orWhere('u.name = :name1', 'u.name = :name2')
280
+            ->andWhere('u.name <> :noname');
281 281
 
282 282
         self::assertValidQueryBuilder($qb, 'SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE (((u.id = :uid OR u.id = :uid2) AND u.id = :uid3) OR u.name = :name1 OR u.name = :name2) AND u.name <> :noname');
283 283
     }
@@ -286,9 +286,9 @@  discard block
 block discarded – undo
286 286
     {
287 287
         $qb = $this->em->createQueryBuilder();
288 288
         $qb->select('u')
289
-           ->from(CmsUser::class, 'u')
290
-           ->where('u.id = :uid')
291
-           ->andWhere($qb->expr()->in('u.id', [1, 2, 3]));
289
+            ->from(CmsUser::class, 'u')
290
+            ->where('u.id = :uid')
291
+            ->andWhere($qb->expr()->in('u.id', [1, 2, 3]));
292 292
 
293 293
         self::assertValidQueryBuilder($qb, 'SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE u.id = :uid AND u.id IN(1, 2, 3)');
294 294
     }
@@ -297,9 +297,9 @@  discard block
 block discarded – undo
297 297
     {
298 298
         $qb = $this->em->createQueryBuilder();
299 299
         $qb->select('u')
300
-           ->from(CmsUser::class, 'u')
301
-           ->where('u.id = :uid')
302
-           ->orWhere($qb->expr()->in('u.id', [1, 2, 3]));
300
+            ->from(CmsUser::class, 'u')
301
+            ->where('u.id = :uid')
302
+            ->orWhere($qb->expr()->in('u.id', [1, 2, 3]));
303 303
 
304 304
         self::assertValidQueryBuilder($qb, 'SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE u.id = :uid OR u.id IN(1, 2, 3)');
305 305
     }
@@ -308,9 +308,9 @@  discard block
 block discarded – undo
308 308
     {
309 309
         $qb = $this->em->createQueryBuilder();
310 310
         $qb->select('u')
311
-           ->from(CmsUser::class, 'u')
312
-           ->where('u.id = :uid')
313
-           ->andWhere($qb->expr()->notIn('u.id', [1, 2, 3]));
311
+            ->from(CmsUser::class, 'u')
312
+            ->where('u.id = :uid')
313
+            ->andWhere($qb->expr()->notIn('u.id', [1, 2, 3]));
314 314
 
315 315
         self::assertValidQueryBuilder($qb, 'SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE u.id = :uid AND u.id NOT IN(1, 2, 3)');
316 316
     }
@@ -319,9 +319,9 @@  discard block
 block discarded – undo
319 319
     {
320 320
         $qb = $this->em->createQueryBuilder();
321 321
         $qb->select('u')
322
-           ->from(CmsUser::class, 'u')
323
-           ->where('u.id = :uid')
324
-           ->orWhere($qb->expr()->notIn('u.id', [1, 2, 3]));
322
+            ->from(CmsUser::class, 'u')
323
+            ->where('u.id = :uid')
324
+            ->orWhere($qb->expr()->notIn('u.id', [1, 2, 3]));
325 325
 
326 326
         self::assertValidQueryBuilder($qb, 'SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE u.id = :uid OR u.id NOT IN(1, 2, 3)');
327 327
     }
@@ -617,8 +617,8 @@  discard block
 block discarded – undo
617 617
     {
618 618
         $qb = $this->em->createQueryBuilder();
619 619
         $qb->select('u')
620
-           ->from(CmsUser::class, 'u')
621
-           ->where($qb->expr()->orX('u.username = :username', 'u.username = :username2'));
620
+            ->from(CmsUser::class, 'u')
621
+            ->where($qb->expr()->orX('u.username = :username', 'u.username = :username2'));
622 622
 
623 623
         $parameters = new ArrayCollection();
624 624
         $parameters->add(new Parameter('username', 'jwage'));
@@ -634,8 +634,8 @@  discard block
 block discarded – undo
634 634
     {
635 635
         $qb = $this->em->createQueryBuilder();
636 636
         $qb->select('u')
637
-           ->from(CmsUser::class, 'u')
638
-           ->where('u.id = :id');
637
+            ->from(CmsUser::class, 'u')
638
+            ->where('u.id = :id');
639 639
 
640 640
         $parameters = new ArrayCollection();
641 641
         $parameters->add(new Parameter('id', 1));
@@ -684,8 +684,8 @@  discard block
 block discarded – undo
684 684
     {
685 685
         $qb = $this->em->createQueryBuilder();
686 686
         $qb->select('u')
687
-           ->from(CmsUser::class, 'u')
688
-           ->orWhere('u.id = :uid', $qb->expr()->eq('u.id', ':uid2'));
687
+            ->from(CmsUser::class, 'u')
688
+            ->orWhere('u.id = :uid', $qb->expr()->eq('u.id', ':uid2'));
689 689
 
690 690
         self::assertValidQueryBuilder($qb, 'SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE u.id = :uid OR u.id = :uid2');
691 691
     }
@@ -698,8 +698,8 @@  discard block
 block discarded – undo
698 698
         $orExpr->add($qb->expr()->in('u.id', [1]));
699 699
 
700 700
         $qb->select('u')
701
-           ->from(CmsUser::class, 'u')
702
-           ->where($orExpr);
701
+            ->from(CmsUser::class, 'u')
702
+            ->where($orExpr);
703 703
 
704 704
         self::assertValidQueryBuilder($qb, 'SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE u.id = :uid3 OR u.id IN(1)');
705 705
     }
@@ -708,8 +708,8 @@  discard block
 block discarded – undo
708 708
     {
709 709
         $qb = $this->em->createQueryBuilder();
710 710
         $qb->select('u')
711
-           ->from(CmsUser::class, 'u')
712
-           ->where($qb->expr()->in('u.name', ['one', 'two', 'three']));
711
+            ->from(CmsUser::class, 'u')
712
+            ->where($qb->expr()->in('u.name', ['one', 'two', 'three']));
713 713
 
714 714
         self::assertValidQueryBuilder($qb, "SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE u.name IN('one', 'two', 'three')");
715 715
 
@@ -723,8 +723,8 @@  discard block
 block discarded – undo
723 723
         $qb   = $this->em->createQueryBuilder();
724 724
         $expr = $this->em->getExpressionBuilder();
725 725
         $qb->select('u')
726
-           ->from(CmsUser::class, 'u')
727
-           ->where($expr->in('u.name', [$expr->literal('one'), $expr->literal('two'), $expr->literal('three')]));
726
+            ->from(CmsUser::class, 'u')
727
+            ->where($expr->in('u.name', [$expr->literal('one'), $expr->literal('two'), $expr->literal('three')]));
728 728
 
729 729
         self::assertValidQueryBuilder($qb, "SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE u.name IN('one', 'two', 'three')");
730 730
 
@@ -742,8 +742,8 @@  discard block
 block discarded – undo
742 742
 
743 743
         $qb = $this->em->createQueryBuilder();
744 744
         $qb->select('u')
745
-           ->from(CmsUser::class, 'u')
746
-           ->where($orExpr);
745
+            ->from(CmsUser::class, 'u')
746
+            ->where($orExpr);
747 747
 
748 748
         self::assertValidQueryBuilder($qb, 'SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE u.id = :uid3 OR NOT(u.id IN(1))');
749 749
     }
@@ -754,8 +754,8 @@  discard block
 block discarded – undo
754 754
         $expr = $this->em->getExpressionBuilder();
755 755
 
756 756
         $qb->select('u')
757
-           ->from(CmsUser::class, 'u')
758
-           ->where($expr->gt('u.id', $expr->all('select a.id from Doctrine\Tests\Models\CMS\CmsArticle a')));
757
+            ->from(CmsUser::class, 'u')
758
+            ->where($expr->gt('u.id', $expr->all('select a.id from Doctrine\Tests\Models\CMS\CmsArticle a')));
759 759
 
760 760
         self::assertValidQueryBuilder($qb, 'SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u WHERE u.id > ALL(select a.id from Doctrine\Tests\Models\CMS\CmsArticle a)');
761 761
     }
@@ -1120,9 +1120,9 @@  discard block
 block discarded – undo
1120 1120
     public function testRebuildsFromParts() : void
1121 1121
     {
1122 1122
         $qb = $this->em->createQueryBuilder()
1123
-          ->select('u')
1124
-          ->from(CmsUser::class, 'u')
1125
-          ->join('u.article', 'a');
1123
+            ->select('u')
1124
+            ->from(CmsUser::class, 'u')
1125
+            ->join('u.article', 'a');
1126 1126
 
1127 1127
         $dqlParts = $qb->getDQLParts();
1128 1128
         $dql      = $qb->getDQL();
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -1043,7 +1043,7 @@
 block discarded – undo
1043 1043
     {
1044 1044
         $qb = $this->em->createQueryBuilder()
1045 1045
             ->add('select', 'u')
1046
-            ->add('from', CmsUser::class . ' u');
1046
+            ->add('from', CmsUser::class.' u');
1047 1047
 
1048 1048
         self::assertEquals('SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u', $qb->getDQL());
1049 1049
     }
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/EntityManagerTest.php 2 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -92,7 +92,7 @@
 block discarded – undo
92 92
     public function testCreateQueryBuilderAliasValid() : void
93 93
     {
94 94
         $q  = $this->em->createQueryBuilder()
95
-             ->select('u')->from(CmsUser::class, 'u');
95
+                ->select('u')->from(CmsUser::class, 'u');
96 96
         $q2 = clone $q;
97 97
 
98 98
         self::assertEquals('SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u', $q->getQuery()->getDql());
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
     public function testThrowsExceptionOnNonObjectValues($methodName) : void
142 142
     {
143 143
         $this->expectException(ORMInvalidArgumentException::class);
144
-        $this->expectExceptionMessage('EntityManager#' . $methodName . '() expects parameter 1 to be an entity object, NULL given.');
144
+        $this->expectExceptionMessage('EntityManager#'.$methodName.'() expects parameter 1 to be an entity object, NULL given.');
145 145
 
146 146
         $this->em->{$methodName}(null);
147 147
     }
@@ -186,7 +186,7 @@  discard block
 block discarded – undo
186 186
     {
187 187
         self::assertSame(
188 188
             $value,
189
-            $this->em->transactional(static function ($em) use ($value) {
189
+            $this->em->transactional(static function($em) use ($value) {
190 190
                 return $value;
191 191
             })
192 192
         );
@@ -219,8 +219,8 @@  discard block
 block discarded – undo
219 219
     public function testTransactionalReThrowsThrowables() : void
220 220
     {
221 221
         try {
222
-            $this->em->transactional(static function () {
223
-                (static function (array $value) {
222
+            $this->em->transactional(static function() {
223
+                (static function(array $value) {
224 224
                     // this only serves as an IIFE that throws a `TypeError`
225 225
                 })(null);
226 226
             });
Please login to merge, or discard this patch.
Doctrine/Tests/ORM/Functional/OneToOneInverseSideLoadAfterDqlQueryTest.php 1 patch
Indentation   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -45,12 +45,12 @@
 block discarded – undo
45 45
 
46 46
         /** @var InverseSide $fetchedInverse */
47 47
         $fetchedInverse = $this->em->createQueryBuilder()
48
-                                   ->select('inverse')
49
-                                   ->from(InverseSide::class, 'inverse')
50
-                                   ->andWhere('inverse.id = :id')
51
-                                   ->setParameter('id', 'inverse')
52
-                                   ->getQuery()
53
-                                   ->getSingleResult();
48
+                                    ->select('inverse')
49
+                                    ->from(InverseSide::class, 'inverse')
50
+                                    ->andWhere('inverse.id = :id')
51
+                                    ->setParameter('id', 'inverse')
52
+                                    ->getQuery()
53
+                                    ->getSingleResult();
54 54
 
55 55
         self::assertInstanceOf(InverseSide::class, $fetchedInverse);
56 56
         self::assertInstanceOf(OwningSide::class, $fetchedInverse->owning);
Please login to merge, or discard this patch.
tests/Doctrine/Performance/ChangeSet/UnitOfWorkComputeChangesBench.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -29,8 +29,8 @@
 block discarded – undo
29 29
             $user           = new CmsUser();
30 30
             $user->id       = $i;
31 31
             $user->status   = 'user';
32
-            $user->username = 'user' . $i;
33
-            $user->name     = 'Mr.Smith-' . $i;
32
+            $user->username = 'user'.$i;
33
+            $user->name     = 'Mr.Smith-'.$i;
34 34
             $this->users[]  = $user;
35 35
 
36 36
             $this->unitOfWork->registerManaged(
Please login to merge, or discard this patch.
Hydration/MixedQueryFetchJoinFullObjectHydrationPerformanceBench.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@
 block discarded – undo
48 48
                 'u__status'      => 'developer',
49 49
                 'u__username'    => 'jwage',
50 50
                 'u__name'        => 'Jonathan',
51
-                'sclr0'          => 'JWAGE' . $i,
51
+                'sclr0'          => 'JWAGE'.$i,
52 52
                 'p__phonenumber' => '91',
53 53
                 'a__id'          => $i,
54 54
             ];
Please login to merge, or discard this patch.
tests/Doctrine/Performance/Hydration/SimpleInsertPerformanceBench.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -39,8 +39,8 @@  discard block
 block discarded – undo
39 39
         for ($i = 1; $i <= 10000; ++$i) {
40 40
             $user           = new CMS\CmsUser();
41 41
             $user->status   = 'user';
42
-            $user->username = 'user' . $i;
43
-            $user->name     = 'Mr.Smith-' . $i;
42
+            $user->username = 'user'.$i;
43
+            $user->name     = 'Mr.Smith-'.$i;
44 44
 
45 45
             $this->users[$i] = $user;
46 46
         }
@@ -53,12 +53,12 @@  discard block
 block discarded – undo
53 53
         // Yes, this is a lot of overhead, but I have no better solution other than
54 54
         // completely mocking out the DB, which would be silly (query impact is
55 55
         // necessarily part of our benchmarks)
56
-        $this->entityManager->getConnection()->executeQuery('DELETE FROM ' . $this->tableName)->execute();
56
+        $this->entityManager->getConnection()->executeQuery('DELETE FROM '.$this->tableName)->execute();
57 57
 
58 58
         foreach ($this->users as $key => $user) {
59 59
             $this->entityManager->persist($user);
60 60
 
61
-            if (! ($key % 20)) {
61
+            if ( ! ($key % 20)) {
62 62
                 $this->entityManager->flush();
63 63
                 $this->entityManager->clear();
64 64
             }
Please login to merge, or discard this patch.
Hydration/MixedQueryFetchJoinPartialObjectHydrationPerformanceBench.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@
 block discarded – undo
63 63
                 'u__status'      => 'developer',
64 64
                 'u__username'    => 'jwage',
65 65
                 'u__name'        => 'Jonathan',
66
-                'sclr0'          => 'JWAGE' . $i,
66
+                'sclr0'          => 'JWAGE'.$i,
67 67
                 'p__phonenumber' => '91',
68 68
             ];
69 69
         }
Please login to merge, or discard this patch.