OhDear\PhpSdk\Actions\ManagesCronChecks

Complexity

Total Complexity

6

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	cronChecks()	0	5	1
A	updateCronCheck()	0	5	1
A	createSimpleCronCheck()	0	16	1
A	deleteCronCheck()	0	3	1
A	createCronCheck()	0	18	1
A	syncCronChecks()	0	7	1

<?php

namespace OhDear\PhpSdk\Actions;

use OhDear\PhpSdk\Resources\CronCheck;

trait ManagesCronChecks
{
    public function cronChecks(int $siteId)
    {
        return $this->transformCollection(

It seems like transformCollection() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            $this->get("sites/{$siteId}/cron-checks")['data'],

It seems like get() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            CronCheck::class
        );
    }

    public function createSimpleCronCheck(
        int $siteId,
        string $name,
        int $frequencyInMinutes,
        int $graceTimeInMinutes,
        string $description
    ): CronCheck {
        $attributes = $this->post("sites/{$siteId}/cron-checks", [

It seems like post() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            'name' => $name,
            'type' => 'simple',
            'frequency_in_minutes' => $frequencyInMinutes,
            'grace_time_in_minutes' => $graceTimeInMinutes,
            'description' => $description,
        ]);

        return new CronCheck($attributes, $this);
    }

    public function createCronCheck(
        int $siteId,
        string $name,
        string $cronExpression,
        int $graceTimeInMinutes,
        string $description,
        string $serverTimezone
    ): CronCheck {
        $attributes = $this->post("sites/{$siteId}/cron-checks", [
            'name' => $name,
            'type' => 'cron',
            'cron_expression' => $cronExpression,
            'grace_time_in_minutes' => $graceTimeInMinutes,
            'description' => $description,
            'server_timezone' => $serverTimezone,
        ]);

        return new CronCheck($attributes, $this);
    }

    public function updateCronCheck(int $cronCheckId, array $payload): CronCheck
    {
        $attributes = $this->put("cron-checks/{$cronCheckId}", $payload);

It seems like put() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return new CronCheck($attributes, $this);
    }

    public function deleteCronCheck(int $cronCheckId): void
    {
        $this->delete("cron-checks/{$cronCheckId}");

The method delete() does not exist on OhDear\PhpSdk\Actions\ManagesCronChecks. Did you maybe mean deleteCronCheck()?

    }

    public function syncCronChecks(int $siteId, array $cronCheckAttributes): array
    {
        $response = $this->post("sites/{$siteId}/cron-checks/sync", ['cron_checks' => $cronCheckAttributes]);

        return $this->transformCollection(
            $response,
            CronCheck::class,
        );
    }
}