chrismou/php-lastfm-wrapper

Method doRequest has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function doRequest($method, array $parameters = [], $doRequestAuth = false)
    {
        // We automatically append a few parameters here.
        $parameters = array_merge(
            [

The method doRequest has a boolean flag argument $doRequestAuth, which is a certain sign of a Single Responsibility Principle violation.

Open

    protected function doRequest($method, array $parameters = [], $doRequestAuth = false)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar($flag = true) {
    }
}

Missing class import via use statement (line '109', column '23').

Open

            throw new \Chrismou\LastFm\Exception\ResponseErrorException("[{$response->error}|{$response->message}] "

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

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

The method get has a boolean flag argument $authRequired, which is a certain sign of a Single Responsibility Principle violation.

Open

    public function get($method, array $parameters = [], $authRequired = false)

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function bar($flag = true) {
    }
}

Missing class import via use statement (line '104', column '23').

Open

            throw new \Chrismou\LastFm\Exception\ResponseMalformedException("Error parsing JSON");

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

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