Page MenuHomePhabricator

Error message with HTML entity insertables as params should display the entity as text
Closed, ResolvedPublicBUG REPORT

Description

Steps to Reproduce:

  1. Start translating a message that uses an HTML entity as an insertable.
  2. Don't add the HTML entity in the translation.

Actual Results:

The HTML entity is converted to HTML and displayed, but the insertable shows the text of the HTML entity.

See below,

validator-framework-issue.png (398×990 px, 30 KB)

Expected Results:
Insertables that are HTML entities should be displayed as is, and not transformed.

Event Timeline

Good catch. The parameter needs additional escaping. Should be simple to fix once you find the place (is it on PHP side or JavaScript side).

Change 546902 had a related patch set uploaded (by Nikerabbit; owner: Nikerabbit):
[mediawiki/extensions/Translate@master] Display error message with HTML entity insertables properly escaped

https://gerrit.wikimedia.org/r/546902

Tested the patch by Niklas locally, and it works well. See screenshot,

image.png (423×960 px, 30 KB)

Change 546902 merged by jenkins-bot:
[mediawiki/extensions/Translate@master] Display error message with HTML entity insertables properly escaped

https://gerrit.wikimedia.org/r/546902

Tested this again locally today, and it works fine. Since this does not need any additional configuration after deployment, resolving this task.

image.png (458×855 px, 28 KB)