Failed Conditions
Pull Request — master (#6762)
by Claudio
19:02
created
tests/Doctrine/Tests/ORM/Functional/PaginationTest.php 2 patches
Doc Comments   +24 added lines patch added patch discarded remove patch
@@ -120,6 +120,10 @@  discard block
 block discarded – undo
120 120
         self::assertCount(3, $paginator->getIterator());
121 121
     }
122 122
 
123
+    /**
124
+     * @param string $baseDql
125
+     * @param string $checkField
126
+     */
123 127
     private function iterateWithOrderAsc($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
124 128
     {
125 129
         // Ascending
@@ -134,6 +138,10 @@  discard block
 block discarded – undo
134 138
         self::assertEquals($checkField . "0", $result[0]->$checkField);
135 139
     }
136 140
 
141
+    /**
142
+     * @param string $baseDql
143
+     * @param string $checkField
144
+     */
137 145
     private function iterateWithOrderAscWithLimit($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
138 146
     {
139 147
         // Ascending
@@ -150,6 +158,10 @@  discard block
 block discarded – undo
150 158
         self::assertEquals($checkField . "0", $result[0]->$checkField);
151 159
     }
152 160
 
161
+    /**
162
+     * @param string $baseDql
163
+     * @param string $checkField
164
+     */
153 165
     private function iterateWithOrderAscWithLimitAndOffset($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
154 166
     {
155 167
         // Ascending
@@ -166,6 +178,10 @@  discard block
 block discarded – undo
166 178
         self::assertEquals($checkField . "3", $result[0]->$checkField);
167 179
     }
168 180
 
181
+    /**
182
+     * @param string $baseDql
183
+     * @param string $checkField
184
+     */
169 185
     private function iterateWithOrderDesc($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
170 186
     {
171 187
         $dql = "$baseDql DESC";
@@ -179,6 +195,10 @@  discard block
 block discarded – undo
179 195
         self::assertEquals($checkField . "8", $result[0]->$checkField);
180 196
     }
181 197
 
198
+    /**
199
+     * @param string $baseDql
200
+     * @param string $checkField
201
+     */
182 202
     private function iterateWithOrderDescWithLimit($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
183 203
     {
184 204
         $dql = "$baseDql DESC";
@@ -194,6 +214,10 @@  discard block
 block discarded – undo
194 214
         self::assertEquals($checkField . "8", $result[0]->$checkField);
195 215
     }
196 216
 
217
+    /**
218
+     * @param string $baseDql
219
+     * @param string $checkField
220
+     */
197 221
     private function iterateWithOrderDescWithLimitAndOffset($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
198 222
     {
199 223
         $dql = "$baseDql DESC";
Please login to merge, or discard this patch.
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
         $iter = $paginator->getIterator();
130 130
         self::assertCount(9, $iter);
131 131
         $result = iterator_to_array($iter);
132
-        self::assertEquals($checkField . "0", $result[0]->$checkField);
132
+        self::assertEquals($checkField."0", $result[0]->$checkField);
133 133
     }
134 134
 
135 135
     private function iterateWithOrderAscWithLimit($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
145 145
         $iter = $paginator->getIterator();
146 146
         self::assertCount(3, $iter);
147 147
         $result = iterator_to_array($iter);
148
-        self::assertEquals($checkField . "0", $result[0]->$checkField);
148
+        self::assertEquals($checkField."0", $result[0]->$checkField);
149 149
     }
150 150
 
151 151
     private function iterateWithOrderAscWithLimitAndOffset($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
@@ -161,7 +161,7 @@  discard block
 block discarded – undo
161 161
         $iter = $paginator->getIterator();
162 162
         self::assertCount(3, $iter);
163 163
         $result = iterator_to_array($iter);
164
-        self::assertEquals($checkField . "3", $result[0]->$checkField);
164
+        self::assertEquals($checkField."3", $result[0]->$checkField);
165 165
     }
166 166
 
167 167
     private function iterateWithOrderDesc($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
@@ -174,7 +174,7 @@  discard block
 block discarded – undo
174 174
         $iter = $paginator->getIterator();
175 175
         self::assertCount(9, $iter);
176 176
         $result = iterator_to_array($iter);
177
-        self::assertEquals($checkField . "8", $result[0]->$checkField);
177
+        self::assertEquals($checkField."8", $result[0]->$checkField);
178 178
     }
179 179
 
180 180
     private function iterateWithOrderDescWithLimit($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
@@ -189,7 +189,7 @@  discard block
 block discarded – undo
189 189
         $iter = $paginator->getIterator();
190 190
         self::assertCount(3, $iter);
191 191
         $result = iterator_to_array($iter);
192
-        self::assertEquals($checkField . "8", $result[0]->$checkField);
192
+        self::assertEquals($checkField."8", $result[0]->$checkField);
193 193
     }
194 194
 
195 195
     private function iterateWithOrderDescWithLimitAndOffset($useOutputWalkers, $fetchJoinCollection, $baseDql, $checkField)
@@ -204,7 +204,7 @@  discard block
 block discarded – undo
204 204
         $iter = $paginator->getIterator();
205 205
         self::assertCount(3, $iter);
206 206
         $result = iterator_to_array($iter);
207
-        self::assertEquals($checkField . "5", $result[0]->$checkField);
207
+        self::assertEquals($checkField."5", $result[0]->$checkField);
208 208
     }
209 209
 
210 210
     /**
@@ -477,8 +477,8 @@  discard block
 block discarded – undo
477 477
     public function testIterateWithFetchJoinOneToManyWithOrderByColumnFromBoth($useOutputWalkers)
478 478
     {
479 479
         $dql = 'SELECT c, d FROM Doctrine\Tests\Models\Pagination\Company c JOIN c.departments d ORDER BY c.name';
480
-        $dqlAsc = $dql . " ASC, d.name";
481
-        $dqlDesc = $dql . " DESC, d.name";
480
+        $dqlAsc = $dql." ASC, d.name";
481
+        $dqlDesc = $dql." DESC, d.name";
482 482
         $this->iterateWithOrderAsc($useOutputWalkers, true, $dqlAsc, "name");
483 483
         $this->iterateWithOrderDesc($useOutputWalkers, true, $dqlDesc, "name");
484 484
     }
@@ -486,8 +486,8 @@  discard block
 block discarded – undo
486 486
     public function testIterateWithFetchJoinOneToManyWithOrderByColumnFromBothWithLimitWithOutputWalker()
487 487
     {
488 488
         $dql = 'SELECT c, d FROM Doctrine\Tests\Models\Pagination\Company c JOIN c.departments d ORDER BY c.name';
489
-        $dqlAsc = $dql . " ASC, d.name";
490
-        $dqlDesc = $dql . " DESC, d.name";
489
+        $dqlAsc = $dql." ASC, d.name";
490
+        $dqlDesc = $dql." DESC, d.name";
491 491
         $this->iterateWithOrderAscWithLimit(true, true, $dqlAsc, "name");
492 492
         $this->iterateWithOrderDescWithLimit(true, true, $dqlDesc, "name");
493 493
     }
@@ -498,8 +498,8 @@  discard block
 block discarded – undo
498 498
         $this->expectExceptionMessage('Cannot select distinct identifiers from query with LIMIT and ORDER BY on a column from a fetch joined to-many association. Use output walkers.');
499 499
 
500 500
         $dql = 'SELECT c, d FROM Doctrine\Tests\Models\Pagination\Company c JOIN c.departments d ORDER BY c.name';
501
-        $dqlAsc = $dql . " ASC, d.name";
502
-        $dqlDesc = $dql . " DESC, d.name";
501
+        $dqlAsc = $dql." ASC, d.name";
502
+        $dqlDesc = $dql." DESC, d.name";
503 503
         $this->iterateWithOrderAscWithLimit(false, true, $dqlAsc, "name");
504 504
         $this->iterateWithOrderDescWithLimit(false, true, $dqlDesc, "name");
505 505
     }
@@ -583,7 +583,7 @@  discard block
 block discarded – undo
583 583
         $users = iterator_to_array($paginator->getIterator());
584 584
         self::assertCount(9, $users);
585 585
         foreach ($users as $i => $user) {
586
-            self::assertEquals("username" . (8 - $i), $user->username);
586
+            self::assertEquals("username".(8 - $i), $user->username);
587 587
         }
588 588
     }
589 589
 
@@ -733,7 +733,7 @@  discard block
 block discarded – undo
733 733
             $company->logo->image_width = 100 + $i;
734 734
             $company->logo->image_height = 100 + $i;
735 735
             $company->logo->company = $company;
736
-            for($j=0;$j<3;$j++) {
736
+            for ($j = 0; $j < 3; $j++) {
737 737
                 $department = new Department();
738 738
                 $department->name = "name$i$j";
739 739
                 $department->company = $company;
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/SQLFilterTest.php 2 patches
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -180,6 +180,9 @@  discard block
 block discarded – undo
180 180
         self::assertFalse($em->getFilters()->isEnabled("foo_filter"));
181 181
     }
182 182
 
183
+    /**
184
+     * @param EntityManager $em
185
+     */
183 186
     protected function configureFilters($em)
184 187
     {
185 188
         // Add filters to the configuration of the EM
@@ -894,6 +897,9 @@  discard block
 block discarded – undo
894 897
         self::assertEquals(2, count($manager->managedContracts->slice(0, 10)));
895 898
     }
896 899
 
900
+    /**
901
+     * @param string $name
902
+     */
897 903
     private function usePersonNameFilter($name)
898 904
     {
899 905
         // Enable the filter
Please login to merge, or discard this patch.
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -568,11 +568,11 @@  discard block
 block discarded – undo
568 568
         $user = $this->_em->find(CmsUser::class, $this->userId);
569 569
 
570 570
         self::assertFalse($user->articles->isInitialized());
571
-        self::assertEquals(2, count($user->articles->slice(0,10)));
571
+        self::assertEquals(2, count($user->articles->slice(0, 10)));
572 572
 
573 573
         $this->useCMSArticleTopicFilter();
574 574
 
575
-        self::assertEquals(1, count($user->articles->slice(0,10)));
575
+        self::assertEquals(1, count($user->articles->slice(0, 10)));
576 576
     }
577 577
 
578 578
     private function useCMSGroupPrefixFilter()
@@ -616,11 +616,11 @@  discard block
 block discarded – undo
616 616
         $user = $this->_em->find(CmsUser::class, $this->userId2);
617 617
 
618 618
         self::assertFalse($user->groups->isInitialized());
619
-        self::assertEquals(2, count($user->groups->slice(0,10)));
619
+        self::assertEquals(2, count($user->groups->slice(0, 10)));
620 620
 
621 621
         $this->useCMSGroupPrefixFilter();
622 622
 
623
-        self::assertEquals(1, count($user->groups->slice(0,10)));
623
+        self::assertEquals(1, count($user->groups->slice(0, 10)));
624 624
     }
625 625
 
626 626
     private function loadFixtureData()
@@ -1106,11 +1106,11 @@  discard block
 block discarded – undo
1106 1106
 {
1107 1107
     public function addFilterConstraint(ClassMetadata $targetEntity, $targetTableAlias)
1108 1108
     {
1109
-        if (!in_array("LocaleAware", $targetEntity->reflClass->getInterfaceNames())) {
1109
+        if ( ! in_array("LocaleAware", $targetEntity->reflClass->getInterfaceNames())) {
1110 1110
             return "";
1111 1111
         }
1112 1112
 
1113
-        return $targetTableAlias.'.locale = ' . $this->getParameter('locale'); // getParam uses connection to quote the value.
1113
+        return $targetTableAlias.'.locale = '.$this->getParameter('locale'); // getParam uses connection to quote the value.
1114 1114
     }
1115 1115
 }
1116 1116
 
@@ -1122,7 +1122,7 @@  discard block
 block discarded – undo
1122 1122
             return "";
1123 1123
         }
1124 1124
 
1125
-        return $targetTableAlias.'.country = ' . $this->getParameter('country'); // getParam uses connection to quote the value.
1125
+        return $targetTableAlias.'.country = '.$this->getParameter('country'); // getParam uses connection to quote the value.
1126 1126
     }
1127 1127
 }
1128 1128
 
@@ -1134,7 +1134,7 @@  discard block
 block discarded – undo
1134 1134
             return "";
1135 1135
         }
1136 1136
 
1137
-        return $targetTableAlias.'.name LIKE ' . $this->getParameter('prefix'); // getParam uses connection to quote the value.
1137
+        return $targetTableAlias.'.name LIKE '.$this->getParameter('prefix'); // getParam uses connection to quote the value.
1138 1138
     }
