@@ -4,7 +4,6 @@ |
||
4 | 4 | |
5 | 5 | use Hogosha\Monitor\Guesser\StatusGuesser; |
6 | 6 | use Hogosha\Monitor\Model\Result; |
7 | -use Hogosha\Monitor\Model\ResultCollection; |
|
8 | 7 | use Hogosha\Monitor\Model\UrlInfo; |
9 | 8 | |
10 | 9 | /** |
@@ -39,7 +39,7 @@ discard block |
||
39 | 39 | * Constructor. |
40 | 40 | * |
41 | 41 | * @param UrlProvider $urlProvider |
42 | - * @param GuzzleClient $client |
|
42 | + * @param Client $client |
|
43 | 43 | */ |
44 | 44 | public function __construct(UrlProvider $urlProvider, Client $client) |
45 | 45 | { |
@@ -50,7 +50,7 @@ discard block |
||
50 | 50 | /** |
51 | 51 | * run. |
52 | 52 | * |
53 | - * @return array |
|
53 | + * @return ResultCollection |
|
54 | 54 | */ |
55 | 55 | public function run() |
56 | 56 | { |
@@ -61,9 +61,9 @@ discard block |
||
61 | 61 | |
62 | 62 | $resultCollection = new ResultCollection(); |
63 | 63 | |
64 | - $requests = function () use ($urls, $client, $resultCollection) { |
|
64 | + $requests = function() use ($urls, $client, $resultCollection) { |
|
65 | 65 | foreach ($urls as $url) { |
66 | - yield function () use ($client, $url, $resultCollection) { |
|
66 | + yield function() use ($client, $url, $resultCollection) { |
|
67 | 67 | return $client->sendAsync( |
68 | 68 | new Request( |
69 | 69 | $url->getMethod(), |
@@ -72,7 +72,7 @@ discard block |
||
72 | 72 | ), |
73 | 73 | [ |
74 | 74 | 'timeout' => $url->getTimeout(), |
75 | - 'on_stats' => function (TransferStats $tranferStats) use ($url, $resultCollection) { |
|
75 | + 'on_stats' => function(TransferStats $tranferStats) use ($url, $resultCollection) { |
|
76 | 76 | |
77 | 77 | if ($tranferStats->hasResponse()) { |
78 | 78 | $statusCode = $tranferStats->getResponse()->getStatusCode(); |
@@ -17,7 +17,6 @@ |
||
17 | 17 | |
18 | 18 | use GuzzleHttp\Client; |
19 | 19 | use GuzzleHttp\Pool; |
20 | -use GuzzleHttp\Promise; |
|
21 | 20 | use GuzzleHttp\Psr7\Request; |
22 | 21 | use GuzzleHttp\TransferStats; |
23 | 22 | use Hogosha\Monitor\Client\GuzzleClient; |
@@ -57,7 +57,7 @@ discard block |
||
57 | 57 | ->addOption('config', 'c', Option::OPTIONAL_VALUE, 'Config file path', rtrim(getcwd(), DIRECTORY_SEPARATOR)) |
58 | 58 | ->beginCommand('init') |
59 | 59 | ->setDescription('Create a default configuration file if you do not have one') |
60 | - ->setHandler(function () use ($configurationLoader, $configurationDumper, $filesystem) { |
|
60 | + ->setHandler(function() use ($configurationLoader, $configurationDumper, $filesystem) { |
|
61 | 61 | return new InitHandler($configurationLoader, $configurationDumper, $filesystem); |
62 | 62 | }) |
63 | 63 | ->setHelp('php <info>bin/monitor</info> init') |
@@ -65,7 +65,7 @@ discard block |
||
65 | 65 | ->end() |
66 | 66 | ->beginCommand('run') |
67 | 67 | ->setDescription('Launch the monitor process') |
68 | - ->setHandler(function () use ($configurationLoader) { |
|
68 | + ->setHandler(function() use ($configurationLoader) { |
|
69 | 69 | return new RunHandler($configurationLoader); |
70 | 70 | }) |
71 | 71 | ->setHelp('php <info>bin/monitor</info> run') |
@@ -107,7 +107,7 @@ |
||
107 | 107 | * |
108 | 108 | * @param string $element |
109 | 109 | * |
110 | - * @return DOMNodeList |
|
110 | + * @return \DOMNodeList |
|
111 | 111 | */ |
112 | 112 | public function xpath($element) |
113 | 113 | { |
@@ -95,7 +95,7 @@ |
||
95 | 95 | * |
96 | 96 | * @param string $value |
97 | 97 | * |
98 | - * @return bool |
|
98 | + * @return null|boolean |
|
99 | 99 | */ |
100 | 100 | public function check($value) |
101 | 101 | { |
@@ -60,19 +60,19 @@ |
||
60 | 60 | ] |
61 | 61 | ); |
62 | 62 | |
63 | - $resolver->setDefault('type', function (Options $options, $value) { |
|
63 | + $resolver->setDefault('type', function(Options $options, $value) { |
|
64 | 64 | if (null === $value) { |
65 | 65 | return; |
66 | 66 | } |
67 | 67 | }); |
68 | 68 | |
69 | - $resolver->setDefault('match', function (Options $options, $value) { |
|
69 | + $resolver->setDefault('match', function(Options $options, $value) { |
|
70 | 70 | if (null === $value) { |
71 | 71 | return; |
72 | 72 | } |
73 | 73 | }); |
74 | 74 | |
75 | - $resolver->setAllowedValues('type', function ($value) { |
|
75 | + $resolver->setAllowedValues('type', function($value) { |
|
76 | 76 | if (in_array($value, ['json', 'xml', 'html'])) { |
77 | 77 | return true; |
78 | 78 | } |