Non-functioning translatable enums

Author: codedmonkey

src/Controller/Dashboard/DashboardRegistryController.php

@@ -67,8 +67,9 @@ public function configureFields(string $pageName): iterable
             ->setTemplatePath('dashboard/fields/registry_package_mirroring.html.twig')
             ->setRequired(true)
             ->setChoices(RegistryPackageMirroring::cases())
+            // This shouldn't be necessary because it's a translatable enum but it is
             ->setFormTypeOption('choice_label', function (RegistryPackageMirroring $choice): string {
-                return "registry.package_mirroring.{$choice->value}";
+                return "registry.package-mirroring.{$choice->value}";
             })
             ->renderExpanded();
 

src/Doctrine/Entity/PackageFetchStrategy.php

@@ -2,8 +2,16 @@
 
 namespace CodedMonkey\Dirigent\Doctrine\Entity;
 
-enum PackageFetchStrategy: string
+use Symfony\Contracts\Translation\TranslatableInterface;
+use Symfony\Contracts\Translation\TranslatorInterface;
+
+enum PackageFetchStrategy: string implements TranslatableInterface
 {
     case Mirror = 'mirror';
     case Vcs = 'vcs';
+
+    public function trans(TranslatorInterface $translator, ?string $locale = null): string
+    {
+        return sprintf('package.fetch-strategy.%s', $this->value);
+    }
 }

src/Doctrine/Entity/RegistryPackageMirroring.php

@@ -2,9 +2,17 @@
 
 namespace CodedMonkey\Dirigent\Doctrine\Entity;
 
-enum RegistryPackageMirroring: string
+use Symfony\Contracts\Translation\TranslatableInterface;
+use Symfony\Contracts\Translation\TranslatorInterface;
+
+enum RegistryPackageMirroring: string implements TranslatableInterface
 {
     case Disabled = 'none';
     case Manual = 'manual';
     case Automatic = 'auto';
+
+    public function trans(TranslatorInterface $translator, ?string $locale = null): string
+    {
+        return sprintf('registry.package-mirroring.%s', $this->value);
+    }
 }

src/Form/PackageFormType.php

@@ -73,8 +73,9 @@ public function onPostSetData(PostSetDataEvent $event): void
                     'class' => PackageFetchStrategy::class,
                     'expanded' => true,
                     'disabled' => !$package->getRepositoryUrl(),
+                    // This shouldn't be necessary because it's a translatable enum but it is
                     'choice_label' => function (PackageFetchStrategy $choice): string {
-                        return "package.fetch_strategy.{$choice->value}";
+                        return "package.fetch-strategy.{$choice->value}";
                     },
                 ]);
         }

templates/dashboard/fields/registry_package_mirroring.html.twig

@@ -1 +1,3 @@
-{{ "registry.package_mirroring.#{field.value.value}"|trans }}
+{# Since it is a translatable enum the following should work but it doesn't #}
+{# {{ field.value|trans }} #}
+{{ "registry.package-mirroring.#{field.value.value}"|trans }}

translations/messages.en.yaml

@@ -61,11 +61,11 @@ credentials:
     gitlab-dt: GitLab deploy token
     gitlab-pat: GitLab personal access token
 package:
-  fetch_strategy:
+  fetch-strategy:
     mirror: Fetch from mirror registry
     vcs: Fetch directly from VCS
 registry:
-  package_mirroring:
+  package-mirroring:
     none: Package mirroring disabled
     manual: Only mirror specified packages
     auto: Automatically mirror packages on request