1139 1139
 }
1140 1140
 
@@ -1146,7 +1146,7 @@  discard block
 block discarded – undo
1146 1146
             return "";
1147 1147
         }
1148 1148
 
1149
-        return $targetTableAlias.'.topic = ' . $this->getParameter('topic'); // getParam uses connection to quote the value.
1149
+        return $targetTableAlias.'.topic = '.$this->getParameter('topic'); // getParam uses connection to quote the value.
1150 1150
     }
1151 1151
 }
1152 1152
 
@@ -1158,7 +1158,7 @@  discard block
 block discarded – undo
1158 1158
             return "";
1159 1159
         }
1160 1160
 
1161
-        return $targetTableAlias.'.name LIKE ' . $this->getParameter('name');
1161
+        return $targetTableAlias.'.name LIKE '.$this->getParameter('name');
1162 1162
     }
1163 1163
 }
1164 1164
 
@@ -1170,7 +1170,7 @@  discard block
 block discarded – undo
1170 1170
             return "";
1171 1171
         }
1172 1172
 
1173
-        return $targetTableAlias.'.completed = ' . $this->getParameter('completed');
1173
+        return $targetTableAlias.'.completed = '.$this->getParameter('completed');
1174 1174
     }
1175 1175
 }
1176 1176
 
@@ -1182,6 +1182,6 @@  discard block
 block discarded – undo
