SAREhub/PHP_Client

View on GitHub
src/SAREhub/Client/Amqp/AmqpChannelWrapperState.php

Summary

Maintainability
A
0 mins
Test Coverage

Missing class import via use statement (line '143', column '19').
Open

        throw new \InvalidArgumentException("consumer with tag: '$consumerTag' is not registered");

MissingImport

Since: 2.7.0

Importing all external classes in a file through use statements makes them clearly visible.

Example

function make() {
    return new \stdClass();
}

Source http://phpmd.org/rules/cleancode.html#MissingImport

Avoid unused local variables such as '$index'.
Open

        foreach ($this->consumers as $index => $consumer) {

UnusedLocalVariable

Since: 0.2

Detects when a local variable is declared and/or assigned, but not used.

Example

class Foo {
    public function doSomething()
    {
        $i = 5; // Unused
    }
}

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

Avoid unused local variables such as '$index'.
Open

        foreach ($this->consumers as $index => $consumer) {

UnusedLocalVariable

Since: 0.2

Detects when a local variable is declared and/or assigned, but not used.

Example

class Foo {
    public function doSomething()
    {
        $i = 5; // Unused
    }
}

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

There must be one blank line after the namespace declaration
Open

namespace SAREhub\Client\Amqp;

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

}

Expected 1 newline at end of file; 0 found
Open

}

There are no issues that match your filters.

Category
Status