ViewResultCountDeriver::getDerivativeDefinitions() - Code Metrics - drupal-graphql/graphql-views - Measure and Improve Code Quality continuously with Scrutinizer

ViewResultCountDeriver::getDerivativeDefinitions() A
last analyzed 2021-03-25 10:44 UTC

↳ Parent: ViewResultCountDeriver

Complexity

Conditions	5
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	30
rs	9.1288
c	0
b	0
f	0
cc	5
nc	2
nop	1

<?php

namespace Drupal\graphql_views\Plugin\Deriver\Fields;

use Drupal\graphql_views\Plugin\Deriver\ViewDeriverBase;
use Drupal\views\Views;

/**
 * Derive fields from configured views.
 */
class ViewResultCountDeriver extends ViewDeriverBase {

  /**
   * {@inheritdoc}
   */
  public function getDerivativeDefinitions($basePluginDefinition) {
    if ($this->entityTypeManager->hasDefinition('view')) {
      $viewStorage = $this->entityTypeManager->getStorage('view');

      foreach (Views::getApplicableViews('graphql_display') as list($viewId, $displayId)) {
        /** @var \Drupal\views\ViewEntityInterface $view */
        $view = $viewStorage->load($viewId);
        /** @var \Drupal\graphql_views\Plugin\views\display\GraphQL $display */
        $display = $this->getViewDisplay($view, $displayId);
        if (!$this->isPaged($display)) {
          continue;
        }

        if (!$this->getRowResolveType($view, $displayId)) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
          continue;
        }

        $id = implode('-', [$viewId, $displayId, 'result', 'count']);
        $this->derivatives[$id] = [
          'id' => $id,
          'type' => 'Int',
          'parents' => [$display->getGraphQLResultName()],
          'view' => $viewId,
          'display' => $displayId,
        ] + $this->getCacheMetadataDefinition($view, $display) + $basePluginDefinition;
      }
    }

    return parent::getDerivativeDefinitions($basePluginDefinition);
  }

}


1			<?php
2
3			namespace Drupal\graphql_views\Plugin\Deriver\Fields;
4
5			use Drupal\graphql_views\Plugin\Deriver\ViewDeriverBase;
6			use Drupal\views\Views;
7
8			/**
9			* Derive fields from configured views.
10			*/
11			class ViewResultCountDeriver extends ViewDeriverBase {
12
13			/**
14			* {@inheritdoc}
15			*/
16			public function getDerivativeDefinitions($basePluginDefinition) {
17			if ($this->entityTypeManager->hasDefinition('view')) {
18			$viewStorage = $this->entityTypeManager->getStorage('view');
19
20			foreach (Views::getApplicableViews('graphql_display') as list($viewId, $displayId)) {
21			/** @var \Drupal\views\ViewEntityInterface $view */
22			$view = $viewStorage->load($viewId);
23			/** @var \Drupal\graphql_views\Plugin\views\display\GraphQL $display */
24			$display = $this->getViewDisplay($view, $displayId);
25			if (!$this->isPaged($display)) {
26			continue;
27			}
28
29			if (!$this->getRowResolveType($view, $displayId)) {
			0 ignored issues – show Bug Best Practice introduced 2017-12-05 07:57 UTC by Report Bug Copy Issue Report The expression `$this->getRowResolveType($view, $displayId)` 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...
30			continue;
31			}
32
33			$id = implode('-', [$viewId, $displayId, 'result', 'count']);
34			$this->derivatives[$id] = [
35			'id' => $id,
36			'type' => 'Int',
37			'parents' => [$display->getGraphQLResultName()],
38			'view' => $viewId,
39			'display' => $displayId,
40			] + $this->getCacheMetadataDefinition($view, $display) + $basePluginDefinition;
41			}
42			}
43
44			return parent::getDerivativeDefinitions($basePluginDefinition);
45			}
46
47			}
48

drupal-graphql / graphql-views

ViewResultCountDeriver::getDerivativeDefinitions() A last analyzed 2021-03-25 10:44 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

ViewResultCountDeriver::getDerivativeDefinitions() A
last analyzed 2021-03-25 10:44 UTC