rtens\domin\delivery\web\ActionForm

Complexity

Total Complexity

9

Size/Duplication

Total Lines	58
Duplicated Lines	12.07 %

Coupling/Cohesion

Components	0
Dependencies	7

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	getModel()	0	3	1
A	getHeadElements()	0	3	1
A	getActionField()	7	7	2
A	readParameters()	0	19	4

<?php
namespace rtens\domin\delivery\web;

use rtens\domin\Action;
use rtens\domin\delivery\FieldRegistry;
use rtens\domin\delivery\ParameterReader;
use rtens\domin\Parameter;
use watoki\reflect\type\ClassType;

class ActionForm {

    /** @var array */
    private $model;

    /** @var array|Element[] */
    private $headElements = [];

    public function __construct(ParameterReader $reader, FieldRegistry $fields, Action $action, $actionId) {
        $actionParameter = new Parameter($actionId, new ClassType(get_class($action)));
        $actionField = $this->getActionField($actionParameter, $fields);

        $this->model = $actionField->render($actionParameter, $this->readParameters($reader, $fields, $action));

It seems like $actionField->render($actionParameter, $this->readParameters($reader, $fields, $action)) of type string is incompatible with the declared type array of property $model.

        $this->headElements = $actionField->headElements($actionParameter);
    }

    public function getModel() {
        return $this->model;
    }

    public function getHeadElements() {
        return $this->headElements;
    }

    /**
     * @param Parameter $actionParameter
     * @param FieldRegistry $fields
     * @return WebField
     * @throws \Exception
     */
    private function getActionField(Parameter $actionParameter, FieldRegistry $fields) {

This method seems to be duplicated in your project.

        $actionField = $fields->getField($actionParameter);
        if ($actionField instanceof WebField) {
            return $actionField;
        }
        throw new \Exception(get_class($actionField) . " must implement WebField");
    }

    private function readParameters(ParameterReader $reader, FieldRegistry $fields, Action $action) {
        $values = [
            'inflated' => [],
            'errors' => []
        ];

        foreach ($action->parameters() as $parameter) {
            if ($reader->has($parameter)) {
                $field = $fields->getField($parameter);

                try {
                    $values['inflated'][$parameter->getName()] = $field->inflate($parameter, $reader->read($parameter));
                } catch (\Exception $e) {
                    $values['errors'][$parameter->getName()] = $e;
                }
            }
        }
        return $values;
    }
}