Merge pull request #3519 from doctrine/fix-unique-constraint-with-empty-name

Fix UniqueConstraint with empty name.

Author: jwage

lib/Doctrine/DBAL/Schema/Table.php

@@ -863,7 +863,7 @@ protected function _addUniqueConstraint(UniqueConstraint $constraint)
         $name = strlen($constraint->getName())
             ? $constraint->getName()
             : $this->_generateIdentifierName(
-                array_merge((array) $this->getName(), $constraint->getLocalColumns()),
+                array_merge((array) $this->getName(), $constraint->getColumns()),
                 'fk',
                 $this->_getMaxIdentifierLength()
             );

phpstan.neon.dist

@@ -56,9 +56,6 @@ parameters:
         # weird class name, represented in stubs as OCI_(Lob|Collection)
         - '~unknown class OCI-(Lob|Collection)~'
 
-        # https://github.com/doctrine/dbal/issues/3236
-        - '~^Call to an undefined method Doctrine\\DBAL\\Schema\\UniqueConstraint::getLocalColumns\(\)~'
-
         # https://github.com/doctrine/dbal/issues/3237
         - '~^Call to an undefined method Doctrine\\DBAL\\Driver\\PDOStatement::nextRowset\(\)~'
 

tests/Doctrine/Tests/DBAL/Schema/TableTest.php

@@ -12,9 +12,10 @@
 use Doctrine\DBAL\Schema\Index;
 use Doctrine\DBAL\Schema\SchemaException;
 use Doctrine\DBAL\Schema\Table;
+use Doctrine\DBAL\Schema\UniqueConstraint;
 use Doctrine\DBAL\Types\Type;
 use Doctrine\Tests\DbalTestCase;
-use function array_shift;
+use function array_keys;
 use function current;
 
 class TableTest extends DbalTestCase
@@ -206,7 +207,11 @@ public function testConstraints() : void
         $constraints = $tableA->getForeignKeys();
 
         self::assertCount(1, $constraints);
-        self::assertSame($constraint, array_shift($constraints));
+
+        $constraintNames = array_keys($constraints);
+
+        self::assertSame('fk_8c736521', $constraintNames[0]);
+        self::assertSame($constraint, $constraints['fk_8c736521']);
     }
 
     public function testOptions() : void
@@ -887,4 +892,33 @@ public static function getNormalizesAssetNames() : iterable
             ['"FOO"'],
         ];
     }
+
+    public function testUniqueConstraintWithEmptyName() : void
+    {
+        $columns = [
+            new Column('column1', Type::getType(Type::STRING)),
+            new Column('column2', Type::getType(Type::STRING)),
+            new Column('column3', Type::getType(Type::STRING)),
+            new Column('column4', Type::getType(Type::STRING)),
+        ];
+
+        $uniqueConstraints = [
+            new UniqueConstraint('', ['column1', 'column2']),
+            new UniqueConstraint('', ['column3', 'column4']),
+        ];
+
+        $table = new Table('test', $columns, [], $uniqueConstraints);
+
+        $constraints = $table->getUniqueConstraints();
+
+        self::assertCount(2, $constraints);
+
+        $constraintNames = array_keys($constraints);
+
+        self::assertSame('fk_d87f7e0c341ce00bad15b1b1', $constraintNames[0]);
+        self::assertSame('fk_d87f7e0cda12812744761484', $constraintNames[1]);
+
+        self::assertSame($uniqueConstraints[0], $constraints['fk_d87f7e0c341ce00bad15b1b1']);
+        self::assertSame($uniqueConstraints[1], $constraints['fk_d87f7e0cda12812744761484']);
+    }
 }