Follow up for #2042143: [META] Fix unsafe core translatable strings

Problem/Motivation

In the FeedStorageController.php file there is one string not passing the locale_string_is_safe test because of the usage of the <description> tag.

Proposed resolution

Use a placeholder.

Credit

This issue was covered during the first Drupal Sprint of Montpellier (France).

CommentFileSizeAuthor
#1 drupal-remove_description_markup_from_translated_string-2046415-1.patch834 bytesDuaelFr
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DuaelFr’s picture

DuaelFr
he/him
Primary language French
Location Montpellier, France
CreditAttribution: DuaelFr commented
Assigned: DuaelFr » Unassigned
Status: Active » Needs review
FileSize
drupal-remove_description_markup_from_translated_string-2046415-1.patch834 bytes

Here is the patch !

DuaelFr’s picture

DuaelFr
he/him
Primary language French
Location Montpellier, France
CreditAttribution: DuaelFr commented
Issue tags: +Novice
opi’s picture

opi
Primary language French
CreditAttribution: opi commented
Status: Needs review » Reviewed & tested by the community

This patch looks fine for me. thanks.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Committed 4501400 and pushed to 8.x. Thanks!

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

Anonymous’s picture

(not verified) CreditAttribution: commented
Issue summary: View changes

Added around