Add strict type

[airbundle] / Form / RegisterType.php

diff --git a/Form/RegisterType.php b/Form/RegisterType.php
index 83c80b59daafc98062cdcfe1ee7dc88b022f86fa..b41944acff58415c3c363b15ac8268dcb37367e3 100644 (file)
--- a/Form/RegisterType.php
+++ b/Form/RegisterType.php
@@ -1,28 +1,58 @@
-<?php
+<?php declare(strict_types=1);
+
+/*
+ * This file is part of the Rapsys PackBundle package.
+ *
+ * (c) Raphaël Gertz <symfony@rapsys.eu>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
 
 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 {
        /**
         * {@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']]);
+       public function buildForm(FormBuilderInterface $builder, array $options): FormBuilderInterface {
+               //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 getName() {
+       public function configureOptions(OptionsResolver $resolver): void {
+               //Call parent configure options
+               parent::configureOptions($resolver);
+
+               //Set defaults
+               $resolver->setDefaults(['phone' => true]);
+
+               //Add extra mail option
+               $resolver->setAllowedTypes('phone', 'boolean');
+       }
+
+
+       /**
+        * {@inheritdoc}
+        */
+       public function getName(): string {
                return 'rapsys_air_register';
        }
 }