ObjectType::getName() - Code Metrics - Inspection of "Refactoring plugins and decoupling them from the s..." - drupal-graphql/graphql - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — 8.x-3.x ( e67c0e...2e01b0 )

by Philipp

created 2017-11-15 13:45 UTC

ObjectType::getName() A

↳ Parent: ObjectType

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

namespace Drupal\graphql\GraphQL\Type;

use Drupal\graphql\GraphQL\CacheableEdgeInterface;
use Drupal\graphql\GraphQL\CacheableEdgeTrait;
use Drupal\graphql\Plugin\GraphQL\Types\TypePluginBase;
use Drupal\graphql\Plugin\GraphQL\TypeSystemPluginReferenceInterface;
use Drupal\graphql\Plugin\GraphQL\TypeSystemPluginReferenceTrait;
use Youshido\GraphQL\Config\Object\ObjectTypeConfig;
use Youshido\GraphQL\Execution\ResolveInfo;
use Youshido\GraphQL\Type\Object\AbstractObjectType;

class ObjectType extends AbstractObjectType implements TypeSystemPluginReferenceInterface, CacheableEdgeInterface  {
  use TypeSystemPluginReferenceTrait;
  use CacheableEdgeTrait;

  /**
   * {@inheritdoc}
   */
  public function __construct(TypePluginBase $plugin, array $config = []) {
    $this->plugin = $plugin;
    $this->config = new ObjectTypeConfig($config, $this, TRUE);
  }

  /**
   * {@inheritdoc}
   */
  public function getName() {
    return $this->getConfigValue('name');
  }

  /**
   * Checks whether this type applies to a given object.
   *
   * @param mixed $object
   *   The object to check against.
   * @param \Youshido\GraphQL\Execution\ResolveInfo|null $info
   *   The resolve info object.
   *
   * @return bool
   *   TRUE if this type applies to the given object, FALSE otherwise.
   */
  public function applies($object, ResolveInfo $info = NULL) {
    if (($plugin = $this->getPluginFromResolveInfo($info)) && $plugin instanceof TypePluginBase) {
function notNullable(stdClass $x) { }

// Unsafe
function withoutCheck(stdClass $x = null) {
    notNullable($x);
}

// Safe - Alternative 1: Adding Additional Type-Check
function withCheck(stdClass $x = null) {
    if ($x instanceof stdClass) {
        notNullable($x);
    }
}

// Safe - Alternative 2: Changing Parameter
function withNonNullableParam(stdClass $x) {
    notNullable($x);
}
      return $plugin->applies($object);
    }

    return FALSE;
  }

  /**
   * {@inheritdoc}
   */
  public function build($config) {
    // Nothing to do here.
  }

}


1			<?php
2
3			namespace Drupal\graphql\GraphQL\Type;
4
5			use Drupal\graphql\GraphQL\CacheableEdgeInterface;
6			use Drupal\graphql\GraphQL\CacheableEdgeTrait;
7			use Drupal\graphql\Plugin\GraphQL\Types\TypePluginBase;
8			use Drupal\graphql\Plugin\GraphQL\TypeSystemPluginReferenceInterface;
9			use Drupal\graphql\Plugin\GraphQL\TypeSystemPluginReferenceTrait;
10			use Youshido\GraphQL\Config\Object\ObjectTypeConfig;
11			use Youshido\GraphQL\Execution\ResolveInfo;
12			use Youshido\GraphQL\Type\Object\AbstractObjectType;
13
14			class ObjectType extends AbstractObjectType implements TypeSystemPluginReferenceInterface, CacheableEdgeInterface {
15			use TypeSystemPluginReferenceTrait;
16			use CacheableEdgeTrait;
17
18			/**
19			* {@inheritdoc}
20			*/
21			public function __construct(TypePluginBase $plugin, array $config = []) {
22			$this->plugin = $plugin;
23			$this->config = new ObjectTypeConfig($config, $this, TRUE);
24			}
25
26			/**
27			* {@inheritdoc}
28			*/
29			public function getName() {
30			return $this->getConfigValue('name');
31			}
32
33			/**
34			* Checks whether this type applies to a given object.
35			*
36			* @param mixed $object
37			* The object to check against.
38			* @param \Youshido\GraphQL\Execution\ResolveInfo\|null $info
39			* The resolve info object.
40			*
41			* @return bool
42			* TRUE if this type applies to the given object, FALSE otherwise.
43			*/
44			public function applies($object, ResolveInfo $info = NULL) {
45			if (($plugin = $this->getPluginFromResolveInfo($info)) && $plugin instanceof TypePluginBase) {
			0 ignored issues – show Bug introduced 2017-11-15 10:20 UTC by Report Bug Copy Issue Report It seems like `$info` defined by parameter `$info` on line `44` can be `null`; however, `Drupal\graphql\Plugin\Gr...PluginFromResolveInfo()` does not accept `null`, maybe add an additional type check? It seems like you allow that null is being passed for a parameter, however the function which is called does not seem to accept null. We recommend to add an additional type check (or disallow null for the parameter): function notNullable(stdClass $x) { } // Unsafe function withoutCheck(stdClass $x = null) { notNullable($x); } // Safe - Alternative 1: Adding Additional Type-Check function withCheck(stdClass $x = null) { if ($x instanceof stdClass) { notNullable($x); } } // Safe - Alternative 2: Changing Parameter function withNonNullableParam(stdClass $x) { notNullable($x); } Loading history...
46			return $plugin->applies($object);
47			}
48
49			return FALSE;
50			}
51
52			/**
53			* {@inheritdoc}
54			*/
55			public function build($config) {
56			// Nothing to do here.
57			}
58
59			}
60

drupal-graphql / graphql

Push — 8.x-3.x ( e67c0e...2e01b0 )

ObjectType::getName() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like