Running a test build and this is what I see at admin/modules/uninstall

CommentFileSizeAuthor
#7 Screen Shot 2014-08-04 at 3.06.50 PM.png96.13 KBCharuAg
#7 uninstall-2310241-6.patch1.3 KBCharuAg
#3 uninstall_page_showing-2310241-3.patch913 bytesalansaviolobo
#1 uninstall_page_showing-2310241-1.patch945 bytesalansaviolobo
Screen Shot 2014-07-26 at 12.03.19 PM.png103.63 KBagentrickard
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alansaviolobo’s picture

alansaviolobo CreditAttribution: alansaviolobo commented
Status: Active » Needs review
FileSize
uninstall_page_showing-2310241-1.patch945 bytes

the html was not wrapped in SafeMarkup::set hence the issue.

Status: Needs review » Needs work

The last submitted patch, 1: uninstall_page_showing-2310241-1.patch, failed testing.

alansaviolobo’s picture

alansaviolobo CreditAttribution: alansaviolobo commented
Status: Needs work » Needs review
FileSize
uninstall_page_showing-2310241-3.patch913 bytes
LinL’s picture

LinL CreditAttribution: LinL commented
Parent issue: » #2297711: Fix HTML escaping due to Twig autoescape
jibran’s picture

jibran
he/him
Primary language Urdu
Location Sydney, Australia
CreditAttribution: jibran commented

Can you share the after screenshot? Thanks

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Needs work

I am sorry but this is not the right fix for that problem: https://groups.drupal.org/node/436473

CharuAg’s picture

CharuAg CreditAttribution: CharuAg commented
Status: Needs work » Needs review
FileSize
uninstall-2310241-6.patch1.3 KB
Screen Shot 2014-08-04 at 3.06.50 PM.png96.13 KB

Addressed comment 6.

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave commented
Status: Needs review » Closed (duplicate)

Duplicate of #2305831: Double escaping on /admin/modules/uninstall

jibran’s picture

jibran
he/him
Primary language Urdu
Location Sydney, Australia
CreditAttribution: jibran commented
Issue tags: +SafeMarkup