Allow phone field to be optional

[airbundle] / Form / RegisterType.php

diff --git a/Form/RegisterType.php b/Form/RegisterType.php
index 83c80b59daafc98062cdcfe1ee7dc88b022f86fa..56e0addba2b20a8f1bd6c700c1faa1f051be5c6c 100644 (file)
--- a/Form/RegisterType.php
+++ b/Form/RegisterType.php
@@ -5,6 +5,7 @@ namespace Rapsys\AirBundle\Form;
 use Symfony\Component\Form\Extension\Core\Type\TelType;
 use Symfony\Component\Form\Extension\Core\Type\UrlType;
 use Symfony\Component\Form\FormBuilderInterface;
+use Symfony\Component\OptionsResolver\OptionsResolver;
 use Symfony\Component\Validator\Constraints\NotBlank;
 
 class RegisterType extends \Rapsys\UserBundle\Form\RegisterType {
@@ -12,13 +13,33 @@ class RegisterType extends \Rapsys\UserBundle\Form\RegisterType {
         * {@inheritdoc}
         */
        public function buildForm(FormBuilderInterface $builder, array $options) {
-               return parent::buildForm($builder, $options)
-                       #TODO: add url ? add text ?
-                       ->add('phone', TelType::class, ['attr' => ['placeholder' => 'Your phone'], 'constraints' => [new NotBlank(['message' => 'Please provide your phone'])]])
-                       ->add('donation', UrlType::class, ['attr' => ['placeholder' => 'Your donation link']])
-                       ->add('site', UrlType::class, ['attr' => ['placeholder' => 'Your website']]);
+               //Call parent build form
+               $form = parent::buildForm($builder, $options);
+
+               //Add extra phone field
+               if (!empty($options['phone'])) {
+                       $form->add('phone', TelType::class, ['attr' => ['placeholder' => 'Your phone'], 'required' => false]);
+               }
+
+               //Return form
+               return $form;
+       }
+
+       /**
+        * {@inheritdoc}
+        */
+       public function configureOptions(OptionsResolver $resolver) {
+               //Call parent configure options
+               parent::configureOptions($resolver);
+
+               //Set defaults
+               $resolver->setDefaults(['phone' => true]);
+
+               //Add extra mail option
+               $resolver->setAllowedTypes('phone', 'boolean');
        }
 
+
        /**
         * {@inheritdoc}
         */