AsyncClient::whoami() - Code Metrics - Inspection of "Resource: Rate limits" - php-api-clients/github - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#16)

by Cees-Jan

created 2017-05-02 20:18 UTC

AsyncClient::whoami() A

↳ Parent: AsyncClient

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	4
ccs	0
cts	2
cp	0
rs	10
c	0
b	0
f	0
cc	1
eloc	2
nc	1
nop	0
crap	2

<?php declare(strict_types=1);

namespace ApiClients\Client\Github;

use ApiClients\Client\Github\CommandBus\Command;
use ApiClients\Foundation\ClientInterface;
.
|-- OtherDir
|   |-- Bar.php
|   `-- Foo.php
`-- SomeDir
    `-- Foo.php
use ApiClients\Foundation\Factory;
use ApiClients\Foundation\Options;
use React\EventLoop\LoopInterface;
use React\Promise\PromiseInterface;
use Rx\Observable;
use Rx\Scheduler;
use function ApiClients\Tools\Rx\unwrapObservableFromPromise;

final class AsyncClient implements AsyncClientInterface
{
    /**
     * @var ClientInterface
     */
    private $client;

    /**
     * @var RateLimitState
     */
    private $rateLimitState;

    /**
     * @param LoopInterface $loop
     * @param AuthenticationInterface $auth
     * @param array $options
     * @return AsyncClient
     */
    public static function create(
        LoopInterface $loop,
        AuthenticationInterface $auth,
        array $options = []
    ): self {
        $options = ApiSettings::getOptions($auth, $options, 'Async');
        $rateLimitState = new RateLimitState();
        $options[Options::CONTAINER_DEFINITIONS][RateLimitState::class] = $rateLimitState;
        $client = Factory::create($loop, $options);

        try {
            Scheduler::setAsyncFactory(function () use ($loop) {
                return new Scheduler\EventLoopScheduler($loop);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
            });
        } catch (\Throwable $t) {

        }

        return new self($client, $rateLimitState);
    }

    /**
     * @internal
     * @param ClientInterface $client
     * @param RateLimitState $rateLimitState
     * @return AsyncClient
     */
    public static function createFromClient(ClientInterface $client, RateLimitState $rateLimitState): self
    {
        return new self($client, $rateLimitState);
    }

    /**
     * @param ClientInterface $client
     * @param RateLimitState $rateLimitState
     */
    private function __construct(ClientInterface $client, RateLimitState $rateLimitState)
    {
        $this->client = $client;
        $this->rateLimitState = $rateLimitState;
    }

    public function meta(): PromiseInterface
    {
        return $this->client->handle(new Command\MetaCommand());
    }

    /**
     * @param string $user
     * @return PromiseInterface
     */
    public function user(string $user): PromiseInterface
    {
        return $this->client->handle(new Command\UserCommand($user));
    }

    /**
     * @return PromiseInterface
     */
    public function whoami(): PromiseInterface
    {
        return $this->client->handle(new Command\UserCommand());
    }

    /**
     * @return Observable
     */
    public function emojis(): Observable
    {
        return unwrapObservableFromPromise($this->client->handle(new Command\EmojisCommand()));
    }

    public function myOrganizations(): Observable
    {
        return unwrapObservableFromPromise($this->client->handle(new Command\MyOrganizationsCommand()));
    }

    public function licenses(): Observable
    {
        return unwrapObservableFromPromise($this->client->handle(new Command\LicensesCommand()));
    }

    /**
     * @return RateLimitState
     */
    public function getRateLimitState(): RateLimitState
    {
        return clone $this->rateLimitState;
    }

    public function rateLimit(): PromiseInterface
    {
        return $this->client->handle(new Command\RateLimitCommand());
    }
}


1		<?php declare(strict_types=1);
2
3		namespace ApiClients\Client\Github;
4
5		use ApiClients\Client\Github\CommandBus\Command;
6		use ApiClients\Foundation\ClientInterface;
		0 ignored issues – show Bug introduced 2017-04-16 14:00 UTC by Report Bug Copy Issue Report This use statement conflicts with another class in this namespace, `ApiClients\Client\Github\ClientInterface`. Let’s assume that you have a directory layout like this: . \|-- OtherDir \| \|-- Bar.php \| `-- Foo.php `-- SomeDir `-- Foo.php and let’s assume the following content of `Bar.php`: // Bar.php namespace OtherDir; use SomeDir\Foo; // This now conflicts the class OtherDir\Foo If both files `OtherDir/Foo.php` and `SomeDir/Foo.php` are loaded in the same runtime, you will see a PHP error such as the following: PHP Fatal error: Cannot use SomeDir\Foo as Foo because the name is already in use in OtherDir/Foo.php However, as `OtherDir/Foo.php` does not necessarily have to be loaded and the error is only triggered if it is loaded before `OtherDir/Bar.php`, this problem might go unnoticed for a while. In order to prevent this error from surfacing, you must import the namespace with a different alias: // Bar.php namespace OtherDir; use SomeDir\Foo as SomeDirFoo; // There is no conflict anymore. Loading history...
