[Dotenv][ErrorHandler][EventDispatcher] Use CPP

Author: OskarStark

Debug/TraceableEventDispatcher.php

@@ -30,25 +30,20 @@
  */
 class TraceableEventDispatcher implements EventDispatcherInterface, ResetInterface
 {
-    protected ?LoggerInterface $logger;
-    protected Stopwatch $stopwatch;
-
     /**
      * @var \SplObjectStorage<WrappedListener, array{string, string}>|null
      */
     private ?\SplObjectStorage $callStack = null;
-    private EventDispatcherInterface $dispatcher;
     private array $wrappedListeners = [];
     private array $orphanedEvents = [];
-    private ?RequestStack $requestStack;
     private string $currentRequestHash = '';
 
-    public function __construct(EventDispatcherInterface $dispatcher, Stopwatch $stopwatch, LoggerInterface $logger = null, RequestStack $requestStack = null)
-    {
-        $this->dispatcher = $dispatcher;
-        $this->stopwatch = $stopwatch;
-        $this->logger = $logger;
-        $this->requestStack = $requestStack;
+    public function __construct(
+        private EventDispatcherInterface $dispatcher,
+        protected Stopwatch $stopwatch,
+        protected ?LoggerInterface $logger = null,
+        private ?RequestStack $requestStack = null,
+    ) {
     }
 
     public function addListener(string $eventName, callable|array $listener, int $priority = 0): void

Debug/WrappedListener.php

@@ -26,21 +26,20 @@ final class WrappedListener
     private string $name;
     private bool $called = false;
     private bool $stoppedPropagation = false;
-    private Stopwatch $stopwatch;
-    private ?EventDispatcherInterface $dispatcher;
     private string $pretty;
     private string $callableRef;
     private ClassStub|string $stub;
-    private ?int $priority = null;
     private static bool $hasClassStub;
 
-    public function __construct(callable|array $listener, ?string $name, Stopwatch $stopwatch, EventDispatcherInterface $dispatcher = null, int $priority = null)
-    {
+    public function __construct(
+        callable|array $listener,
+        ?string $name,
+        private Stopwatch $stopwatch,
+        private ?EventDispatcherInterface $dispatcher = null,
+        private ?int $priority = null,
+    ) {
         $this->listener = $listener;
         $this->optimizedListener = $listener instanceof \Closure ? $listener : (\is_callable($listener) ? $listener(...) : null);
-        $this->stopwatch = $stopwatch;
-        $this->dispatcher = $dispatcher;
-        $this->priority = $priority;
 
         if (\is_array($listener)) {
             [$this->name, $this->callableRef] = $this->parseListener($listener);

DependencyInjection/AddEventAliasesPass.php

@@ -21,11 +21,9 @@
  */
 class AddEventAliasesPass implements CompilerPassInterface
 {
-    private array $eventAliases;
-
-    public function __construct(array $eventAliases)
-    {
-        $this->eventAliases = $eventAliases;
+    public function __construct(
+        private array $eventAliases,
+    ) {
     }
 
     public function process(ContainerBuilder $container): void

GenericEvent.php

@@ -25,19 +25,16 @@
  */
 class GenericEvent extends Event implements \ArrayAccess, \IteratorAggregate
 {
-    protected mixed $subject;
-    protected array $arguments;
-
     /**
      * Encapsulate an event with $subject and $arguments.
      *
      * @param mixed $subject   The subject of the event, usually an object or a callable
      * @param array $arguments Arguments to store in the event
      */
-    public function __construct(mixed $subject = null, array $arguments = [])
-    {
-        $this->subject = $subject;
-        $this->arguments = $arguments;
+    public function __construct(
+        protected mixed $subject = null,
+        protected array $arguments = [],
+    ) {
     }
 
     /**

ImmutableEventDispatcher.php

@@ -18,11 +18,9 @@
  */
 class ImmutableEventDispatcher implements EventDispatcherInterface
 {
-    private EventDispatcherInterface $dispatcher;
-
-    public function __construct(EventDispatcherInterface $dispatcher)
-    {
-        $this->dispatcher = $dispatcher;
+    public function __construct(
+        private EventDispatcherInterface $dispatcher,
+    ) {
     }
 
     public function dispatch(object $event, string $eventName = null): object