Merge pull request #28 from tominventisbe/feature/allow-disabling-for…

…m-error-when-filled-in Allow disabling form error if honeypot is filled in
eymengunay · Sep 19, 2019 · 1faf275 · 1faf275
2 parents fbb3f8d + 2fe41b6
commit 1faf275
Showing 1 changed file with 9 additions and 6 deletions.
diff --git a/Form/Type/HoneypotType.php b/Form/Type/HoneypotType.php
@@ -68,7 +68,7 @@ public function buildForm(FormBuilderInterface $builder, array $options)
         $honeypotManager = $this->honeypotManager;
         $eventDispatcher = $this->eventDispatcher;
 
-        $builder->addEventListener(FormEvents::PRE_SUBMIT, function(FormEvent $event) use ($request, $honeypotManager, $eventDispatcher) {
+        $builder->addEventListener(FormEvents::PRE_SUBMIT, function(FormEvent $event) use ($request, $honeypotManager, $eventDispatcher, $options) {
             $data = $event->getData();
             $form = $event->getForm();
 
@@ -85,7 +85,9 @@ public function buildForm(FormBuilderInterface $builder, array $options)
             // Save prey
             $honeypotManager->save($prey);
 
-            $form->getParent()->addError(new FormError('Form is invalid.'));
+            if ($options['causesError']) {
+                $form->getParent()->addError(new FormError('Form is invalid.'));
+            }
         });
     }
 
@@ -95,10 +97,11 @@ public function buildForm(FormBuilderInterface $builder, array $options)
     public function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefaults(array(
-            'required' => false,
-            'mapped'   => false,
-            'data'     => '',
-            'attr'     => array(
+            'required'    => false,
+            'mapped'      => false,
+            'data'        => '',
+            'causesError' => true,
+            'attr'        => array(
                 // autocomplete="off" does not work in some cases, random strings always do
                 'autocomplete' => 'nope',
                 // Make the field unfocusable for keyboard users