Output::outputWithState() - Code Metrics - lookyman/rundeck-api - Measure and Improve Code Quality continuously with Scrutinizer

Output::outputWithState() A
last analyzed 2016-12-13 16:35 UTC

↳ Parent: Output

Complexity

Conditions	2
Paths	1

Size

Total Lines	6
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	6
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	3
nc	1
nop	2

<?php
declare(strict_types=1);

namespace Lookyman\Rundeck\Api\Endpoints\Execution;

use GuzzleHttp\Promise\PromiseInterface;
use GuzzleHttp\Psr7\Request;
use Lookyman\Rundeck\Api\Client;

class Output
{
	/**
	 * @var Client
	 */
	private $client;

	/**
	 * @param Client $client
	 */
	public function __construct(Client $client)
	{
		$this->client = $client;
	}

	/**
	 * @param string $id
	 * @param string|null $node
	 * @param string|null $step
	 * @param array $params
	 * @return PromiseInterface
	 */
	public function output(string $id, string $node = null, string $step = null, array $params = []): PromiseInterface
	{
		if ($node && $step) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
			$path = sprintf('/execution/%s/output/node/%s/step/%s', urlencode($id), urlencode($node), urlencode($step));
		} elseif ($node && !$step) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
			$path = sprintf('/execution/%s/output/node/%s', urlencode($id), urlencode($node));
		} elseif (!$node && $step) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
			$path = sprintf('/execution/%s/output/step/%s', urlencode($id), urlencode($step));
		} else {
			$path = sprintf('/execution/%s/output', urlencode($id));
		}

		return $this->client->getConfiguration()->getGuzzle()->sendAsync(
			new Request('GET', $this->client->getConfiguration()->getBaseUri() . $path, [], $this->client->getConfiguration()->getFormat()->formatParams($params))
		);
	}

	/**
	 * @param string $id
	 * @param bool $stateOnly
	 * @return PromiseInterface
	 */
	public function outputWithState(string $id, bool $stateOnly = false): PromiseInterface
	{
		return $this->client->getConfiguration()->getGuzzle()->sendAsync(
			new Request('GET', $this->client->getConfiguration()->getBaseUri() . sprintf('/execution/%s/output/state%s', urlencode($id), $stateOnly ? '?stateOnly=true' : ''))
		);
	}
}


1			<?php
2			declare(strict_types=1);
3
4			namespace Lookyman\Rundeck\Api\Endpoints\Execution;
5
6			use GuzzleHttp\Promise\PromiseInterface;
7			use GuzzleHttp\Psr7\Request;
8			use Lookyman\Rundeck\Api\Client;
9
10			class Output
11			{
12			/**
13			* @var Client
14			*/
15			private $client;
16
17			/**
18			* @param Client $client
19			*/
20			public function __construct(Client $client)
21			{
22			$this->client = $client;
23			}
24
25			/**
26			* @param string $id
27			* @param string\|null $node
28			* @param string\|null $step
29			* @param array $params
30			* @return PromiseInterface
31			*/
32			public function output(string $id, string $node = null, string $step = null, array $params = []): PromiseInterface
33			{
34			if ($node && $step) {
			0 ignored issues – show Bug Best Practice introduced 2016-12-13 15:27 UTC by Report Bug Copy Issue Report The expression `$node` of type `null\|string` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history... Bug Best Practice introduced 2016-12-13 15:27 UTC by Report Bug Copy Issue Report The expression `$step` of type `null\|string` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
35			$path = sprintf('/execution/%s/output/node/%s/step/%s', urlencode($id), urlencode($node), urlencode($step));
36			} elseif ($node && !$step) {
			0 ignored issues – show Bug Best Practice introduced 2016-12-13 15:27 UTC by Report Bug Copy Issue Report The expression `$node` of type `null\|string` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history... Bug Best Practice introduced 2016-12-13 15:27 UTC by Report Bug Copy Issue Report The expression `$step` of type `null\|string` is loosely compared to `false`; this is ambiguous if the string can be empty. You might want to explicitly use `=== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
37			$path = sprintf('/execution/%s/output/node/%s', urlencode($id), urlencode($node));
38			} elseif (!$node && $step) {
			0 ignored issues – show Bug Best Practice introduced 2016-12-13 15:27 UTC by Report Bug Copy Issue Report The expression `$node` of type `null\|string` is loosely compared to `false`; this is ambiguous if the string can be empty. You might want to explicitly use `=== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history... Bug Best Practice introduced 2016-12-13 15:27 UTC by Report Bug Copy Issue Report The expression `$step` of type `null\|string` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
39			$path = sprintf('/execution/%s/output/step/%s', urlencode($id), urlencode($step));
40			} else {
41			$path = sprintf('/execution/%s/output', urlencode($id));
42			}
43
44			return $this->client->getConfiguration()->getGuzzle()->sendAsync(
45			new Request('GET', $this->client->getConfiguration()->getBaseUri() . $path, [], $this->client->getConfiguration()->getFormat()->formatParams($params))
46			);
47			}
48
49			/**
50			* @param string $id
51			* @param bool $stateOnly
52			* @return PromiseInterface
53			*/
54			public function outputWithState(string $id, bool $stateOnly = false): PromiseInterface
55			{
56			return $this->client->getConfiguration()->getGuzzle()->sendAsync(
57			new Request('GET', $this->client->getConfiguration()->getBaseUri() . sprintf('/execution/%s/output/state%s', urlencode($id), $stateOnly ? '?stateOnly=true' : ''))
58			);
59			}
60			}
61

lookyman / rundeck-api

Output::outputWithState() A last analyzed 2016-12-13 16:35 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

Output::outputWithState() A
last analyzed 2016-12-13 16:35 UTC