Problem/Motivation

SafeMarkup::format() is deprecated and other core loggers (syslog, dblog) use the logger.log_message_parser service to parse message placeholders.

Proposed resolution

Use the logger.log_message_parser service to parse message placeholders.

Maybe we could add a setting to let site admin decide to replace or not message placeholders (not included in this patch).

Remaining tasks

Review patch.

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#2 replace-2649016-2.patch813 byteslussoluca
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lussoluca created an issue. See original summary.

lussoluca’s picture

lussoluca
he/him
Primary language Italian
Location Italy
CreditAttribution: lussoluca as a volunteer commented
FileSize
replace-2649016-2.patch813 bytes
lussoluca’s picture

lussoluca
he/him
Primary language Italian
Location Italy
CreditAttribution: lussoluca as a volunteer commented
Status: Active » Needs work

Forgot NR

lussoluca’s picture

lussoluca
he/him
Primary language Italian
Location Italy
CreditAttribution: lussoluca as a volunteer commented
Status: Needs work » Needs review

No it's NR, sorry for this noise

ParisLiakos’s picture

ParisLiakos CreditAttribution: ParisLiakos commented
Status: Needs review » Fixed

thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.