Mailcode_Traits_Commands_Validation_URLDecode::validateSyntax_urldecode()

Complexity

Conditions	3
Paths	3

Size

Total Lines	10
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	Mailcode_Traits_Commands_Validation_URLDecode::setURLDecoding()	0	3	1

<?php
/**
 * File containing the {@see \Mailcode\Mailcode_Traits_Commands_Validation_URLDecode} trait.
 *
 * @package Mailcode
 * @subpackage Validation
 * @see \Mailcode\Mailcode_Traits_Commands_Validation_URLDecode
 */

declare(strict_types=1);

namespace Mailcode;

use Mailcode\Commands\ParamsException;

/**
 * Command validation drop-in: checks for the presence
 * of a "urldecode:" keyword.
 *
 * @package Mailcode
 * @subpackage Validation
 * @author Sebastian Mordziol <s.mordziol@mistralys.eu>
 *
 * @see Mailcode_Interfaces_Commands_Validation_URLDecode
 */
trait Mailcode_Traits_Commands_Validation_URLDecode
{
    public function getURLDecodeToken() : ?Mailcode_Parser_Statement_Tokenizer_Token_Keyword
    {
        return $this->getEncodingToken(Mailcode_Commands_Keywords::TYPE_URLDECODE);

It seems like getEncodingToken() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * @param bool $decode
     * @return $this
     * @throws Mailcode_Exception
     */
    public function setURLDecoding(bool $decode=true) : self
    {
        return $this->setEncodingEnabled(Mailcode_Commands_Keywords::TYPE_URLDECODE, $decode);

It seems like setEncodingEnabled() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    public function isURLDecoded() : bool
    {
        return $this->getURLDecodeToken() !== null;
    }
}