AcquiaCloudApi\Endpoints\Logs

Complexity

Total Complexity

4

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	snapshot()	0	4	1
A	stream()	0	4	1
A	getAll()	0	4	1
A	download()	0	3	1

<?php

namespace AcquiaCloudApi\Endpoints;

use AcquiaCloudApi\Connector\ClientInterface;
use AcquiaCloudApi\Response\EnvironmentResponse;
use AcquiaCloudApi\Response\EnvironmentsResponse;
use AcquiaCloudApi\Response\LogstreamResponse;
use AcquiaCloudApi\Response\LogsResponse;
use AcquiaCloudApi\Response\OperationResponse;

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

    /**
     * Returns a list of log files available for download.
     *
     * @param string $environmentUuid
     * @return LogsResponse
     */
    public function getAll($environmentUuid)
    {
        return new LogsResponse(
            $this->client->request('get', "/environments/${environmentUuid}/logs")

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

        );
    }

    /**
     * Downloads a log file.
     *
     * @param string $environmentUuid
     * @param string $logType
     * @return object
     */
    public function download($environmentUuid, $logType)
    {
        return $this->client->request('get', "/environments/${environmentUuid}/logs/${logType}");

The expression return $this->client->request('get', '/environments/'.$environmentUuid.'/logs/'.$logType) also could return the type array which is incompatible with the documented return type object.

    }

    /**
     * Creates a log file snapshot.
     *
     * @param string $environmentUuid
     * @param string $logType
     * @return OperationResponse
     */
    public function snapshot($environmentUuid, $logType)
    {
        return new OperationResponse(
            $this->client->request('post', "/environments/${environmentUuid}/logs/${logType}")

It seems like $this->client->request('post', '/environments/'.$environmentUuid.'/logs/'.$logType) 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?

        );
    }

    /**
     * Returns logstream WSS stream information.
     *
     * @param string $environmentUuid
     * @return LogstreamResponse
     */
    public function stream($environmentUuid)
    {
        return new LogstreamResponse(
            $this->client->request('get', "/environments/${environmentUuid}/logstream")

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

        );
    }
}