Failed Conditions
Pull Request — 2.6 (#7875)
by
unknown
07:16
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/PostgreSqlSchemaToolTest.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@  discard block
 block discarded – undo
14 14
         parent::setUp();
15 15
 
16 16
         if ($this->_em->getConnection()->getDatabasePlatform()->getName() !== 'postgresql') {
17
-            $this->markTestSkipped('The ' . __CLASS__ .' requires the use of postgresql.');
17
+            $this->markTestSkipped('The '.__CLASS__.' requires the use of postgresql.');
18 18
         }
19 19
     }
20 20
 
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
     {
148 148
         return [
149 149
             ['/^(?!pg_entity_to_r)/', null],
150
-            [null, function ($assetName) : bool {
150
+            [null, function($assetName) : bool {
151 151
                 return $assetName !== 'pg_entity_to_remove';
152 152
             }]
153 153
         ];
@@ -158,11 +158,11 @@  discard block
 block discarded – undo
158 158
      */
159 159
     public function testUpdateSchemaSqlWithSchemaAssetFilter(?string $filterRegex, ?callable $filterCallback)
160 160
     {
161
-        if ($filterRegex && !method_exists(Configuration::class, 'setFilterSchemaAssetsExpression')) {
161
+        if ($filterRegex && ! method_exists(Configuration::class, 'setFilterSchemaAssetsExpression')) {
162 162
             $this->markTestSkipped(sprintf("Test require %s::setFilterSchemaAssetsExpression method", Configuration::class));
163 163
         }
164 164
 
165
-        if ($filterCallback && !method_exists(Configuration::class, 'setSchemaAssetsFilter')) {
165
+        if ($filterCallback && ! method_exists(Configuration::class, 'setSchemaAssetsFilter')) {
166 166
             $this->markTestSkipped(sprintf("Test require %s::setSchemaAssetsFilter method", Configuration::class));
167 167
         }
168 168
 
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
@@ -14,7 +14,7 @@  discard block
 block discarded – undo
14 14
         parent::setUp();
15 15
 
16 16
         if ($this->_em->getConnection()->getDatabasePlatform()->getName() !== 'mysql') {
17
-            $this->markTestSkipped('The ' . __CLASS__ .' requires the use of mysql.');
17
+            $this->markTestSkipped('The '.__CLASS__.' requires the use of mysql.');
18 18
         }
19 19
     }
20 20
 
@@ -122,7 +122,7 @@  discard block
 block discarded – undo
122 122
     {
123 123
         return [
124 124
             ['/^(?!entity_to_r)/', null],
125
-            [null, function ($assetName) : bool {
125
+            [null, function($assetName) : bool {
126 126
                 return $assetName !== 'entity_to_remove';
127 127
             }]
128 128
         ];
@@ -133,11 +133,11 @@  discard block
 block discarded – undo
133 133
      */
134 134
     public function testUpdateSchemaSqlWithSchemaAssetFilter(?string $filterRegex, ?callable $filterCallback)
135 135
     {
136
-        if ($filterRegex && !method_exists(Configuration::class, 'setFilterSchemaAssetsExpression')) {
136
+        if ($filterRegex && ! method_exists(Configuration::class, 'setFilterSchemaAssetsExpression')) {
137 137
             $this->markTestSkipped(sprintf("Test require %s::setFilterSchemaAssetsExpression method", Configuration::class));
138 138
         }
139 139
 
140
-        if ($filterCallback && !method_exists(Configuration::class, 'setSchemaAssetsFilter')) {
140
+        if ($filterCallback && ! method_exists(Configuration::class, 'setSchemaAssetsFilter')) {
141 141
             $this->markTestSkipped(sprintf("Test require %s::setSchemaAssetsFilter method", Configuration::class));
142 142
         }
143 143
 
Please login to merge, or discard this patch.