greenlion/PHP-SQL-Parser

PHP 8.0 Fatal error in ExpressionToken::setNoQuotes

Closed this issue · 1 comments

PHP 8 does not allow optional parameters to come before mandatory parameters

public function setNoQuotes($token, $qchars = null, Options $options) {

Looking at the code it appears that changing the function signature to

public function setNoQuotes($token, $qchars = null, Options $options.= null) {

Looking at the calls it looks like a null Option will be handled correctly throughout the rest of the stack, it eventually gets passed to the DefaultProcessor constructor, which is already defined to default the options to null if they are not provided

dupe of #331