Sign upLogin

t1gor/Robots.txt-Parser-Class

View on GitHub
Star

Showing 1,364 of 1,364 total issues

Opening brace should be on a new line
Open

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

Opening brace should be on a new line
Open

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

Opening brace should be on a new line
Open

    protected function __construct() {
Severity: Minor
Found in source/Stream/GeneratorBasedReader.php by phpcodesniffer

Opening brace should be on a new line
Open

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

Opening brace should be on a new line
Open

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

Opening brace should be on a new line
Open

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

Opening brace should be on a new line
Open

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

Opening brace should be on a new line
Open

    public function getContentIterated(): \Generator {
Severity: Minor
Found in source/Stream/GeneratorBasedReader.php by phpcodesniffer

Opening brace should be on a new line
Open

    public function getContentRaw(): string {
Severity: Minor
Found in source/Stream/GeneratorBasedReader.php by phpcodesniffer

Opening brace should be on a new line
Open

    public static function fromStream($stream): self {
Severity: Minor
Found in source/Stream/GeneratorBasedReader.php by phpcodesniffer

Opening brace should be on a new line
Open

    public static function fromString(string $input = ''): self {
Severity: Minor
Found in source/Stream/GeneratorBasedReader.php by phpcodesniffer

Opening brace should be on a new line
Open

    public function __destruct() {
Severity: Minor
Found in source/Stream/GeneratorBasedReader.php by phpcodesniffer

Opening brace should be on a new line
Open

    public function setEncoding(string $encoding) {
Severity: Minor
Found in source/Stream/GeneratorBasedReader.php by phpcodesniffer

Opening brace should be on a new line
Open

    protected function setStream($stream): GeneratorBasedReader {
Severity: Minor
Found in source/Stream/GeneratorBasedReader.php by phpcodesniffer

The variable $error_msg is not named in camelCase.
Open

    private function checkHostRule($rule) {
        if (!isset($this->url)) {
            $error_msg = WarmingMessages::INLINED_HOST;
            $this->log($error_msg, [], LogLevel::ERROR);
            return false;
Severity: Minor
Found in source/RobotsTxtParser.php by phpmd

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

The variable $error_msg is not named in camelCase.
Open

    private function checkHostRule($rule) {
        if (!isset($this->url)) {
            $error_msg = WarmingMessages::INLINED_HOST;
            $this->log($error_msg, [], LogLevel::ERROR);
            return false;
Severity: Minor
Found in source/RobotsTxtParser.php by phpmd

CamelCaseVariableName

Since: 0.2

It is considered best practice to use the camelCase notation to name variables.

Example

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

Source

Define a constant instead of duplicating this literal "logger" 3 times.
Open

                && isset($this->params['logger'])
Severity: Critical
Found in source/Stream/Filters/SkipEndOfCommentedLineFilter.php by sonar-php

Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.

On the other hand, constants can be referenced from many places, but only need to be updated in a single place.

Noncompliant Code Example

With the default threshold of 3:

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

Compliant Solution

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Exceptions

To prevent generating some false-positives, literals having less than 5 characters are excluded.

Define a constant instead of duplicating this literal "logger" 3 times.
Open

                && isset($this->params['logger'])
Severity: Critical
Found in source/Stream/Filters/SkipCommentedLinesFilter.php by sonar-php

Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.

On the other hand, constants can be referenced from many places, but only need to be updated in a single place.

Noncompliant Code Example

With the default threshold of 3:

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

Compliant Solution

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Exceptions

To prevent generating some false-positives, literals having less than 5 characters are excluded.

Define a constant instead of duplicating this literal "{useragent}" 3 times.
Open

                '{useragent}' => $currentUserAgent,
Severity: Critical
Found in source/Parser/DirectiveProcessors/AbstractAllowanceProcessor.php by sonar-php

Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.

On the other hand, constants can be referenced from many places, but only need to be updated in a single place.

Noncompliant Code Example

With the default threshold of 3:

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

Compliant Solution

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Exceptions

To prevent generating some false-positives, literals having less than 5 characters are excluded.

Define a constant instead of duplicating this literal "logger" 5 times.
Open

            if (isset($this->params['logger']) && $this->params['logger'] instanceof LoggerInterface) {
Severity: Critical
Found in source/Stream/Filters/SkipDirectivesWithInvalidValuesFilter.php by sonar-php

Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.

On the other hand, constants can be referenced from many places, but only need to be updated in a single place.

Noncompliant Code Example

With the default threshold of 3:

function run() {
  prepare('action1');                              // Non-Compliant - 'action1' is duplicated 3 times
  execute('action1');
  release('action1');
}

Compliant Solution

ACTION_1 = 'action1';

function run() {
  prepare(ACTION_1);
  execute(ACTION_1);
  release(ACTION_1);
}

Exceptions

To prevent generating some false-positives, literals having less than 5 characters are excluded.

Severity
Category
Status
Source
Language