feat: improved validation of personal data

src/BusProNet/Model/Address.php

@@ -2,13 +2,23 @@
 
 namespace App\BusProNet\Model;
 
+use Symfony\Component\Validator\Constraints as Assert;
+
 class Address
 {
-    public ?string $street = null;
+    #[Assert\NotBlank(message: 'Bitte angeben', groups: ['personal_data'])]
-    public ?string $postCode = null;
+    public ?string $street = '';
-    public ?string $city = null;
+
-    public ?string $district = null;
+    #[Assert\NotBlank(message: 'Bitte angeben', groups: ['personal_data'])]
-    public ?string $country = null;
+    public ?string $postCode = '';
+
+    #[Assert\NotBlank(message: 'Bitte angeben', groups: ['personal_data'])]
+    public ?string $city = '';
+
+    public ?string $district = '';
+
+    #[Assert\NotBlank(message: 'Bitte angeben', groups: ['personal_data'])]
+    public ?string $country = '';
 
     public function toPayload(): array
     {
@@ -20,4 +30,4 @@ class Address
             'land' => $this->country,
         ];
     }
-}
+}

src/BusProNet/Model/Communication.php

@@ -2,11 +2,19 @@
 
 namespace App\BusProNet\Model;
 
+use Symfony\Component\Validator\Constraints as Assert;
+
 class Communication
 {
-    public ?string $phone = null;
+    public ?string $phone = '';
-    public ?string $mobile = null;
+
-    public ?string $email = null;
+    #[Assert\NotBlank(message: 'Bitte angeben', groups: ['personal_data'])]
+    public ?string $mobile = '';
+
+    #[Assert\NotBlank(message: 'Bitte angeben', groups: ['personal_data'])]
+    #[Assert\Email(message: 'Bitte eine gültige Adresse angeben', mode: 'strict', groups: ['personal_data'])]
+    public ?string $email = '';
+
     public bool $newsletter = false;
 
     public function toPayload(): array
@@ -18,4 +26,4 @@ class Communication
             'newsletter' => $this->newsletter ? 'True' : 'False',
         ];
     }
-}
+}

src/BusProNet/Model/PersonalData.php

@@ -9,29 +9,35 @@ class PersonalData
     public ?int $addressId = null;
     public ?int $personId = null;
     public bool $mutable = false;
-    public ?string $status = null;
+    public ?string $status = '';
 
-    #[Assert\NotBlank(message: 'Bitte angeben')]
+    #[Assert\NotBlank(message: 'Bitte angeben', groups: ['Default', 'personal_data'])]
-    public ?string $name = null;
+    public ?string $name = '';
 
-    #[Assert\NotBlank(message: 'Bitte angeben')]
+    #[Assert\NotBlank(message: 'Bitte angeben', groups: ['Default', 'personal_data'])]
-    public ?string $firstName = null;
+    public ?string $firstName = '';
 
-    public ?string $salutation = null;
+    public ?string $salutation = '';
-    public ?string $title = null;
+    public ?string $title = '';
-    public ?string $gender = null;
+    public ?string $gender = '';
-    public ?string $nationality = null;
+    public ?string $nationality = '';
-    public ?string $height = null;
+    public ?string $height = '';
-    public ?string $shoeSize = null;
+    public ?string $shoeSize = '';
-    public ?string $weight = null;
+    public ?string $weight = '';
     public ?\DateTimeImmutable $dateOfBirth = null;
-    public ?string $remarks = null;
+    public ?string $remarks = '';
 
-    #[Assert\Valid()]
+    #[Assert\Valid(groups: ['personal_data'])]
-    public ?Address $address = null;
+    public Address $address;
 
-    #[Assert\Valid()]
+    #[Assert\Valid(groups: ['personal_data'])]
-    public ?Communication $communication = null;
+    public Communication $communication;
+
+    public function __construct()
+    {
+        $this->address = new Address();
+        $this->communication = new Communication();
+    }
 
     public function toPayload(): array
     {

src/Controller/PersonalDataController.php

@@ -55,6 +55,7 @@ class PersonalDataController extends AbstractController
 
         $personalDataForm = $this->createForm(PersonalDataType::class, $personalData, [
             'attr' => ['novalidate' => 'novalidate'],
+            'validation_groups' => ['personal_data'],
         ]);
         $personalDataForm->handleRequest($request);
 

src/Form/PersonalDataType.php

@@ -38,6 +38,7 @@ class PersonalDataType extends AbstractType
             ])
             ->add('phone', TextType::class, [
                 'label' => 'Telefon',
+                'required' => false,
                 'property_path' => 'communication.phone',
             ])
             ->add('mobile', TextType::class, [