Function performTasks has a Cognitive Complexity of 29 (exceeds 5 allowed). Consider refactoring.

Open

    private function performTasks(App\EventHandler $eventHandler, $condition = [])
    {
        $recordModel = $eventHandler->getRecordModel();
        $moduleName = $eventHandler->getModuleName();
        if (!isset($this->workflows[$moduleName])) {

• Read up
  Read up

Method performTasks has 59 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private function performTasks(App\EventHandler $eventHandler, $condition = [])
    {
        $recordModel = $eventHandler->getRecordModel();
        $moduleName = $eventHandler->getModuleName();
        if (!isset($this->workflows[$moduleName])) {

The method performTasks() has an NPath complexity of 314. The configured NPath complexity threshold is 200.

Open

    private function performTasks(App\EventHandler $eventHandler, $condition = [])
    {
        $recordModel = $eventHandler->getRecordModel();
        $moduleName = $eventHandler->getModuleName();
        if (!isset($this->workflows[$moduleName])) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    function bar() {
        // lots of complicated code
    }
}

The method performTasks() has a Cyclomatic Complexity of 21. The configured cyclomatic complexity threshold is 10.

Open

    private function performTasks(App\EventHandler $eventHandler, $condition = [])
    {
        $recordModel = $eventHandler->getRecordModel();
        $moduleName = $eventHandler->getModuleName();
        if (!isset($this->workflows[$moduleName])) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

Refactor this function to reduce its Cognitive Complexity from 22 to the 15 allowed.

Open

    private function performTasks(App\EventHandler $eventHandler, $condition = [])

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

Missing class import via use statement (line '86', column '15').

Open

            $wfs = new VTWorkflowManager();

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Missing class import via use statement (line '133', column '16').

Open

                    throw new \App\Exceptions\AppException('Should never come here! Execution Condition:' . $workflow->executionCondition);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

The method entityChangeState uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            $this->performTasks($eventHandler, [
                VTWorkflowManager::$ON_EVERY_SAVE
            ]);
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method performTasks uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                    } else {
                        $doEvaluate = true;
                    }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method performTasks uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                    } else {
                        $doEvaluate = false;
                    }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Reference to undeclared property \Workflow->executionCondition

Open

            if ($condition && !\in_array($workflow->executionCondition, $condition)) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \Workflow->executionCondition

Open

                    throw new \App\Exceptions\AppException('Should never come here! Execution Condition:' . $workflow->executionCondition);

• Exclude checks
  • Disable engine
  • Disable check

Call with 2 arg(s) to \Workflow::evaluate() which only takes 1 arg(s) defined at /code/modules/com_vtiger_workflow/Workflow.php:168

Open

            if ($doEvaluate && $workflow->evaluate($recordModel, $recordModel->getId())) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \Workflow->executionCondition

Open

                if (VTWorkflowManager::$ONCE == $workflow->executionCondition) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \Workflow->executionCondition

Open

            switch ($workflow->executionCondition) {

• Exclude checks
  • Disable engine
  • Disable check

This case's code block is the same as the block for the case on line 108.

Open

                case VTWorkflowManager::$ON_DELETE:
                    $doEvaluate = true;
                    break;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

switch ($i) {
  case 1:
    doSomething();
    break;
  case 2:
    doSomethingDifferent();
    break;
  case 3:  // Noncompliant; duplicates case 1's implementation
    doSomething();
    break;
  default:
    doTheRest();
}

if ($a >= 0 && $a < 10) {
  doTheThing();
else if ($a >= 10 && $a < 20) {
  doTheOtherThing();
}
else if ($a >= 20 && $a < 50) {
  doTheThing();  // Noncompliant; duplicates first condition
}
else {
  doTheRest();
}

if ($b == 0) {
  doOneMoreThing();
}
else {
  doOneMoreThing(); // Noncompliant; duplicates then-branch
}

var b = a ? 12 > 4 : 4;  // Noncompliant; always results in the same value

switch ($i) {
  case 1:
  case 3:
    doSomething();
    break;
  case 2:
    doSomethingDifferent();
    break;
  default:
    doTheRest();
}

if (($a >= 0 && $a < 10) || ($a >= 20 && $a < 50)) {
  doTheThing();
else if ($a >= 10 && $a < 20) {
  doTheOtherThing();
}
else {
  doTheRest();
}

doOneMoreThing();

b = 4;

switch ($i) {
  case 1:
    doSomething();
    break;
  case 2:
    doSomethingDifferent();
    break;
  case 3:
    doThirdThing();
    break;
  default:
    doTheRest();
}

if ($a >= 0 && $a < 10) {
  doTheThing();
else if ($a >= 10 && $a < 20) {
  doTheOtherThing();
}
else if ($a >= 20 && $a < 50) {
  doTheThirdThing();
}
else {
  doTheRest();
}

if ($b == 0) {
  doOneMoreThing();
}
else {
  doTheRest();
}

int b = a ? 12 > 4 : 8;

This case's code block is the same as the block for the case on line 114.

Open

                case VTWorkflowManager::$TRIGGER:
                    $doEvaluate = false;
                    break;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

switch ($i) {
  case 1:
    doSomething();
    break;
  case 2:
    doSomethingDifferent();
    break;
  case 3:  // Noncompliant; duplicates case 1's implementation
    doSomething();
    break;
  default:
    doTheRest();
}

if ($a >= 0 && $a < 10) {
  doTheThing();
else if ($a >= 10 && $a < 20) {
  doTheOtherThing();
}
else if ($a >= 20 && $a < 50) {
  doTheThing();  // Noncompliant; duplicates first condition
}
else {
  doTheRest();
}

if ($b == 0) {
  doOneMoreThing();
}
else {
  doOneMoreThing(); // Noncompliant; duplicates then-branch
}

var b = a ? 12 > 4 : 4;  // Noncompliant; always results in the same value

switch ($i) {
  case 1:
  case 3:
    doSomething();
    break;
  case 2:
    doSomethingDifferent();
    break;
  default:
    doTheRest();
}

if (($a >= 0 && $a < 10) || ($a >= 20 && $a < 50)) {
  doTheThing();
else if ($a >= 10 && $a < 20) {
  doTheOtherThing();
}
else {
  doTheRest();
}

doOneMoreThing();

b = 4;

switch ($i) {
  case 1:
    doSomething();
    break;
  case 2:
    doSomethingDifferent();
    break;
  case 3:
    doThirdThing();
    break;
  default:
    doTheRest();
}

if ($a >= 0 && $a < 10) {
  doTheThing();
else if ($a >= 10 && $a < 20) {
  doTheOtherThing();
}
else if ($a >= 20 && $a < 50) {
  doTheThirdThing();
}
else {
  doTheRest();
}

if ($b == 0) {
  doOneMoreThing();
}
else {
  doTheRest();
}

int b = a ? 12 > 4 : 8;

This case's code block is the same as the block for the case on line 114.

Open

                case VTWorkflowManager::$ON_SCHEDULE:
                    $doEvaluate = false;
                    break;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

switch ($i) {
  case 1:
    doSomething();
    break;
  case 2:
    doSomethingDifferent();
    break;
  case 3:  // Noncompliant; duplicates case 1's implementation
    doSomething();
    break;
  default:
    doTheRest();
}

if ($a >= 0 && $a < 10) {
  doTheThing();
else if ($a >= 10 && $a < 20) {
  doTheOtherThing();
}
else if ($a >= 20 && $a < 50) {
  doTheThing();  // Noncompliant; duplicates first condition
}
else {
  doTheRest();
}

if ($b == 0) {
  doOneMoreThing();
}
else {
  doOneMoreThing(); // Noncompliant; duplicates then-branch
}

var b = a ? 12 > 4 : 4;  // Noncompliant; always results in the same value

switch ($i) {
  case 1:
  case 3:
    doSomething();
    break;
  case 2:
    doSomethingDifferent();
    break;
  default:
    doTheRest();
}

if (($a >= 0 && $a < 10) || ($a >= 20 && $a < 50)) {
  doTheThing();
else if ($a >= 10 && $a < 20) {
  doTheOtherThing();
}
else {
  doTheRest();
}

doOneMoreThing();

b = 4;

switch ($i) {
  case 1:
    doSomething();
    break;
  case 2:
    doSomethingDifferent();
    break;
  case 3:
    doThirdThing();
    break;
  default:
    doTheRest();
}

if ($a >= 0 && $a < 10) {
  doTheThing();
else if ($a >= 10 && $a < 20) {
  doTheOtherThing();
}
else if ($a >= 20 && $a < 50) {
  doTheThirdThing();
}
else {
  doTheRest();
}

if ($b == 0) {
  doOneMoreThing();
}
else {
  doTheRest();
}

int b = a ? 12 > 4 : 8;

This case's code block is the same as the block for the case on line 114.

Open

                case VTWorkflowManager::$BLOCK_EDIT:
                    $doEvaluate = false;
                    break;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

switch ($i) {
  case 1:
    doSomething();
    break;
  case 2:
    doSomethingDifferent();
    break;
  case 3:  // Noncompliant; duplicates case 1's implementation
    doSomething();
    break;
  default:
    doTheRest();
}

if ($a >= 0 && $a < 10) {
  doTheThing();
else if ($a >= 10 && $a < 20) {
  doTheOtherThing();
}
else if ($a >= 20 && $a < 50) {
  doTheThing();  // Noncompliant; duplicates first condition
}
else {
  doTheRest();
}

if ($b == 0) {
  doOneMoreThing();
}
else {
  doOneMoreThing(); // Noncompliant; duplicates then-branch
}

var b = a ? 12 > 4 : 4;  // Noncompliant; always results in the same value

switch ($i) {
  case 1:
  case 3:
    doSomething();
    break;
  case 2:
    doSomethingDifferent();
    break;
  default:
    doTheRest();
}

if (($a >= 0 && $a < 10) || ($a >= 20 && $a < 50)) {
  doTheThing();
else if ($a >= 10 && $a < 20) {
  doTheOtherThing();
}
else {
  doTheRest();
}

doOneMoreThing();

b = 4;

switch ($i) {
  case 1:
    doSomething();
    break;
  case 2:
    doSomethingDifferent();
    break;
  case 3:
    doThirdThing();
    break;
  default:
    doTheRest();
}

if ($a >= 0 && $a < 10) {
  doTheThing();
else if ($a >= 10 && $a < 20) {
  doTheOtherThing();
}
else if ($a >= 20 && $a < 50) {
  doTheThirdThing();
}
else {
  doTheRest();
}

if ($b == 0) {
  doOneMoreThing();
}
else {
  doTheRest();
}

int b = a ? 12 > 4 : 8;

This case's code block is the same as the block for the case on line 114.

Open

                case VTWorkflowManager::$ON_RELATED:
                    $doEvaluate = false;
                    break;

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

switch ($i) {
  case 1:
    doSomething();
    break;
  case 2:
    doSomethingDifferent();
    break;
  case 3:  // Noncompliant; duplicates case 1's implementation
    doSomething();
    break;
  default:
    doTheRest();
}

if ($a >= 0 && $a < 10) {
  doTheThing();
else if ($a >= 10 && $a < 20) {
  doTheOtherThing();
}
else if ($a >= 20 && $a < 50) {
  doTheThing();  // Noncompliant; duplicates first condition
}
else {
  doTheRest();
}

if ($b == 0) {
  doOneMoreThing();
}
else {
  doOneMoreThing(); // Noncompliant; duplicates then-branch
}

var b = a ? 12 > 4 : 4;  // Noncompliant; always results in the same value

switch ($i) {
  case 1:
  case 3:
    doSomething();
    break;
  case 2:
    doSomethingDifferent();
    break;
  default:
    doTheRest();
}

if (($a >= 0 && $a < 10) || ($a >= 20 && $a < 50)) {
  doTheThing();
else if ($a >= 10 && $a < 20) {
  doTheOtherThing();
}
else {
  doTheRest();
}

doOneMoreThing();

b = 4;

switch ($i) {
  case 1:
    doSomething();
    break;
  case 2:
    doSomethingDifferent();
    break;
  case 3:
    doThirdThing();
    break;
  default:
    doTheRest();
}

if ($a >= 0 && $a < 10) {
  doTheThing();
else if ($a >= 10 && $a < 20) {
  doTheOtherThing();
}
else if ($a >= 20 && $a < 50) {
  doTheThirdThing();
}
else {
  doTheRest();
}

if ($b == 0) {
  doOneMoreThing();
}
else {
  doTheRest();
}

int b = a ? 12 > 4 : 8;

Each class must be in a namespace of at least one level (a top-level vendor name)

Open

class Vtiger_Workflow_Handler

• Exclude checks
  • Disable engine
  • Disable check

A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute logic with side effects, but should not do both. The first symbol is defined on line 14 and the first side effect is on line 3.

Open

<?php

• Exclude checks
  • Disable engine
  • Disable check

The class Vtiger_Workflow_Handler is not named in CamelCase.

Open

class Vtiger_Workflow_Handler
{
    private $workflows;

    /**

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class class_name {
}

Spaces must be used for alignment; tabs are not allowed

Open

        $this->performTasks($eventHandler, [

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @param int[]             $condition

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $recordModel = $eventHandler->getRecordModel();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        } else {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * EntityAfterSave function.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            VTWorkflowManager::$ONCE,

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            VTWorkflowManager::$ON_FIRST_SAVE,

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * EntityChangeState handler function.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $this->performTasks($eventHandler, [

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    public function entityAfterSave(App\EventHandler $eventHandler)

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            ]);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            ]);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $this->performTasks($eventHandler, [

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $this->performTasks($eventHandler, [

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $this->entityAfterSave($eventHandler);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @param App\EventHandler $eventHandler

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            VTWorkflowManager::$ON_DELETE

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @param App\EventHandler $eventHandler

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    private $workflows;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        if ('Trash' === $eventHandler->getRecordModel()->get('deleted')) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    public function entityAfterDelete(App\EventHandler $eventHandler)

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        ]);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    public function entityChangeState(App\EventHandler $eventHandler)

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        ]);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @param App\EventHandler $eventHandler

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        $moduleName = $eventHandler->getModuleName();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                VTWorkflowManager::$ON_DELETE

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            VTWorkflowManager::$ON_MODIFY

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    private function performTasks(App\EventHandler $eventHandler, $condition = [])

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

 * @package        Handler

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    /**

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     */

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                VTWorkflowManager::$ON_EVERY_SAVE

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * EntityAfterDelete handler function.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * Perform workflow tasks.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            VTWorkflowManager::$ON_EVERY_SAVE,

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * UserAfterSave function.

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    public function userAfterSave(App\EventHandler $eventHandler)

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     *

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @param \App\EventHandler $eventHandler

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    break;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    break;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    $workflow->markAsCompletedForRecord($recordModel->getId());

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @param App\EventHandler $eventHandler

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    if ($workflow->isCompletedForRecord($recordModel->getId())) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            switch ($workflow->executionCondition) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    break;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                case VTWorkflowManager::$ON_DELETE:

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $wfs = new VTWorkflowManager();

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                        $doEvaluate = true;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    break;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                $workflow->performTasks($recordModel);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                        $doEvaluate = true;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                case VTWorkflowManager::$ON_EVERY_SAVE:

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                case VTWorkflowManager::$MANUAL:

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    $doEvaluate = false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    $doEvaluate = false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

     * @throws \Exception

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                continue;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        if (!isset($this->workflows[$moduleName])) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            $this->workflows[$moduleName] = $wfs->getWorkflowsForModule($moduleName);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                case VTWorkflowManager::$ONCE:

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    $doEvaluate = false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                case VTWorkflowManager::$TRIGGER:

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    $doEvaluate = false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    break;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    $doEvaluate = false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                case VTWorkflowManager::$ON_FIRST_SAVE:

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    } else {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    break;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    break;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    throw new \App\Exceptions\AppException('Should never come here! Execution Condition:' . $workflow->executionCondition);

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

        foreach ($this->workflows[$moduleName] as &$workflow) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                case VTWorkflowManager::$ON_SCHEDULE:

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    $doEvaluate = true;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    break;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                if (VTWorkflowManager::$ONCE == $workflow->executionCondition) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                        $doEvaluate = false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                        $doEvaluate = false;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    } else {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    $doEvaluate = true;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    break;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                case VTWorkflowManager::$BLOCK_EDIT:

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                case VTWorkflowManager::$ON_RELATED:

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    break;

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    if ($recordModel->isNew()) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                default:

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            if ($condition && !\in_array($workflow->executionCondition, $condition)) {

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                case VTWorkflowManager::$ON_MODIFY:

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

                    $doEvaluate = !$recordModel->isNew() && !empty($recordModel->getPreviousValue());

• Exclude checks
  • Disable engine
  • Disable check

Spaces must be used for alignment; tabs are not allowed

Open

            if ($doEvaluate && $workflow->evaluate($recordModel, $recordModel->getId())) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 139 characters

Open

                    throw new \App\Exceptions\AppException('Should never come here! Execution Condition:' . $workflow->executionCondition);

• Exclude checks
  • Disable engine
  • Disable check

Class name "Vtiger_Workflow_Handler" is not in camel caps format

Open

class Vtiger_Workflow_Handler

• Exclude checks
  • Disable engine
  • Disable check