Page MenuHomePhabricator

Use postprocessing cache for Discussion Tools
Open, LowPublic

Description

There are a number of DT issues like T371125 that are caused by the ParserAfterTidy hook being run too often and in too many contexts: it is run for sub-parses, for interface messages, for random text on special pages, etc.

Eventually we might solve this by moving processing into OutputTransform, but keeping it pre-cache.

But in the near term, the issue is mostly that core doesn't have an unambiguous "at top level" indicator to flag when the ParserOutput has the complete top level contents of an article page, ie not a sub-parse or interface text, etc. This could be addressed by adding an ArticleAfterParse hook (bikeshed bikeshed) like we added ArticleParserOptions to allow DiscussionTools to post process *only complete pages* not partial parses.

Related Objects

StatusSubtypeAssignedTask
OpenNone
Openihurbain
Openihurbain
Resolvedihurbain
Resolvedihurbain
Resolvedcscott
Resolvedihurbain
Resolvedcscott
Resolvedihurbain
OpenNone
Opencscott
OpenBUG REPORTNone
Opencscott
OpenNone
OpenNone
ResolvedKrinkle
ResolvedKrinkle
ResolvedDAlangi_WMF
Opencscott
Openihurbain
Resolvedihurbain
Resolvedihurbain
Openihurbain
Resolvedcscott
Openihurbain
OpenNone
ResolvedPRODUCTION ERRORcscott
Openihurbain

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
MSantos edited projects, added MW-1.43-release, Content-Transform-Team-WIP; removed Parsoid.
ihurbain subscribed.

This is the path we currently see to be able to have legacy DT in the post-processing cache, AND to be able to move away from modifying cache-varying options during the post-processing, so I'm moving this back to "current work" and doing that now.

cscott renamed this task from Create new AfterArticleParse hook for Discussion Tools to Use postprocessing cache for Discussion Tools.Mon, Jan 26, 4:14 PM

Change #1233222 had a related patch set uploaded (by Isabelle Hurbain-Palatin; author: Isabelle Hurbain-Palatin):

[mediawiki/core@master] Clarify the post-processing cache status of Article

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

Change #1233222 merged by jenkins-bot:

[mediawiki/core@master] Clarify the post-processing cache status of Article

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

Change #1235841 had a related patch set uploaded (by Isabelle Hurbain-Palatin; author: Isabelle Hurbain-Palatin):

[mediawiki/core@master] Sets title in Parser-generated ParserOutput

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

Change #1235841 merged by jenkins-bot:

[mediawiki/core@master] Sets title in Parser-generated ParserOutput

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