d6lts / drupal

Fork of Drupal core for Drupal 6 LTS support.
https://www.drupal.org
GNU General Public License v2.0
130 stars 45 forks source link

PHP 8 Update for bootstrap.inc #81

Open Dorson opened 8 months ago

Dorson commented 8 months ago

Updated syntax / variable definition to the PHP 8 version standard.

Makes the bootstrap.inc run on php 8.2

old format was ${VAR} , new PHP8 format is {$VAR}

Simple , one letter change makes it work :-)

Dorson commented 8 months ago

This really solved my issues. Drupal d6lts would not run on my PHP 8.2 version.

The variable syntax change was suggested by PHP 8.2 itself , because it would not execute the code otherwise.

Here is the explanation from the PHP 8.2 documentation :

https://php.watch/versions/8.2/$%7Bvar%7D-string-interpolation-deprecated

Here is the search for all the similar / possilbe bugs of the same type in this repo :

https://github.com/search?q=repo%3Ad6lts%2Fdrupal%20%2F%24%7B%2F&type=code

Hope it helps !