Merge pull request #21 from dsavina/feature/optional-index-name

Revert parameter `$indexName` for column method `unique`
thecodingmachine · Jan 6, 2021 · 1aacb60 · 1aacb60
2 parents 64d63f6 + e42a057
commit 1aacb60
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/composer.json b/composer.json
@@ -10,7 +10,7 @@
     ],
     "require": {
         "php": ">=7.1",
-        "thecodingmachine/dbal-fluid-schema-builder": "^1.6",
+        "thecodingmachine/dbal-fluid-schema-builder": "^1.6.1",
         "doctrine/dbal": "^2.5"
     },
     "require-dev": {

diff --git a/src/TdbmFluidColumnOptions.php b/src/TdbmFluidColumnOptions.php
@@ -51,9 +51,9 @@ public function null(): self
      *
      * @return self
      */
-    public function unique(?string $indexName = null): self
+    public function unique(): self
     {
-        $this->fluidColumnOptions->unique($indexName);
+        $this->fluidColumnOptions->unique();
         return $this;
     }
 

diff --git a/tests/TdbmFluidColumnOptionsTest.php b/tests/TdbmFluidColumnOptionsTest.php
@@ -26,8 +26,8 @@ public function testOptions()
         $columnOptions->notNull();
         $this->assertSame(true, $dbalColumn->getNotnull());
 
-        $columnOptions->unique('unique_foo');
-        $this->assertSame(true, $posts->getDbalTable()->getIndex('unique_foo')->isUnique());
+        $columnOptions->unique();
+        $this->assertSame(true, $dbalColumn->getCustomSchemaOption('unique'));
 
         $columnOptions->comment('foo');
         $this->assertSame('foo', $dbalColumn->getComment());