HasFormFields::setField() - Code Metrics - Inspection of "update form factory" - delboy1978uk/form - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( ca4e2d...533f2e )

by Derek Stephen

created 2025-09-02 17:48 UTC

HasFormFields::setField() C

↳ Parent: HasFormFields

Complexity

Conditions	16
Paths	16

Size

Total Lines	31
Code Lines	28

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
eloc	28
c	2
b	0
f	0
dl	0
loc	31
rs	5.5666
cc	16
nc	16
nop	3

How to fix Complexity

<?php

declare(strict_types=1);

namespace Del\Form\Traits;

use Del\Form\Field\Attributes\Field;
use Del\Form\FormInterface;
use ReflectionClass;
use function array_shift;
use function count;
use function explode;
use function str_contains;
use function ucfirst;

trait HasFormFields
{
    public function populate(FormInterface $form): void
    {
        $data = $form->getValues(true);
        $mirror = new ReflectionClass($this);
        $properties = $mirror->getProperties();

        foreach ($properties as $property) {
            $fieldName = $property->getName();
            $attributes = $property->getAttributes(Field::class);

            if (count($attributes) > 0) {
                $rules = $attributes[0]->newInstance()->rules;

                if (str_contains($rules, '|')) {
                    $rules = explode('|', $rules);
                    $fieldType = array_shift($rules);
                } else {
                    $fieldType = $rules;
                    $rules = [];

                }

                if (str_contains($fieldType, 'date_format')) {
                    $fieldType = str_contains($fieldType, 'H:i')
                        ? 'datetime'
                        : 'date';
                }

                $value = $data[$fieldName];
                $this->setField($fieldName, $fieldType, $value);
            }
        }
    }

    private function setField(string $fieldName, string $fieldType, mixed $value): void
    {
        $setter = 'set' . ucfirst($fieldName);

        switch ($fieldType) {
            case 'checkbox':
                $this->$setter((bool) $value);
                break;
            case 'email':
            case 'file':
            case 'hidden':
            case 'password':
            case 'string':
            case 'text':
            case 'textarea':
                $this->$setter((string) $value);
                break;
            case 'integer':
                $this->$setter((int) $value);
                break;
            case 'float':
                $this->$setter((float) $value);
                break;
            case 'date':
            case 'datetime':
            case 'multiselect':
            case 'radio':
            case 'select':
            default:
                $this->$setter($value);
                break;
        }
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Del\Form\Traits;
6
7			use Del\Form\Field\Attributes\Field;
8			use Del\Form\FormInterface;
9			use ReflectionClass;
10			use function array_shift;
11			use function count;
12			use function explode;
13			use function str_contains;
14			use function ucfirst;
15
16			trait HasFormFields
17			{
18			public function populate(FormInterface $form): void
19			{
20			$data = $form->getValues(true);
21			$mirror = new ReflectionClass($this);
22			$properties = $mirror->getProperties();
23
24			foreach ($properties as $property) {
25			$fieldName = $property->getName();
26			$attributes = $property->getAttributes(Field::class);
27
28			if (count($attributes) > 0) {
29			$rules = $attributes[0]->newInstance()->rules;
30
31			if (str_contains($rules, '\|')) {
32			$rules = explode('\|', $rules);
33			$fieldType = array_shift($rules);
34			} else {
35			$fieldType = $rules;
36			$rules = [];
			0 ignored issues – show Unused Code introduced 2025-07-06 10:31 UTC by Report Bug Copy Issue Report The assignment to `$rules` is dead and can be removed. Loading history...
37			}
38
39			if (str_contains($fieldType, 'date_format')) {
40			$fieldType = str_contains($fieldType, 'H:i')
41			? 'datetime'
42			: 'date';
43			}
44
45			$value = $data[$fieldName];
46			$this->setField($fieldName, $fieldType, $value);
47			}
48			}
49			}
50
51			private function setField(string $fieldName, string $fieldType, mixed $value): void
52			{
53			$setter = 'set' . ucfirst($fieldName);
54
55			switch ($fieldType) {
56			case 'checkbox':
57			$this->$setter((bool) $value);
58			break;
59			case 'email':
60			case 'file':
61			case 'hidden':
62			case 'password':
63			case 'string':
64			case 'text':
65			case 'textarea':
66			$this->$setter((string) $value);
67			break;
68			case 'integer':
69			$this->$setter((int) $value);
70			break;
71			case 'float':
72			$this->$setter((float) $value);
73			break;
74			case 'date':
75			case 'datetime':
76			case 'multiselect':
77			case 'radio':
78			case 'select':
79			default:
80			$this->$setter($value);
81			break;
82			}
83			}
84			}
85

delboy1978uk / form

Push — master ( ca4e2d...533f2e )

HasFormFields::setField() C

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like