Vanderlee\Comprehend\Parser\Structure\SpacingTrait

Complexity

Total Complexity

8

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	popSpacer()	0	4	2
A	pushSpacer()	0	4	2
A	spacing()	0	11	4

<?php

namespace Vanderlee\Comprehend\Parser\Structure;

use Vanderlee\Comprehend\Core\ArgumentsTrait;
use Vanderlee\Comprehend\Core\Context;
use Vanderlee\Comprehend\Parser\Parser;

/**
 * Classes implementing this can scan
 *
 * @author Martijn
 */
trait SpacingTrait
{

    use ArgumentsTrait;

    /**
     * Parser used for scanning the text
     * @var Parser
     */
    private $spacer = false;

    private function pushSpacer(Context $context)
    {
        if ($this->spacer !== false) {

The condition $this->spacer is always true. If $this->spacer can have other possible types, add them to src/Parser/Structure/SpacingTrait.php:20

            $context->pushSpacer($this->spacer);
        }
    }

    private function popSpacer(Context $context)
    {
        if ($this->spacer !== false) {

The condition $this->spacer is always true. If $this->spacer can have other possible types, add them to src/Parser/Structure/SpacingTrait.php:20

            $context->popSpacer();
        }
    }

    /**
     * Set a spacing parser for this parser or disable or enable (if a previous
     * spacing parser is enabled) spacing parsing.
     *
     * @param Parser|bool|null $spacer
     * @return $this
     */
    public function spacing($spacer = true)
    {
        if ($spacer === true) {
            $this->spacer = true;

It seems like true of type true is incompatible with the declared type Vanderlee\Comprehend\Parser\Parser of property $spacer.

        } elseif ($spacer === null || $spacer === false) {

The condition $spacer === false is always true.

            $this->spacer = null;
        } else {
            $this->spacer = self::getArgument($spacer);
        }

        return $this;
    }

}