wodby / drupal-php

PHP docker container image for Drupal
https://wodby.com/stacks/drupal
MIT License
60 stars 80 forks source link

Drush gives errors on PHP 8.2 #95

Closed back-2-95 closed 1 year ago

back-2-95 commented 1 year ago

Error is

Deprecated function: Using ${var} in strings is deprecated, use {$var} instead in include() (line 21 of /var/www/html/public/sites/default/settings.php)

And that line has the include '/var/www/conf/wodby.settings.php';

And when I checked that file, there is at the very end this where it has ${server}:

if (!empty($wodby['solr_cloud']['password'])) {
    $server = $wodby['solr_cloud']['server'];
    $config["search_api.server.${server}"]['backend_config']['connector_config']['password'] = $wodby['solr_cloud']['password'];
}
csandanov commented 1 year ago

Great find, thank you!

back-2-95 commented 1 year ago

Next stack update will have these?

csandanov commented 1 year ago

yes

back-2-95 commented 1 year ago

@csandanov fyi https://github.com/drush-ops/drush-launcher/issues/102

The project has not been updated since Feb 2022 and cannot see much answer on the issues either. I wonder if drush-launcher is active anymore...