mcorten87\rabbitmq_api\mappers\JobClusterNameUpdateMapper

Complexity

Total Complexity

3

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	mapMethod()	0	3	1
A	mapUrl()	0	4	1
A	mapConfig()	0	10	1

<?php
declare(strict_types=1);
namespace mcorten87\rabbitmq_api\mappers;

use mcorten87\rabbitmq_api\jobs\JobBase;
use mcorten87\rabbitmq_api\jobs\JobBindingListAll;
use mcorten87\rabbitmq_api\jobs\JobClusterNameList;
use mcorten87\rabbitmq_api\jobs\JobClusterNameUpdate;
use mcorten87\rabbitmq_api\objects\Method;
use mcorten87\rabbitmq_api\objects\Url;
use mcorten87\rabbitmq_api\services\MqManagementConfig;

class JobClusterNameUpdateMapper  extends BaseMapper

Expected 1 space after class name; 2 found

Expected 1 space before extends keyword; 2 found

{

    /**
     * @return Method
     */
    protected function mapMethod() : Method {
        return new Method(Method::PUT);
    }

    /**
     * @param JobClusterNameUpdate $job
     * @return Url
     */
    protected function mapUrl(JobBase $job) : Url {
        $url = 'cluster-name';
        return new Url($url);
    }

    /**
     * @param JobClusterNameUpdate $job
     * @return array
     */
    protected function mapConfig(JobBase $job): array
    {
        $body = [
            'name' => $job->getName(),

It seems like you code against a specific sub-type and not the parent class mcorten87\rabbitmq_api\jobs\JobBase as the method getName() does only exist in the following sub-classes of mcorten87\rabbitmq_api\jobs\JobBase: mcorten87\rabbitmq_api\jobs\JobClusterNameUpdate. Maybe you want to instanceof check for one of these explicitly?

        ];

        return array_merge(parent::mapConfig($job), [
            'json'      =>  $body,
        ]);
    }
}