AcquiaCloudApi\Endpoints\Ides

Complexity

Total Complexity

4

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getAll()	0	6	1
A	delete()	0	4	1
A	get()	0	6	1
A	create()	0	11	1

<?php

namespace AcquiaCloudApi\Endpoints;

use AcquiaCloudApi\Connector\ClientInterface;
use AcquiaCloudApi\Response\IdesResponse;
use AcquiaCloudApi\Response\IdeResponse;
use AcquiaCloudApi\Response\OperationResponse;

/**
 * Class Ides
 * @package AcquiaCloudApi\CloudApi
 */
class Ides extends CloudApiBase implements CloudApiInterface
{

    /**
     * Returns a list of remote IDEs.
     *
     * @param string $applicationUuid The application ID
     * @return IdesResponse
     */
    public function getAll($applicationUuid)
    {
        return new IdesResponse(
            $this->client->request(

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

                'get',
                "/applications/${applicationUuid}/ides"
            )
        );
    }

    /**
     * Get remote IDE info.
     *
     * @param string $ideUuid The Remote IDE universally unique identifier.
     * @return IdeResponse
     */
    public function get($ideUuid)
    {
        return new IdeResponse(
            $this->client->request(

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

                'get',
                "/ides/${ideUuid}"
            )
        );
    }

    /**
     * Creates a new remote IDE.
     *
     * @param string $applicationUuid
     * @param string $name
     * @return OperationResponse
     */
    public function create($applicationUuid, $name)
    {

        $options = [
            'form_params' => [
                'name' => $name,
            ],
        ];

        return new OperationResponse(
            $this->client->request('post', "/applications/${applicationUuid}/ides", $options)

It seems like $this->client->request('post', '/applications/'.$applicationUuid.'/ides', $options) 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?

        );
    }

    /**
     * De-provisions a specific Remote IDE.
     *
     * @param string $ideUuid
     * @return OperationResponse
     */
    public function delete($ideUuid)

The parameter $ideUuid is not used and could be removed.

    {
        return new OperationResponse(
            $this->client->request('delete', "/ides/{ideUuid}")

It seems like $this->client->request('delete', '/ides/{ideUuid}') 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?

        );
    }
}