Sebaks\Crud\Controller\UpdateController

Complexity

Total Complexity

5

Size/Duplication

Total Lines	79
Duplicated Lines	6.33 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes	6
Bugs	0	Features	4

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	14	1
B	onDispatch()	5	23	4

<?php

namespace Sebaks\Crud\Controller;

use Zend\Mvc\MvcEvent;

use Zend\Mvc\Controller\AbstractActionController;
use T4webDomainInterface\Service\UpdaterInterface;
use Sebaks\Crud\View\Model\UpdateViewModelInterface;

class UpdateController extends AbstractActionController
{
    /**
     * @var int
     */
    private $id;

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

    /**
     * @var UpdaterInterface
     */
    private $updater;

    /**
     * @var UpdateViewModelInterface
     */
    private $viewModel;

    /**
     * @var string
     */
    private $redirectTo;

    /**
     * @param $id
     * @param array $data
     * @param UpdaterInterface $updater
     * @param UpdateViewModelInterface $viewModel
     * @param null $redirectTo
     */
    public function __construct(
        $id,
        array $data,
        UpdaterInterface $updater,
        UpdateViewModelInterface $viewModel,
        $redirectTo = null
    )
    {
        $this->id = $id;
        $this->data = $data;
        $this->updater = $updater;
        $this->viewModel = $viewModel;
        $this->redirectTo = $redirectTo;
    }

    /**
     * Execute the request
     *
     * @param  MvcEvent $e
     * @return UpdateViewModelInterface|\Zend\Http\Response
     */
    public function onDispatch(MvcEvent $e)
    {
        $entity = $this->updater->update($this->id, $this->data);

        if (!$entity) {
            return $this->notFoundAction();
        }

        $this->viewModel->setEntity($entity);
        $e->setResult($this->viewModel);

        if ($this->updater->hasErrors()) {

This code seems to be duplicated across your project.

            $this->viewModel->setErrors($this->updater->getErrors());
            $this->viewModel->setInputData($this->data);
            return $this->viewModel;
        }

        if ($this->redirectTo) {
            return $this->redirect()->toRoute($this->redirectTo);
        }

        return $this->viewModel;
    }
}