Page MenuHomePhabricator

OOUI PHP: No simple way to add HTML-escaped text to a Element
Closed, ResolvedPublic

Description

There's no simple way to add HTML-escaped text to a Element. OoUiTag::appendContent() doesn't escape strings.

I say we should add a appendText() method and use it where applicable.


Version: unspecified
Severity: major

Details

Reference
bz72542

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 3:52 AM
bzimport added a project: OOUI.
bzimport set Reference to bz72542.

We currently have some bugs where JS and PHP escape things differently because of this.

gerritadmin wrote:

Change 169106 had a related patch set uploaded by Bartosz Dziewoński:
OoUiTag: HTML-escape everything by default

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

gerritadmin wrote:

Change 169106 merged by jenkins-bot:
OoUiTag: HTML-escape everything by default

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