cs-fix; phpstan

ecamp · Aug 8, 2024 · 5cf86d5 · 5cf86d5
1 parent 630b9ab
commit 5cf86d5
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/api/src/Validator/ChecklistItem/AssertBelongsToSameCampValidator.php b/api/src/Validator/ChecklistItem/AssertBelongsToSameCampValidator.php
@@ -15,9 +15,9 @@
 
 class AssertBelongsToSameCampValidator extends ConstraintValidator {
     use GetCampFromContentNodeTrait;
-    
+
     public function __construct(
-        public RequestStack $requestStack, 
+        public RequestStack $requestStack,
         private EntityManagerInterface $em,
         private ChecklistItemRepository $checklistItemRepository,
     ) {}
@@ -42,13 +42,14 @@ public function validate($value, Constraint $constraint): void {
 
         $camp = $this->getCampFromInterface($object, $this->em);
 
-        foreach($value as $checklistItemId) {
-            /** @var ChecklistItem $checklistItem */
+        foreach ($value as $checklistItemId) {
+            /** @var ?ChecklistItem $checklistItem */
             $checklistItem = $this->checklistItemRepository->find($checklistItemId);
 
             if ($camp != $checklistItem?->getCamp()) {
                 $this->context->buildViolation($constraint->message)
-                    ->addViolation();
+                    ->addViolation()
+                ;
             }
         }
     }

diff --git a/api/tests/Api/ChecklistItems/UpdateChecklistItemTest.php b/api/tests/Api/ChecklistItems/UpdateChecklistItemTest.php
@@ -124,7 +124,7 @@ public function testPatchhChecklistItemValidatesNoParentLoop() {
                 'headers' => ['Content-Type' => 'application/merge-patch+json'],
             ]
         );
-        
+
         $this->assertResponseStatusCodeSame(422);
         $this->assertJsonContains([
             'title' => 'An error occurred',