Romm\Formz\Middleware\Item\Persistence\PersistenceFetchingMiddleware

Complexity

Total Complexity

12

Size/Duplication

Total Lines	82
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	initializeMiddleware()	0	14	3
A	before()	0	14	4
A	getFormIdentifierHash()	0	15	4
A	injectService()	0	5	1

<?php
/*
 * 2017 Romain CANON <romain.hydrocanon@gmail.com>
 *
 * This file is part of the TYPO3 FormZ project.
 * It is free software; you can redistribute it and/or modify it
 * under the terms of the GNU General Public License, either
 * version 3 of the License, or any later version.
 *
 * For the full copyright and license information, see:
 * http://www.gnu.org/licenses/gpl-3.0.html
 */

namespace Romm\Formz\Middleware\Item\Persistence;

use Romm\Formz\Form\Service\DataObject\WrongFormIdentifierObjectDataException;
use Romm\Formz\Middleware\Argument\Arguments;
use Romm\Formz\Middleware\Item\AbstractMiddleware;
use Romm\Formz\Middleware\Item\FormInjection\FormInjectionSignal;
use Romm\Formz\Middleware\Signal\Before;
use Romm\Formz\Middleware\State\PresetMiddlewareInterface;

/**
 * This middleware will try to fetch a form instance using the persistence
 * manager.
 *
 * If a form identifier hash is found in the request arguments, it is used to
 * search for a form instance in every persistence service bound to the form
 * object.
 */
class PersistenceFetchingMiddleware extends AbstractMiddleware implements Before, FormInjectionSignal, PresetMiddlewareInterface
{
    /**
     * @var \Romm\Formz\Middleware\Item\Persistence\PersistenceFetchingMiddlewareOption
     */
    protected $options;

    /**
     * @var PersistenceMiddlewareService
     */
    protected $service;

    /**
     * Checks for a form identifier hash in the request arguments, and tries to
     * get a form identifier object with it.
     */
    public function initializeMiddleware()
    {
        $identifier = $this->getFormIdentifierHash();

        if ($identifier) {

The expression $identifier of type string|null is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

            try {
                $identifierObject = $this->getFormObject()->getFormIdentifierObject($identifier);
                $this->service->setIdentifierObject($identifierObject);
            } catch (WrongFormIdentifierObjectDataException $exception) {
                // @todo forward
                $action = $this->options->getForwardToActionOnHashError();

$action is not used, you could remove the assignment.

            }
        }
    }

    /**
     * @see PersistenceFetchingMiddleware
     *
     * @param Arguments $arguments
     */
    public function before(Arguments $arguments)
    {
        if (false === $this->getFormObject()->formWasSubmitted()
            && $this->service->hasIdentifierObject()
        ) {
            $form = $this->getFormObject()
                ->getPersistenceManager()
                ->fetchFirst($this->service->getIdentifierObject());

            if ($form) {
                $this->getFormObject()->setForm($form);
            }
        }
    }

    /**
     * This function tries to fetch the form identifier hash for the current
     * form, in the request arguments.
     *
     * @return string|null
     */
    protected function getFormIdentifierHash()
    {
        if ($this->getRequest()->hasArgument('fz-identifier')) {
            $formName = $this->getFormObject()->getName();
            $identifierList = $this->getRequest()->getArgument('fz-identifier');

            if (is_array($identifierList)
                && isset($identifierList[$formName])
            ) {
                return (string)$identifierList[$formName];
            }
        }

        return null;
    }

    /**
     * @param PersistenceMiddlewareService $service
     */
    public function injectService(PersistenceMiddlewareService $service)
    {
        $this->service = $service;
        $this->service->reset();
    }
}