final-notification-transport-email-decorator.php

<?php

declare(strict_types=1);

namespace App\Notification\Implementation;

use App\Notification\NotificationInterface;
use App\Notification\TransportInterface;

/**
 * {@inheritDoc}
 *
 * @template-implements TransportInterface<TransportInterface>
 */
final class RepeatableTransportDecorator implements TransportInterface
{
    private TransportInterface $transport;
    private int $retryCount;
    
    public function __construct(TransportInterface $transport, int $retryCount)
    {
        $this->transport = $transport;
        $this->retryCount = $retryCount;
    }

    public function supports(NotificationInterface $notification): bool
    {
        return $this->transport->supports($notification);
    }
    
    /**
     * {@inheritDoc}
     */
    public function deliver(NotificationInterface $notification): void
    {
        $tryCounter = 0;
        
        do {
            try {
                $this->transport->deliver($notification);

                return;
            } catch (\Exception $exception) {
                ++$tryCounter;
            }
        } while ($tryCounter < $this->retryCount);
        
        throw NotificationSendException::duringDelivery($exception);
    }
}

Fix:

public function deliver(NotificationInterface $notification): void
    {
        $tryCounter = 0;
        
        do {
            try {
                $this->transport->deliver($notification);

                return;
            } catch (\Exception $exception) {
                ++$tryCounter;
            }
        } while ($tryCounter < $this->retryCount);
        
        throw NotificationSendException::duringDelivery($exception);
    }

Thx