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

Completed

Push — master ( db632b...7eca44 )

by Freek

created 2016-11-15 08:52 UTC

SiteCollection::getPromises() A

↳ Parent: SiteCollection

Complexity

Conditions	2
Paths	2

Size

Total Lines	20
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
eloc	11
nc	2
nop	0
dl	0
loc	20
rs	9.4285
c	0
b	0
f	0

<?php

namespace Spatie\UptimeMonitor;

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

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

        (new EachPromise($this->getPromises(), [
            'concurrency' => config('laravel-uptime-monitor.concurrent_uptime_checks'),
            'fulfilled' => function (ResponseInterface $response, $index) {
                $site = $this->items[$index];

                uptimeMonitorConsoleOutput()->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];

                uptimeMonitorConsoleOutput()->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) {
            uptimeMonitorConsoleOutput()->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->uptime_check_method,
                $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;
4
5			use Generator;
6			use GuzzleHttp\Client;
7			use GuzzleHttp\Exception\RequestException;
8			use GuzzleHttp\Promise\EachPromise;
9			use Illuminate\Support\Collection;
10			use Psr\Http\Message\ResponseInterface;
11
12			class SiteCollection extends Collection
13			{
14			public function checkUptime()
15			{
16			$this->resetItemKeys();
17
18			(new EachPromise($this->getPromises(), [
19			'concurrency' => config('laravel-uptime-monitor.concurrent_uptime_checks'),
20			'fulfilled' => function (ResponseInterface $response, $index) {
21			$site = $this->items[$index];
22
23			uptimeMonitorConsoleOutput()->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...
24
25			$site->pingSucceeded($response->getBody());
26			},
27
28			'rejected' => function (RequestException $exception, $index) {
29			$site = $this->items[$index];
30
31			uptimeMonitorConsoleOutput()->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...
32
33			$site->pingFailed($exception->getMessage());
34			},
35			]))->promise()->wait();
36			}
37
38			protected function getPromises() : Generator
39			{
40			$client = new Client([
41			'headers' => [
42			'User-Agent' => 'spatie/laravel-uptime-monitor uptime checker',
43			],
44			]);
45
46			foreach ($this->items as $site) {
47			uptimeMonitorConsoleOutput()->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...
48
49			$promise = $client->requestAsync(
50			$site->uptime_check_method,
51			$site->url,
52			['connect_timeout' => 10]
53			);
54
55			yield $promise;
56			}
57			}
58
59			/**
60			* Make sure the keys are in consecutive order without gaps.
61			*/
62			protected function resetItemKeys()
63			{
64			$this->items = $this->values()->all();
65			}
66			}
67

spatie / laravel-uptime-monitor

GitHub Access Token became invalid

Push — master ( db632b...7eca44 )

SiteCollection::getPromises() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like