Sign upLogin

pluf/shop

View on GitHub
Star
src/Notifier/Engine/NoNotify.php

Summary

Maintainability
A
0 mins
Test Coverage

Avoid unused parameters such as '$notification'.
Open

    public function send($notification)
Severity: Minor
Found in src/Notifier/Engine/NoNotify.php by phpmd

UnusedFormalParameter

Since: 0.2

Avoid passing parameters to methods or constructors and then not using those parameters.

Example

class Foo
{
    private function bar($howdy)
    {
        // $howdy is not used
    }
}

Source https://phpmd.org/rules/unusedcode.html#unusedformalparameter

Each class must be in a namespace of at least one level (a top-level vendor name)
Open

class Notifier_Engine_NoNotify extends Notifier_Engine
Severity: Minor
Found in src/Notifier/Engine/NoNotify.php by phpcodesniffer

The class Notifier_Engine_NoNotify is not named in CamelCase.
Open

class Notifier_Engine_NoNotify extends Notifier_Engine
{

    /*
     *
Severity: Minor
Found in src/Notifier/Engine/NoNotify.php by phpmd

CamelCaseClassName

Since: 0.2

It is considered best practice to use the CamelCase notation to name classes.

Example

class class_name {
}

Source

Class name "Notifier_Engine_NoNotify" is not in camel caps format
Open

class Notifier_Engine_NoNotify extends Notifier_Engine
Severity: Minor
Found in src/Notifier/Engine/NoNotify.php by phpcodesniffer

Whitespace found at end of line
Open

 *
Severity: Minor
Found in src/Notifier/Engine/NoNotify.php by phpcodesniffer

There are no issues that match your filters.

Category
Status