Wallhaven\WallpaperList

Complexity

Total Complexity

17

Size/Duplication

Total Lines	134
Duplicated Lines	3.73 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes	3
Bugs	0	Features	2

8 Methods

Rating	Name	Duplication	Size	Complexity
A	offsetExists()	0	4	1
A	offsetGet()	0	4	2
A	offsetUnset()	0	4	1
A	getIterator()	0	4	1
A	count()	0	4	1
B	downloadAll()	5	39	6
A	offsetSet()	0	12	3
A	addAll()	0	6	2

<?php

namespace Wallhaven;

use GuzzleHttp\Client;
use GuzzleHttp\Pool;
use Wallhaven\Exceptions\DownloadException;
use Wallhaven\Exceptions\WallhavenException;

/**
 * Wallpaper list.
 *
 * @package Wallhaven
 */
class WallpaperList implements \ArrayAccess, \IteratorAggregate, \Countable
{

    /**
     * @var Wallpaper[] Wallpapers.
     */
    private $wallpapers = [];

    /**
     * Download all wallpapers in list.
     *
     * @param string $directory Where to download wallpapers.
     *
     * @throws DownloadException Thrown if the download directory cannot be created.
     */
    public function downloadAll($directory)
    {
        if (!file_exists($directory)) {

This code seems to be duplicated across your project.

            if (!@mkdir($directory, null, true)) {
                throw new DownloadException("The download directory cannot be created.");
            }
        }

        $client = new Client();

        $requests = [];
        foreach ($this->wallpapers as $w) {
            $url = $w->getImageUrl(true);

            $requests[] = $client->createRequest('GET', $url, [

The method createRequest() does not exist on GuzzleHttp\Client. Did you maybe mean request()?

                'save_to' => $directory . '/' . basename($url)
            ]);
        }

        $results = Pool::batch($client, $requests);

        // Retry with PNG
        $retryRequests = [];
        foreach ($results->getFailures() as $e) {

The method getFailures cannot be called on $results (of type array).

            // Delete failed files
            unlink($directory . '/' . basename($e->getRequest()->getUrl()));

            $urlPng = str_replace('.jpg', '.png', $e->getRequest()->getUrl());
            $statusCode = $e->getResponse()->getStatusCode();

            if ($statusCode == 404) {
                $retryRequests[] = $client->createRequest('GET', $urlPng, [

The method createRequest() does not exist on GuzzleHttp\Client. Did you maybe mean request()?

                    'save_to' => $directory . '/' . basename($urlPng)
                ]);
            }
        }

        Pool::batch($client, $retryRequests);
    }


    /**
     *
     * @param $offset
     *
     * @return bool
     */
    public function offsetExists($offset)
    {
        return isset($this->wallpapers[$offset]);
    }

    /**
     *
     * @param $offset
     *
     * @return null|Wallpaper
     */
    public function offsetGet($offset)
    {
        return isset($this->wallpapers[$offset]) ? $this->wallpapers[$offset] : null;
    }

    /**
     *
     * @param           $offset
     * @param Wallpaper $value
     *
     * @throws WallhavenException
     */
    public function offsetSet($offset, $value)
    {
        if (!$value instanceof Wallpaper) {
            throw new WallhavenException("Not a Wallpaper object.");
        }

        if (is_null($offset)) {
            $this->wallpapers[] = $value;
        } else {
            $this->wallpapers[$offset] = $value;
        }
    }

    /**
     * @param $offset
     */
    public function offsetUnset($offset)
    {
        unset($this->wallpapers[$offset]);
    }

    /**
     * @return \ArrayIterator
     */
    public function getIterator()
    {
        return new \ArrayIterator($this->wallpapers);
    }

    /**
     * @return int Wallpaper count.
     */
    public function count()
    {
        return count($this->wallpapers);
    }

    /**
     * All all items from the given list to the current list.
     *
     * @param WallpaperList $wallpaperList
     */
    public function addAll(WallpaperList $wallpaperList)
    {
        foreach ($wallpaperList as $wallpaper) {
            $this->wallpapers[] = $wallpaper;
        }
    }
}