Add alter enum basic support

Author: pfilsx

.gitignore

@@ -1,4 +1,2 @@
 /vendor
-/composer.lock
-/tools
-.phpunit.result.cache
+/composer.lock

.php-cs-fixer.php

@@ -24,6 +24,7 @@
         'declare_strict_types' => true,
         'no_superfluous_phpdoc_tags' => false,
         'yoda_style' => ['equal' => false, 'identical'=>false, 'less_and_greater' => false],
+        'no_unused_imports' => true,
     ])
     ->setUsingCache(false)
     ->setFinder($finder)

src/DBAL/Platform/PostgreSQLPlatform.php

@@ -6,6 +6,7 @@
 
 use Doctrine\DBAL\Connection;
 use Doctrine\DBAL\Exception;
+use Doctrine\DBAL\Exception\InvalidArgumentException;
 use Doctrine\DBAL\Platforms\PostgreSQLPlatform as BasePlatform;
 use Pfilsx\PostgreSQLDoctrine\DBAL\Schema\EnumTypeAsset;
 use Pfilsx\PostgreSQLDoctrine\DBAL\Schema\PostgreSQLSchemaManager;
@@ -44,16 +45,45 @@ public function getCommentOnTypeSql(EnumTypeAsset $type): string
         return "COMMENT ON TYPE {$type->getQuotedName($this)} IS '{$type->getEnumClass()}'";
     }
 
-    public function getAlterTypeSql(EnumTypeAsset $from, EnumTypeAsset $to): string
+    /**
+     * @param EnumTypeAsset $from
+     * @param EnumTypeAsset $to
+     * @throws Exception
+     * @return string[]
+     */
+    public function getAlterTypeSql(EnumTypeAsset $from, EnumTypeAsset $to): array
     {
-        throw Exception::notSupported(__METHOD__);
+        $fromLabels = $from->getLabels();
+        $toLabels = $to->getLabels();
+
+        $result = [];
+        foreach (array_diff($toLabels, $fromLabels) as $label) {
+            $result[] = "ALTER TYPE {$to->getQuotedName($this)} ADD VALUE {$this->quoteEnumLabel($label)}";
+        }
+
+        if (count(array_diff($fromLabels, $toLabels)) > 0) {
+            throw new Exception('Enum labels reduction is not supported in automatic generation');
+        }
+
+        return $result;
     }
 
     public function getDropTypeSql(EnumTypeAsset $type): string
     {
         return "DROP TYPE {$type->getQuotedName($this)}";
     }
 
+    public function quoteEnumLabel(mixed $label): int|string
+    {
+        if (\is_string($label)) {
+            return $this->quoteStringLiteral($label);
+        } elseif (\is_int($label)) {
+            return $label;
+        } else {
+            throw new InvalidArgumentException('Invalid custom type labels specified. Only string and integers are supported');
+        }
+    }
+
     protected function initializeDoctrineTypeMappings()
     {
         parent::initializeDoctrineTypeMappings();

src/DBAL/Schema/EnumTypeAsset.php

@@ -8,6 +8,7 @@
 use Doctrine\DBAL\Platforms\AbstractPlatform;
 use Doctrine\DBAL\Schema\AbstractAsset;
 use Pfilsx\PostgreSQLDoctrine\DBAL\Contract\EnumInterface;
+use Pfilsx\PostgreSQLDoctrine\DBAL\Platform\PostgreSQLPlatform;
 use Pfilsx\PostgreSQLDoctrine\Tools\EnumTool;
 
 final class EnumTypeAsset extends AbstractAsset
@@ -69,15 +70,13 @@ public function getEnumClass(): string
      */
     public function getQuotedLabels(AbstractPlatform $platform): array
     {
+        if (!$platform instanceof PostgreSQLPlatform) {
+            return array_map(static fn ($label) => $platform->quoteStringLiteral((string) $label), $this->labels);
+        }
+
         $result = [];
         foreach ($this->labels as $label) {
-            if (\is_string($label)) {
-                $result[] = $platform->quoteStringLiteral($label);
-            } elseif (\is_int($label)) {
-                $result[] = $label;
-            } else {
-                throw new InvalidArgumentException('Invalid custom type labels specified. Only string and integers are supported');
-            }
+            $result[] = $platform->quoteEnumLabel($label);
         }
 
         return $result;

src/DBAL/Schema/SchemaDiff.php

@@ -70,7 +70,10 @@ public function toSql(AbstractPlatform $platform): array
         }
 
         foreach ($this->getAlteredTypes() as $alterTypeArray) {
-            $sql[] = $platform->getAlterTypeSql($alterTypeArray['from'], $alterTypeArray['to']);
+            $sql = array_merge(
+                $sql,
+                $platform->getAlterTypeSql($alterTypeArray['from'], $alterTypeArray['to'])
+            );
         }
 
         $sql = \array_merge($sql, $this->_toSql($platform, false));