[Notifier] Use CPP

symfony · nicolas-grekas · Jan 2, 2024 · Dec 27, 2023 · Jan 2, 2024 · dbe43a52a020c55e89dd2180bc81372af843c466
commit dbe43a52a020c55e89dd2180bc81372af843c466
@@ -22,14 +22,10 @@
  */
 final class BrowserChannel implements ChannelInterface
 {
-    private RequestStack $stack;
-
-    private FlashMessageImportanceMapperInterface $mapper;
-
-    public function __construct(RequestStack $stack, FlashMessageImportanceMapperInterface $mapper = new DefaultFlashMessageImportanceMapper())
-    {
-        $this->stack = $stack;
-        $this->mapper = $mapper;
+    public function __construct(
+        private RequestStack $stack,
+        private FlashMessageImportanceMapperInterface $mapper = new DefaultFlashMessageImportanceMapper(),
+    ) {
     }
 
     public function notify(Notification $notification, RecipientInterface $recipient, string $transportName = null): void

@@ -18,11 +18,9 @@
  */
 final class ChannelPolicy implements ChannelPolicyInterface
 {
-    private array $policy;
-
-    public function __construct(array $policy)
-    {
-        $this->policy = $policy;
+    public function __construct(
+        private array $policy,
+    ) {
     }
 
     public function getChannels(string $importance): array

@@ -29,21 +29,19 @@
  */
 class EmailChannel implements ChannelInterface
 {
-    private ?TransportInterface $transport;
-    private ?MessageBusInterface $bus;
     private string|Address|null $from;
-    private ?Envelope $envelope;
 
-    public function __construct(TransportInterface $transport = null, MessageBusInterface $bus = null, string $from = null, Envelope $envelope = null)
-    {
+    public function __construct(
+        private ?TransportInterface $transport = null,
+        private ?MessageBusInterface $bus = null,
+        string $from = null,
+        private ?Envelope $envelope = null,
+    ) {
         if (null === $transport && null === $bus) {
             throw new LogicException(sprintf('"%s" needs a Transport or a Bus but both cannot be "null".', static::class));
         }
 
-        $this->transport = $transport;
-        $this->bus = $bus;
         $this->from = $from ?: $envelope?->getSender();
-        $this->envelope = $envelope;
     }
 
     /**

@@ -23,15 +23,11 @@
  */
 final class Chatter implements ChatterInterface
 {
-    private TransportInterface $transport;
-    private ?MessageBusInterface $bus;
-    private ?EventDispatcherInterface $dispatcher;
-
-    public function __construct(TransportInterface $transport, MessageBusInterface $bus = null, EventDispatcherInterface $dispatcher = null)
-    {
-        $this->transport = $transport;
-        $this->bus = $bus;
-        $this->dispatcher = $dispatcher;
+    public function __construct(
+        private TransportInterface $transport,
+        private ?MessageBusInterface $bus = null,
+        private ?EventDispatcherInterface $dispatcher = null,
+    ) {
     }
 
     public function __toString(): string

@@ -22,11 +22,9 @@
  */
 final class NotificationDataCollector extends DataCollector
 {
-    private NotificationLoggerListener $logger;
-
-    public function __construct(NotificationLoggerListener $logger)
-    {
-        $this->logger = $logger;
+    public function __construct(
+        private NotificationLoggerListener $logger,
+    ) {
     }
 
     public function collect(Request $request, Response $response, \Throwable $exception = null): void

@@ -19,13 +19,10 @@
  */
 final class FailedMessageEvent extends Event
 {
-    private MessageInterface $message;
-    private \Throwable $error;
-
-    public function __construct(MessageInterface $message, \Throwable $error)
-    {
-        $this->message = $message;
-        $this->error = $error;
+    public function __construct(
+        private MessageInterface $message,
+        private \Throwable $error,
+    ) {
     }
 
     public function getMessage(): MessageInterface

@@ -19,13 +19,10 @@
  */
 final class MessageEvent extends Event
 {
-    private MessageInterface $message;
-    private bool $queued;
-
-    public function __construct(MessageInterface $message, bool $queued = false)
-    {
-        $this->message = $message;
-        $this->queued = $queued;
+    public function __construct(
+        private MessageInterface $message,
+        private bool $queued = false,
+    ) {
     }
 
     public function getMessage(): MessageInterface

@@ -19,11 +19,9 @@
  */
 final class SentMessageEvent extends Event
 {
-    private SentMessage $message;
-
-    public function __construct(SentMessage $message)
-    {
-        $this->message = $message;
+    public function __construct(
+        private SentMessage $message,
+    ) {
     }
 
     public function getMessage(): SentMessage

@@ -24,11 +24,9 @@
  */
 class SendFailedMessageToNotifierListener implements EventSubscriberInterface
 {
-    private Notifier $notifier;
-
-    public function __construct(Notifier $notifier)
-    {
-        $this->notifier = $notifier;
+    public function __construct(
+        private Notifier $notifier,
+    ) {
     }
 
     public function onMessageFailed(WorkerMessageFailedEvent $event): void

@@ -16,11 +16,11 @@
  */
 class IncompleteDsnException extends InvalidArgumentException
 {
-    private ?string $dsn;
-
-    public function __construct(string $message, string $dsn = null, \Throwable $previous = null)
-    {
-        $this->dsn = $dsn;
+    public function __construct(
+        string $message,
+        private ?string $dsn = null,
+        \Throwable $previous = null,
+    ) {
         if ($dsn) {
             $message = sprintf('Invalid "%s" notifier DSN: %s', $dsn, $message);
         }

@@ -18,12 +18,14 @@
  */
 class TransportException extends RuntimeException implements TransportExceptionInterface
 {
-    private ResponseInterface $response;
     private string $debug = '';
 
-    public function __construct(string $message, ResponseInterface $response, int $code = 0, \Throwable $previous = null)
-    {
-        $this->response = $response;
+    public function __construct(
+        string $message,
+        private ResponseInterface $response,
+        int $code = 0,
+        \Throwable $previous = null,
+    ) {
         $this->debug .= $response->getInfo('debug') ?? '';
 
         parent::__construct($message, $code, $previous);

@@ -19,14 +19,12 @@
 class ChatMessage implements MessageInterface, FromNotificationInterface
 {
     private ?string $transport = null;
-    private string $subject;
-    private ?MessageOptionsInterface $options;
     private ?Notification $notification = null;
 
-    public function __construct(string $subject, MessageOptionsInterface $options = null)
-    {
-        $this->subject = $subject;
-        $this->options = $options;
+    public function __construct(
+        private string $subject,
+        private ?MessageOptionsInterface $options = null,
+    ) {
     }
 
     public static function fromNotification(Notification $notification): self

@@ -25,14 +25,12 @@
  */
 class EmailMessage implements MessageInterface, FromNotificationInterface
 {
-    private RawMessage $message;
-    private ?Envelope $envelope;
     private ?Notification $notification = null;
 
-    public function __construct(RawMessage $message, Envelope $envelope = null)
-    {
-        $this->message = $message;
-        $this->envelope = $envelope;
+    public function __construct(
+        private RawMessage $message,
+        private ?Envelope $envelope = null,
+    ) {
     }
 
     public static function fromNotification(Notification $notification, EmailRecipientInterface $recipient): self

@@ -19,16 +19,13 @@
 class PushMessage implements MessageInterface, FromNotificationInterface
 {
     private ?string $transport = null;
-    private string $subject;
-    private string $content;
-    private ?MessageOptionsInterface $options;
     private ?Notification $notification = null;
 
-    public function __construct(string $subject, string $content, MessageOptionsInterface $options = null)
-    {
-        $this->subject = $subject;
-        $this->content = $content;
-        $this->options = $options;
+    public function __construct(
+        private string $subject,
+        private string $content,
+        private ?MessageOptionsInterface $options = null,
+    ) {
     }
 
     public static function fromNotification(Notification $notification): self

@@ -16,14 +16,12 @@
  */
 class SentMessage
 {
-    private MessageInterface $original;
-    private string $transport;
     private ?string $messageId = null;
 
-    public function __construct(MessageInterface $original, string $transport)
-    {
-        $this->original = $original;
-        $this->transport = $transport;
+    public function __construct(
+        private MessageInterface $original,
+        private string $transport,
+    ) {
     }
 
     public function getOriginalMessage(): MessageInterface

@@ -21,22 +21,17 @@
 class SmsMessage implements MessageInterface, FromNotificationInterface
 {
     private ?string $transport = null;
-    private string $subject;
-    private string $phone;
-    private string $from;
-    private ?MessageOptionsInterface $options;
     private ?Notification $notification = null;
 
-    public function __construct(string $phone, string $subject, string $from = '', MessageOptionsInterface $options = null)
-    {
+    public function __construct(
+        private string $phone,
+        private string $subject,
+        private string $from = '',
+        private ?MessageOptionsInterface $options = null,
+    ) {
         if ('' === $phone) {
             throw new InvalidArgumentException(sprintf('"%s" needs a phone number, it cannot be empty.', __CLASS__));
         }
-
-        $this->subject = $subject;
-        $this->phone = $phone;
-        $this->from = $from;
-        $this->options = $options;
     }
 
     public static function fromNotification(Notification $notification, SmsRecipientInterface $recipient): self

@@ -20,11 +20,9 @@
  */
 final class MessageHandler
 {
-    private TransportInterface $transport;
-
-    public function __construct(TransportInterface $transport)
-    {
-        $this->transport = $transport;
+    public function __construct(
+        private TransportInterface $transport,
+    ) {
     }
 
     public function __invoke(MessageInterface $message): ?SentMessage

@@ -27,16 +27,14 @@
 final class Notifier implements NotifierInterface
 {
     private array $adminRecipients = [];
-    private array|ContainerInterface $channels;
-    private ?ChannelPolicyInterface $policy;
 
     /**
      * @param ChannelInterface[]|ContainerInterface $channels
      */
-    public function __construct(array|ContainerInterface $channels, ChannelPolicyInterface $policy = null)
-    {
-        $this->channels = $channels;
-        $this->policy = $policy;
+    public function __construct(
+        private array|ContainerInterface $channels,
+        private ?ChannelPolicyInterface $policy = null,
+    ) {
     }
 
     public function send(Notification $notification, RecipientInterface ...$recipients): void

@@ -19,15 +19,11 @@
  */
 final class NotificationCount extends Constraint
 {
-    private int $expectedValue;
-    private ?string $transport;
-    private bool $queued;
-
-    public function __construct(int $expectedValue, string $transport = null, bool $queued = false)
-    {
-        $this->expectedValue = $expectedValue;
-        $this->transport = $transport;
-        $this->queued = $queued;
+    public function __construct(
+        private int $expectedValue,
+        private ?string $transport = null,
+        private bool $queued = false,
+    ) {
     }
 
     public function toString(): string

@@ -19,11 +19,9 @@
  */
 final class NotificationSubjectContains extends Constraint
 {
-    private string $expectedText;
-
-    public function __construct(string $expectedText)
-    {
-        $this->expectedText = $expectedText;
+    public function __construct(
+        private string $expectedText,
+    ) {
     }
 
     public function toString(): string

@@ -19,11 +19,9 @@
  */
 final class NotificationTransportIsEqual extends Constraint
 {
-    private ?string $expectedText;
-
-    public function __construct(?string $expectedText)
-    {
-        $this->expectedText = $expectedText;
+    public function __construct(
+        private ?string $expectedText,
+    ) {
     }
 
     public function toString(): string