phpmyadmin / sql-parser

src/Components/ParameterDefinition.php

<?php

declare(strict_types=1);

namespace PhpMyAdmin\SqlParser\Components;

use PhpMyAdmin\SqlParser\Component;
use PhpMyAdmin\SqlParser\Context;

use function trim;

/**
 * The definition of a parameter of a function or procedure.
 */
final class ParameterDefinition implements Component
{
    /**
     * The name of the new column.
     */
    public string|null $name = null;

    /**
     * Parameter's direction (IN, OUT or INOUT).
     */
    public string|null $inOut = null;

    /**
     * The data type of thew new column.
     */
    public DataType|null $type = null;

    /**
     * @param string|null   $name  parameter's name
     * @param string|null   $inOut parameter's directional type (IN / OUT or None)
     * @param DataType|null $type  parameter's type
     */
    public function __construct(string|null $name = null, string|null $inOut = null, DataType|null $type = null)
    {
        $this->name = $name;
        $this->inOut = $inOut;
        $this->type = $type;
    }

    public function build(): string
    {
        $tmp = '';
        if (! empty($this->inOut)) {
            $tmp .= $this->inOut . ' ';
        }

        return trim(
            $tmp . Context::escape($this->name) . ' ' . $this->type,

It seems like $this->name can also be of type null; however, parameter $str of PhpMyAdmin\SqlParser\Context::escape() does only seem to accept string, maybe add an additional type check?

        );
    }

    public function __toString(): string
    {
        return $this->build();
    }
}