AcquiaCloudApi\Endpoints\Crons

Complexity

Total Complexity

7

Size/Duplication

Total Lines	132
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	disable()	0	6	1
A	delete()	0	4	1
A	getAll()	0	6	1
A	get()	0	6	1
A	update()	0	14	1
A	enable()	0	6	1
A	create()	0	14	1

<?php

namespace AcquiaCloudApi\Endpoints;

use AcquiaCloudApi\Connector\ClientInterface;
use AcquiaCloudApi\Response\CronsResponse;
use AcquiaCloudApi\Response\CronResponse;
use AcquiaCloudApi\Response\OperationResponse;

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

    /**
     * Show all cron tasks for an environment.
     *
     * @param string $environmentUuid The environment ID
     * @return CronsResponse
     */
    public function getAll($environmentUuid)
    {
        return new CronsResponse(
            $this->client->request(

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

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

    /**
     * Get information about a cron task.
     *
     * @param string $environmentUuid The environment ID
     * @param int    $cronId
     * @return CronResponse
     */
    public function get($environmentUuid, $cronId)
    {
        return new CronResponse(
            $this->client->request(

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

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

    /**
     * Add a cron task.
     *
     * @param string $environmentUuid
     * @param string $command
     * @param string $frequency
     * @param string $label
     * @return OperationResponse
     */
    public function create($environmentUuid, $command, $frequency, $label, $serverId = null)
    {

        $options = [
            'form_params' => [
                'command' => $command,
                'frequency' => $frequency,
                'label' => $label,
                'server_id' => $serverId
            ],
        ];

        return new OperationResponse(
            $this->client->request('post', "/environments/${environmentUuid}/crons", $options)

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

        );
    }

    /**
     * Update a cron task.
     *
     * @param string $environmentUuid
     * @param string $command
     * @param string $frequency
     * @param string $label
     * @return OperationResponse
     */
    public function update($environmentUuid, $cronId, $command, $frequency, $label, $serverId = null)
    {

        $options = [
            'form_params' => [
                'command' => $command,
                'frequency' => $frequency,
                'label' => $label,
                'server_id' => $serverId
            ],
        ];

        return new OperationResponse(
            $this->client->request('post', "/environments/${environmentUuid}/crons/${cronId}", $options)

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

        );
    }

    /**
     * Delete a cron task.
     *
     * @param string $environmentUuid
     * @param int    $cronId
     * @return OperationResponse
     */
    public function delete($environmentUuid, $cronId)
    {
        return new OperationResponse(
            $this->client->request('delete', "/environments/${environmentUuid}/crons/${cronId}")

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

        );
    }

    /**
     * Disable a cron task.
     *
     * @param string $environmentUuid
     * @param int    $cronId
     * @return OperationResponse
     */
    public function disable($environmentUuid, $cronId)
    {
        return new OperationResponse(
            $this->client->request(

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

                'post',
                "/environments/${environmentUuid}/crons/${cronId}/actions/disable"
            )
        );
    }

    /**
     * Enable a cron task.
     *
     * @param string $environmentUuid
     * @param int    $cronId
     * @return OperationResponse
     */
    public function enable($environmentUuid, $cronId)
    {
        return new OperationResponse(
            $this->client->request(

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

                'post',
                "/environments/${environmentUuid}/crons/${cronId}/actions/enable"
            )
        );
    }
}