ohdearapp / ohdear-php-sdk

src/Actions/ManagesSites.php

<?php

namespace OhDear\PhpSdk\Actions;

use OhDear\PhpSdk\Resources\Site;

trait ManagesSites
{
    public function sites(): 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')['data'],

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

            Site::class,
        );
    }

    public function site(int $siteId): Site
    {
        $siteAttributes = $this->get("sites/{$siteId}");

        return new Site($siteAttributes, $this);
    }

    public function siteByUrl(string $siteUrl): Site
    {
        $siteAttributes = $this->get("sites/url/{$siteUrl}");

        return new Site($siteAttributes, $this);
    }

    public function createSite(array $data): Site
    {
        $siteAttributes = $this->post('sites', $data);

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

        return new Site($siteAttributes, $this);
    }

    public function deleteSite(int $siteId)
    {
        $this->delete("sites/$siteId");

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

    }
}