replaced generic HttpExceptions with more specific subclasses

Author: mvdbos

EventListener/FormatListener.php

@@ -13,6 +13,7 @@
 
 use Symfony\Component\HttpKernel\Event\GetResponseEvent;
 use Symfony\Component\HttpKernel\Exception\HttpException;
+use Symfony\Component\HttpKernel\Exception\NotAcceptableHttpException;
 use Symfony\Component\HttpKernel\HttpKernelInterface;
 
 use FOS\RestBundle\Util\Codes;
@@ -65,7 +66,7 @@ public function onKernelRequest(GetResponseEvent $event)
 
         if (null === $format) {
             if ($event->getRequestType() === HttpKernelInterface::MASTER_REQUEST) {
-                throw new HttpException(Codes::HTTP_NOT_ACCEPTABLE, "No matching accepted Response format could be determined");
+                throw new NotAcceptableHttpException("No matching accepted Response format could be determined");
             }
 
             return;

Request/ParamFetcher.php

@@ -17,6 +17,7 @@
 use FOS\RestBundle\Controller\Annotations\RequestParam;
 use Doctrine\Common\Util\ClassUtils;
 use Symfony\Component\HttpFoundation\Request;
+use Symfony\Component\HttpKernel\Exception\BadRequestHttpException;
 use Symfony\Component\HttpKernel\Exception\HttpException;
 
 /**
@@ -113,7 +114,7 @@ public function get($name, $strict = null)
 
             if (null !== $failMessage) {
                 if ($strict) {
-                    throw new HttpException(Codes::HTTP_BAD_REQUEST, $failMessage);
+                    throw new BadRequestHttpException($failMessage);
                 }
 
                 return $default;
@@ -133,8 +134,9 @@ public function get($name, $strict = null)
                     $paramType = $config instanceof QueryParam ? 'Query' : 'Request';
                     $problem = empty($param) ? 'empty' : 'not a scalar';
 
-                    throw new HttpException(Codes::HTTP_BAD_REQUEST,
-                        sprintf('%s parameter "%s" is %s', $paramType, $name, $problem));
+                    throw new BadRequestHttpException(
+                        sprintf('%s parameter "%s" is %s', $paramType, $name, $problem)
+                    );
                 }
 
                 return $this->cleanParamWithRequirements($config, $param, $strict);
@@ -165,7 +167,7 @@ public function cleanParamWithRequirements(Param $config, $param, $strict)
             if ($strict) {
                 $paramType = $config instanceof QueryParam ? 'Query' : 'Request';
 
-                throw new HttpException(Codes::HTTP_BAD_REQUEST, $paramType . " parameter value '$param', does not match requirements '{$config->requirements}'");
+                throw new BadRequestHttpException($paramType . " parameter value '$param', does not match requirements '{$config->requirements}'");
             }
 
             return null === $default ? '' : $default;

Request/RequestBodyParamConverter.php

@@ -12,7 +12,9 @@
 namespace FOS\RestBundle\Request;
 
 use Symfony\Component\HttpFoundation\Request;
+use Symfony\Component\HttpKernel\Exception\BadRequestHttpException;
 use Symfony\Component\HttpKernel\Exception\HttpException;
+use Symfony\Component\HttpKernel\Exception\UnsupportedMediaTypeHttpException;
 use Symfony\Component\OptionsResolver\OptionsResolver;
 use Symfony\Component\Serializer\Exception\Exception as SymfonySerializerException;
 use Symfony\Component\Serializer\SerializerInterface as SymfonySerializerInterface;
@@ -108,11 +110,11 @@ public function apply(Request $request, ConfigurationInterface $configuration)
                 $context
             );
         } catch (UnsupportedFormatException $e) {
-            throw new HttpException(Codes::HTTP_UNSUPPORTED_MEDIA_TYPE, $e->getMessage());
+            throw new UnsupportedMediaTypeHttpException($e->getMessage());
         } catch (JMSSerializerException $e) {
-            throw new HttpException(Codes::HTTP_BAD_REQUEST, $e->getMessage());
+            throw new BadRequestHttpException($e->getMessage());
         } catch (SymfonySerializerException $e) {
-            throw new HttpException(Codes::HTTP_BAD_REQUEST, $e->getMessage());
+            throw new BadRequestHttpException($e->getMessage());
         }
 
         $request->attributes->set($configuration->getName(), $object);

View/JsonpHandler.php

@@ -13,6 +13,7 @@
 
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\HttpFoundation\Request;
+use Symfony\Component\HttpKernel\Exception\BadRequestHttpException;
 use Symfony\Component\HttpKernel\Exception\HttpException;
 
 use FOS\RestBundle\Util\Codes;
@@ -39,7 +40,7 @@ protected function getCallback(Request $request)
 
         if ($this->callbackFilter && !preg_match($this->callbackFilter, $callback)) {
             $msg = "Callback '$callback' does not match '{$this->callbackFilter}'";
-            throw new HttpException(Codes::HTTP_BAD_REQUEST, $msg);
+            throw new BadRequestHttpException($msg);
         }
 
         return $callback;

View/ViewHandler.php

@@ -17,7 +17,7 @@
 use Symfony\Component\HttpFoundation\RedirectResponse;
 use Symfony\Component\HttpFoundation\Response;
 use Symfony\Component\HttpFoundation\Request;
-use Symfony\Component\HttpKernel\Exception\HttpException;
+use Symfony\Component\HttpKernel\Exception\UnsupportedMediaTypeHttpException;
 use Symfony\Component\DependencyInjection\ContainerAware;
 use Symfony\Component\Form\FormInterface;
 use Symfony\Bundle\FrameworkBundle\Templating\TemplateReference;
@@ -289,7 +289,7 @@ public function handle(View $view, Request $request = null)
 
         if (!$this->supports($format)) {
             $msg = "Format '$format' not supported, handler must be implemented";
-            throw new HttpException(Codes::HTTP_UNSUPPORTED_MEDIA_TYPE, $msg);
+            throw new UnsupportedMediaTypeHttpException($msg);
         }
 
         if (isset($this->customHandlers[$format])) {