Page MenuHomePhabricator

Status::getHTML should return HTML.
Closed, ResolvedPublic

Description

Status::getHTML() currently calls MessageCache::transform(), which does not generate HTML. This appears to be an oversight. It should call MessageCache::parse() instead.

Status::getHTML() is not used in the core at the moment. It appears that the only usage (for which this method was created) was removed as a workaround to the problem described here.


Version: 1.21.x
Severity: normal

Details

Reference
bz45844

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 1:26 AM
bzimport added a project: MediaWiki-Parser.
bzimport set Reference to bz45844.
daniel created this task.Mar 7 2013, 2:49 PM
Anomie added a comment.Dec 6 2013, 5:02 PM

I just ran into this issue. Are you still working on this? If not, I'm going to fix it.

Go ahead, I had already forgotten about this :)

Change 99798 had a related patch set uploaded by Anomie:
Status::getHTML should actually return HTML

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

Change 99798 merged by jenkins-bot:
Status::getHTML should actually return HTML

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