SimpleSAML\Module\ldapPasswordReset\MagicLink::sendMagicLink()

Complexity

Conditions	1
Paths	1

Size

Total Lines	14
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function sendMagicLink(string $email, string $token, int $validUntil): void
    {
        $url = Module::getModuleURL('ldapPasswordReset/validateMagicLink', ['t' => $token]);

        $mail = new Utils\EMail(
            $this->moduleConfig->getOptionalString('email.subject', 'Password reset'),

It seems like $this->moduleConfig->getOptionalString('email.subject', 'Password reset') can also be of type null; however, parameter $subject of SimpleSAML\Utils\EMail::__construct() does only seem to accept string, maybe add an additional type check?

            $this->moduleConfig->getOptionalString('email.from', $this->config->getString('technicalcontact_email')),
            $email,
            'ldapPasswordReset:mailtxt.twig',
            'ldapPasswordReset:mailhtml.twig'
        );
        $mail->setData(['url' => $url, 'validUntil' => $validUntil]);
        $mail->setText('{url}');
        $mail->send();
    }