From 61a8afdde7486d1f777f16c4c83511c812c951ab Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bj=C3=B6rn=20Fromme?= <fromme@dreipunktnull.com>
Date: Mon, 1 Dec 2025 10:38:55 +0100
Subject: [PATCH] fix: prevent null reference errors

---
 src/Controller/Api/TravelController.php | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/src/Controller/Api/TravelController.php b/src/Controller/Api/TravelController.php
index b6f475b..8007b3d 100644
--- a/src/Controller/Api/TravelController.php
+++ b/src/Controller/Api/TravelController.php
@@ -46,6 +46,10 @@ class TravelController extends AbstractController
     {
         $travel = $this->loadCached($travelId, $hotelId);
 
+        if (null === $travel) {
+            return new JsonResponse(['message' => 'Not found'], Response::HTTP_NOT_FOUND);
+        }
+
         return $this->json($travel, Response::HTTP_OK, [], ['groups' => ['api:list', 'api:single']]);
     }
 
@@ -68,6 +72,10 @@ class TravelController extends AbstractController
 
         $travel = $this->loadCached($travelId, $hotelId);
 
+        if (null === $travel) {
+            return new JsonResponse(['message' => 'Not found'], Response::HTTP_NOT_FOUND);
+        }
+
         return $this->json($travel, Response::HTTP_OK, [], ['groups' => ['api:list', 'api:single']]);
     }
 
@@ -82,7 +90,7 @@ class TravelController extends AbstractController
                 $travel = $this->travelXmlLoader->loadById($travelId, $hotelId);
 
                 if (null === $travel) {
-                    return new JsonResponse(['message' => 'Not found'], Response::HTTP_NOT_FOUND);
+                    return null;
                 }
 
                 $this->pickupXmlLoader->patchPickupsDetails($travel);