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 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

        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

          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

          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

          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

          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/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

          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

          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

          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

          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

          Headers should be surrounded by blank lines
          Open

          ## Methods
          Severity: Info
          Found in docs/supported-modules/VisitsSummaryModule.md by markdownlint

          MD022 - Headers should be surrounded by blank lines

          Tags: headers, blank_lines

          Aliases: blanks-around-headers

          This rule is triggered when headers (any style) are either not preceded or not followed by a blank line:

          # Header 1
Some text

Some more text
## Header 2

          To fix this, ensure that all headers have a blank line both before and after (except where the header is at the beginning or end of the document):

          # Header 1

Some text

Some more text

## Header 2

          Rationale: Aside from aesthetic reasons, some parsers, including kramdown, will not parse headers that don't have a blank line before, and will parse them as regular text.

          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

          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