Sign upLogin

t1gor/Robots.txt-Parser-Class

View on GitHub
Star

Showing 1,364 of 1,364 total issues

Avoid variables with short names like $in. Configured minimum length is 3.
Open

    public function filter($in, $out, &$consumed, $closing) {
Severity: Minor
Found in source/Stream/Filters/SkipEndOfCommentedLineFilter.php by phpmd

Avoid variables with short names like $a. Configured minimum length is 3.
Open

                    usort($value, function ($a, $b) {
Severity: Minor
Found in source/RobotsTxtParser.php by phpmd

Avoid variables with short names like $in. Configured minimum length is 3.
Open

    public function filter($in, $out, &$consumed, $closing);
Severity: Minor
Found in source/Stream/CustomFilterInterface.php by phpmd

Avoid variables with short names like $in. Configured minimum length is 3.
Open

    public function __construct(string $in) {
Severity: Minor
Found in source/Parser/Url.php by phpmd

Avoid variables with short names like $in. Configured minimum length is 3.
Open

    public function filter($in, $out, &$consumed, $closing) {
Severity: Minor
Found in source/Stream/Filters/TrimSpacesLeftFilter.php by phpmd

Avoid variables with short names like $in. Configured minimum length is 3.
Open

    public function filter($in, $out, &$consumed, $closing) {
Severity: Minor
Found in source/Stream/Filters/SkipUnsupportedDirectivesFilter.php by phpmd

Function parse has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.
Open

    protected function parse(string $url) {
        $parsed = parse_url($url);
 
        if ($parsed === false) {
            $this->log("Failed to parse URL from {$url}");
Severity: Minor
Found in source/Parser/Url.php - About 45 mins to fix

The class RobotsTxtParser has a coupling between objects value of 15. Consider to reduce the number of dependencies under 13.
Open

class RobotsTxtParser implements LoggerAwareInterface {
 
    use LogsIfAvailableTrait;
 
    // default encoding
Severity: Minor
Found in source/RobotsTxtParser.php by phpmd

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

        $content,
        string $encoding = self::DEFAULT_ENCODING,
        ?TreeBuilderInterface $treeBuilder = null,
        ?ReaderInterface $reader = null,
        ?UserAgentMatcherInterface $userAgentMatcher = null
Severity: Minor
Found in source/RobotsTxtParser.php - About 35 mins to fix

    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

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

      class DisallowProcessor extends AbstractAllowanceProcessor implements DirectiveProcessorInterface {
      Severity: Minor
      Found in source/Parser/DirectiveProcessors/DisallowProcessor.php by phpcodesniffer

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

      class SitemapProcessor extends AbstractDirectiveProcessor implements DirectiveProcessorInterface {
      Severity: Minor
      Found in source/Parser/DirectiveProcessors/SitemapProcessor.php by phpcodesniffer

      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 CleanParamProcessor extends AbstractDirectiveProcessor implements DirectiveProcessorInterface {
      Severity: Minor
      Found in source/Parser/DirectiveProcessors/CleanParamProcessor.php by phpcodesniffer

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

      abstract class AbstractDirectiveProcessor implements DirectiveProcessorInterface {
      Severity: Minor
      Found in source/Parser/DirectiveProcessors/AbstractDirectiveProcessor.php by phpcodesniffer

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

          public function getHost(?string $userAgent = null) {
              $this->buildTree();
       
              if (!is_null($userAgent)) {
                  $userAgent = $this->userAgentMatcher->getMatching($userAgent, array_keys($this->tree));
      Severity: Minor
      Found in source/RobotsTxtParser.php - About 35 mins to fix

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

      class AllowProcessor extends AbstractAllowanceProcessor implements DirectiveProcessorInterface {
      Severity: Minor
      Found in source/Parser/DirectiveProcessors/AllowProcessor.php by phpcodesniffer

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

      class HostProcessor extends AbstractDirectiveProcessor implements DirectiveProcessorInterface {
      Severity: Minor
      Found in source/Parser/DirectiveProcessors/HostProcessor.php by phpcodesniffer

      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

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

      trait LogsIfAvailableTrait {
      Severity: Minor
      Found in source/LogsIfAvailableTrait.php by phpcodesniffer
      Severity
      Category
      Status
      Source
      Language