Vanderlee\Comprehend\Parser\Structure\SpacingTrait

Complexity

Total Complexity

8

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Test Coverage

Coverage

100%

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 !== false is always true.

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

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

The condition $this->spacer !== false is always true.

            $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;
    }
}