Jabe\Engine\Impl\Util\ModificationUtil

Complexity

Total Complexity

8

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
B	handleChildRemovalInScope()	0	28	8

<?php

namespace Jabe\Engine\Impl\Util;

use Jabe\Engine\Impl\Persistence\Entity\ExecutionEntity;
use Jabe\Engine\Impl\Pvm\Delegate\ModificationObserverBehaviorInterface;
use Jabe\Engine\Impl\Pvm\Process\{
    ActivityImpl,
    ScopeImpl
};
use Jabe\Engine\Impl\Pvm\Runtime\PvmExecutionImpl;

class ModificationUtil
{
    public static function handleChildRemovalInScope(ExecutionEntity $removedExecution): void
    {
        $activity = $removedExecution->getActivity();
        if ($activity === null) {
            if ($removedExecution->getSuperExecution() !== null) {
                $removedExecution = $removedExecution->getSuperExecution();
                $activity = $removedExecution->getActivity();
                if ($activity == null) {
                    return;
                }
            } else {
                return;
            }
        }
        $flowScope = $activity->getFlowScope();

        $scopeExecution = $removedExecution->getParentScopeExecution(false);
        $executionInParentScope = $removedExecution->isConcurrent() ? $removedExecution : $removedExecution->getParent();

        if ($flowScope->getActivityBehavior() != null && $flowScope->getActivityBehavior() instanceof ModificationObserverBehaviorInterface) {
            // let child removal be handled by the scope itself
            $behavior = $flowScope->getActivityBehavior();
            $behavior->destroyInnerInstance($executionInParentScope);

It seems like $executionInParentScope can also be of type null; however, parameter $concurrentExecution of Jabe\Engine\Impl\Pvm\Delegate\ModificationObserverBehaviorInterface::destroyInnerInstance() does only seem to accept Jabe\Engine\Impl\Pvm\Delegate\ActivityExecutionInterface, maybe add an additional type check?

        } else {
            if ($executionInParentScope->isConcurrent()) {

The method isConcurrent() does not exist on null.

                $executionInParentScope->remove();
                $scopeExecution->tryPruneLastConcurrentChild();
                $scopeExecution->forceUpdate();
            }
        }
    }
}