LaravelWorkcast\Endpoints\WithRestFullRead

Complexity

Total Complexity

5

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	callPagination()	0	5	1
A	list()	0	10	1
A	get()	0	9	3

<?php

namespace LaravelWorkcast\Endpoints;

use Illuminate\Http\Client\Response;
use LaravelWorkcast\WorkcastException;
use LaravelWorkcast\WorkcastPagination;

trait WithRestFullRead
{
    /**
     * @inheritDoc
     */
    public function callPagination(string $link): WorkcastPagination
    {
        $response = $this->httpClient()->get($link);

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

        return new WorkcastPagination($response, $this->key());
    }

    /**
     * @inheritDoc
     */
    public function list(array $query = []): WorkcastPagination
    {
        $query = array_merge([
            'limit' => 100,
            'skip'  => 0,
        ], $query);

        $response = $this->httpClient()->get($this->baseUrl(), $query);

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

        return new WorkcastPagination($response, $this->key());
    }

    /**
     * @inheritDoc
     */
    public function get($id): Response
    {
        $response = $this->httpClient()->get($this->baseUrl() . "/{$id}");

        if (!$response->ok() || $response->json('responseCode') != 200) {
            throw new WorkcastException('Request error', $response->status());
        }

        return $response;
    }

    abstract public function key():string;
}