Sign upLogin

RobBrazier/Laravel_Piwik

View on GitHub
Star

Showing 302 of 309 total issues

Piwik has 32 functions (exceeds 20 allowed). Consider refactoring.
Open

class Piwik
{
    use ConfigTrait {
        ConfigTrait::__construct as private __configConstruct;
    }
Severity: Minor
Found in src/RobBrazier/Piwik/Piwik.php - About 4 hrs to fix

    Method getParsedLastVisitsDetails has 27 lines of code (exceeds 25 allowed). Consider refactoring.
    Open

        private function getParsedLastVisitsDetails(array $visits): array
    {
        $result = [];
        foreach ($visits as $visit) {
            $actionDetails = (array) $this->safeGetArrayEntry($visit, 'actionDetails');
    Severity: Minor
    Found in src/RobBrazier/Piwik/Module/LiveModule.php - About 1 hr to fix

      Method addUser has 6 arguments (exceeds 4 allowed). Consider refactoring.
      Open

          public function addUser($userLogin, $password, $email, $alias = null, $initialIdSite = null, $format = null)
      Severity: Minor
      Found in src/RobBrazier/Piwik/Module/UsersManagerModule.php - About 45 mins to fix

        Method createAppSpecificTokenAuth has 6 arguments (exceeds 4 allowed). Consider refactoring.
        Open

            public function createAppSpecificTokenAuth($userLogin, $password, $description, $expireDate = '', $expireHours = 0, $format = null)
        Severity: Minor
        Found in src/RobBrazier/Piwik/Module/UsersManagerModule.php - About 45 mins to fix

          There must be one blank line after the namespace declaration
          Open

          namespace RobBrazier\Piwik\Module;
          Severity: Minor
          Found in src/RobBrazier/Piwik/Module/APIModule.php by phpcodesniffer

          Function closing brace must go on the next line following the body; found 1 blank lines before brace
          Open

              }
          Severity: Minor
          Found in src/RobBrazier/Piwik/Module/UsersManagerModule.php by phpcodesniffer

          There must be one blank line after the namespace declaration
          Open

          namespace RobBrazier\Piwik\Module;
          Severity: Minor
          Found in src/RobBrazier/Piwik/Module/ReferrersModule.php by phpcodesniffer

          There must be one blank line after the namespace declaration
          Open

          namespace RobBrazier\Piwik\Module;
          Severity: Minor
          Found in src/RobBrazier/Piwik/Module/SitesManagerModule.php by phpcodesniffer

          There must be one blank line after the namespace declaration
          Open

          namespace RobBrazier\Piwik\Module;
          Severity: Minor
          Found in src/RobBrazier/Piwik/Module/UsersManagerModule.php by phpcodesniffer

          There must be one blank line after the namespace declaration
          Open

          namespace RobBrazier\Piwik\Module;
          Severity: Minor
          Found in src/RobBrazier/Piwik/Module/ActionsModule.php by phpcodesniffer

          Function closing brace must go on the next line following the body; found 1 blank lines before brace
          Open

              }
          Severity: Minor
          Found in src/RobBrazier/Piwik/Module/UsersManagerModule.php by phpcodesniffer

          The closing brace for the trait must go on the next line after the body
          Open

          }
          Severity: Minor
          Found in src/RobBrazier/Piwik/Traits/ArrayAccessTrait.php by phpcodesniffer

          Spaces must be used to indent lines; tabs are not allowed
          Open
          Severity: Minor
          Found in src/RobBrazier/Piwik/Module/UsersManagerModule.php by phpcodesniffer

          Line exceeds 120 characters; contains 121 characters
          Open

               * @param string $access     user access role ({@see https://matomo.org/docs/manage-users/#advanced-user-management})
          Severity: Minor
          Found in src/RobBrazier/Piwik/Module/UsersManagerModule.php by phpcodesniffer

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

              /**
          Severity: Minor
          Found in src/RobBrazier/Piwik/Module/UsersManagerModule.php by phpcodesniffer

          Line exceeds 120 characters; contains 135 characters
          Open

              public function createAppSpecificTokenAuth($userLogin, $password, $description, $expireDate = '', $expireHours = 0, $format = null)
          Severity: Minor
          Found in src/RobBrazier/Piwik/Module/UsersManagerModule.php by phpcodesniffer

          Multiple headers with the same content
          Open

          Severity: Info
          Found in docs/changelog.md by markdownlint

          MD024 - Multiple headers with the same content

          Tags: headers

          Aliases: no-duplicate-header

          Parameters: allowdifferentnesting (boolean; default false)

          This rule is triggered if there are multiple headers in the document that have the same text:

          # Some text

## Some text

          To fix this, ensure that the content of each header is different:

          # Some text

## Some more text

          Rationale: Some markdown parses generate anchors for headers based on the header name, and having headers with the same content can cause problems with this.

          If the parameter allow_different_nesting is set to true, header duplication under different nesting is allowed, like it usually happens in change logs:

          # Change log

## 2.0.0

### Bug fixes

### Features

## 1.0.0

### Bug fixes

          Trailing spaces
          Open

          Unless otherwise specified, the methods will accept the following arguments:
          Severity: Info
          Found in docs/supported-modules/index.md by markdownlint

          MD009 - Trailing spaces

          Tags: whitespace

          Aliases: no-trailing-spaces

          Parameters: br_spaces (number; default: 0)

          This rule is triggered on any lines that end with whitespace. To fix this, find the line that is triggered and remove any trailing spaces from the end.

          The brspaces parameter allows an exception to this rule for a specific amount of trailing spaces used to insert an explicit line break/br element. For example, set brspaces to 2 to allow exactly 2 spaces at the end of a line.

          Note: you have to set brspaces to 2 or higher for this exception to take effect - you can't insert a br element with just a single trailing space, so if you set brspaces to 1, the exception will be disabled, just as if it was set to the default of 0.

          Multiple headers with the same content
          Open

          #### Verbose Example
          Severity: Info
          Found in docs/supported-modules/ActionsModule.md by markdownlint

          MD024 - Multiple headers with the same content

          Tags: headers

          Aliases: no-duplicate-header

          Parameters: allowdifferentnesting (boolean; default false)

          This rule is triggered if there are multiple headers in the document that have the same text:

          # Some text

## Some text

          To fix this, ensure that the content of each header is different:

          # Some text

## Some more text

          Rationale: Some markdown parses generate anchors for headers based on the header name, and having headers with the same content can cause problems with this.

          If the parameter allow_different_nesting is set to true, header duplication under different nesting is allowed, like it usually happens in change logs:

          # Change log

## 2.0.0

### Bug fixes

### Features

## 1.0.0

### Bug fixes

          Multiple headers with the same content
          Open

          #### Parameters
          Severity: Info
          Found in docs/supported-modules/ActionsModule.md by markdownlint

          MD024 - Multiple headers with the same content

          Tags: headers

          Aliases: no-duplicate-header

          Parameters: allowdifferentnesting (boolean; default false)

          This rule is triggered if there are multiple headers in the document that have the same text:

          # Some text

## Some text

          To fix this, ensure that the content of each header is different:

          # Some text

## Some more text

          Rationale: Some markdown parses generate anchors for headers based on the header name, and having headers with the same content can cause problems with this.

          If the parameter allow_different_nesting is set to true, header duplication under different nesting is allowed, like it usually happens in change logs:

          # Change log

## 2.0.0

### Bug fixes

### Features

## 1.0.0

### Bug fixes
          Severity
          Category
          Status
          Source
          Language