AcquiaCloudApi\Endpoints\Notifications

Complexity

Total Complexity

3

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

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

<?php

namespace AcquiaCloudApi\Endpoints;

use AcquiaCloudApi\Connector\ClientInterface;
use AcquiaCloudApi\Response\NotificationResponse;
use AcquiaCloudApi\Response\NotificationsResponse;

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

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

    /**
     * Client constructor.
     *
     * @param ClientInterface $client
     */
    public function __construct(ClientInterface $client)
    {
        $this->client = $client;
    }
    /**
     * Returns details about a notification.
     *
     * @return NotificationResponse
     */
    public function get($notificationUuid)
    {
        return new NotificationResponse(
            $this->client->request(

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

                'get',
                "/notifications/${notificationUuid}"
            )
        );
    }

    /**
     * Returns a list of notifications.
     *
     * @param string $applicationUuid
     *
     * @return NotificationsResponse
     */
    public function getAll($applicationUuid)
    {
        return new NotificationsResponse(
            $this->client->request(

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

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