SfCod\EmailEngineBundle\Example\Params\TestMessage

Complexity

Total Complexity

3

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getName()	0	4	1
A	getValue()	0	4	1
A	getDescription()	0	4	1

<?php

namespace SfCod\EmailEngineBundle\Example\Params;

use SfCod\EmailEngineBundle\Example\TestTemplateOptions;
use SfCod\EmailEngineBundle\Template\Params\ParameterInterface;
use SfCod\EmailEngineBundle\Template\TemplateOptionsInterface;

/**
 * Class TestMessage
 *
 * @author Virchenko Maksim <muslim1992@gmail.com>
 *
 * @package SfCod\EmailEngineBundle\Example\Params
 */
class TestMessage implements ParameterInterface
{
    /**
     * Get parameter name
     *
     * @return string
     */
    public static function getName(): string
    {
        return 'message';
    }

    /**
     * Get parameter value
     *
     * @param TestTemplateOptions|TemplateOptionsInterface $options
     *
     * @return mixed
     */
    public function getValue(TemplateOptionsInterface $options)
    {
        return '<b>' . $options->message . '</b>';

Accessing message on the interface SfCod\EmailEngineBundle\Template\TemplateOptionsInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

    }

    /**
     * Get parameter description
     *
     * @return string
     */
    public static function getDescription(): string
    {
        return 'Test message.';
    }
}