Passed
Pull Request — 2.6 (#7875)
by
unknown
10:22
created
lib/Doctrine/ORM/Tools/SchemaTool.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -228,7 +228,7 @@  discard block
 block discarded – undo
228 228
                             $targetEntity = current(
229 229
                                 array_filter(
230 230
                                     $classes,
231
-                                    function (ClassMetadata $class) use ($idMapping) : bool {
231
+                                    function(ClassMetadata $class) use ($idMapping) : bool {
232 232
                                         return $class->name === $idMapping['targetEntity'];
233 233
                                     }
234 234
                                 )
@@ -648,8 +648,8 @@  discard block
 block discarded – undo
648 648
 
649 649
             if ( ! $definingClass) {
650 650
                 throw new \Doctrine\ORM\ORMException(
651
-                    'Column name `' . $joinColumn['referencedColumnName'] . '` referenced for relation from '
652
-                    . $mapping['sourceEntity'] . ' towards ' . $mapping['targetEntity'] . ' does not exist.'
651
+                    'Column name `'.$joinColumn['referencedColumnName'].'` referenced for relation from '
652
+                    . $mapping['sourceEntity'].' towards '.$mapping['targetEntity'].' does not exist.'
653 653
                 );
654 654
             }
655 655
 
@@ -713,7 +713,7 @@  discard block
 block discarded – undo
713 713
 
714 714
         $compositeName = $theJoinTable->getName().'.'.implode('', $localColumns);
715 715
 
716
-        if (! $this->platform->supportsForeignKeyConstraints()) {
716
+        if ( ! $this->platform->supportsForeignKeyConstraints()) {
717 717
             return;
718 718
         }
719 719
 
@@ -749,7 +749,7 @@  discard block
 block discarded – undo
749 749
      */
750 750
     private function gatherColumnOptions(array $mapping) : array
751 751
     {
752
-        if (! isset($mapping['options'])) {
752
+        if ( ! isset($mapping['options'])) {
753 753
             return [];
754 754
         }
755 755
 
@@ -855,7 +855,7 @@  discard block
 block discarded – undo
855 855
                 if ($table->hasPrimaryKey()) {
856 856
                     $columns = $table->getPrimaryKey()->getColumns();
857 857
                     if (count($columns) == 1) {
858
-                        $checkSequence = $table->getName() . '_' . $columns[0] . '_seq';
858
+                        $checkSequence = $table->getName().'_'.$columns[0].'_seq';
859 859
                         if ($fullSchema->hasSequence($checkSequence)) {
860 860
                             $visitor->acceptSequence($fullSchema->getSequence($checkSequence));
861 861
                         }
@@ -921,7 +921,7 @@  discard block
 block discarded – undo
921 921
 
922 922
             if ($filter !== null) {
923 923
                 // whitelist assets we already know about in $toSchema, use the existing filter otherwise
924
-                $config->setSchemaAssetsFilter(static function ($asset) use ($filter, $toSchema) : bool {
924
+                $config->setSchemaAssetsFilter(static function($asset) use ($filter, $toSchema) : bool {
925 925
                     $assetName = $asset instanceof AbstractAsset ? $asset->getName() : $asset;
926 926
 
927 927
                     return $toSchema->hasTable($assetName) || $toSchema->hasSequence($assetName) || $filter($asset);
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/SchemaTool/SqliteSchemaToolTest.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
         parent::setUp();
21 21
 
22 22
         if ($this->_em->getConnection()->getDatabasePlatform()->getName() !== 'sqlite') {
23
-            $this->markTestSkipped('The ' . self::class . ' requires the use of sqlite.');
23
+            $this->markTestSkipped('The '.self::class.' requires the use of sqlite.');
24 24
         }
25 25
     }
26 26
 
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
 
35 35
     private function filterCreateTable(array $sqls, string $tableName) : array
36 36
     {
37
-        return array_filter($sqls, static function (string $sql) use ($tableName) : bool {
37
+        return array_filter($sqls, static function(string $sql) use ($tableName) : bool {
38 38
             return strpos($sql, sprintf('CREATE TABLE %s (', $tableName)) === 0;
39 39
         });
40 40
     }
@@ -51,7 +51,7 @@  discard block
 block discarded – undo
51 51
 
52 52
         $this->_em->getConnection()->exec(current($sqls));
53 53
         $sqls = $tool->getUpdateSchemaSql($classes);
54
-        $sqls = array_filter($sqls, static function (string $sql) : bool {
54
+        $sqls = array_filter($sqls, static function(string $sql) : bool {
55 55
             return (bool) strpos($sql, 'my_entity');
56 56
         });
57 57
         $this->assertCount(0, $sqls);
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
68 68
             ['/^(?!my_enti)/', null],
69 69
             [
70 70
                 null,
71
-                static function ($assetName) : bool {
71
+                static function($assetName) : bool {
72 72
                     return $assetName !== 'my_entity';
73 73
                 },
74 74
             ],
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
      */
81 81
     public function testUpdateSchemaSqlWithSchemaAssetFilter(?string $filterRegex, ?callable $filterCallback)
82 82
     {
83
-        if (! method_exists(Configuration::class, 'setSchemaAssetsFilter')) {
83
+        if ( ! method_exists(Configuration::class, 'setSchemaAssetsFilter')) {
84 84
             $this->markTestSkipped(sprintf('Test require %s::setSchemaAssetsFilter method', Configuration::class));
85 85
         }
86 86
 
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
         }
102 102
 
103 103
         $sqls = $tool->getUpdateSchemaSql($classes);
104
-        $sqls = array_filter($sqls, static function (string $sql) : bool {
104
+        $sqls = array_filter($sqls, static function(string $sql) : bool {
105 105
             return (bool) strpos($sql, 'my_entity');
106 106
         });
107 107
         $this->assertCount(0, $sqls);
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/SchemaTool/MySqlSchemaToolTest.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
         parent::setUp();
20 20
 
21 21
         if ($this->_em->getConnection()->getDatabasePlatform()->getName() !== 'mysql') {
22
-            $this->markTestSkipped('The ' . __CLASS__ .' requires the use of mysql.');
22
+            $this->markTestSkipped('The '.__CLASS__.' requires the use of mysql.');
23 23
         }
24 24
     }
25 25
 
@@ -134,7 +134,7 @@  discard block
 block discarded – undo
134 134
             ['/^(?!entity_to_r)/', null],
135 135
             [
136 136
                 null,
137
-                static function ($assetName) : bool {
137
+                static function($assetName) : bool {
138 138
                     return $assetName !== 'entity_to_remove';
139 139
                 },
140 140
             ],
@@ -146,7 +146,7 @@  discard block
 block discarded – undo
146 146
      */
147 147
     public function testUpdateSchemaSqlWithSchemaAssetFilter(?string $filterRegex, ?callable $filterCallback)
148 148
     {
149
-        if (! method_exists(Configuration::class, 'setSchemaAssetsFilter')) {
149
+        if ( ! method_exists(Configuration::class, 'setSchemaAssetsFilter')) {
150 150
             $this->markTestSkipped(sprintf('Test require %s::setSchemaAssetsFilter method', Configuration::class));
151 151
         }
152 152
 
@@ -179,7 +179,7 @@  discard block
 block discarded – undo
179 179
 
180 180
     private function filterSqls(array $sqls, array $needles) : array
181 181
     {
182
-        return array_filter($sqls, static function ($sql) use ($needles) {
182
+        return array_filter($sqls, static function($sql) use ($needles) {
183 183
             foreach ($needles as $needle) {
184 184
                 if (strpos($sql, $needle) !== false) {
185 185
                     return true;
Please login to merge, or discard this patch.
tests/Doctrine/Tests/ORM/Functional/SchemaTool/PostgreSqlSchemaToolTest.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -18,7 +18,7 @@  discard block
 block discarded – undo
18 18
         parent::setUp();
19 19
 
20 20
         if ($this->_em->getConnection()->getDatabasePlatform()->getName() !== 'postgresql') {
21
-            $this->markTestSkipped('The ' . __CLASS__ .' requires the use of postgresql.');
21
+            $this->markTestSkipped('The '.__CLASS__.' requires the use of postgresql.');
22 22
         }
23 23
     }
24 24
 
@@ -153,7 +153,7 @@  discard block
 block discarded – undo
153 153
             ['/^(?!pg_entity_to_r)/', null],
154 154
             [
155 155
                 null,
156
-                static function ($assetName) : bool {
156
+                static function($assetName) : bool {
157 157
                     return $assetName !== 'pg_entity_to_remove';
158 158
                 },
159 159
             ],
@@ -165,7 +165,7 @@  discard block
 block discarded – undo
165 165
      */
166 166
     public function testUpdateSchemaSqlWithSchemaAssetFilter(?string $filterRegex, ?callable $filterCallback)
167 167
     {
168
-        if (! method_exists(Configuration::class, 'setSchemaAssetsFilter')) {
168
+        if ( ! method_exists(Configuration::class, 'setSchemaAssetsFilter')) {
169 169
             $this->markTestSkipped(sprintf('Test require %s::setSchemaAssetsFilter method', Configuration::class));
170 170
         }
171 171
 
@@ -186,7 +186,7 @@  discard block
 block discarded – undo
186 186
         }
187 187
 
188 188
         $sqls = $tool->getUpdateSchemaSql($classes);
189
-        $sqls = array_filter($sqls, static function ($sql) {
189
+        $sqls = array_filter($sqls, static function($sql) {
190 190
             return strpos($sql, 'pg_entity_to_remove') !== false;
191 191
         });
192 192
         $this->assertCount(0, $sqls);
Please login to merge, or discard this patch.