Running php-cs-fixer

Running php-cs-fixer to fix CS drift

Author: JimTools

AsyncEventDispatcher.php

@@ -4,7 +4,7 @@
 
 class AsyncEventDispatcher extends AbstractAsyncEventDispatcher
 {
-    public function dispatch(object $event, string $eventName = null): object
+    public function dispatch(object $event, ?string $eventName = null): object
     {
         $this->parentDispatch($event, $eventName);
 

AsyncProcessor.php

@@ -25,11 +25,7 @@ public function __construct(Registry $registry, EventDispatcherInterface $dispat
         $this->registry = $registry;
 
         if (false == $dispatcher instanceof AsyncEventDispatcher) {
-            throw new \InvalidArgumentException(sprintf(
-                'The dispatcher argument must be instance of "%s" but got "%s"',
-                AsyncEventDispatcher::class,
-                get_class($dispatcher)
-            ));
+            throw new \InvalidArgumentException(sprintf('The dispatcher argument must be instance of "%s" but got "%s"', AsyncEventDispatcher::class, $dispatcher::class));
         }
 
         $this->dispatcher = $dispatcher;

Commands.php

@@ -4,5 +4,5 @@
 
 final class Commands
 {
-    const DISPATCH_ASYNC_EVENTS = 'symfony.dispatch_async_events';
+    public const DISPATCH_ASYNC_EVENTS = 'symfony.dispatch_async_events';
 }

DependencyInjection/AsyncTransformersPass.php

@@ -7,9 +7,6 @@
 
 class AsyncTransformersPass implements CompilerPassInterface
 {
-    /**
-     * {@inheritdoc}
-     */
     public function process(ContainerBuilder $container)
     {
         if (false == $container->hasDefinition('enqueue.events.registry')) {

EventTransformer.php

@@ -12,15 +12,12 @@ interface EventTransformer
      *
      * @return Message
      */
-    public function toMessage($eventName, Event $event = null);
+    public function toMessage($eventName, ?Event $event = null);
 
     /**
      * If you able to transform message back to event return it.
      * If you failed to transform for some reason you can return a string status.
      *
-     * @param mixed $eventNAme
-     * @param mixed $eventName
-     *
      * @return Event|string|object
      *
      * @see Process constants) or an object that implements __toString method.

PhpSerializerEventTransformer.php

@@ -6,7 +6,7 @@
 
 class PhpSerializerEventTransformer extends AbstractPhpSerializerEventTransformer implements EventTransformer
 {
-    public function toMessage($eventName, Event $event = null)
+    public function toMessage($eventName, ?Event $event = null)
     {
         return $this->context->createMessage(serialize($event));
     }

SimpleRegistry.php

@@ -24,9 +24,6 @@ public function __construct(array $eventsMap, array $transformersMap)
         $this->transformersMap = $transformersMap;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getTransformerNameForEvent($eventName)
     {
         $transformerName = null;
@@ -53,9 +50,6 @@ public function getTransformerNameForEvent($eventName)
         return $transformerName;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function getTransformer($name)
     {
         if (false == array_key_exists($name, $this->transformersMap)) {
@@ -64,12 +58,8 @@ public function getTransformer($name)
 
         $transformer = $this->transformersMap[$name];
 
-        if (false == $transformer instanceof  EventTransformer) {
-            throw new \LogicException(sprintf(
-                'The container must return instance of %s but got %s',
-                EventTransformer::class,
-                is_object($transformer) ? get_class($transformer) : gettype($transformer)
-            ));
+        if (false == $transformer instanceof EventTransformer) {
+            throw new \LogicException(sprintf('The container must return instance of %s but got %s', EventTransformer::class, is_object($transformer) ? $transformer::class : gettype($transformer)));
         }
 
         return $transformer;

Tests/SimpleRegistryTest.php

@@ -21,7 +21,7 @@ public function testShouldImplementRegistryInterface()
     public function testShouldAllowGetTransportNameByEventName()
     {
         $registry = new SimpleRegistry([
-                'fooEvent' => 'fooTrans',
+            'fooEvent' => 'fooTrans',
         ], []);
 
         $this->assertEquals('fooTrans', $registry->getTransformerNameForEvent('fooEvent'));