GraphQLPromiseAdapter::convertThenable() - Code Metrics - Inspection of "Promise" - overblog/GraphQLBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#73)

by Jérémiah

created 2017-01-26 15:15 UTC

GraphQLPromiseAdapter::convertThenable() A

↳ Parent: GraphQLPromiseAdapter

Complexity

Conditions	2
Paths	2

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	2

Importance

Changes

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

<?php

/*
 * This file is part of the OverblogGraphQLBundle package.
 *
 * (c) Overblog <http://github.com/overblog/>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Overblog\GraphQLBundle\Executor\Promise\Adapter;

use GraphQL\Executor\Promise\Adapter\SyncPromise;
use GraphQL\Executor\Promise\Adapter\SyncPromiseAdapter as BaseSyncPromiseAdapter;
use GraphQL\Executor\Promise\Promise;
use Overblog\GraphQLBundle\Executor\Promise\PromiseAdapterInterface;

class GraphQLPromiseAdapter extends BaseSyncPromiseAdapter implements PromiseAdapterInterface
{
    /**
     * {@inheritdoc}
     */
    public function isThenable($value)
    {
        $valueOrPromise = $value instanceof Promise ? $value->adoptedPromise : $value;

        return parent::isThenable($valueOrPromise) || $valueOrPromise instanceof SyncPromise;
    }

    /**
     * {@inheritdoc}
     */
    public function convertThenable($thenable)
    {
        if ($thenable instanceof Promise) {
            return $thenable;
        }

        return parent::convertThenable($thenable);
    }
}


1		<?php
2
3		/*
4		* This file is part of the OverblogGraphQLBundle package.
5		*
6		* (c) Overblog <http://github.com/overblog/>
7		*
8		* For the full copyright and license information, please view the LICENSE
9		* file that was distributed with this source code.
10		*/
11
12		namespace Overblog\GraphQLBundle\Executor\Promise\Adapter;
13
14		use GraphQL\Executor\Promise\Adapter\SyncPromise;
15		use GraphQL\Executor\Promise\Adapter\SyncPromiseAdapter as BaseSyncPromiseAdapter;
16		use GraphQL\Executor\Promise\Promise;
17		use Overblog\GraphQLBundle\Executor\Promise\PromiseAdapterInterface;
18
19		class GraphQLPromiseAdapter extends BaseSyncPromiseAdapter implements PromiseAdapterInterface
20		{
21		/**
22		* {@inheritdoc}
23		*/
24	34	public function isThenable($value)
25		{
26	34	$valueOrPromise = $value instanceof Promise ? $value->adoptedPromise : $value;
27
28	34	return parent::isThenable($valueOrPromise) \|\| $valueOrPromise instanceof SyncPromise;
29		}
30
31		/**
32		* {@inheritdoc}
33		*/
34	8	public function convertThenable($thenable)
35		{
36	8	if ($thenable instanceof Promise) {
37	2	return $thenable;
38		}
39
40	8	return parent::convertThenable($thenable);
41		}
42		}
43

overblog / GraphQLBundle

Scrutinizer GitHub App not installed

Pull Request — master (#73)

GraphQLPromiseAdapter::convertThenable() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like