Mention relevant CR in an improved trigger_error() message in FormattableMarkup::placeholderFormat()

Created on 9 December 2016, over 7 years ago
Updated 10 March 2023, over 1 year ago

Follow-up to #2807705: FormattableMarkup::placeholderFormat() can result in unsafe replacements →

Enhance trigger_error with an improved message and a link to the CR → . Suggestion, courtesy of @alexpott:

elseif (strpos($string, $key) !== FALSE) {
  trigger_error("Fallthrough for unrecognised placeholders to %variable has been removed. See https://www.drupal.org/node/2605274. Invalid placeholder ($key) in string: $string', E_USER_DEPRECATED);
}
📌 Task
Status

Needs review

Version

10.1 ✨

Component
Base  →

Last updated about 1 hour ago

Created by

🇺🇸United States cilefen

Live updates comments and jobs are added and updated live.
  • Novice

    It would make a good project for someone who is new to the Drupal contribution process. It's preferred over Newbie.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.69.0 2024