Как исправить «ord(): Passing null to parameter #1 ($character) of type string is deprecated»?

Question

[Slash]

Здравствуйте!
После перехода на PHP 8.1.6 сразу получил ошибку: Error: ord(): Passing null to parameter #1 ($character) of type string is deprecated - include/library/jsmin/jsmin2.class.php (339), на строку:
if (ord($get) <= self::ORD_LF) { // EOL reached
Весь код метода выглядит так:

protected function singleLineComment()
    {
        $comment = '';
        while (true) {
            $get = $this->get();
            $comment .= $get;
            if (ord($get) <= self::ORD_LF) { // EOL reached
                // if IE conditional comment
                if (preg_match('/^\\/@(?:cc_on|if|elif|else|end)\\b/', $comment)) {
                    return "/{$comment}";
                }
                return $get;
            }
        }
    }

Подскажите, как это можно исправить?
Спасибо.

Comments

[SagePtr]

Что возвращает метод $this->get() ?

[Slash]

Либо первую букву на латинском, видимо, какой строки кода, либо пробел.

Я думаю, что ошибка появляется при пробеле.

[Slash]

Вот метод get

protected function get()
    {
        $c = $this->lookAhead;
        $this->lookAhead = null;
        if ($c === null) {
            if ($this->inputIndex < $this->inputLength) {
                $c = $this->input[$this->inputIndex];
                $this->inputIndex += 1;
            } else {
                return null;
            }
        }
        if ($c === "\r" || $c === "\n") {
            return "\n";
        }
        if (ord($c) < self::ORD_SPACE) { // control char
            return ' ';
        }
        return $c;
    }

[scottparker]

Slash, а null не возвращает?

[Slash]

scottparker, похоже, что возвращает.

[scottparker]

Slash, а в ошибке видишь что написано?)

[Slash]

Вижу.

Answer 1

[Rsa97]

У вас метод get() может вернуть null, а ord() в этом случае выдаст предупреждение.
Первый вариант решения, самый быстрый - проверять вернувшееся значение
$get = $this->get() ?? "\x00";
Второй - отказаться от ord() и заменить числовую константу ORD_LF на строковую с нужным значением, вместо 10 использовать "\x0A". Придётся просмотреть все места, где константа используется.
Третий - вместо null возвращать символ с кодом 0, то есть "\x00". Но надо проверить другие места использования get() и заменить в них проверку на null.

Comments

[Slash]

Спасибо за решение:
$get = $this->get() ?? "\x00";
Ответ отмечен, как решение!