DNADesign\ElementalUserForms\Control\ElementFormController

Complexity

Total Complexity

7

Size/Duplication

Total Lines	68
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	Link()	0	15	4
A	Form()	0	8	1
A	process()	0	8	1
A	finished()	0	14	1

<?php

namespace DNADesign\ElementalUserForms\Control;

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

class ElementFormController extends ElementController
{
    private static $allowed_actions = [

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

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

    /**
     * @return SilverStripe\Forms\Form

The type DNADesign\ElementalUserForms\Control\SilverStripe\Forms\Form was not found. Did you mean SilverStripe\Forms\Form? If so, make sure to prefix the type with \.

     */
    public function Form()
    {
        $request = $this->getRequest();

        $user = UserDefinedFormController::create($this->element);
        $user->setRequest($request);

        return $user->Form();
    }

    public function process($data)
    {
        $request = $this->getRequest();

        $user = UserDefinedFormController::create($this->element);
        $user->setRequest($request);

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

    public function finished()
    {
        $request = $this->getRequest();

        $user = UserDefinedFormController::create($this->element);
        $user->setRequest($request);
        $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;
    }
}