Failed Conditions
Pull Request — 2.6 (#7875)
by
unknown
07:53
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
 
@@ -139,7 +139,7 @@  discard block
 block discarded – undo
139 139
     {
140 140
         return [
141 141
             ['/^(?!pg_entity_to_r)/', null],
142
-            [null, function ($assetName): bool {
142
+            [null, function($assetName): bool {
143 143
                 return $assetName != 'pg_entity_to_remove';
144 144
             }]
145 145
         ];
@@ -150,11 +150,11 @@  discard block
 block discarded – undo
150 150
      */
151 151
     public function testUpdateSchemaSqlWithSchemaAssetFilter(?string $filterRegex, ?callable $filterCallback): void
152 152
     {
153
-        if ($filterRegex && !method_exists(Configuration::class, 'setFilterSchemaAssetsExpression')) {
153
+        if ($filterRegex && ! method_exists(Configuration::class, 'setFilterSchemaAssetsExpression')) {
154 154
             $this->markTestSkipped(sprintf("Test require %s::setFilterSchemaAssetsExpression method", Configuration::class));
155 155
         }
156 156
 
157
-        if ($filterCallback && !method_exists(Configuration::class, 'setSchemaAssetsFilter')) {
157
+        if ($filterCallback && ! method_exists(Configuration::class, 'setSchemaAssetsFilter')) {
158 158
             $this->markTestSkipped(sprintf("Test require %s::setSchemaAssetsFilter method", Configuration::class));
159 159
         }
160 160
 
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
@@ -12,7 +12,7 @@  discard block
 block discarded – undo
12 12
     protected function setUp() {
13 13
         parent::setUp();
14 14
         if ($this->_em->getConnection()->getDatabasePlatform()->getName() !== 'mysql') {
15
-            $this->markTestSkipped('The ' . __CLASS__ .' requires the use of mysql.');
15
+            $this->markTestSkipped('The '.__CLASS__.' requires the use of mysql.');
16 16
         }
17 17
     }
18 18
 
@@ -113,7 +113,7 @@  discard block
 block discarded – undo
113 113
     {
114 114
         return [
115 115
             ['/^(?!entity_to_r)/', null],
116
-            [null, function ($assetName): bool {
116
+            [null, function($assetName): bool {
117 117
                 return $assetName != 'entity_to_remove';
118 118
             }]
119 119
         ];
@@ -124,11 +124,11 @@  discard block
 block discarded – undo
124 124
      */
125 125
     public function testUpdateSchemaSqlWithSchemaAssetFilter(?string $filterRegex, ?callable $filterCallback): void
126 126
     {
127
-        if ($filterRegex && !method_exists(Configuration::class, 'setFilterSchemaAssetsExpression')) {
127
+        if ($filterRegex && ! method_exists(Configuration::class, 'setFilterSchemaAssetsExpression')) {
128 128
             $this->markTestSkipped(sprintf("Test require %s::setFilterSchemaAssetsExpression method", Configuration::class));
129 129
         }
130 130
 
131
-        if ($filterCallback && !method_exists(Configuration::class, 'setSchemaAssetsFilter')) {
131
+        if ($filterCallback && ! method_exists(Configuration::class, 'setSchemaAssetsFilter')) {
132 132
             $this->markTestSkipped(sprintf("Test require %s::setSchemaAssetsFilter method", Configuration::class));
133 133
         }
134 134
 
Please login to merge, or discard this patch.