DNADesign\ElementalUserForms\Control\ElementFormController

Complexity

Total Complexity

11

Size/Duplication

Total Lines	96
Duplicated Lines	0 %

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	setUserFormController()	0	4	1
A	Form()	0	3	1
A	getUserFormController()	0	3	1
A	init()	0	9	2
A	Link()	0	15	4
A	process()	0	5	1
A	finished()	0	12	1

<?php

namespace DNADesign\ElementalUserForms\Control;

use DNADesign\Elemental\Controllers\ElementController;
use SilverStripe\Control\Controller;
use SilverStripe\Control\Director;
use SilverStripe\Core\Injector\Injector;
use SilverStripe\UserForms\Control\UserDefinedFormController;
use SilverStripe\UserForms\Form\UserForm;

class ElementFormController extends ElementController
{
    private static $allowed_actions = [

The private property $allowed_actions is not used, and could be removed.

        'Form',
        'process',
        'finished'
    ];

    /**
     * @var UserDefinedFormController
     */
    protected $userFormController;

    protected function init()
    {
        parent::init();

        $controller = $this->getUserFormController() ?: UserDefinedFormController::create($this->element);
        $controller->setRequest($this->getRequest());
        $controller->doInit();

        $this->setUserFormController($controller);
    }

    /**
     * @return UserForm
     */
    public function Form()
    {
        return $this->getUserFormController()->Form();
    }

    public function process($data)
    {
        $user = $this->getUserFormController();

        return $user->process($data, $user->Form());
    }

    public function finished()
    {
        $user = $this->getUserFormController();

        $user->finished();

        $page = $this->getPage();

The method getPage() does not exist on DNADesign\ElementalUserForms\Control\ElementFormController. Since you implemented __call, consider adding a @method annotation.

        $controller = Injector::inst()->create($page->getControllerName(), $this->element->getPage());
        $element = $this->element;

        return $controller->customise([
            'Content' => $element->renderWith($element->getRenderTemplates('_ReceivedFormSubmission')),
        ]);
    }

    /**
     * @param string $action
     *
     * @return string
     */
    public function Link($action = null)
    {
        $id = $this->element->ID;
        $segment = Controller::join_links('element', $id, $action);
        $page = Director::get_current_page();

        if ($page && !($page instanceof ElementController)) {
            return $page->Link($segment);
        }

        if ($controller = $this->getParentController()) {

The method getParentController() does not exist on DNADesign\ElementalUserForms\Control\ElementFormController. Since you implemented __call, consider adding a @method annotation.

            return $controller->Link($segment);
        }

        return $segment;
    }

    /**
     * Return the associated UserDefinedFormController
     *
     * @return UserDefinedFormController
     */
    public function getUserFormController()
    {
        return $this->userFormController;
    }

    /**
     * Set the associated UserDefinedFormController
     *
     * @param UserDefinedFormController $controller
     * @return $this
     */
    public function setUserFormController(UserDefinedFormController $controller)
    {
        $this->userFormController = $controller;
        return $this;
    }
}