Page MenuHomePhabricator

Parsoid read views doesn't support -{T|...}- page title markup
Open, LowPublic

Description

Need to investigate root cause here, but if parsoid read views are enabled and (say) the following is on the page:

-{T|en:Pig Latin Test (en);en-x-piglatin:Pig Latin Test (pig latin);sr:Pig Latin Test (sr);sr-ec:Pig Latin Test (sr-ec);sr-el:Pig Latin Test (sr-el)}-

The title is shown as:

en:Pig Latin Test (en);en-x-piglatin:Pig Latin Test (pig latin);sr:Pig Latin Test (sr);sr-ec:Pig Latin Test (sr-ec);sr-el:Pig Latin Test (sr-el)

That is, the -{T|...}- seems to be being parsed and someone is calling the appropriate method in ParserOutput to update the page title, but rather than actually selecting the proper title based on the current variant, the entire string is being made the title.

Event Timeline

ssastry triaged this task as Medium priority.Jun 14 2022, 10:39 PM

Note that we likely *do* support the {{DISPLAYTITLE}} markup, since that is processed by CoreParserFunctions in integrated mode and writes directly into the ParserOutput. However, we should double check that handling while we're writing test cases for -{T|....}-.

MSantos lowered the priority of this task from Medium to Low.Tue, Sep 19, 3:54 PM