Cv\Controller\ManageController::formAction()

Complexity

Conditions	12
Paths	24

Size

Total Lines	113
Code Lines	61

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	156

Importance

Changes

0

<?php
    public function formAction()
    {
        $repositories = $this->repositories;
        /* @var $cvRepository \Cv\Repository\Cv */
        $cvRepository = $repositories->get('Cv/Cv');
        $user = $this->auth()->getUser();

The method getUser() does not exist on Laminas\Stdlib\DispatchableInterface. It seems like you code against a sub-type of Laminas\Stdlib\DispatchableInterface such as Laminas\Mvc\Controller\AbstractController.

The method auth() does not exist on Cv\Controller\ManageController. Since you implemented __call, consider adding a @method annotation.

        /* @var $cv Cv */
        $cv = $this->getCv($cvRepository, $user);
        $params = $this->params();
        
        if (empty($cv)) {
            // create draft CV
            $cv = $cvRepository->create();
            $cv->setIsDraft(true);
            $cv->setContact($user->getRole() == User::ROLE_USER ? $user->getInfo() : new Contact());
            $cv->setUser($user);
            $repositories->store($cv);
        }
        
        if (($status = $params->fromQuery('status')) != '') {
            $this->acl('Cv/Status', 'change');

The method acl() does not exist on Cv\Controller\ManageController. Since you implemented __call, consider adding a @method annotation.

            return $this->changeStatus($cv, $status);
        }
        
        /* @var $container \Core\Form\Container */
        $container = $this->formElements
            ->get('CvContainer')
            ->setEntity($cv);

        // process post method
        if ($this->getRequest()->isPost()) {

The method isPost() does not exist on Laminas\Stdlib\RequestInterface. It seems like you code against a sub-type of Laminas\Stdlib\RequestInterface such as Laminas\Http\Request.

            $form = $container->getForm($params->fromQuery('form'));

            if ($form) {
                $form->setData(array_merge(
                    $params->fromPost(),
                    $params->fromFiles()

It seems like $params->fromFiles() can also be of type ArrayAccess and null; however, parameter $arrays of array_merge() does only seem to accept array, maybe add an additional type check?

                ));
                $formId = $params->fromQuery('form');
                $viewHelperManager = $this->viewHelper;
                $uploadHandler = $this->uploadHandler;

                if('image' === $formId){
                    // handles image upload
                    $uploadHandler->handleImageUpload($cv, $_FILES['image']);
                }
                elseif('attachments' === $formId){
                    // handles attachment upload
                    $attachment = $uploadHandler->handleAttachmentUpload($cv, $_FILES['attachments']);
                    $content = $viewHelperManager->get('basepath')
                        ->__invoke($attachment->getUri());
                    return new JsonModel([
                        'valid' => true,
                        'content' => $content,
                    ]);
                }
                else{
                    if (!$form->isValid()) {
                        return new JsonModel([
                            'valid' => false,
                            'errors' => $form->getMessages()
                        ]);
                    };

                    /*
 * @todo This is a workaround for GeoJSON data insertion
 * until we figured out, what we really want it to be.
 */

//                if ('preferredJob' == $formId) {
//                    $locElem = $form->getBaseFieldset()->get('geo-location');
//                    if ($locElem instanceof GeoText) {
//                        $loc = $locElem->getValue('entity');
//                        $locations = $cv->getPreferredJob()->getDesiredLocations();
//                        if (count($locations)) {
//                            $locations->clear();
//                        }
//                        $locations->add($loc);
//                        $cv->getPreferredJob()->setDesiredLocation($locElem->getValue());
//                    }
//                }
                    $this->validateCv($cv);
                    $repositories->store($cv);
                }

                if ('file-uri' === $params->fromPost('return')) {
                    $content = $viewHelperManager->get('basepath')
                        ->__invoke($form->getHydrator()->getLastUploadedFile()->getUri());

The method getLastUploadedFile() does not exist on Laminas\Hydrator\HydratorInterface. It seems like you code against a sub-type of Laminas\Hydrator\HydratorInterface such as Core\Entity\Hydrator\FileCollectionUploadHydrator.

The method getHydrator() does not exist on Core\Form\Container.

                } else {
                    if ($form instanceof SummaryFormInterface) {
                        $form->setRenderMode(SummaryFormInterface::RENDER_SUMMARY);
                        $viewHelper = 'summaryForm';
                    } else {
                        $viewHelper = 'form';
                    }
                    
                    // render form
                    $content = $viewHelperManager->get($viewHelper)
                        ->__invoke($form);
                }
                
                return new JsonModel([
                    'valid' => true,
                    'content' => $content
                ]);
            } elseif (($action = $params->fromQuery('action')) !== null) {
                return new JsonModel($container->executeAction($action, $params->fromPost()));
            }
        }// end of process post method

        return [
            'container' => $container,
            'cv' => $cv
        ];