AcquiaCloudApi\Endpoints\Insights

Complexity

Total Complexity

3

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

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

<?php

namespace AcquiaCloudApi\Endpoints;

use AcquiaCloudApi\Connector\ClientInterface;
use AcquiaCloudApi\Response\InsightsResponse;
use AcquiaCloudApi\Response\InsightResponse;
use AcquiaCloudApi\Response\OperationResponse;

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

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

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

    /**
     * Returns Insight data for all sites associated with the application by its UUID.
     *
     * @param string $applicationUuid
     * @return InsightsResponse
     */
    public function getAll($applicationUuid)
    {
        return new InsightsResponse(
            $this->client->request(

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

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

    /**
     * Returns insight data for a particular site.
     *
     * @param string $siteId
     * @return InsightResponse
     */
    public function get($siteId)
    {
        return new InsightResponse(
            $this->client->request(

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

                'get',
                "/insight/${siteId}"
            )
        );
    }
}