Page MenuHomePhabricator

DiscussionTools Echo hook should use Parsoid parse (not legacy parse)
Closed, ResolvedPublic

Description

In an effort to transition DT to 100% parsoid, both for read views and for post-edit tasks, the DiscussionTools EchoHook class should use the Parsoid parse of the page instead of the legacy parser parse.

CF this note in DataUpdatesHooks.php:

// TODO Deduplicate work between this and the Echo hook (make it use Parsoid too)

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change 989952 had a related patch set uploaded (by C. Scott Ananian; author: C. Scott Ananian):

[mediawiki/extensions/DiscussionTools@master] Use Parsoid HTML to drive notifications

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

Change 989952 merged by jenkins-bot:

[mediawiki/extensions/DiscussionTools@master] Use Parsoid HTML to drive notifications

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

matmarex assigned this task to cscott.