Fix BC break

Author: Jérôme Deuchnord

src/EventListener/AddFormatListener.php

@@ -41,7 +41,7 @@ final class AddFormatListener
     /**
      * @throws InvalidArgumentException
      */
-    public function __construct(Negotiator $negotiator, /* FormatsProviderInterface */ $formatsProvider, EventDispatcherInterface $dispatcher)
+    public function __construct(Negotiator $negotiator, /* FormatsProviderInterface */ $formatsProvider, EventDispatcherInterface $dispatcher = null)
     {
         $this->negotiator = $negotiator;
         if (\is_array($formatsProvider)) {
@@ -77,9 +77,16 @@ public function onKernelRequest(GetResponseEvent $event)
         }
 
         $this->populateMimeTypes();
-        $this->dispatcher->dispatch(PreAddFormatEvent::NAME, new PreAddFormatEvent($this->formats));
+
+        if (null !== $this->dispatcher) {
+            $this->dispatcher->dispatch(PreAddFormatEvent::NAME, new PreAddFormatEvent($this->formats));
+        }
+
         $this->addRequestFormats($request, $this->formats);
-        $this->dispatcher->dispatch(PostAddFormatEvent::NAME, new PostAddFormatEvent($this->formats));
+
+        if (null !== $this->dispatcher) {
+            $this->dispatcher->dispatch(PostAddFormatEvent::NAME, new PostAddFormatEvent($this->formats));
+        }
 
         // Empty strings must be converted to null because the Symfony router doesn't support parameter typing before 3.2 (_format)
         if (null === $routeFormat = $request->attributes->get('_format') ?: null) {

src/EventListener/DeserializeListener.php

@@ -42,7 +42,7 @@ final class DeserializeListener
     /**
      * @throws InvalidArgumentException
      */
-    public function __construct(SerializerInterface $serializer, SerializerContextBuilderInterface $serializerContextBuilder, /* FormatsProviderInterface */ $formatsProvider, EventDispatcherInterface $dispatcher)
+    public function __construct(SerializerInterface $serializer, SerializerContextBuilderInterface $serializerContextBuilder, /* FormatsProviderInterface */ $formatsProvider, EventDispatcherInterface $dispatcher = null)
     {
         $this->serializer = $serializer;
         $this->serializerContextBuilder = $serializerContextBuilder;
@@ -95,14 +95,20 @@ public function onKernelRequest(GetResponseEvent $event)
             $context[AbstractNormalizer::OBJECT_TO_POPULATE] = $data;
         }
 
-        $this->dispatcher->dispatch(PreDeserializeEvent::NAME, new PreDeserializeEvent($requestContent));
+        if (null !== $this->dispatcher) {
+            $this->dispatcher->dispatch(PreDeserializeEvent::NAME, new PreDeserializeEvent($requestContent));
+        }
+
         $request->attributes->set(
             'data',
             $this->serializer->deserialize(
                 $requestContent, $attributes['input_class'], $format, $context
             )
         );
-        $this->dispatcher->dispatch(PostDeserializeEvent::NAME, new PostDeserializeEvent($requestContent));
+
+        if (null !== $this->dispatcher) {
+            $this->dispatcher->dispatch(PostDeserializeEvent::NAME, new PostDeserializeEvent($requestContent));
+        }
     }
 
     /**

src/EventListener/ReadListener.php

@@ -95,7 +95,9 @@ public function onKernelRequest(GetResponseEvent $event)
         try {
             $identifiers = $this->extractIdentifiers($request->attributes->all(), $attributes);
 
-            $this->dispatcher->dispatch(PreReadEvent::NAME, new PreReadEvent($data));
+            if (null !== $this->dispatcher) {
+                $this->dispatcher->dispatch(PreReadEvent::NAME, new PreReadEvent($data));
+            }
 
             if (isset($attributes['item_operation_name'])) {
                 $data = $this->getItemData($identifiers, $attributes, $context);
@@ -111,7 +113,9 @@ public function onKernelRequest(GetResponseEvent $event)
             $data = null;
         }
 
-        $this->dispatcher->dispatch(PostReadEvent::NAME, new PostReadEvent($data));
+        if (null !== $this->dispatcher) {
+            $this->dispatcher->dispatch(PostReadEvent::NAME, new PostReadEvent($data));
+        }
 
         if (null === $data) {
             throw new NotFoundHttpException('Not Found');

src/EventListener/RespondListener.php

@@ -77,14 +77,18 @@ public function onKernelView(GetResponseForControllerResultEvent $event)
             }
         }
 
-        $this->dispatcher->dispatch(PreRespondEvent::NAME, new PreRespondEvent($event));
+        if (null !== $this->dispatcher) {
+            $this->dispatcher->dispatch(PreRespondEvent::NAME, new PreRespondEvent($event));
+        }
 
         $event->setResponse(new Response(
             $controllerResult,
             self::METHOD_TO_CODE[$request->getMethod()] ?? Response::HTTP_OK,
             $headers
         ));
 
-        $this->dispatcher->dispatch(PostRespondEvent::NAME, new PostRespondEvent($event));
+        if (null !== $this->dispatcher) {
+            $this->dispatcher->dispatch(PostRespondEvent::NAME, new PostRespondEvent($event));
+        }
     }
 }

src/EventListener/SerializeListener.php

@@ -39,7 +39,7 @@ final class SerializeListener
     /** @var EventDispatcherInterface */
     private $dispatcher;
 
-    public function __construct(SerializerInterface $serializer, SerializerContextBuilderInterface $serializerContextBuilder, EventDispatcherInterface $dispatcher)
+    public function __construct(SerializerInterface $serializer, SerializerContextBuilderInterface $serializerContextBuilder, EventDispatcherInterface $dispatcher = null)
     {
         $this->serializer = $serializer;
         $this->serializerContextBuilder = $serializerContextBuilder;
@@ -76,11 +76,15 @@ public function onKernelView(GetResponseForControllerResultEvent $event)
 
         $request->attributes->set('_api_normalization_context', $context);
 
-        $this->dispatcher->dispatch(PreSerializeEvent::NAME, new PreSerializeEvent($controllerResult));
+        if (null !== $this->dispatcher) {
+            $this->dispatcher->dispatch(PreSerializeEvent::NAME, new PreSerializeEvent($controllerResult));
+        }
 
         $event->setControllerResult($this->serializer->serialize($controllerResult, $request->getRequestFormat(), $context));
 
-        $this->dispatcher->dispatch(PostSerializeEvent::NAME, new PostSerializeEvent($controllerResult));
+        if (null !== $this->dispatcher) {
+            $this->dispatcher->dispatch(PostSerializeEvent::NAME, new PostSerializeEvent($controllerResult));
+        }
 
         $request->attributes->set('_api_respond', true);
         $request->attributes->set('_resources', $request->attributes->get('_resources', []) + (array) $resources);

src/EventListener/WriteListener.php

@@ -54,14 +54,19 @@ public function onKernelView(GetResponseForControllerResultEvent $event)
             return;
         }
 
-        $this->dispatcher->dispatch(PreWriteEvent::NAME, new PreWriteEvent($request->getMethod(), $controllerResult));
+        if (null !== $this->dispatcher) {
+            $this->dispatcher->dispatch(PreWriteEvent::NAME, new PreWriteEvent($request->getMethod(), $controllerResult));
+        }
 
         switch ($request->getMethod()) {
             case 'PUT':
             case 'PATCH':
             case 'POST':
                 $persistResult = $this->dataPersister->persist($controllerResult);
-                $this->dispatcher->dispatch(PostWriteEvent::NAME, new PostWriteEvent($request->getMethod(), $controllerResult));
+
+                if (null !== $this->dispatcher) {
+                    $this->dispatcher->dispatch(PostWriteEvent::NAME, new PostWriteEvent($request->getMethod(), $controllerResult));
+                }
 
                 if (null === $persistResult) {
                     @trigger_error(sprintf('Returning void from %s::persist() is deprecated since API Platform 2.3 and will not be supported in API Platform 3, an object should always be returned.', DataPersisterInterface::class), E_USER_DEPRECATED);