1182 1182
             return "";
1183 1183
         }
1184 1184
 
1185
-        return $targetTableAlias.'.id = ' . $this->getParameter('id');
1185
+        return $targetTableAlias.'.id = '.$this->getParameter('id');
1186 1186
     }
1187 1187
 }
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Performance/SecondLevelCacheTest.php 1 patch
Doc Comments   +12 added lines patch added patch discarded remove patch
@@ -127,6 +127,9 @@  discard block
 block discarded – undo
127 127
         self::assertEquals(503, $this->countQuery($em));
128 128
     }
129 129
 
130
+    /**
131
+     * @param string $label
132
+     */
130 133
     private function queryEntity(EntityManagerInterface $em, $label)
131 134
     {
132 135
         $times        = 100;
@@ -158,6 +161,9 @@  discard block
 block discarded – undo
158 161
         printf("\n%s\n", str_repeat('-', 50));
159 162
     }
160 163
 
164
+    /**
165
+     * @param string $label
166
+     */
161 167
     public function findEntityOneToMany(EntityManagerInterface $em, $label)
162 168
     {
163 169
         $times        = 50;
@@ -217,6 +223,9 @@  discard block
 block discarded – undo
217 223
         printf("\n%s\n", str_repeat('-', 50));
218 224
     }
219 225
 
