Form::addElements() - Code Metrics - Inspection of "[update] use new name for fields" - x-systems/epesi-core - Measure and Improve Code Quality continuously with Scrutinizer

Test Setup Failed

Push — master ( eda994...e2bd49 )

by Georgi

created 2022-02-02 07:36 UTC

Form::addElements() B

↳ Parent: Form

Complexity

Conditions	11
Paths	14

Size

Total Lines	58
Code Lines	34

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	11
eloc	34
nc	14
nop	2
dl	0
loc	58
rs	7.3166
c	1
b	0
f	0

How to fix Long Method Complexity

<?php

namespace Epesi\Core\System\View;

use atk4\ui\Form as BaseForm;
use Epesi\Core\System\Modules\Concerns\Notifies;
use atk4\ui\Label;

class Form extends BaseForm
{
	use Notifies;
	
	public $buttonSave = null;
	
	protected $controlRules = [];
	protected $validationRules = [];
	
	public function addElements($elements, $parent = null) {
		$parent = $parent?: $this;
		
		foreach ($elements as $name => $desc) {
			$name = $desc['name']?? $name;
			
			$this->addControlRules($name, $desc['rules']?? []);
			
			switch ($desc['type']?? 'field') {
				case 'field':
					$desc = is_string($desc)? [
						'decorator' => [$desc]
					]: $desc;
					
					$field = $parent->addControl($name, $desc['decorator']?? [], $desc['options']?? []);
					
					if ($default = $desc['default']) {
						$field->set($default);
					}
					
					if ($desc['display']?? false) {
						$this->addControlDisplayRules([$name => $desc['display']]);
					}
					break;
					
				case 'group':
					$seed = $desc['seed']?? [$name];
					
					$group = $parent->addGroup($seed);
					
					$this->addElements($desc['elements'], $group);
					
					if ($desc['display']?? false) {
						$this->addGroupDisplayRules([$name => $desc['display']]);
					}
					break;
					
				case 'header':
					$seed = $desc['seed']?? [$name];
					
					$parent->addHeader($seed);
					$parent->addHeader(/** @scrutinizer ignore-type */ $seed);
					break;
					
				case 'view':
					$seed = $desc['seed']?? [Label::class, $name];
					
					$region = $desc['region']?? null;
					
					$this->add($seed, $region);
					$this->add(/** @scrutinizer ignore-type */ $seed, $region);
					break;
					
				default:
					;
					break;
			}
		}
		
		return $this;
	}
	
	public function addControlDisplayRules($controlDisplayRules) {
		$this->setControlsDisplayRules(array_merge($this->controlDisplayRules?: [], $controlDisplayRules));
	}
	
	public function addGroupDisplayRules($groupDisplayRules) {
		$controlDisplayRules = $this->controlDisplayRules;
		
		$this->setGroupDisplayRules($groupDisplayRules);
		
		$this->addControlDisplayRules($controlDisplayRules);
	}
	
	public function addControlRules($field, $rules = []) {
		if (! $rules) return;
		
		$this->controlRules[$field] = $rules['rules']?? [
				'identifier' => $field,
				'rules' => $rules
		];
		
		return $this;
	}
	
	public function validate($callback)
	{
		$this->setApiConfig([
				'beforeSend' => new \atk4\ui\JsFunction(['settings'], [
						new \atk4\ui\JsExpression('return $(this).form("is valid")')
				]),
		]);
		
		$this->setFormConfig([
				'fields' => $this->controlRules
		]);
		
		$this->onSubmit(function ($form) use ($callback) {
			$errors = [];
			foreach ($this->validationRules?: [] as $ruleCallback) {
				if (! is_callable($ruleCallback)) continue;
				
				$ruleErrors = $ruleCallback($form);
				
				$errors = array_merge($errors, $ruleErrors?: []);
			}
			
			return $errors?: $callback($this);
		});
			
		return $this;
	}
	
	public function submit()
	{
		return $this->js()->form('submit');
	}
	
	public function confirmLeave($confirm = true)
	{
		$this->canLeave = ! $confirm;
		
		return $this;
	}
}

x-systems / epesi-core

Push — master ( eda994...e2bd49 )

Form::addElements() B

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like