Failed Conditions
Pull Request — master (#7242)
by Gabriel
08:46
created
tests/Doctrine/Tests/ORM/UnitOfWorkTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -506,7 +506,7 @@  discard block
 block discarded – undo
506 506
             'empty string, single field'     => [$emptyString, ''],
507 507
             'non-empty string, single field' => [$nonEmptyString, $nonEmptyString->id],
508 508
             'empty strings, two fields'      => [$emptyStrings, ' '],
509
-            'non-empty strings, two fields'  => [$nonEmptyStrings, $nonEmptyStrings->id1 . ' ' . $nonEmptyStrings->id2],
509
+            'non-empty strings, two fields'  => [$nonEmptyStrings, $nonEmptyStrings->id1.' '.$nonEmptyStrings->id2],
510 510
             'boolean true'                   => [$booleanTrue, '1'],
511 511
             'boolean false'                  => [$booleanFalse, ''],
512 512
         ];
@@ -777,7 +777,7 @@  discard block
 block discarded – undo
777 777
 
778 778
     protected function onPropertyChanged($propName, $oldValue, $newValue)
779 779
     {
780
-        if (! $this->listeners) {
780
+        if ( ! $this->listeners) {
781 781
             return;
782 782
         }
783 783
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Query/QueryExpressionVisitorTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@
 block discarded – undo
37 37
     public function testWalkComparison(CriteriaComparison $criteriaExpr, $queryExpr, ?Parameter $parameter = null) : void
38 38
     {
39 39
         self::assertEquals($queryExpr, $this->visitor->walkComparison($criteriaExpr));
40
-        if (! $parameter) {
40
+        if ( ! $parameter) {
41 41
             return;
42 42
         }
43 43
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/PostLoadEventTest.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -231,8 +231,8 @@  discard block
 block discarded – undo
231 231
         $eventManager->addEventListener([Events::postLoad], $listener);
232 232
 
233 233
         $this->em->find(CmsUser::class, $this->userId);
234
-        self::assertSame(1, $listener->countHandledEvents(CmsUser::class), CmsUser::class . ' should be handled once!');
235
-        self::assertSame(1, $listener->countHandledEvents(CmsEmail::class), CmsEmail::class . ' should be handled once!');
234
+        self::assertSame(1, $listener->countHandledEvents(CmsUser::class), CmsUser::class.' should be handled once!');
235
+        self::assertSame(1, $listener->countHandledEvents(CmsEmail::class), CmsEmail::class.' should be handled once!');
236 236
     }
237 237
 
238 238
     private function loadFixture()
@@ -290,7 +290,7 @@  discard block
 block discarded – undo
290 290
     public function postLoad(LifecycleEventArgs $event)
291 291
     {
292 292
         $object = $event->getObject();
293
-        if (! ($object instanceof CmsUser)) {
293
+        if ( ! ($object instanceof CmsUser)) {
294 294
             return;
295 295
         }
296 296
 
@@ -310,12 +310,12 @@  discard block
 block discarded – undo
310 310
     {
311 311
         $object = $event->getObject();
312 312
         $class  = StaticClassNameConverter::getClass($object);
313
-        if (! isset($this->firedByClasses[$class])) {
313
+        if ( ! isset($this->firedByClasses[$class])) {
314 314
             $this->firedByClasses[$class] = 1;
315 315
         } else {
316 316
             $this->firedByClasses[$class]++;
317 317
         }
318
-        if (! ($object instanceof CmsUser)) {
318
+        if ( ! ($object instanceof CmsUser)) {
319 319
             return;
320 320
         }
321 321
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/SchemaTool/MySqlSchemaToolTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@
 block discarded – undo
18 18
             return;
19 19
         }
20 20
 
21
-        $this->markTestSkipped('The ' . __CLASS__ . ' requires the use of mysql.');
21
+        $this->markTestSkipped('The '.__CLASS__.' requires the use of mysql.');
22 22
     }
23 23
 
24 24
     public function testGetCreateSchemaSql() : void
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/SchemaTool/PostgreSqlSchemaToolTest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@  discard block
 block discarded – undo
23 23
             return;
24 24
         }
25 25
 
26
-        $this->markTestSkipped('The ' . __CLASS__ . ' requires the use of postgresql.');
26
+        $this->markTestSkipped('The '.__CLASS__.' requires the use of postgresql.');
27 27
     }
28 28
 
29 29
     public function testPostgresMetadataSequenceIncrementedBy10() : void
@@ -135,7 +135,7 @@  discard block
 block discarded – undo
135 135
         $tool->createSchema($classes);
136 136
 
137 137
         $sql = $tool->getUpdateSchemaSql($classes);
138
-        $sql = array_filter($sql, function ($sql) {
138
+        $sql = array_filter($sql, function($sql) {
139 139
             return strpos($sql, 'DROP SEQUENCE stonewood.') === 0;
140 140
         });
141 141
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/Ticket/DDC3160Test.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@
 block discarded – undo
57 57
 
58 58
         foreach ($uow->getScheduledEntityInsertions() as $entity) {
59 59
             $this->inserts++;
60
-            if (! ($entity instanceof CmsUser)) {
60
+            if ( ! ($entity instanceof CmsUser)) {
61 61
                 continue;
62 62
             }
63 63
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/Ticket/DDC440Test.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -115,7 +115,7 @@
 block discarded – undo
115 115
     public function setClient(DDC440Client $value, $update_inverse = true)
116 116
     {
117 117
         $this->client = $value;
118
-        if (! $update_inverse) {
118
+        if ( ! $update_inverse) {
119 119
             return;
120 120
         }
121 121
 
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/Ticket/DDC2825Test.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -66,7 +66,7 @@
 block discarded – undo
66 66
 
67 67
         // Checks sequence name validity
68 68
         self::assertEquals(
69
-            str_replace('"', '', $fullTableName) . '_' . $property->getColumnName() . '_seq',
69
+            str_replace('"', '', $fullTableName).'_'.$property->getColumnName().'_seq',
70 70
             $idSequenceName
71 71
         );
72 72
     }
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/Ticket/DDC2996Test.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -74,7 +74,7 @@
 block discarded – undo
74 74
         $em  = $event->getEntityManager();
75 75
         $uow = $em->getUnitOfWork();
76 76
 
77
-        if (! $uow->getOriginalEntityData($this->user)) {
77
+        if ( ! $uow->getOriginalEntityData($this->user)) {
78 78
             return;
79 79
         }
80 80
 
Please login to merge, or discard this patch.