226
+    /**
227
+     * @param string $label
228
+     */
220 229
     private function findEntity(EntityManagerInterface $em, $label)
221 230
     {
222 231
         $times        = 10;
@@ -252,6 +261,9 @@  discard block
 block discarded – undo
252 261
         printf("\n%s\n", str_repeat('-', 50));
253 262
     }
254 263
 
264
+    /**
265
+     * @param string $label
266
+     */
255 267
     private function findAllEntity(EntityManagerInterface $em, $label)
256 268
     {
257 269
         $times        = 100;
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Mapping/ClassMetadataFactoryTest.php 2 patches
Doc Comments   +7 added lines, -1 removed lines patch added patch discarded remove patch
@@ -227,6 +227,9 @@  discard block
 block discarded – undo
227 227
         self::assertEquals([], $metadata);
228 228
     }
229 229
 
230
+    /**
231
+     * @param \PHPUnit_Framework_MockObject_MockObject $conn
232
+     */
230 233
     protected function _createEntityManager($metadataDriver, $conn = null)
231 234
     {
232 235
         $driverMock = new DriverMock();
@@ -255,7 +258,6 @@  discard block
 block discarded – undo
255 258
     }
256 259
 
257 260
     /**
258
-     * @param string $class
259 261
      * @return ClassMetadata
260 262
      */
261 263
     protected function _createValidClassMetadata()
@@ -469,6 +471,10 @@  discard block
 block discarded – undo
469 471
         return $this->mockMetadata[$className];
470 472
     }
471 473
 
474
+    /**
475
+     * @param string $className
476
+     * @param ClassMetadata $metadata
477
+     */
472 478
     public function setMetadataForClass($className, $metadata)
