Applications\Controller\ApiApplyController

Complexity

Total Complexity

9

Size/Duplication

Total Lines	121
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	0	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
A	noJobFoundModel()	0	9	1
A	noPostRequestModel()	0	9	1
A	indexAction()	0	54	4
A	__construct()	0	10	1
A	invalidApplicationDataModel()	0	10	1
A	invalidDataModel()	0	9	1

<?php

/**
 * YAWIK
 *
 * @filesource
 * @copyright 2020 CROSS Solution <https://www.cross-solution.de>
 * @license MIT
 */

declare(strict_types=1);

namespace Applications\Controller;

use Applications\Entity\Contact;
use Applications\Entity\Hydrator\ApiApplicationHydrator;
use Applications\Repository\Application as ApplicationsRepository;
use Auth\Entity\AnonymousUser;
use Jobs\Repository\Job as JobsRepository;
use Laminas\Mvc\Controller\AbstractActionController;
use Laminas\View\Model\JsonModel;

/**
 * TODO: description
 *
 * @author Mathias Gelhausen <gelhausen@cross-solution.de>
 * TODO: write tests
 */
class ApiApplyController extends AbstractActionController
{
    private $appRepository;
    private $jobRepository;
    private $formContainer;
    private $hydrator;

    public function __construct(
        ApplicationsRepository $appRepository,
        JobsRepository $jobRepository,
        $formContainer,
        ApiApplicationHydrator $hydrator
    ) {
        $this->appRepository = $appRepository;
        $this->jobRepository = $jobRepository;
        $this->formContainer = $formContainer;
        $this->hydrator = $hydrator;
    }

    public function indexAction()
    {
        /** @var \Laminas\Http\PhpEnvironment\Request $request */
        $request = $this->getRequest();
        if (!$request->isPost()) {
            return $this->noPostRequestModel();
        }

        $applyId = $this->params()->fromQuery('applyId');
        $job = $this->jobRepository->findOneBy(['applyId' => $applyId]);

        if (!$job) {

$job is of type object, thus it always evaluated to true.

            return $this->noJobFoundModel($applyId);
        }

        $data = array_merge_recursive(
            $request->getPost()->toArray(),
            $request->getFiles()->toArray()
        );


        $application = $this->appRepository->create();
        $application->setJob($job);
        $user = $this->auth()->getUser();

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

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.

        $application->setUser($user);
        $application->setContact(new Contact());

        $hydrator = $this->hydrator;
        $hydrator->hydrate($data, $application);
        //$application->getContact()->setFirstName($data['contact']['firstName']);


        $this->appRepository->store($application);

        $result = [
            'status' => 'OK',
            'id' => $application->getId(),
            'entity' => $hydrator->extract($application)
        ];

        if ($user instanceof AnonymousUser) {
            $result['track'] =
                $this->url()->fromRoute(
                    'lang/applications/detail',
                    ['id' => $application->getId()],
                    ['force_canonical' => true],
                    true
                )
                . '?token=' . $user->getToken()
            ;
        }

        $model = new JsonModel($result);
        return $model;
    }

    private function noPostRequestModel()
    {
        /** @var \Laminas\Http\PhpEnvironment\Response $response */
        $response = $this->getResponse();
        $response->setStatusCode($response::STATUS_CODE_405);

        return new JsonModel([
            'status' => 'error',
            'message' => 'Invalid request method. Only POST allowed.'
        ]);
    }

    private function invalidDataModel($e)

The method invalidDataModel() is not used, and could be removed.

    {
        /** @var \Laminas\Http\PhpEnvironment\Response $response */
        $response = $this->getResponse();
        $response->setStatusCode($response::STATUS_CODE_405);

        return new JsonModel([
            'status' => 'error',
            'message' => 'Invalid json data: ' . $e->getMessage()
        ]);
    }

    private function noJobFoundModel($applyId)
    {
        /** @var \Laminas\Http\PhpEnvironment\Response $response */
        $response = $this->getResponse();
        $response->setStatusCode($response::STATUS_CODE_400);

        return new JsonModel([
            'status' => 'error',
            'message' => 'No job found with apply id "' . $applyId . '"',
        ]);
    }

    private function invalidApplicationDataModel()

The method invalidApplicationDataModel() is not used, and could be removed.

    {
        /** @var \Laminas\Http\PhpEnvironment\Response $response */
        $response = $this->getResponse();
        $response->setStatusCode($response::STATUS_CODE_400);

        return new JsonModel([
            'status' => 'error',
            'message' => 'Invalid application data.',
            'errrors' => $this->formContainer->getMessages()
        ]);
    }
}