OhDear\PhpSdk\Actions\ManagesSites

Complexity

Total Complexity

5

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Importance

Changes	3
Bugs	0	Features	2

5 Methods

Rating	Name	Duplication	Size	Complexity
A	siteByUrl()	0	5	1
A	createSite()	0	5	1
A	site()	0	5	1
A	sites()	0	5	1
A	deleteSite()	0	3	1

<?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()?

    }
}