fix: Name validation error while trying to save an edited command (ba…

…ckport for 5.x) (#105) Co-authored-by: Christopher Georg <[email protected]>
Dukecity · Feb 2, 2024 · 705bbf3 · 705bbf3
1 parent 227246e
commit 705bbf3
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 8 deletions.
diff --git a/Controller/DetailController.php b/Controller/DetailController.php
@@ -19,12 +19,16 @@ class DetailController extends AbstractBaseController
      */
     public function edit(Request $request, $id = null): Response
     {
+        $validationGroups = [];
         $scheduledCommand = $id ? $this->getDoctrineManager()->getRepository(ScheduledCommand::class)->find($id) : null;
         if (!$scheduledCommand) {
             $scheduledCommand = new ScheduledCommand();
+            $validationGroups[] = 'new';
         }
 
-        $form = $this->createForm(ScheduledCommandType::class, $scheduledCommand);
+        $form = $this->createForm(ScheduledCommandType::class, $scheduledCommand, [
+            'validation_groups' => $validationGroups
+        ]);
         $form->handleRequest($request);
 
         if ($form->isSubmitted() && $form->isValid()) {

diff --git a/Entity/ScheduledCommand.php b/Entity/ScheduledCommand.php
@@ -18,7 +18,7 @@
  */
 #[ORM\Entity(repositoryClass: ScheduledCommandRepository::class)]
 #[ORM\Table(name: "scheduled_command")]
-#[UniqueEntity(fields: ["name"])]
+#[UniqueEntity(fields: ["name"], groups: ['new'])]
 class ScheduledCommand
 {
     #[ORM\Id, ORM\Column(type: Types::INTEGER), ORM\GeneratedValue(strategy: 'AUTO')]

diff --git a/Form/Type/ScheduledCommandType.php b/Form/Type/ScheduledCommandType.php
@@ -9,6 +9,8 @@
 use Symfony\Component\Form\Extension\Core\Type\SubmitType;
 use Symfony\Component\Form\Extension\Core\Type\TextType;
 use Symfony\Component\Form\FormBuilderInterface;
+use Symfony\Component\Form\FormEvent;
+use Symfony\Component\Form\FormEvents;
 use Symfony\Component\OptionsResolver\OptionsResolver;
 
 /**
@@ -20,14 +22,24 @@ class ScheduledCommandType extends AbstractType
 {
     public function buildForm(FormBuilderInterface $builder, array $options): void
     {
-        $builder->add(
-            'name',
-            TextType::class,
-            [
+        $builder->addEventListener(FormEvents::PRE_SET_DATA, function (FormEvent $event): void
+        {
+            /** @var ScheduledCommand $scheduledCommand */
+            $scheduledCommand = $event->getData();
+            $form = $event->getForm();
+            $options = [
                 'label' => 'detail.name',
                 'required' => true,
-            ]
-        );
+            ];
+
+            if (! is_null($scheduledCommand->getId())) {
+                $options['attr'] = ['readonly' => 'readonly'];
+            }
+
+            $form->add('name',
+                TextType::class,
+                $options);
+        });
 
         $builder->add(
             'command',