Groundskeeper\Tokens\DocType

Complexity

Total Complexity

5

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

92.31%

Importance

Changes	2
Bugs	0	Features	2

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	validate()	0	8	2
A	toString()	0	8	2

<?php

namespace Groundskeeper\Tokens;

use Groundskeeper\Configuration;

class DocType extends AbstractValuedToken
{
    /**
     * Constructor
     */
    public function __construct(Token $parent = null, $value = null)
    {
        parent::__construct(Token::DOCTYPE, $parent, $value);
    }

    public function validate(Configuration $configuration)
    {
        parent::validate($configuration);

        if ($this->isValid === true) {
            $this->isValid = $this->getParent() === null;

The property $isValid was declared of type integer, but $this->getParent() === null is of type boolean. Maybe add a type cast?

        }
    }

    public function toString(Configuration $configuration, $prefix = '', $suffix = '')
    {
        if (!$this->isValid) {
            return '';
        }

        return $prefix . '<!DOCTYPE ' . $this->getValue() . '>' . $suffix;
    }
}