App\Contact\ContactMailer

Complexity

Total Complexity

7

Size/Duplication

Total Lines	63
Duplicated Lines	0 %

Test Coverage

Coverage

67.74%

Importance

Changes	3
Bugs	1	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	12	1
B	send()	0	41	6

<?php

declare(strict_types=1);

namespace App\Contact;

use Exception;
use Psr\Http\Message\ServerRequestInterface;
use Psr\Log\LoggerInterface;
use Yiisoft\Form\FormModelInterface;
use Yiisoft\Mailer\File;
use Yiisoft\Mailer\MailerInterface;
use Yiisoft\Mailer\MessageBodyTemplate;
use Yiisoft\Session\Flash\FlashInterface;

/**
 * ContactMailer sends an email from the contact form
 */
final class ContactMailer
{
    private FlashInterface $flash;
    private LoggerInterface $logger;
    private MailerInterface $mailer;
    private string $sender;
    private string $to;

    public function __construct(
        FlashInterface $flash,
        LoggerInterface $logger,
        MailerInterface $mailer,
        string $sender,
        string $to
    ) {
        $this->flash = $flash;
        $this->logger = $logger;
        $this->mailer = $mailer->withTemplate(new MessageBodyTemplate(__DIR__ . '/mail/'));
        $this->sender = $sender;
        $this->to = $to;
    }

    public function send(FormModelInterface $form, ServerRequestInterface $request)
    {
        $message = $this->mailer->compose(
            'contact-email',
            [
                'content' => $form->getAttributeValue('body'),
            ]
        )
            ->withSubject($form->getAttributeValue('subject'))
            ->withFrom([$form->getAttributeValue('email') => $form->getAttributeValue('name')])
            ->withSender($this->sender)
            ->withTo($this->to);

        $attachFiles = $request->getUploadedFiles();
        foreach ($attachFiles as $attachFile) {
            foreach ($attachFile as $file) {
                if ($file[0]?->getError() === UPLOAD_ERR_OK) {
                    $message = $message->withAttached(
                        File::fromContent(
                            (string)$file->getStream(),
                            $file->getClientFilename(),
                            $file->getClientMediaType()
                        ),
                    );
                }
            }
        }

        try {
            $this->mailer->send($message);
            $flashMsg = 'Thank you for contacting us, we\'ll get in touch with you as soon as possible.';
        } catch (Exception $e) {
            $flashMsg = $e->getMessage();
            $this->logger->error($flashMsg);
        } finally {
            $this->flash->add(
                isset($e) ? 'danger' : 'success',
                [
                    'body' => $flashMsg,

The variable $flashMsg does not seem to be defined for all execution paths leading up to this point.

                ],
                true
            );
        }
    }
}