Completed
Push — master ( cd535a...da42ba )
by Vladimir
02:31
created

IntentManager::find()   B

Complexity

Conditions 5
Paths 4

Size

Total Lines 13
Code Lines 6

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
c 0
b 0
f 0
dl 0
loc 13
rs 8.8571
cc 5
eloc 6
nc 4
nop 1
1
<?php
2
3
declare(strict_types=1);
4
5
namespace FondBot\Conversation;
6
7
use FondBot\Contracts\Conversation\Intent;
0 ignored issues
show
Bug introduced by
This use statement conflicts with another class in this namespace, FondBot\Conversation\Intent.

Let’s assume that you have a directory layout like this:

.
|-- OtherDir
|   |-- Bar.php
|   `-- Foo.php
`-- SomeDir
    `-- Foo.php

and let’s assume the following content of Bar.php:

// Bar.php
namespace OtherDir;

use SomeDir\Foo; // This now conflicts the class OtherDir\Foo

If both files OtherDir/Foo.php and SomeDir/Foo.php are loaded in the same runtime, you will see a PHP error such as the following:

PHP Fatal error:  Cannot use SomeDir\Foo as Foo because the name is already in use in OtherDir/Foo.php

However, as OtherDir/Foo.php does not necessarily have to be loaded and the error is only triggered if it is loaded before OtherDir/Bar.php, this problem might go unnoticed for a while. In order to prevent this error from surfacing, you must import the namespace with a different alias:

// Bar.php
namespace OtherDir;

use SomeDir\Foo as SomeDirFoo; // There is no conflict anymore.
Loading history...
8
use FondBot\Contracts\Channels\ReceivedMessage;
9
10
class IntentManager
11
{
12
    /** @var Intent[] */
13
    private $intents = [];
14
15
    /** @var Intent */
16
    private $fallbackIntent;
17
18
    /**
19
     * Find intent.
20
     *
21
     * @param ReceivedMessage $message
22
     *
23
     * @return Intent|null
24
     */
25
    public function find(ReceivedMessage $message): ?Intent
26
    {
27
        foreach ($this->intents as $intent) {
28
            foreach ($intent->activators() as $activator) {
29
                if ($activator->matches($message) && $intent->passesAuthorization()) {
30
                    return $intent;
31
                }
32
            }
33
        }
34
35
        // Otherwise, return fallback intent
36
        return $this->fallbackIntent;
37
    }
38
39
    /**
40
     * Add intent.
41
     *
42
     * @param Intent $intent
43
     */
44
    public function add(Intent $intent): void
45
    {
46
        if (!in_array($intent, $this->intents, true)) {
47
            $this->intents[] = $intent;
48
        }
49
    }
50
51
    /**
52
     * Set fallback intent.
53
     *
54
     * @param Intent $intent
55
     */
56
    public function setFallbackIntent(Intent $intent): void
57
    {
58
        $this->fallbackIntent = $intent;
59
    }
60
}
61