When installing D8 the user is presented with an error if some required PHP extensions are missing.
This error message is displayed as HTML code and not as rendered HTML.
After some searching I found out that the problem happens because of system.install:118 where a concatenation with a string previously passed from t() happens. The concatenation introduces a <ul><li>.....</ul> list that breaks the output. (see attached screenshots)

I tried to overcome the problem with many ways, but I couldn't do so while maintaining the list. So I attempted to change the output to something simpler, though still easy to read and understand by the end user.

The attached patch solves the problem.
I hope you like the approach

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bserem’s picture

bserem’s picture

Issue summary: View changes
bserem’s picture

Status: Active » Needs review
LinL’s picture

Thanks @bserem!

Looks like this is part of #2297711: Fix HTML escaping due to Twig autoescape

LinL’s picture

bserem’s picture

Thanks! I wasn't aware of it.

bserem’s picture

bserem’s picture

Attached patch solves this the drupal way for php-extensions.

I am continuing on the related issues.

Zekvyrin’s picture

Status: Needs review » Closed (duplicate)

This issue is relevant to issue #2346287 and should be solved there.
https://www.drupal.org/node/2346287

Marking as duplicate.

jibran’s picture

Issue tags: +SafeMarkup