Replace !placeholder with @placeholder in Config Translation module [#2559449]

In order to make #2506445: Replace !placeholder with @placeholder in t() and format_string() for non-URLs in tests approachable, we need to break it up into smaller chunks. This issue address !placeholder in the Config Translation module

Replace !placeholder with @placeholder in the Config Translation module.

core/modules/config_translation/*

Replace !placeholder with @placeholder. Refer to patch in #2506445-85: Replace !placeholder with @placeholder in t() and format_string() for non-URLs in tests as that patch should have related update
Ensure tests come back clean
Manually test the update and post screen shot after patch, review source for any difference in escaping.

Comment	File	Size	Author
#3	interdiff-2559449-2-3.txt	5.59 KB	geertvd
#3	replace_placeholder-2559449-3.patch	8.17 KB	geertvd
#3	8.0.x: PHP 5.5 & MySQL 5.5 13,702 pass
#2	replace_placeholder-2559449-2.patch	13.47 KB	joelpittet
#2	8.0.x: PHP 5.5 & MySQL 5.5 13,556 pass

Comments

28 August 2015 at 18:34

English

Vancouver

CreditAttribution: joelpittet as a volunteer commented 28 August 2015 at 20:59

Status:

Active

» Needs review

File	Size
replace_placeholder-2559449-2.patch	13.47 KB
8.0.x: PHP 5.5 & MySQL 5.5 13,556 pass

geertvd CreditAttribution: geertvd at XIO commented 6 September 2015 at 18:27

justAChris CreditAttribution: justAChris as a volunteer commented 10 September 2015 at 15:16

Status:

Needs review

» Postponed

justAChris CreditAttribution: justAChris as a volunteer commented 11 September 2015 at 21:29

Status:

Postponed

» Closed (duplicate)

Closing this, splitting by module was not the ideal approach to removing !placeholder. Marking as duplicate of #2506427: [meta] !placeholder causes strings to be escaped and makes the sanitization API harder to understand, since the chosen approach is / will be outlined there, please refer to it for any additional action.