tinify/tinify-php

PHP 8.2: Deprecated ${} string interpolation.

kevin-appelt opened this issue · 4 comments

kevin-appelt commented

${var} String Interpolation Deprecated
PHP has always supported string variable interpolation with "foo {$bar}" pattern, and along with an alternative syntax that the dollar sign is placed outside the curly braces (e.g. "foo ${bar}").
In PHP 8.2, the alternative syntax of placing the dollar sign outside the curly braces is deprecated.

Deprecated in PHP 8.2 Recommended syntax for PHP 8.2
Hello ${name}; Hello {$name}
echo "Hello ${$var}"; echo "Hello {$var}";

Source: https://php.watch/versions/8.2

For detailed information see PHP RFC: https://wiki.php.net/rfc/deprecate_dollar_brace_string_interpolation

rkoopmans commented

Thanks for your report, we have prepared a change!

kevin-appelt commented

@rkoopmans Thanks for the change. LGTM and I am looking forward for the release. :)

kevin-appelt commented

@rkoopmans AFAICS you merged the changes to master and changed the bumped the version in the code, but there is a tag missing to get the release f.e. via packagist.

Mind to add the 1.6.1 tag? :)

If done, this issues can be closed 👍

rkoopmans commented

hello Kevin, This is done now ;-)