UserFormsFieldList::processNext() - Code Metrics - silverstripe/silverstripe-userforms - Measure and Improve Code Quality continuously with Scrutinizer

UserFormsFieldList::processNext() A
last analyzed 2020-07-07 03:40 UTC

↳ Parent: UserFormsFieldList

Complexity

Conditions	3
Paths	3

Size

Total Lines	14
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

namespace SilverStripe\UserForms\FormField;

use SilverStripe\Forms\FieldList;
use SilverStripe\UserForms\Model\EditableFormField;

/**
 * A list of formfields which allows for iterative processing of nested composite fields
 */
class UserFormsFieldList extends FieldList implements UserFormsFieldContainer
{
    public function processNext(EditableFormField $field)
    {
        $formField = $field->getFormField();
        if (!$formField) {

            return $this;
        }

        $this->push($formField);

        if ($formField instanceof UserFormsFieldContainer) {
            return $formField->setParent($this);

        }

        return $this;
interface HasName {
    /** @return string */
    public function getName();
}

class Name {
    public $name;
}

class User implements HasName {
    /** @return string|Name */
    public function getName() {
        return new Name('foo'); // This is a violation of the ``HasName`` interface
                                // which only allows a string value to be returned.
    }
}
    }

    public function getParent()
    {
        // Field list does not have a parent
        return null;
    }

    public function setParent(UserFormsFieldContainer $parent)
    {
        return $this;
    }

    /**
     * Remove all empty steps
     */
    public function clearEmptySteps()
    {
        foreach ($this as $field) {
            if ($field instanceof UserFormsStepField && count($field->getChildren()) === 0) {
                $this->remove($field);
            }
        }
    }
}


1			<?php
2
3			namespace SilverStripe\UserForms\FormField;
4
5			use SilverStripe\Forms\FieldList;
6			use SilverStripe\UserForms\Model\EditableFormField;
7
8			/**
9			* A list of formfields which allows for iterative processing of nested composite fields
10			*/
11			class UserFormsFieldList extends FieldList implements UserFormsFieldContainer
12			{
13			public function processNext(EditableFormField $field)
14			{
15			$formField = $field->getFormField();
16			if (!$formField) {
			0 ignored issues – show introduced 2018-10-22 18:35 UTC by Report Bug Copy Issue Report `$formField` is of type `SilverStripe\Forms\FormField`, thus it always evaluated to `true`. Loading history...
17			return $this;
18			}
19
20			$this->push($formField);
21
22			if ($formField instanceof UserFormsFieldContainer) {
23			return $formField->setParent($this);
			0 ignored issues – show Bug Best Practice introduced 2018-04-28 12:44 UTC by Report Bug Copy Issue Report The expression `return $formField->setParent($this)` also could return the type `SilverStripe\Forms\FormF...UserFormsFieldContainer` which is incompatible with the return type mandated by `SilverStripe\UserForms\F...ontainer::processNext()` of `SilverStripe\UserForms\F...\EditableContainerField`. Loading history...
24			}
25
26			return $this;
			0 ignored issues – show Bug Best Practice introduced 2017-11-20 04:54 UTC by Report Bug Copy Issue Report The expression `return $this` returns the type `SilverStripe\UserForms\F...ield\UserFormsFieldList` which is incompatible with the return type mandated by `SilverStripe\UserForms\F...ontainer::processNext()` of `SilverStripe\UserForms\F...\EditableContainerField`. In the issue above, the returned value is violating the contract defined by the mentioned interface. Let's take a look at an example: interface HasName { /** @return string / public function getName(); } class Name { public $name; } class User implements HasName { /* @return string\|Name */ public function getName() { return new Name('foo'); // This is a violation of the ``HasName`` interface // which only allows a string value to be returned. } } Loading history...
27			}
28
29			public function getParent()
30			{
31			// Field list does not have a parent
32			return null;
33			}
34
35			public function setParent(UserFormsFieldContainer $parent)
36			{
37			return $this;
38			}
39
40			/**
41			* Remove all empty steps
42			*/
43			public function clearEmptySteps()
44			{
45			foreach ($this as $field) {
46			if ($field instanceof UserFormsStepField && count($field->getChildren()) === 0) {
47			$this->remove($field);
48			}
49			}
50			}
51			}
52

silverstripe / silverstripe-userforms

UserFormsFieldList::processNext() A last analyzed 2020-07-07 03:40 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

UserFormsFieldList::processNext() A
last analyzed 2020-07-07 03:40 UTC