phpmyadmin / phpmyadmin

libraries/classes/Triggers/Trigger.php

<?php

declare(strict_types=1);

namespace PhpMyAdmin\Triggers;

use PhpMyAdmin\Dbal\TableName;
use Webmozart\Assert\Assert;
use Webmozart\Assert\InvalidArgumentException;

final class Trigger
{
    public function __construct(
        public readonly string $name,
        public readonly Timing $timing,
        public readonly Event $event,
        public readonly TableName $table,
        public readonly string $statement,
        public readonly string $definer,
    ) {
    }

    /** @param mixed[] $trigger */
    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);
            $timing = Timing::tryFrom($timing);

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

            Assert::notNull($timing);
            Assert::string($event);
            $event = Event::tryFrom($event);

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

            Assert::notNull($event);
            Assert::string($table);
            $table = TableName::tryFromValue($table);
            Assert::notNull($table);
            Assert::string($statement);
            Assert::string($definer);

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

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

It seems like $table can also be of type null; however, parameter $table of PhpMyAdmin\Triggers\Trigger::__construct() does only seem to accept PhpMyAdmin\Dbal\TableName, 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 PhpMyAdmin\Triggers\Timing, maybe add an additional type check?

        } catch (InvalidArgumentException) {
            return null;
        }
    }
}