OhDear\PhpSdk\Actions\ManagesMaintenancePeriods

Complexity

Total Complexity

5

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	maintenancePeriods()	0	7	1
A	startSiteMaintenance()	0	8	1
A	stopSiteMaintenance()	0	4	1
A	createSiteMaintenance()	0	12	1
A	deleteSiteMaintenance()	0	4	1

<?php

namespace OhDear\PhpSdk\Actions;

use OhDear\PhpSdk\Resources\MaintenancePeriod;

trait ManagesMaintenancePeriods
{
    /**
     * @param int $siteId
     *
     * @return array
     */
    public function maintenancePeriods(int $siteId): array
    {
        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}/maintenance-periods")['data'],

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

            MaintenancePeriod::class
        );
    }

    /**
     * @param int $siteId
     * @param int $stopMaintenanceAfterSeconds Stops after one hour by default
     *
     * @return MaintenancePeriod
     */
    public function startSiteMaintenance(int $siteId, int $stopMaintenanceAfterSeconds = 60 * 60): MaintenancePeriod
    {
        $attributes = $this->post("sites/{$siteId}/start-maintenance", [

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

            'stop_maintenance_after_seconds' => $stopMaintenanceAfterSeconds,
        ]);

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

    /**
     * @param int $siteId
     */
    public function stopSiteMaintenance(int $siteId)
    {
        $this->post("sites/{$siteId}/stop-maintenance");

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

    }

    /**
     * @param int $siteId
     * @param string $startsAt Y:m:d H:i
     * @param string $endsAt Y:m:d H:i
     *
     * @return MaintenancePeriod
     */
    public function createSiteMaintenance(int $siteId, string $startsAt, string $endsAt): MaintenancePeriod
    {
        $payload = [
            'site_id' => $siteId,
            'starts_at' => $startsAt,
            'ends_at' => $endsAt,
        ];

        $attributes = $this->post('maintenance-periods', $payload);

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

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

    /**
     * @param int $maintenancePeriodId
     */
    public function deleteSiteMaintenance(int $maintenancePeriodId)
    {
        $this->delete("maintenance-periods/{$maintenancePeriodId}");

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

    }
}