SiteCollection - Code Metrics - Inspection of "wip" - spatie/laravel-uptime-monitor - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 9e170a...fa9406 )

by Freek

created 2016-09-25 18:08 UTC

SiteCollection A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

Metric	Value
dl	0
loc	55
rs	10
c	0
b	0
f	0
wmc	4
lcom	1
cbo	6

3 Methods

Rating	Name	Size	Complexity
A	checkUptime()	23	1
A	getPromises()	20	2
A	resetItemKeys()	4	1

<?php

namespace Spatie\UptimeMonitor\Services\PingMonitors;

use Cache;
use Generator;
use GuzzleHttp\Client;
use GuzzleHttp\Exception\RequestException;
use GuzzleHttp\Promise\EachPromise;
use Illuminate\Support\Collection;
use Log;
use Psr\Http\Message\ResponseInterface;

class SiteCollection extends Collection
{
    public function checkUptime()
    {
        $this->resetItemKeys();

        (new EachPromise($this->getPromises(), [
            'concurrency' => 100,
            'fulfilled' => function (ResponseInterface $response, $index) {
                $site = $this->items[$index];

                consoleOutput()->info("Could reach {$site->url}");
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }

                $site->pingSucceeded($response->getBody());
            },

            'rejected' => function (RequestException $exception, $index) {
                $site = $this->items[$index];

                consoleOutput()->error("Could not reach {$site->url} error: `{$exception->getMessage()}`");
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }

                $site->pingFailed($exception->getMessage());
            },
        ]))->promise()->wait();
    }

    protected function getPromises() : Generator
    {
        $client = new Client([
            'headers' => [
                'User-Agent' => 'spatie/laravel-uptime-monitor uptime checker',
            ],
        ]);

        foreach ($this->items as $site) {
            consoleOutput()->info("checking {$site->url}");
class ParentClass {
    private $data = array();

    public function __call($method, array $args) {
        if (0 === strpos($method, 'get')) {
            return $this->data[strtolower(substr($method, 3))];
        }

        throw new \LogicException(sprintf('Unsupported method: %s', $method));
    }
}

/**
 * If this class knows which fields exist, you can specify the methods here:
 *
 * @method string getName()
 */
class SomeClass extends ParentClass { }

            $promise = $client->requestAsync(
                $site->getPingRequestMethod(),
                $site->url,
                ['connect_timeout' => 10]
            );

            yield $promise;
        }
    }

    /**
     * Make sure the keys are in consecutive order without gaps.
     */
    protected function resetItemKeys()
    {
        $this->items = $this->values()->all();
    }
}


1			<?php
2
3			namespace Spatie\UptimeMonitor\Services\PingMonitors;
4
5			use Cache;
6			use Generator;
7			use GuzzleHttp\Client;
8			use GuzzleHttp\Exception\RequestException;
9			use GuzzleHttp\Promise\EachPromise;
10			use Illuminate\Support\Collection;
11			use Log;
12			use Psr\Http\Message\ResponseInterface;
13
14			class SiteCollection extends Collection
15			{
16			public function checkUptime()
17			{
18			$this->resetItemKeys();
19
20			(new EachPromise($this->getPromises(), [
21			'concurrency' => 100,
22			'fulfilled' => function (ResponseInterface $response, $index) {
23			$site = $this->items[$index];
24
25			consoleOutput()->info("Could reach {$site->url}");
			0 ignored issues – show Documentation Bug introduced 2016-09-24 22:03 UTC by Report Bug Copy Issue Report The method `info` does not exist on `object<Spatie\UptimeMoni...\Helpers\ConsoleOutput>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
26
27			$site->pingSucceeded($response->getBody());
28			},
29
30			'rejected' => function (RequestException $exception, $index) {
31			$site = $this->items[$index];
32
33			consoleOutput()->error("Could not reach {$site->url} error: `{$exception->getMessage()}`");
			0 ignored issues – show Documentation Bug introduced 2016-09-24 22:03 UTC by Report Bug Copy Issue Report The method `error` does not exist on `object<Spatie\UptimeMoni...\Helpers\ConsoleOutput>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
34
35			$site->pingFailed($exception->getMessage());
36			},
37			]))->promise()->wait();
38			}
39
40			protected function getPromises() : Generator
41			{
42			$client = new Client([
43			'headers' => [
44			'User-Agent' => 'spatie/laravel-uptime-monitor uptime checker',
45			],
46			]);
47
48			foreach ($this->items as $site) {
49			consoleOutput()->info("checking {$site->url}");
			0 ignored issues – show Documentation Bug introduced 2016-09-24 22:03 UTC by Report Bug Copy Issue Report The method `info` does not exist on `object<Spatie\UptimeMoni...\Helpers\ConsoleOutput>`? Since you implemented `__call`, maybe consider adding a @method annotation. If you implement `__call` and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class. This is often the case, when `__call` is implemented by a parent class and only the child class knows which methods exist: class ParentClass { private $data = array(); public function __call($method, array $args) { if (0 === strpos($method, 'get')) { return $this->data[strtolower(substr($method, 3))]; } throw new \LogicException(sprintf('Unsupported method: %s', $method)); } } /** * If this class knows which fields exist, you can specify the methods here: * * @method string getName() */ class SomeClass extends ParentClass { } Loading history...
50
51			$promise = $client->requestAsync(
52			$site->getPingRequestMethod(),
53			$site->url,
54			['connect_timeout' => 10]
55			);
56
57			yield $promise;
58			}
59			}
60
61			/**
62			* Make sure the keys are in consecutive order without gaps.
63			*/
64			protected function resetItemKeys()
65			{
66			$this->items = $this->values()->all();
67			}
68			}
69

spatie / laravel-uptime-monitor

GitHub Access Token became invalid

Push — master ( 9e170a...fa9406 )

SiteCollection A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

3 Methods

Duplication Side-by-Side

Filter issues like