Scriptotek\GoogleBooks\Volumes

Complexity

Total Complexity

9

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	search()	0	7	2
A	fetchSearch()	0	6	2
A	chunk()	0	5	1
A	firstOrNull()	0	5	1
A	get()	0	4	1
A	byIsbn()	0	6	1

<?php

namespace Scriptotek\GoogleBooks;


class Volumes
{
    protected $client;
    protected $chunk = false;

    public function __construct(GoogleBooks $client)
    {
        $this->client = $client;
    }

    /**
     * @param $query
     * @return \Generator|Volume || array
     */
    public function search($query)
    {
        if ($this->chunk) {
            return $this->client->chunk($this->fetchSearch($query), $this->chunk);

$this->chunk is of type boolean, but the function expects a integer.

        }
        return $this->fetchSearch($query);
    }

    /**
     * @param $query
     * @return \Generator|Volume
     */
    private function fetchSearch($query)
    {
        foreach ($this->client->listItems('volumes', ['q' => $query]) as $item) {
            yield new Volume($this->client, $item);
        }
    }

    /**
     * @param $chunk
     * @return Volumes
     */
    public function chunk(int $chunk)
    {
        $this->chunk = $chunk;

The property $chunk was declared of type boolean, but $chunk is of type integer. Maybe add a type cast?

        return $this;
    }

    public function firstOrNull($query)
    {
        $res = $this->search($query)->current();
        return $res;
    }

    public function get($id)
    {
        return new Volume($this->client, $this->client->getItem("volumes/$id"), true);
    }

    public function byIsbn($isbn)
    {
        $isbn = preg_replace('/[^0-9Xx]/', '', $isbn);

        return $this->firstOrNull('isbn:' . $isbn);
    }
}