FondBot\Conversation\Traits\Transitions

Complexity

Total Complexity

5

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	jump()	0	14	2
B	restart()	0	24	3

<?php

declare(strict_types=1);

namespace FondBot\Conversation\Traits;

use RuntimeException;
use InvalidArgumentException;
use FondBot\Application\Kernel;
use FondBot\Conversation\Intent;
use FondBot\Conversation\Interaction;

trait Transitions
{
    /** @var Kernel */
    protected $kernel;

    /**
     * Whether any transition run.
     *
     * @var bool
     */
    protected $transitioned = false;

    /**
     * Jump to another interaction.
     *
     * @param string $interaction
     *
     * @throws \InvalidArgumentException
     */
    protected function jump(string $interaction): void
    {
        /** @var Interaction $instance */
        $instance = $this->kernel->resolve($interaction);

        if (!$instance instanceof Interaction) {
            throw new InvalidArgumentException('Invalid interaction `'.$interaction.'`');
        }

        // Run interaction
        $this->kernel->converse($instance);

        $this->transitioned = true;
    }

    /**
     * Restart current intent or interaction.
     */
    protected function restart(): void
    {
        switch (true) {
            case $this instanceof Intent:
                $this->kernel->clearContext();

The property kernel cannot be accessed from this context as it is declared protected in class FondBot\Conversation\Traits\InteractsWithContext.

                $this->kernel->converse($this);

The property kernel cannot be accessed from this context as it is declared protected in class FondBot\Conversation\Traits\InteractsWithContext.

                $this->transitioned = true;

The property transitioned cannot be accessed from this context as it is declared protected in class FondBot\Conversation\Traits\Transitions.

                break;
            case $this instanceof Interaction:
                $context = $this->kernel->getContext();

The property kernel cannot be accessed from this context as it is declared protected in class FondBot\Conversation\Traits\InteractsWithContext.

                $context->setInteraction(null);
                $context->setValues([]);
                $this->kernel->setContext($context);

The property kernel cannot be accessed from this context as it is declared protected in class FondBot\Conversation\Traits\InteractsWithContext.

It seems like $context defined by $this->kernel->getContext() on line 61 can be null; however, FondBot\Application\Kernel::setContext() does not accept null, maybe add an additional type check?

                $this->transitioned = true;

The property transitioned cannot be accessed from this context as it is declared protected in class FondBot\Conversation\Traits\Transitions.

                $this->kernel->converse($this);

The property kernel cannot be accessed from this context as it is declared protected in class FondBot\Conversation\Traits\InteractsWithContext.

                break;
            default:
                throw new RuntimeException('Only conversable instances can be restarted.');
        }
    }
}