JDR\Mailer\Part\TemplateResolver

Complexity

Total Complexity

3

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	addPart()	0	6	1
A	render()	0	5	1

<?php declare(strict_types = 1);

namespace JDR\Mailer\Part;

use JDR\Mailer\Email\Email;
use JDR\Mailer\Email\Message as EmailMessage;
use JDR\Mailer\Template\Engine;

class TemplateResolver implements EmailPartResolver
{
    /**
     * @var Engine
     */
    private $templateEngine;

    /**
     * Constructor.
     *
     * @param Engine $templateEngine
     */
    public function __construct(Engine $templateEngine)
    {
        $this->templateEngine = $templateEngine;
    }

    /**
     * {@inheritdoc}
     */
    public function addPart(Email $email, EmailPart $part)
    {
        $body = $this->render($part);

$part of type object<JDR\Mailer\Part\EmailPart> is not a sub-type of object<JDR\Mailer\Part\Template>. It seems like you assume a concrete implementation of the interface JDR\Mailer\Part\EmailPart to be always present.

        $contentType = $part->getContentType();

It seems like you code against a concrete implementation and not the interface JDR\Mailer\Part\EmailPart as the method getContentType() does only exist in the following implementations of said interface: JDR\Mailer\Part\Message, JDR\Mailer\Part\Template.

        $email->addMessage(new EmailMessage($contentType, $body));
    }

    /**
     * Render the template.
     *
     * @param Template $part
     *
     * @return string
     */
    private function render(Template $part)
    {
        return $this->templateEngine
            ->render($part->getPath(), $part->getParameters());
    }
}