Split add field and add event in form type for image and video elements

monsieurbiz · Apr 12, 2024 · 73a4066 · 73a4066
1 parent 1d8e66d
commit 73a4066
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 2 deletions.
diff --git a/src/Form/Type/UiElement/ImageType.php b/src/Form/Type/UiElement/ImageType.php
@@ -22,14 +22,19 @@
 use Symfony\Component\Form\FormEvent;
 use Symfony\Component\Form\FormEvents;
 use Symfony\Component\Validator\Constraints as Assert;
-use Webmozart\Assert\Assert as AssertAssert;
 
 class ImageType extends AbstractType
 {
     /**
      * @inheritdoc
      */
     public function buildForm(FormBuilderInterface $builder, array $options): void
+    {
+        $this->addFields($builder, $options);
+        $this->addEvents($builder, $options);
+    }
+
+    public function addFields(FormBuilderInterface $builder, array $options): void
     {
         $builder
             ->add('image', FileType::class, [
@@ -53,7 +58,7 @@ public function buildForm(FormBuilderInterface $builder, array $options): void
                     new Assert\AtLeastOneOf([
                         'includeInternalMessages' => false,
                         'message' => 'monsieurbiz_richeditor_plugin.not_valid_url',
-                        'constraints'=> [
+                        'constraints' => [
                             new Assert\Url(['protocols' => ['http', 'https'], 'relativeProtocol' => true]),
                             new Assert\Regex(['pattern' => '`^(#|/[^/])`']),
                         ],
@@ -62,7 +67,10 @@ public function buildForm(FormBuilderInterface $builder, array $options): void
             ])
             ->add('align', AlignmentType::class)
         ;
+    }
 
+    public function addEvents(FormBuilderInterface $builder, array $options): void
+    {
         $builder->addEventListener(FormEvents::PRE_SUBMIT, function (FormEvent $event): void {
             // Change image field constraints depending on submitted value
             $options = $event->getForm()->get('image')->getConfig()->getOptions();

diff --git a/src/Form/Type/UiElement/VideoType.php b/src/Form/Type/UiElement/VideoType.php
@@ -27,6 +27,12 @@ class VideoType extends AbstractType
      * @inheritdoc
      */
     public function buildForm(FormBuilderInterface $builder, array $options): void
+    {
+        $this->addFields($builder, $options);
+        $this->addEvents($builder, $options);
+    }
+
+    public function addFields(FormBuilderInterface $builder, array $options): void
     {
         $builder
             ->add('video', FileType::class, [
@@ -43,7 +49,10 @@ public function buildForm(FormBuilderInterface $builder, array $options): void
             ])
             ->add('align', AlignmentType::class)
         ;
+    }
 
+    public function addEvents(FormBuilderInterface $builder, array $options): void
+    {
         $builder->addEventListener(FormEvents::PRE_SUBMIT, function (FormEvent $event): void {
             // Change video field constraints depending on submitted value
             $options = $event->getForm()->get('video')->getConfig()->getOptions();