Resova\Endpoints\Availability\Instances

Complexity

Total Complexity

2

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	10	1
A	pricing()	0	13	1

<?php

namespace Resova\Endpoints\Availability;

use Resova\Endpoints\Availability;
use Resova\Models\Instance;
use Resova\Models\InstancePricing;
use Resova\Models\Pricing;

class Instances extends Availability
{
    /**
     * Retrieve an instance
     * Retrieve the details of a specific instance.
     *
     * @param string $instance_id The Instance Id
     *
     * @return $this
     */
    public function __invoke(string $instance_id): self
    {
        $this->instance_id = $instance_id;

The property instance_id does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        // Set HTTP params
        $this->type     = 'get';
        $this->endpoint = '/availability/instance/' . $instance_id;
        $this->response = Instance::class;

The property response does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        return $this;
    }

    /**
     * Retrieve instance pricing
     * Retrieve the pricing for an instance for the quantities passed.
     *
     * @param null|Pricing $pricing
     *
     * @return $this
     */
    public function pricing(Pricing $pricing): self
    {
        $pricing->setRequired([
            'quantities'
        ]);

        // Set HTTP params
        $this->type     = 'post';
        $this->endpoint = '/availability/instance/' . $this->instance_id . '/pricing';
        $this->params   = $pricing;
        $this->response = InstancePricing::class;

The property response does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        return $this;
    }

}