AcquiaCloudApi\Endpoints\Servers

Complexity

Total Complexity

4

Size/Duplication

Total Lines	63
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	update()	0	8	1
A	get()	0	6	1
A	getAll()	0	6	1
A	__construct()	0	3	1

<?php

namespace AcquiaCloudApi\Endpoints;

use AcquiaCloudApi\Connector\ClientInterface;
use AcquiaCloudApi\Response\OperationResponse;
use AcquiaCloudApi\Response\ServersResponse;
use AcquiaCloudApi\Response\ServerResponse;

/**
 * Class Client
 * @package AcquiaCloudApi\CloudApi
 */
class Servers implements CloudApi
{

    /** @var ClientInterface The API client. */
    protected $client;

    /**
     * Client constructor.
     *
     * @param ClientInterface $client
     */
    public function __construct(ClientInterface $client)
    {
        $this->client = $client;
    }

    /**
     * Gets information about a single server.
     *
     * @param string $environmentUuid
     * @return ServerResponse
     */
    public function get($environmentUuid, $serverId)
    {
        return new ServerResponse(
            $this->client->request(

It seems like $this->client->request('get', '/environments/'.$environmentUuid.'/servers/'.$serverId) can also be of type array; however, parameter $server of AcquiaCloudApi\Response\ServerResponse::__construct() does only seem to accept object, maybe add an additional type check?

                'get',
                "/environments/${environmentUuid}/servers/${serverId}"
            )
        );
    }

    /**
     * Modifies configuration settings for a server.
     *
     * @param string $environmentUuid
     * @param array $config
     * @return OperationResponse
     */
    public function update($environmentUuid, $serverId, array $config)
    {

        return new OperationResponse(
            $this->client->request(

It seems like $this->client->request('put', '/environments/'.$environmentUuid.'/servers/'.$serverId, $config) can also be of type array; however, parameter $operation of AcquiaCloudApi\Response\OperationResponse::__construct() does only seem to accept object, maybe add an additional type check?

                'put',
                "/environments/${environmentUuid}/servers/${serverId}",
                $config
            )
        );
    }

    /**
     * Show all servers associated with an environment.
     *
     * @param string $environmentUuid
     * @return ServersResponse
     */
    public function getAll($environmentUuid)
    {
        return new ServersResponse(
            $this->client->request(

It seems like $this->client->request('get', '/environments/'.$environmentUuid.'/servers') can also be of type object; however, parameter $servers of AcquiaCloudApi\Response\ServersResponse::__construct() does only seem to accept array, maybe add an additional type check?

                'get',
                "/environments/${environmentUuid}/servers"
            )
        );
    }
}