Symbiote\AdvancedWorkflow\Controllers\AdvancedWorkflowActionController

Complexity

Total Complexity

8

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	transition()	0	46	8

<?php

namespace Symbiote\AdvancedWorkflow\Controllers;

use SilverStripe\Control\Controller;
use SilverStripe\Control\Director;
use SilverStripe\Core\Convert;
use SilverStripe\ORM\DataObject;
use SilverStripe\Security\Security;
use Symbiote\AdvancedWorkflow\DataObjects\WorkflowInstance;
use Symbiote\AdvancedWorkflow\DataObjects\WorkflowTransition;
use Symbiote\AdvancedWorkflow\Services\WorkflowService;

/**
 * Handles actions triggered from external sources, eg emails or web frontend
 *
 * @author marcus@symbiote.com.au
 * @license BSD License http://silverstripe.org/bsd-license/
 */
class AdvancedWorkflowActionController extends Controller
{
    public function transition($request)

The parameter $request is not used and could be removed.

    {
        if (!Security::getCurrentUser()) {
            return Security::permissionFailure(
                $this,
                _t(
                    'AdvancedWorkflowActionController.ACTION_ERROR',
                    "You must be logged in"
                )
            );
        }

        $id = $this->request->requestVar('id');
        $transition = $this->request->requestVar('transition');

        $instance = DataObject::get_by_id(WorkflowInstance::class, (int) $id);
        if ($instance && $instance->canEdit()) {
            $transition = DataObject::get_by_id(WorkflowTransition::class, (int) $transition);
            if ($transition) {
                if ($this->request->requestVar('comments')) {
                    $action = $instance->CurrentAction();
                    $action->Comment = $this->request->requestVar('comments');
                    $action->write();
                }

                singleton(WorkflowService::class)->executeTransition($instance->getTarget(), $transition->ID);
                $result = array(
                    'success' => true,
                    'link'    => $instance->getTarget()->AbsoluteLink()
                );
                if (Director::is_ajax()) {
                    return json_encode($result);
                }
                return $this->redirect($instance->getTarget()->Link());
            }
        }

        if (Director::is_ajax()) {
            $result = array(
                'success' => false,
            );
            return json_encode($result);
        }

        return $this->redirect($instance->getTarget()->Link());
    }
}