Fix variable substitution in Drush log messages

Created on 15 October 2023, about 1 year ago
Updated 16 October 2023, about 1 year ago

As already reported by ressa β†’ in πŸ› Update README wording for SEARCH_API_SERVER_ID Fixed , variable replacement in Drush log messages currently doesn’t work.
Markus responded there by saying the Search API uses the same code, but this was either out-dated or a mistake: while Search API Solr uses $this->logger()->success($msg, $vars), the Search API has an additional nested dt() call (or $this->t(), really, but redirecting to dt()). It seems \Drush\Log\DrushLoggerManager::success() just doesn’t support variable substitutions.

πŸ› Bug report
Status

Fixed

Version

4.0

Component

Code

Created by

πŸ‡¦πŸ‡ΉAustria drunken monkey Vienna, Austria

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.71.5 2024