MenuLinkLinks::resolveValues() - Code Metrics - Inspection of "Fix language handling and add language switch link..." - drupal-graphql/graphql - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — 8.x-3.x (#509)

by Sebastian

created 2018-02-13 11:16 UTC

MenuLinkLinks::resolveValues() A

↳ Parent: MenuLinkLinks

Complexity

Conditions	4
Paths	3

Size

Total Lines	14
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	4
eloc	8
nc	3
nop	3
dl	0
loc	14
rs	9.2
c	0
b	0
f	0

<?php

namespace Drupal\graphql_core\Plugin\GraphQL\Fields\MenuLink;

use Drupal\Core\Menu\MenuLinkInterface;
use Drupal\Core\Menu\MenuLinkTreeElement;
use Drupal\graphql\Plugin\GraphQL\Fields\FieldPluginBase;
use Youshido\GraphQL\Execution\ResolveInfo;

/**
 * Retrieve a menu links children.
 *
 * @GraphQLField(
 *   id = "menu_link_links",
 *   secure = true,
 *   name = "links",
 *   type = "[MenuLink]",
 *   parents = {"MenuLink"}
 * )
 */
class MenuLinkLinks extends FieldPluginBase {


  /**
   * {@inheritdoc}
   */
  public function resolveValues($value, array $args, ResolveInfo $info) {
    if ($value instanceof MenuLinkTreeElement) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
      $items = array_filter($value->subtree, function(MenuLinkTreeElement $item) {
        if ($item->link instanceof MenuLinkInterface) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
          return $item->link->isEnabled();
        }
        return TRUE;
      });

      foreach ($items as $branch) {
        yield $branch;
      }
    }
  }

}


Pull Request — 8.x-3.x (#509)

MenuLinkLinks::resolveValues() A

Complexity

Size

Duplication

Importance

1. Missing dependencies

2. Missing use statement

1. Missing dependencies

2. Missing use statement

1			<?php
2
3			namespace Drupal\graphql_core\Plugin\GraphQL\Fields\MenuLink;
4
5			use Drupal\Core\Menu\MenuLinkInterface;
6			use Drupal\Core\Menu\MenuLinkTreeElement;
7			use Drupal\graphql\Plugin\GraphQL\Fields\FieldPluginBase;
8			use Youshido\GraphQL\Execution\ResolveInfo;
9
10			/**
11			* Retrieve a menu links children.
12			*
13			* @GraphQLField(
14			* id = "menu_link_links",
15			* secure = true,
16			* name = "links",
17			* type = "[MenuLink]",
18			* parents = {"MenuLink"}
19			* )
20			*/
21			class MenuLinkLinks extends FieldPluginBase {
			0 ignored issues – show Bug introduced 2017-11-14 15:57 UTC by Report Bug Copy Issue Report There is one abstract method `getPluginDefinition` in this class; you could implement it, or declare this class as abstract. Loading history...
22
23			/**
24			* {@inheritdoc}
25			*/
26			public function resolveValues($value, array $args, ResolveInfo $info) {
27			if ($value instanceof MenuLinkTreeElement) {
			0 ignored issues – show Bug introduced 2017-10-17 14:57 UTC by Report Bug Copy Issue Report The class `Drupal\Core\Menu\MenuLinkTreeElement` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
28			$items = array_filter($value->subtree, function(MenuLinkTreeElement $item) {
29			if ($item->link instanceof MenuLinkInterface) {
			0 ignored issues – show Bug introduced 2017-10-17 14:57 UTC by Report Bug Copy Issue Report The class `Drupal\Core\Menu\MenuLinkInterface` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
30			return $item->link->isEnabled();
31			}
32			return TRUE;
33			});
34
35			foreach ($items as $branch) {
36			yield $branch;
37			}
38			}
39			}
40
41			}
42

drupal-graphql / graphql

Pull Request — 8.x-3.x (#509)

MenuLinkLinks::resolveValues() A

Complexity

Size

Duplication

Importance

1. Missing dependencies

2. Missing use statement

1. Missing dependencies

2. Missing use statement

Duplication Side-by-Side

Filter issues like