PiedWeb\SeoPocketCrawler\CrawlerRestart

Complexity

Total Complexity

5

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getHarvest()	0	16	3
A	loadFromPreviousCrawl()	0	3	1

<?php

namespace PiedWeb\SeoPocketCrawler;

use PiedWeb\UrlHarvester\Harvest;
use PiedWeb\Curl\ResponseFromCache;

class CrawlerRestart extends CrawlerContinue
{
    public function __construct(string $id, bool $fromCache = false)
    {
        $this->fromCache = $fromCache;

        parent::__construct($id);
    }

    protected function loadFromPreviousCrawl(string $startUrl)
    {
        $this->urls[$startUrl] = null;
    }

    protected function getHarvest(Url $url)
    {
        if (true === $this->fromCache) {
            $filePath = $this->recorder->getCacheFilePath($url);
            if (file_exists($filePath)) {
                $response = new ResponseFromCache(
                    $filePath,

It seems like $filePath can also be of type null; however, parameter $filePathOrContent of PiedWeb\Curl\ResponseFromCache::__construct() does only seem to accept string, maybe add an additional type check?

                    $this->base.$url->uri,
                    json_decode(file_get_contents($filePath.'---info'), true)
                );

                return new Harvest($response);
            }
        }

        return parent::getHarvest($url);
    }
}