Gskema\TypeSniff\Sniffs\CodeElement\FqcnDescriptionSniff

Complexity

Total Complexity

13

Size/Duplication

Total Lines	69
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
B	process()	0	16	9
A	register()	0	6	1
A	configure()	0	14	3

<?php

namespace Gskema\TypeSniff\Sniffs\CodeElement;

use Gskema\TypeSniff\Core\CodeElement\Element\AbstractFqcnElement;
use Gskema\TypeSniff\Core\CodeElement\Element\ClassElement;
use Gskema\TypeSniff\Core\CodeElement\Element\CodeElementInterface;
use Gskema\TypeSniff\Core\CodeElement\Element\InterfaceElement;
use Gskema\TypeSniff\Core\CodeElement\Element\TraitElement;
use Gskema\TypeSniff\Core\SniffHelper;
use PHP_CodeSniffer\Files\File;

class FqcnDescriptionSniff implements CodeElementSniffInterface
{
    protected const CODE = 'FqcnDescriptionSniff';

    /** @var string[] */
    protected array $invalidPatterns = [
        '^(Class|Trait|Interface)\s+\w+\s*$',
    ];

    /** @var string[] */
    protected array $invalidTags = [
        '@package',
    ];

    protected string $reportType = 'warning';

    protected bool $addViolationId = false;

    /**
     * @inheritDoc
     */
    public function configure(array $config): void
    {
        $this->invalidPatterns = array_merge($this->invalidPatterns, $config['invalidPatterns'] ?? []);
        foreach ($this->invalidPatterns as &$invalidPattern) {
            $invalidPattern = '#' . $invalidPattern . '#i';
        }

        $this->invalidTags = array_merge($this->invalidTags, $config['invalidTags'] ?? []);
        foreach ($this->invalidTags as &$invalidTag) {
            $invalidTag = substr($invalidTag, 1);
        }

        $this->reportType = (string)($config['reportType'] ?? 'warning');
        $this->addViolationId = (bool)($config['addViolationId'] ?? false);
    }

    /**
     * @inheritDoc
     */
    public function register(): array
    {
        return [
            ClassElement::class,
            InterfaceElement::class,
            TraitElement::class,
        ];
    }

    /**
     * @inheritDoc
     * @param AbstractFqcnElement $element
     */
    public function process(File $file, CodeElementInterface $element, CodeElementInterface $parentElement): void
    {
        foreach ($element->getDocBlock()->getDescriptionLines() as $lineNum => $descriptionLine) {
            foreach ($this->invalidPatterns as $invalidPattern) {
                if (preg_match($invalidPattern, $descriptionLine)) {
                    $originId = $this->addViolationId ? $element->getFqcn() . $descriptionLine : null;

The method getFqcn() does not exist on Gskema\TypeSniff\Core\CodeElement\Element\CodeElementInterface. It seems like you code against a sub-type of Gskema\TypeSniff\Core\CodeElement\Element\CodeElementInterface such as Gskema\TypeSniff\Core\CodeElement\Element\AbstractFqcnElement.

                    SniffHelper::addViolation($file, 'Useless description', $lineNum, static::CODE, $this->reportType, $originId);
                }
            }
        }

        foreach ($element->getDocBlock()->getTags() as $tag) {
            foreach ($this->invalidTags as $invalidTagName) {
                if ($tag->getName() === $invalidTagName) {
                    $originId = $this->addViolationId ? $element->getFqcn() . $invalidTagName : null;
                    SniffHelper::addViolation($file, 'Useless tag', $tag->getLine(), static::CODE, $this->reportType, $originId);
                }
            }
        }
    }
}