From fe06a2a27a71036cadfeb086fc3db272323a2da2 Mon Sep 17 00:00:00 2001 From: Kouril42 Date: Thu, 16 Feb 2023 18:33:47 +0100 Subject: [PATCH] Change ValueError (500) to preconditionFailed (412) --- src/Controller/MapApiController.php | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/Controller/MapApiController.php b/src/Controller/MapApiController.php index 8624c66..a9f5575 100644 --- a/src/Controller/MapApiController.php +++ b/src/Controller/MapApiController.php @@ -6,6 +6,7 @@ use Symfony\Bundle\FrameworkBundle\Controller\AbstractController; use Symfony\Component\HttpFoundation\JsonResponse; use Symfony\Component\HttpKernel\Exception\NotFoundHttpException; use Symfony\Component\HttpKernel\Exception\UnprocessableEntityHttpException; +use Symfony\Component\HttpKernel\Exception\AccessDeniedHttpException; use Symfony\Component\Routing\Annotation\Route; use Doctrine\Common\Collections\ArrayCollection; @@ -157,7 +158,7 @@ class MapApiController extends AbstractController else{ $names = new ArrayCollection(); foreach($places[0]->getNames() as $plName) { $names[] = $plName->getName(); } - if ($names->contains($value)) { throw new ValueError('Warning: \''.$value.'\' is already a name for place '.$placeId);} + if ($names->contains($value)) { throw new AccessDeniedHttpException('Warning: \''.$value.'\' is already a name for place '.$placeId);} else { $pN = new PlaceName(); $pN->setPlace($places[0]); @@ -186,7 +187,7 @@ class MapApiController extends AbstractController else{ $names = new ArrayCollection(); foreach($places[0]->getRoomUsers() as $rUser) { $names[] = $rUser->getUserName(); } - if ($names->contains($value)) { throw new ValueError('Warning: \''.$value.'\' is already a name for place '.$placeId);} + if ($names->contains($value)) { throw new AccessDeniedHttpException('Warning: \''.$value.'\' is already a name for place '.$placeId);} else { $rU = new RoomUserName(); $rU->setPlace($places[0]);