- Issue created by @bbrala
- First commit to issue fork.
- Status changed to Postponed: needs info
about 1 year ago 7:59am 17 April 2024 - 🇭🇺Hungary Gábor Hojtsy Hungary
Hm, we have this in our coverage list since August 15, 2022: https://git.drupalcode.org/project/deprecation_status/-/commit/1ab775092...
Is there something wrong with this?
- 🇭🇺Hungary Gábor Hojtsy Hungary
Ah the coverage list has
\Drupal
but the error you cited above does not have the backslash before Drupal. Messages are not consistent in this. I can't find a contrib that has this, so went to check Drupal 9.5.x: https://git.drupalcode.org/project/drupal/-/blob/9.5.x/core/modules/user... That also has\Drupal
in both the@deprecated
and in thetrigger_error()
, so looks like our coverage information is correct in deprecation_status?Is there something wrong in deprecation_status?
- Status changed to Fixed
about 1 year ago 8:19am 17 April 2024 - 🇭🇺Hungary Gábor Hojtsy Hungary
Duh, looks like we modify that to make unified:
$error_string = preg_replace('!(u|U)se \\\\Drupal!', '\1se Drupal', $error_string);
So yeah the listing is wrong, and it needs to be without the backslash.
Fixed that in https://git.drupalcode.org/project/deprecation_status/-/commit/b7a12ae57...
Automatically closed - issue fixed for 2 weeks with no activity.