473 479
     {
474 480
         $this->mockMetadata[$className] = $metadata;
Please login to merge, or discard this patch.
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -108,15 +108,15 @@  discard block
 block discarded – undo
108 108
     {
109 109
         require_once __DIR__."/../../Models/Global/GlobalNamespaceModel.php";
110 110
 
111
-        $metadataDriver = $this->createAnnotationDriver([__DIR__ . '/../../Models/Global/']);
111
+        $metadataDriver = $this->createAnnotationDriver([__DIR__.'/../../Models/Global/']);
112 112
 
113 113
         $entityManager = $this->_createEntityManager($metadataDriver);
114 114
 
115 115
         $mf = $entityManager->getMetadataFactory();
116 116
         $m1 = $mf->getMetadataFor(DoctrineGlobal_Article::class);
117 117
         $h1 = $mf->hasMetadataFor(DoctrineGlobal_Article::class);
118
-        $h2 = $mf->hasMetadataFor('\\' . DoctrineGlobal_Article::class);
119
-        $m2 = $mf->getMetadataFor('\\' . DoctrineGlobal_Article::class);
118
+        $h2 = $mf->hasMetadataFor('\\'.DoctrineGlobal_Article::class);
119
+        $m2 = $mf->getMetadataFor('\\'.DoctrineGlobal_Article::class);
120 120
 
121 121
         self::assertNotSame($m1, $m2);
122 122
         self::assertFalse($h2);
@@ -171,7 +171,7 @@  discard block
 block discarded – undo
171 171
     public function testAddDefaultDiscriminatorMap()
172 172
     {
173 173
         $cmf = new ClassMetadataFactory();
174
-        $driver = $this->createAnnotationDriver([__DIR__ . '/../../Models/JoinedInheritanceType/']);
174
+        $driver = $this->createAnnotationDriver([__DIR__.'/../../Models/JoinedInheritanceType/']);
175 175
         $em = $this->_createEntityManager($driver);
176 176
         $cmf->setEntityManager($em);
177 177
 
@@ -211,7 +211,7 @@  discard block
 block discarded – undo
211 211
     {
212 212
         // DDC-3551
213 213
         $conn = $this->createMock(Connection::class);
214
-        $mockDriver    = new MetadataDriverMock();
214
+        $mockDriver = new MetadataDriverMock();
215 215
         $em = $this->_createEntityManager($mockDriver, $conn);
216 216
 
217 217
         $conn->expects($this->any())
@@ -231,10 +231,10 @@  discard block
 block discarded – undo
231 231
     {
232 232
         $driverMock = new DriverMock();
233 233
         $config = new Configuration();
234
-        $config->setProxyDir(__DIR__ . '/../../Proxies');
234
+        $config->setProxyDir(__DIR__.'/../../Proxies');
235 235
         $config->setProxyNamespace('Doctrine\Tests\Proxies');
236 236
         $eventManager = new EventManager();
237
-        if (!$conn) {
237
+        if ( ! $conn) {
238 238
             $conn = new ConnectionMock([], $driverMock, $config, $eventManager);
239 239
         }
240 240
         $config->setMetadataDriverImpl($metadataDriver);
@@ -288,7 +288,7 @@  discard block
 block discarded – undo
288 288
     public function testQuoteMetadata()
289 289
     {
290 290
         $cmf    = new ClassMetadataFactory();
291
-        $driver = $this->createAnnotationDriver([__DIR__ . '/../../Models/Quote/']);
291
+        $driver = $this->createAnnotationDriver([__DIR__.'/../../Models/Quote/']);
292 292
         $em     = $this->_createEntityManager($driver);
293 293
         $cmf->setEntityManager($em);
294 294
 
@@ -383,7 +383,7 @@  discard block
 block discarded – undo
383 383
         $listener
384 384
             ->expects($this->any())
385 385
             ->method('onClassMetadataNotFound')
386
-            ->will($this->returnCallback(function (OnClassMetadataNotFoundEventArgs $args) use ($metadata, $em, $test) {
386
+            ->will($this->returnCallback(function(OnClassMetadataNotFoundEventArgs $args) use ($metadata, $em, $test) {
387 387
                 $test->assertNull($args->getFoundMetadata());
388 388
                 $test->assertSame('Foo', $args->getClassName());
389 389
                 $test->assertSame($em, $args->getObjectManager());
@@ -443,7 +443,7 @@  discard block
 block discarded – undo
443 443
     public function testInheritsIdGeneratorMappingFromEmbeddable()
444 444
     {
445 445
         $cmf = new ClassMetadataFactory();
446
-        $driver = $this->createAnnotationDriver([__DIR__ . '/../../Models/DDC4006/']);
446
+        $driver = $this->createAnnotationDriver([__DIR__.'/../../Models/DDC4006/']);
447 447
         $em = $this->_createEntityManager($driver);
448 448
         $cmf->setEntityManager($em);
449 449
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/EntityManagerTest.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -103,7 +103,7 @@
 block discarded – undo
103 103
     public function testCreateQueryBuilderAliasValid()
104 104
     {
105 105
         $q = $this->_em->createQueryBuilder()
106
-             ->select('u')->from(CmsUser::class, 'u');
106
+                ->select('u')->from(CmsUser::class, 'u');
107 107
         $q2 = clone $q;
108 108
 
109 109
         self::assertEquals('SELECT u FROM Doctrine\Tests\Models\CMS\CmsUser u', $q->getQuery()->getDql());
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Query/LanguageRecognitionTest.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
             $this->addToAssertionCount(1);
28 28
         } catch (QueryException $e) {
29 29
             if ($debug) {
30
-                echo $e->getTraceAsString() . PHP_EOL;
30
+                echo $e->getTraceAsString().PHP_EOL;
31 31
             }
32 32
 
33 33
             $this->fail($e->getMessage());
@@ -42,8 +42,8 @@  discard block
 block discarded – undo
42 42
             $this->fail('No syntax errors were detected, when syntax errors were expected');
43 43
         } catch (QueryException $e) {
44 44
             if ($debug) {
45
-                echo $e->getMessage() . PHP_EOL;
46
-                echo $e->getTraceAsString() . PHP_EOL;
45
+                echo $e->getMessage().PHP_EOL;
46
+                echo $e->getTraceAsString().PHP_EOL;
47 47
             }
48 48
             $this->addToAssertionCount(1);
49 49
         }
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
             ['SELECT foo: FROM Doctrine\Tests\Models\CMS\CmsUser foo:'],
113 113
 
114 114
             /* Checks for invalid AbstractSchemaName */
115
-            ['SELECT u FROM UnknownClass u'],  // unknown
115
+            ['SELECT u FROM UnknownClass u'], // unknown
116 116
             ['SELECT u FROM Unknown\Class u'], // unknown with namespace
117 117
             ['SELECT u FROM \Unknown\Class u'], // unknown, leading backslash
118 118
             ['SELECT u FROM Unknown\\\\Class u'], // unknown, syntactically bogus (duplicate \\)
@@ -585,7 +585,7 @@  discard block
 block discarded – undo
585 585
      */
586 586
     public function testDQLKeywordInJoinIsAllowed()
587 587
     {
588
-        self::assertValidDQL('SELECT u FROM ' . __NAMESPACE__ . '\DQLKeywordsModelUser u JOIN u.group g');
588
+        self::assertValidDQL('SELECT u FROM '.__NAMESPACE__.'\DQLKeywordsModelUser u JOIN u.group g');
589 589
     }
590 590
 
591 591
     /**
@@ -593,7 +593,7 @@  discard block
 block discarded – undo
593 593
      */
594 594
     public function testDQLKeywordInConditionIsAllowed()
595 595
     {
596
-        self::assertValidDQL('SELECT g FROM ' . __NAMESPACE__ . '\DQLKeywordsModelGroup g WHERE g.from=0');
596
+        self::assertValidDQL('SELECT g FROM '.__NAMESPACE__.'\DQLKeywordsModelGroup g WHERE g.from=0');
597 597
     }
598 598
 
599 599
     /* The exception is currently thrown in the SQLWalker, not earlier.
@@ -698,7 +698,7 @@  discard block
 block discarded – undo
698 698
      */
699 699
     public function testNewLiteralExpression()
700 700
     {
701
-        self::assertValidDQL("SELECT new " . __NAMESPACE__ . "\\DummyStruct(u.id, 'foo', 1, true) FROM Doctrine\Tests\Models\CMS\CmsUser u");
701
+        self::assertValidDQL("SELECT new ".__NAMESPACE__."\\DummyStruct(u.id, 'foo', 1, true) FROM Doctrine\Tests\Models\CMS\CmsUser u");
702 702
     }
703 703
 
704 704
     /**
@@ -706,7 +706,7 @@  discard block
 block discarded – undo
706 706
      */
707 707
     public function testNewLiteralWithSubselectExpression()
708 708
     {
709
-        self::assertValidDQL("SELECT new " . __NAMESPACE__ . "\\DummyStruct(u.id, 'foo', (SELECT 1 FROM Doctrine\Tests\Models\CMS\CmsUser su), true) FROM Doctrine\Tests\Models\CMS\CmsUser u");
709
+        self::assertValidDQL("SELECT new ".__NAMESPACE__."\\DummyStruct(u.id, 'foo', (SELECT 1 FROM Doctrine\Tests\Models\CMS\CmsUser su), true) FROM Doctrine\Tests\Models\CMS\CmsUser u");
710 710
     }
711 711
 }
712 712
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Query/CustomTreeWalkersJoinTest.php 2 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@
 block discarded – undo
28 28
         try {
29 29
             $query = $this->em->createQuery($dqlToBeTested);
30 30
             $query->setHint(Query::HINT_CUSTOM_TREE_WALKERS, [CustomTreeWalkerJoin::class])
31
-                  ->useQueryCache(false);
31
+                    ->useQueryCache(false);
32 32
 
33 33
             self::assertEquals($sqlToBeConfirmed, $query->getSql());
34 34
             $query->free();
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
             self::assertEquals($sqlToBeConfirmed, $query->getSql());
34 34
             $query->free();
35 35
         } catch (\Exception $e) {
36
-            $this->fail($e->getMessage() . ' at "' . $e->getFile() . '" on line ' . $e->getLine());
36
+            $this->fail($e->getMessage().' at "'.$e->getFile().'" on line '.$e->getLine());
37 37
 
38 38
         }
39 39
     }
@@ -74,9 +74,9 @@  discard block
 block discarded – undo
74 74
     {
75 75
         $rangeVariableDecl       = $identificationVariableDecl->rangeVariableDeclaration;
76 76
         $joinAssocPathExpression = new Query\AST\JoinAssociationPathExpression($rangeVariableDecl->aliasIdentificationVariable, 'address');
77
-        $joinAssocDeclaration    = new Query\AST\JoinAssociationDeclaration($joinAssocPathExpression, $rangeVariableDecl->aliasIdentificationVariable . 'a', null);
77
+        $joinAssocDeclaration    = new Query\AST\JoinAssociationDeclaration($joinAssocPathExpression, $rangeVariableDecl->aliasIdentificationVariable.'a', null);
78 78
         $join                    = new Query\AST\Join(Query\AST\Join::JOIN_TYPE_LEFT, $joinAssocDeclaration);
79
-        $selectExpression        = new Query\AST\SelectExpression($rangeVariableDecl->aliasIdentificationVariable . 'a', null, false);
79
+        $selectExpression        = new Query\AST\SelectExpression($rangeVariableDecl->aliasIdentificationVariable.'a', null, false);
80 80
 
81 81
         $identificationVariableDecl->joins[]                = $join;
82 82
         $selectStatement->selectClause->selectExpressions[] = $selectExpression;
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
         $userMetadata    = $entityManager->getClassMetadata(CmsUser::class);
86 86
         $addressMetadata = $entityManager->getClassMetadata(CmsAddress::class);
87 87
 
88
-        $this->setQueryComponent($rangeVariableDecl->aliasIdentificationVariable . 'a',
88
+        $this->setQueryComponent($rangeVariableDecl->aliasIdentificationVariable.'a',
89 89
             [
90 90
                 'metadata'     => $addressMetadata,
91 91
                 'parent'       => $rangeVariableDecl->aliasIdentificationVariable,
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Query/ExprTest.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -231,7 +231,7 @@
 block discarded – undo
231 231
      */
232 232
     public function testLiteralExprProperlyQuotesStrings()
233 233
     {
234
-       self::assertEquals("'00010001'", (string) $this->_expr->literal('00010001'));
234
+        self::assertEquals("'00010001'", (string) $this->_expr->literal('00010001'));
235 235
     }
236 236
 
237 237
     public function testGreaterThanOrEqualToExpr()
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Query/ParserTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -29,9 +29,9 @@
 block discarded – undo
29 29
      */
30 30
     public function testAbstractSchemaNameSupportsClassnamesWithLeadingBackslash()
31 31
     {
32
-        $parser = $this->createParser('\\' . CmsUser::class);
32
+        $parser = $this->createParser('\\'.CmsUser::class);
33 33
 
34
-        self::assertEquals('\\' . CmsUser::class, $parser->AbstractSchemaName());
34
+        self::assertEquals('\\'.CmsUser::class, $parser->AbstractSchemaName());
35 35
     }
36 36
 
37 37
     /**
Please login to merge, or discard this patch.