PhpMyAdmin\Triggers\Trigger::tryFromArray()

Complexity

Conditions	2
Paths	14

Size

Total Lines	19
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	16
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

<?php
    public static function tryFromArray(array $trigger): self|null
    {
        try {
            $name = $trigger['Trigger'] ?? $trigger['TRIGGER_NAME'] ?? null;
            $timing = $trigger['Timing'] ?? $trigger['ACTION_TIMING'] ?? null;
            $event = $trigger['Event'] ?? $trigger['EVENT_MANIPULATION'] ?? null;
            $table = $trigger['Table'] ?? $trigger['EVENT_OBJECT_TABLE'] ?? null;
            $statement = $trigger['Statement'] ?? $trigger['ACTION_STATEMENT'] ?? null;
            $definer = $trigger['Definer'] ?? $trigger['DEFINER'] ?? null;
            Assert::string($name);
            Assert::string($timing);
            Assert::string($event);
            Assert::string($table);
            Assert::string($statement);
            Assert::string($definer);

            return new self($name, $timing, $event, $table, $statement, $definer);

It seems like $table can also be of type null; however, parameter $table of PhpMyAdmin\Triggers\Trigger::__construct() does only seem to accept string, maybe add an additional type check?

It seems like $name can also be of type null; however, parameter $name of PhpMyAdmin\Triggers\Trigger::__construct() does only seem to accept string, maybe add an additional type check?

It seems like $definer can also be of type null; however, parameter $definer of PhpMyAdmin\Triggers\Trigger::__construct() does only seem to accept string, maybe add an additional type check?

It seems like $timing can also be of type null; however, parameter $timing of PhpMyAdmin\Triggers\Trigger::__construct() does only seem to accept string, maybe add an additional type check?

It seems like $event can also be of type null; however, parameter $event of PhpMyAdmin\Triggers\Trigger::__construct() does only seem to accept string, maybe add an additional type check?

It seems like $statement can also be of type null; however, parameter $statement of PhpMyAdmin\Triggers\Trigger::__construct() does only seem to accept string, maybe add an additional type check?

        } catch (InvalidArgumentException) {
            return null;
        }