FlightLog\Http\Web\Controller\PilotEditController

Complexity

Total Complexity

9

Size/Duplication

Total Lines	71
Duplicated Lines	26.76 %

Coupling/Cohesion

Components	1
Dependencies	7

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
B	index()	19	45	6
A	handle()	0	4	1
A	getHandler()	0	3	1
A	getPilotRepository()	0	4	1

<?php


namespace FlightLog\Http\Web\Controller;


use FlightLog\Application\Damage\Command\CreateDamageCommand;
use FlightLog\Application\Damage\Command\CreateDamageCommandHandler;
use FlightLog\Application\Pilot\Command\CreateUpdatePilotInformationCommand;
use FlightLog\Application\Pilot\Command\CreateUpdatePilotInformationCommandHandler;
use FlightLog\Domain\Pilot\ValueObject\PilotId;
use FlightLog\Http\Web\Form\DamageCreationForm;
use FlightLog\Http\Web\Form\PilotForm;
use FlightLog\Infrastructure\Pilot\Repository\PilotRepository;

final class PilotEditController extends WebController
{

    public function index(){
        $id = $this->request->getParam('id');

        if($id === null){
            $this->renderHtml('<p>Paramètre ID non fournis.</p>');
            return;
        }

        $command = new CreateUpdatePilotInformationCommand($id);

        if($this->getPilotRepository()->exist(PilotId::create($id))){
            $command->fromPilot($this->getPilotRepository()->getById(PilotId::create($id)));
        }

        $form = new PilotForm('pilot', $this->db);
        $form->bind($command);

$command is of type object<FlightLog\Application\Pilot\Command\CreateUpdatePilotInformationCommand>, but the function expects a object<stdClass>.

        $user = new \User($this->db);
        $user->fetch($id);

        if($this->request->isPost()){

This code seems to be duplicated across your project.

            $form->setData($this->request->getPostParameters());

            if(!$form->validate()){
                return $this->render('pilot/edit.phtml', [
                    'form' => $form,
                ]);
            }

            try{
                $this->handle($form->getObject());

It seems like $form->getObject() targeting FlightLog\Http\Web\Form\PilotForm::getObject() can also be of type null or object<stdClass>; however, FlightLog\Http\Web\Controller\PilotEditController::handle() does only seem to accept object<FlightLog\Application\Pilot\Command\CreateUpdatePilotInformationCommand>, maybe add an additional type check?

            }catch(\Exception $e){
                print $e->getMessage();
                dol_syslog($e->getMessage(), LOG_ERR);
            }


            return $this->redirect($_SERVER["PHP_SELF"].'?id='.$id.'&r=edit_pilot');
        }

        return $this->render('pilot/edit.phtml', [
            'pilotForm' => $form,
            'pilot' => $user,
        ]);
    }

    /**
     * @param CreateUpdatePilotInformationCommand $command
     *
     * @throws \Exception
     */
    private function handle(CreateUpdatePilotInformationCommand $command)
    {
        $this->getHandler()->__invoke($command);
    }

    /**
     * @return CreateUpdatePilotInformationCommandHandler()

The doc-type CreateUpdatePilotInformationCommandHandler() could not be parsed: Expected "|" or "end of type", but got "(" at position 42. (view supported doc-types)

     */
    private function getHandler(){
        return new CreateUpdatePilotInformationCommandHandler($this->getPilotRepository());
    }

    private function getPilotRepository()
    {
        return new PilotRepository($this->db);
    }
}