Sign upLogin

t1gor/Robots.txt-Parser-Class

View on GitHub
Star
source/Parser/TreeBuilder.php

Summary

Maintainability
A
1 hr
Test Coverage

Showing 80 of 80 total issues

Method processDirective has 5 arguments (exceeds 4 allowed). Consider refactoring.
Open

    protected function processDirective(string $directive, string $line, &$tree, string &$userAgent, string $prevLine = '') {
Severity: Minor
Found in source/Parser/TreeBuilder.php - About 35 mins to fix

    Function build has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.
    Open

        public function build(): array {
            $currentUserAgent = '*';
            $tree             = [];
            $prevLine         = '';
     
     
    Severity: Minor
    Found in source/Parser/TreeBuilder.php - About 35 mins to fix

    Opening brace of a class must be on the line after the definition
    Open

    class TreeBuilder implements TreeBuilderInterface {
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

         */
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

        public function getContent(): \Iterator {
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

                            $processor->getDirectiveName(),
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

            if (empty($processors)) {
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

        /**
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

                $this->log(strtr('{directive} met, but no processor found for it. Skipping.', [
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

        }
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

            return $tree;
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

        }
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

                            $tree,
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

                    }
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

                $prevLine = $line;
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

            $this->logger = $logger;
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

            }
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

            if (!isset($this->processors[$directive])) {
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

         * @todo check for multibyte support?
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

                return;
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

         */
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

            $this->log('Building directives tree...');
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

                // override
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

            foreach ($processors as $processor) {
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

        }
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

        }
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

            foreach ($this->content as $line) {
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

         * @var DirectiveProcessorInterface[]
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

        }
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

         * @param \Iterator $content
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

        /**
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

         */
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

        public function build(): array {
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

                            $currentUserAgent,
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

            }
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

         * @param LoggerInterface|null          $logger
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

            // reformat processors
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

                $this->processors[$processor->getDirectiveName()] = $processor;
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

         * Wrapper to check that processor is available
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

            $currentUserAgent = '*';
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

                            $prevLine
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

                        );
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Line exceeds 120 characters; contains 125 characters
    Open

        protected function processDirective(string $directive, string $line, &$tree, string &$userAgent, string $prevLine = '') {
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

         * @return \Iterator
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

        public function setContent(\Iterator $content): void {
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

         * @return array
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

                        $this->processDirective(
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

        public function __construct(array $processors, ?LoggerInterface $logger) {
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

        /**
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

            $tree             = [];
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

                            $line,
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

                        break;
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

                }
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

        protected array     $processors;
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

            }
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

            $this->processors[$directive]->process($line, $tree, $userAgent, $prevLine);
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

        /**
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

         */
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

                    if ($processor->matches($line)) {
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

        use LogsIfAvailableTrait;
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

        /**
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

        /**
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

        protected function processDirective(string $directive, string $line, &$tree, string &$userAgent, string $prevLine = '') {
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

                    '{directive}' => $directive,
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

                foreach ($this->processors as $processor) {
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

        protected \Iterator $content;
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

         * @param DirectiveProcessorInterface[] $processors
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

                $this->log("Seems like you've passed an empty processors array.", [], LogLevel::WARNING);
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

            return $this->content;
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

            $prevLine         = '';
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

         */
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

         */
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

            }
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

                ]));
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Spaces must be used to indent lines; tabs are not allowed
    Open

            $this->content = $content;
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Opening brace should be on a new line
    Open

        public function __construct(array $processors, ?LoggerInterface $logger) {
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Opening brace should be on a new line
    Open

        public function build(): array {
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Opening brace should be on a new line
    Open

        protected function processDirective(string $directive, string $line, &$tree, string &$userAgent, string $prevLine = '') {
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Opening brace should be on a new line
    Open

        public function setContent(\Iterator $content): void {
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    Opening brace should be on a new line
    Open

        public function getContent(): \Iterator {
    Severity: Minor
    Found in source/Parser/TreeBuilder.php by phpcodesniffer

    There are no issues that match your filters.

    Category
    Status