7		use ApiClients\Foundation\Factory;
8		use ApiClients\Foundation\Options;
9		use React\EventLoop\LoopInterface;
10		use React\Promise\PromiseInterface;
11		use Rx\Observable;
12		use Rx\Scheduler;
13		use function ApiClients\Tools\Rx\unwrapObservableFromPromise;
14
15		final class AsyncClient implements AsyncClientInterface
16		{
17		/**
18		* @var ClientInterface
19		*/
20		private $client;
21
22		/**
23		* @var RateLimitState
24		*/
25		private $rateLimitState;
26
27		/**
28		* @param LoopInterface $loop
29		* @param AuthenticationInterface $auth
30		* @param array $options
31		* @return AsyncClient
32		*/
33	1	public static function create(
34		LoopInterface $loop,
35		AuthenticationInterface $auth,
36		array $options = []
37		): self {
38	1	$options = ApiSettings::getOptions($auth, $options, 'Async');
39	1	$rateLimitState = new RateLimitState();
40	1	$options[Options::CONTAINER_DEFINITIONS][RateLimitState::class] = $rateLimitState;
41	1	$client = Factory::create($loop, $options);
42
43		try {
44	1	Scheduler::setAsyncFactory(function () use ($loop) {
45		return new Scheduler\EventLoopScheduler($loop);
		0 ignored issues – show Documentation introduced 2017-03-25 16:00 UTC by Report Bug Copy Issue Report `$loop` is of type `object<React\EventLoop\LoopInterface>`, but the function expects a `callable`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
46	1	});
47		} catch (\Throwable $t) {
		0 ignored issues – show Coding Style Comprehensibility introduced 2017-03-25 16:00 UTC by Report Bug Copy Issue Report Consider adding a comment why this CATCH block is empty. Loading history...
48		}
49
50	1	return new self($client, $rateLimitState);
51		}
52
53		/**
54		* @internal
55		* @param ClientInterface $client
56		* @param RateLimitState $rateLimitState
57		* @return AsyncClient
58		*/
59	1	public static function createFromClient(ClientInterface $client, RateLimitState $rateLimitState): self
60		{
61	1	return new self($client, $rateLimitState);
62		}
63
64		/**
65		* @param ClientInterface $client
66		* @param RateLimitState $rateLimitState
67		*/
68	2	private function __construct(ClientInterface $client, RateLimitState $rateLimitState)
69		{
70	2	$this->client = $client;
71	2	$this->rateLimitState = $rateLimitState;
72	2	}
73
74		public function meta(): PromiseInterface
75		{
76		return $this->client->handle(new Command\MetaCommand());
77		}
78
79		/**
80		* @param string $user
81		* @return PromiseInterface
82		*/
83		public function user(string $user): PromiseInterface
84		{
85		return $this->client->handle(new Command\UserCommand($user));
86		}
87
88		/**
89		* @return PromiseInterface
90		*/
91		public function whoami(): PromiseInterface
92		{
93		return $this->client->handle(new Command\UserCommand());
94		}
95
96		/**
97		* @return Observable
98		*/
99		public function emojis(): Observable
100		{
101		return unwrapObservableFromPromise($this->client->handle(new Command\EmojisCommand()));
102		}
103
104		public function myOrganizations(): Observable
105		{
106		return unwrapObservableFromPromise($this->client->handle(new Command\MyOrganizationsCommand()));
107		}
108
109		public function licenses(): Observable
110		{
111		return unwrapObservableFromPromise($this->client->handle(new Command\LicensesCommand()));
112		}
113
114		/**
115		* @return RateLimitState
116		*/
117	2	public function getRateLimitState(): RateLimitState
118		{
119	2	return clone $this->rateLimitState;
120		}
121
122		public function rateLimit(): PromiseInterface
123		{
124		return $this->client->handle(new Command\RateLimitCommand());
125		}
126		}
127

php-api-clients / github

GitHub Access Token became invalid

Pull Request — master (#16)

AsyncClient::whoami() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like