MailService::sendEmail() - Code Metrics - Inspection of "remove laminas, use symfony mailer" - delboy1978uk/bone-mail - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 6b1ba5...96756e )

by Derek Stephen

created 2025-10-10 18:05 UTC

MailService::sendEmail() A

↳ Parent: MailService

Complexity

Conditions	1
Paths	1

Size

Total Lines	10
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	7
nc	1
nop	1
dl	0
loc	10
ccs	2
cts	2
cp	1
crap	1
rs	10
c	1
b	0
f	0

<?php declare(strict_types=1);

namespace Bone\Mail\Service;

use Bone\View\ViewEngine;
use Bone\Server\SiteConfig;
use Bone\Server\SiteConfigAwareInterface;
use Bone\Server\Traits\HasSiteConfigTrait;
use Bone\View\Traits\HasViewTrait;
use Bone\View\ViewAwareInterface;
use Bone\Mail\EmailMessage;

use Laminas\Mime\Message;
use Laminas\Mime\Part;
use Symfony\Component\Mailer\Mailer;
use Symfony\Component\Mailer\Transport\TransportInterface;

class MailService implements SiteConfigAwareInterface, ViewAwareInterface
{

    use HasSiteConfigTrait;
    use HasViewTrait;

    private Mailer $mailer;

    public function setMailer(Mailer $mailer): void
    {
        $this->mailer = $mailer;
    }

    private function renderEmail(string $template, array $data): string
    {
        return $this->view->render($template, $data);
    }

    public function sendEmail(EmailMessage $message): bool
    {
        $config = $this->getSiteConfig();
        $message->setFrom($config->getServerEmail(), $config->getCompany());
        $body = $this->renderEmail($message->getTemplate(), $message->getViewData());
        $message->html($body);
        $mailer = new Mailer($this->transport);

        $mailer->send($message);

        return true;
    }
}


1		<?php declare(strict_types=1);
2
3		namespace Bone\Mail\Service;
4
5		use Bone\View\ViewEngine;
6		use Bone\Server\SiteConfig;
7		use Bone\Server\SiteConfigAwareInterface;
8		use Bone\Server\Traits\HasSiteConfigTrait;
9		use Bone\View\Traits\HasViewTrait;
10		use Bone\View\ViewAwareInterface;
11		use Bone\Mail\EmailMessage;
12
13		use Laminas\Mime\Message;
14		use Laminas\Mime\Part;
15		use Symfony\Component\Mailer\Mailer;
16		use Symfony\Component\Mailer\Transport\TransportInterface;
17
18		class MailService implements SiteConfigAwareInterface, ViewAwareInterface
19		{
20
21		use HasSiteConfigTrait;
22		use HasViewTrait;
23
24		private Mailer $mailer;
25
26		public function setMailer(Mailer $mailer): void
27		{
28	2	$this->mailer = $mailer;
29		}
30	2
31		private function renderEmail(string $template, array $data): string
32		{
33		return $this->view->render($template, $data);
34		}
35
36		public function sendEmail(EmailMessage $message): bool
37		{
38	1	$config = $this->getSiteConfig();
39		$message->setFrom($config->getServerEmail(), $config->getCompany());
40	1	$body = $this->renderEmail($message->getTemplate(), $message->getViewData());
41		$message->html($body);
42		$mailer = new Mailer($this->transport);
		0 ignored issues – show Bug Best Practice introduced 2025-10-10 18:18 UTC by Report Bug Copy Issue Report The property `transport` does not exist on `Bone\Mail\Service\MailService`. Did you maybe forget to declare it? Loading history...
43		$mailer->send($message);
44
45		return true;
46		}
47		}
48

delboy1978uk / bone-mail

Push — master ( 6b1ba5...96756e )

MailService::sendEmail() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like