EntityBrowserListBuilder::render() - Code Metrics - Inspection of "Config ui" - drupal-media/entity_browser - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — 8.x-1.x (#137)

unknown

created 2016-01-12 09:45 UTC

EntityBrowserListBuilder::render() A

↳ Parent: EntityBrowserListBuilder

Complexity

Conditions	1
Paths	1

Size

Total Lines	7
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	1	Features	0

Metric	Value
c	1
b	1
f	0
dl	0
loc	7
rs	9.4286
cc	1
eloc	5
nc	1
nop	0

<?php

/**
 * @file
 * Contains \Drupal\entity_browser\Entity\Controllers\EntityBrowserListBuilder.
 */

namespace Drupal\content_entity_example\Entity\Controller;

use Drupal\Core\Entity\EntityInterface;
use Drupal\Core\Entity\EntityListBuilder;


/**
 * Provides a list controller for entity browser.
 *
 * @ingroup entity_browser
 */
class EntityBrowserListBuilder extends EntityListBuilder {

  /**
   * {@inheritdoc}
   *
   * We override ::render() so that we can add our own content above the table.
   * parent::render() is where EntityListBuilder creates the table using our
   * buildHeader() and buildRow() implementations.
   */
  public function render() {
    $build['description'] = array(
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
      '#markup' => $this->t('Entity Browser'),
    );
    $build['table'] = parent::render();
    return $build;
  }

  /**
   * {@inheritdoc}
   *
   * Building the header and content lines for the entity browser list.
   *
   * Calling the parent::buildHeader() adds a column for the possible actions
   * and inserts the 'edit' and 'delete' links as defined for the entity type.
   */
  public function buildHeader() {
    $header['id'] = $this->t('EB ID');
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
    $header['name'] = $this->t('Name');
    return $header + parent::buildHeader();
  }

  /**
   * {@inheritdoc}
   */
  public function buildRow(EntityInterface $entity) {
    /* @var $entity \Drupal\entity_browser\Entity\EntityBrowser */
    $row['id'] = $entity->id();
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
    $row['name'] = $entity->link();
    return $row + parent::buildRow($entity);
  }

}


1			<?php
2
3			/**
4			* @file
5			* Contains \Drupal\entity_browser\Entity\Controllers\EntityBrowserListBuilder.
6			*/
7
8			namespace Drupal\content_entity_example\Entity\Controller;
9
10			use Drupal\Core\Entity\EntityInterface;
11			use Drupal\Core\Entity\EntityListBuilder;
12
13
14			/**
15			* Provides a list controller for entity browser.
16			*
17			* @ingroup entity_browser
18			*/
19			class EntityBrowserListBuilder extends EntityListBuilder {
20
21			/**
22			* {@inheritdoc}
23			*
24			* We override ::render() so that we can add our own content above the table.
25			* parent::render() is where EntityListBuilder creates the table using our
26			* buildHeader() and buildRow() implementations.
27			*/
28			public function render() {
29			$build['description'] = array(
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-01-04 12:22 UTC by Report Bug Copy Issue Report `$build` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$build = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
30			'#markup' => $this->t('Entity Browser'),
31			);
32			$build['table'] = parent::render();
33			return $build;
34			}
35
36			/**
37			* {@inheritdoc}
38			*
39			* Building the header and content lines for the entity browser list.
40			*
41			* Calling the parent::buildHeader() adds a column for the possible actions
42			* and inserts the 'edit' and 'delete' links as defined for the entity type.
43			*/
44			public function buildHeader() {
45			$header['id'] = $this->t('EB ID');
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-01-04 12:22 UTC by Report Bug Copy Issue Report `$header` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$header = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
46			$header['name'] = $this->t('Name');
47			return $header + parent::buildHeader();
48			}
49
50			/**
51			* {@inheritdoc}
52			*/
53			public function buildRow(EntityInterface $entity) {
54			/* @var $entity \Drupal\entity_browser\Entity\EntityBrowser */
55			$row['id'] = $entity->id();
			0 ignored issues – show Coding Style Comprehensibility introduced 2016-01-04 12:22 UTC by Report Bug Copy Issue Report `$row` was never initialized. Although not strictly required by PHP, it is generally a good practice to add `$row = array();` before regardless. Adding an explicit array definition is generally preferable to implicit array definition as it guarantees a stable state of the code. Let’s take a look at an example: foreach ($collection as $item) { $myArray['foo'] = $item->getFoo(); if ($item->hasBar()) { $myArray['bar'] = $item->getBar(); } // do something with $myArray } As you can see in this example, the array `$myArray` is initialized the first time when the foreach loop is entered. You can also see that the value of the `bar` key is only written conditionally; thus, its value might result from a previous iteration. This might or might not be intended. To make your intention clear, your code more readible and to avoid accidental bugs, we recommend to add an explicit initialization `$myArray = array()` either outside or inside the foreach loop. Loading history...
56			$row['name'] = $entity->link();
57			return $row + parent::buildRow($entity);
58			}
59
60			}
61

drupal-media / entity_browser

Pull Request — 8.x-1.x (#137)

EntityBrowserListBuilder::render() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like