Make it possible to instantiate MailerExtender like Mailer?

[sypets]

It would be more convenient to have same constructor as Mailer.

This extension overrides Mailer (via Xclass) but has different constructor. This way, it is not easily possible to instantiate it via DI.

TYPO3 Mailer:

public function __construct(
        protected ?TransportInterface $transport = null,
        protected readonly ?EventDispatcherInterface $eventDispatcher = null,
    ) 

mail_logger: MailerExtender:

 public function __construct(
        protected LoggingTransportFactory $loggingTransportFactory,
        ?TransportInterface $transport = null,
        ?EventDispatcherInterface $eventDispatcher = null,
    ) {

Or for example, if you instantiate Mailer like this:

$mailer = GeneralUtility::makeInstance(Mailer::class);

you get exception because of not passed argument $loggingTransportFactory. I use Mailer in several extensions and had to add checks for mail_logger.

[sypets]

I am currently using this:

 protected function instantiateMailer(): Mailer
    {
        $typo3Version = GeneralUtility::makeInstance(Typo3Version::class);
        if ($typo3Version->getMajorVersion() > 12
            && ExtensionManagementUtility::isLoaded('mail_logger')
            && isset($GLOBALS['TYPO3_CONF_VARS']['SYS']['Objects'][Mailer::class]['className'])
            /**
             * even in a plain PHP file where it works, ::class doesn't require the class to exist - it just
             * resolves the name based on the current namespace and use statements, and substitutes in the resulting string
             *
             * Use string here because otherwise phpstan will complain.
             */
            && $GLOBALS['TYPO3_CONF_VARS']['SYS']['Objects'][Mailer::class]['className'] === 'Pluswerk\\MailLogger\\Logging\\MailerExtender'
        ) {
            /** @phpstan-ignore-next-line  */
            $loggingTransportFactory = GeneralUtility::makeInstance('Pluswerk\\MailLogger\\Logging\\LoggingTransportFactory');
            /** @phpstan-ignore-next-line  */
            if (!$loggingTransportFactory) {
                $this->logger->error('Could not instantiate pluswerk/mail_logger: LoggingTransportFactory');
                throw new \RuntimeException('Could not instantiate pluswerk/mail_logger: LoggingTransportFactory');
            }
            $mailer = GeneralUtility::makeInstance(Mailer::class, $loggingTransportFactory);
        } else {
            $mailer = GeneralUtility::makeInstance(Mailer::class);
        }
        return $mailer;
    }