bug symfony#52232 [Messenger] declare constructor argument as optional for backwards compatibility (xabbuh)

This PR was merged into the 6.3 branch.

Discussion
----------

[Messenger] declare constructor argument as optional for backwards compatibility

| Q             | A
| ------------- | ---
| Branch?       | 6.3
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | Fix symfony#51468 (comment)
| License       | MIT

Commits
-------

3810053 declare constructor argument as optional for backwards compatibility

Author: nicolas-grekas

src/Symfony/Component/Messenger/Exception/DelayedMessageHandlingException.php

@@ -22,9 +22,9 @@
 class DelayedMessageHandlingException extends RuntimeException
 {
     private array $exceptions;
-    private Envelope $envelope;
+    private ?Envelope $envelope;
 
-    public function __construct(array $exceptions, Envelope $envelope)
+    public function __construct(array $exceptions, Envelope $envelope = null)
     {
         $this->envelope = $envelope;
 
@@ -49,7 +49,7 @@ public function getExceptions(): array
         return $this->exceptions;
     }
 
-    public function getEnvelope(): Envelope
+    public function getEnvelope(): ?Envelope
     {
         return $this->envelope;
     }

src/Symfony/Component/Messenger/Worker.php

@@ -187,7 +187,7 @@ private function ack(): bool
                     $receiver->reject($envelope);
                 }
 
-                if ($e instanceof HandlerFailedException || $e instanceof DelayedMessageHandlingException) {
+                if ($e instanceof HandlerFailedException || ($e instanceof DelayedMessageHandlingException && null !== $e->getEnvelope())) {
                     $envelope = $e->getEnvelope();
                 }