ElementalCMSMainExtension - Code Metrics - silverstripe/silverstripe-elemental - Measure and Improve Code Quality continuously with Scrutinizer

ElementalCMSMainExtension A
last analyzed 2022-06-09 00:13 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	14
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	4
c	1
b	0
f	0
dl	0
loc	14
rs	10
wmc	2

1 Method

Rating	Name	Duplication	Size	Complexity
A	updateSearchForm()	0	6	2

<?php

namespace DNADesign\Elemental\Extensions;

use SilverStripe\Core\Extension;
use SilverStripe\Forms\DropdownField;
use SilverStripe\Forms\Form;

class ElementalCMSMainExtension extends Extension
{
    /**
     * Remove the empty default string on the class filter, which adds "All pages" again. This is already
     * added by ElementSiteTreeFilterSearch.
     *
     * @param Form $form
     */
    public function updateSearchForm(Form $form)
    {
        /** @var DropdownField $filterField */
        $filterField = $form->Fields()->fieldByName('Search__FilterClass');
class A
{
    function getObject()
    {
        return null;
    }

}

$a = new A();
$object = $a->getObject();
        if ($filterField) {

            $filterField->setEmptyString('')->setHasEmptyDefault(false);
        }
    }
}


1			<?php
2
3			namespace DNADesign\Elemental\Extensions;
4
5			use SilverStripe\Core\Extension;
6			use SilverStripe\Forms\DropdownField;
7			use SilverStripe\Forms\Form;
8
9			class ElementalCMSMainExtension extends Extension
10			{
11			/**
12			* Remove the empty default string on the class filter, which adds "All pages" again. This is already
13			* added by ElementSiteTreeFilterSearch.
14			*
15			* @param Form $form
16			*/
17			public function updateSearchForm(Form $form)
18			{
19			/** @var DropdownField $filterField */
20			$filterField = $form->Fields()->fieldByName('Search__FilterClass');
			0 ignored issues – show Bug introduced 2021-02-28 23:44 UTC by Report Bug Copy Issue Report Are you sure the assignment to `$filterField` is correct as `$form->Fields()->fieldBy...('Search__FilterClass')` targeting `SilverStripe\Forms\FieldList::fieldByName()` seems to always return null. This check looks for function or method calls that always return null and whose return value is assigned to a variable. class A { function getObject() { return null; } } $a = new A(); $object = $a->getObject(); The method `getObject()` can return nothing but null, so it makes no sense to assign that value to a variable. The reason is most likely that a function or method is imcomplete or has been reduced for debug purposes. Loading history...
21			if ($filterField) {
			0 ignored issues – show introduced 2021-02-28 23:44 UTC by Report Bug Copy Issue Report `$filterField` is of type `SilverStripe\Forms\DropdownField`, thus it always evaluated to `true`. Loading history...
22			$filterField->setEmptyString('')->setHasEmptyDefault(false);
23			}
24			}
25			}
26

silverstripe / silverstripe-elemental

ElementalCMSMainExtension A last analyzed 2022-06-09 00:13 UTC

Complexity

Size/Duplication

Importance

1 Method

Duplication Side-by-Side

Filter issues like

ElementalCMSMainExtension A
last analyzed 2022-06-09 00:13 UTC