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

Comments

alansaviolobo’s picture

alansaviolobo commented
Status: Active » Needs review
StatusFileSize
new 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 commented
Status: Needs work » Needs review
StatusFileSize
new uninstall_page_showing-2310241-3.patch913 bytes
LinL’s picture

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

jibran
he/him
Primary language Urdu
Location Toronto, Canada
commented

Can you share the after screenshot? Thanks

dawehner’s picture

dawehner
Primary language German
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 commented
Status: Needs work » Needs review
StatusFileSize
new uninstall-2310241-6.patch1.3 KB
new 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
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 Toronto, Canada
commented
Issue tags: +SafeMarkup