Fixes the problem with variadic arguments in column type methods (#45)

Co-authored-by: Aleksei Gagarin <[email protected]>
# Conflicts:
#	composer.json

Author: butschster

Diff for: composer.json

@@ -5,7 +5,7 @@
   "license": "MIT",
   "require": {
     "php": ">=8.0",
-    "cycle/database": "^2.0",
+    "cycle/database": "^2.3",
     "spiral/core": "^2.7",
     "spiral/files": "^2.7",
     "spiral/tokenizer": "^2.7",

Diff for: src/Operation/Column/Column.php

@@ -43,21 +43,27 @@ protected function declareColumn(AbstractTable $schema): AbstractColumn
         //Type configuring
         if (method_exists($column, $this->type)) {
             $arguments = [];
+            $variadic = false;
 
             $method = new \ReflectionMethod($column, $this->type);
             foreach ($method->getParameters() as $parameter) {
                 if ($this->hasOption($parameter->getName())) {
-                    $arguments[] = $this->getOption($parameter->getName());
+                    $arguments[$parameter->getName()] = $this->getOption($parameter->getName());
                 } elseif (!$parameter->isOptional()) {
                     throw new ColumnException(
                         "Option '{$parameter->getName()}' are required to define column with type '{$this->type}'"
                     );
-                } else {
-                    $arguments[] = $parameter->getDefaultValue();
+                } elseif ($parameter->isDefaultValueAvailable()) {
+                    $arguments[$parameter->getName()] = $parameter->getDefaultValue();
+                } elseif ($parameter->isVariadic()) {
+                    $variadic = true;
                 }
             }
 
-            call_user_func_array([$column, $this->type], $arguments);
+            \call_user_func_array(
+                [$column, $this->type],
+                $variadic ? $arguments + $this->options + $column->getAttributes() : $arguments,
+            );
         } else {
             $column->type($this->type);
         }

Diff for: tests/Migrations/AtomizerTest.php

@@ -310,15 +310,18 @@ public function testCreateDatetimeNowColumn(): void
         $this->migrator->configure();
 
         $schema = $this->schema('sample');
-        $column = $schema->datetime('value');
+        $column = $schema->datetime('value', size: 2, foo: 'bar');
         $column->defaultValue(new Fragment($column::DATETIME_NOW));
 
         $this->atomize('migration1', [$schema]);
 
         $this->migrator->run();
 
         $this->assertTrue($this->db->hasTable('sample'));
-        $this->assertSame((string)$column->getDefaultValue(), (string)$this->schema('sample')->column('value')->getDefaultValue());
+        $this->assertSame(
+            (string)$column->getDefaultValue(),
+            (string)$this->schema('sample')->column('value')->getDefaultValue()
+        );
 
         $this->migrator->rollback();
         $this->assertFalse($this->db->hasTable('sample'));