Vanderlee\Comprehend\Parser\Terminal\CaseSensitiveTrait

Complexity

Total Complexity

7

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	caseSensitive()	0	5	1
A	caseInsensitive()	0	5	1
A	popCaseSensitivityFromContext()	0	4	2
A	setCaseSensitivity()	0	5	1
A	pushCaseSensitivityToContext()	0	4	2

<?php

namespace Vanderlee\Comprehend\Parser\Terminal;

use Vanderlee\Comprehend\Core\Context;

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

    /**
     * @var boolean
     */
    private $caseSensitivity = null;

    private function pushCaseSensitivityToContext(Context $context)
    {
        if ($this->caseSensitivity !== null) {
            $context->pushCaseSensitivity($this->caseSensitivity);
        }
    }

    private function popCaseSensitivityFromContext(Context $context)
    {
        if ($this->caseSensitivity !== null) {
            $context->popCaseSensitivity();
        }
    }

    public function setCaseSensitivity(string $preference)
    {
        $this->caseSensitivity = $preference;

The property $caseSensitivity was declared of type boolean, but $preference is of type string. Maybe add a type cast?

        return $this;
    }

    public function caseSensitive()
    {
        $this->caseSensitivity = true;

        return $this;
    }

    public function caseInsensitive()
    {
        $this->caseSensitivity = false;

        return $this;
    }

}