bug #52075 [Messenger] Fix DoctrineOpenTransactionLoggerMiddleware (ro0NL)

This PR was squashed before being merged into the 5.4 branch.

Discussion
----------

[Messenger] Fix DoctrineOpenTransactionLoggerMiddleware

| Q             | A
| ------------- | ---
| Branch?       | 5.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exists, explain below instead -->
| License       | MIT

When a message is routed via `sync://` we hit the middleware stack twice (meh, but OK).

A side effect is we always get an error log due open transaction, even though it was closed properly. But it's an inner transaction.

This solves it.

Commits
-------

fa661b5 [Messenger] Fix DoctrineOpenTransactionLoggerMiddleware

Author: nicolas-grekas

src/Symfony/Bridge/Doctrine/Messenger/DoctrineOpenTransactionLoggerMiddleware.php

@@ -26,6 +26,8 @@
 class DoctrineOpenTransactionLoggerMiddleware extends AbstractDoctrineMiddleware
 {
     private $logger;
+    /** @var bool */
+    private $isHandling = false;
 
     public function __construct(ManagerRegistry $managerRegistry, string $entityManagerName = null, LoggerInterface $logger = null)
     {
@@ -36,6 +38,12 @@ public function __construct(ManagerRegistry $managerRegistry, string $entityMana
 
     protected function handleForManager(EntityManagerInterface $entityManager, Envelope $envelope, StackInterface $stack): Envelope
     {
+        if ($this->isHandling) {
+            return $stack->next()->handle($envelope, $stack);
+        }
+
+        $this->isHandling = true;
+
         try {
             return $stack->next()->handle($envelope, $stack);
         } finally {
@@ -44,6 +52,7 @@ protected function handleForManager(EntityManagerInterface $entityManager, Envel
                     'message' => $envelope->getMessage(),
                 ]);
             }
+            $this->isHandling = false;
         }
     }
 }