Page MenuHomePhabricator

Followup to fix corner cases in ToC language-conversion
Closed, ResolvedPublic

Description

We moved the ToC insertion from the parser to ParserOutput::getText() in T287767 but forgot to ensure that the ToC contents are properly language converted. This was hacked into ParserOutput::getText() in T295187 but some of the state from Parser isn't available to ParserOutput.

This is the follow-up task to make sure the new code matches the old behavior precisely. In particular:

  • ensure that ParserOptions::disableContentConversion() is respected. (It seems like this could actually be deprecated and a new Content type created for Conversion tables to have the same effect.)
  • ensure that __NOCONTENTCONVERT__ is respected
  • ensure that any differences between Parser::getTargetLanguage() and ::getContentLanguage() are resolved.

Event Timeline

cscott renamed this task from ToC needs to be language-converted to Follow to fix corner cases in ToC language-conversion.Nov 6 2021, 12:55 AM
cscott updated the task description. (Show Details)
cscott renamed this task from Follow to fix corner cases in ToC language-conversion to Followup to fix corner cases in ToC language-conversion.Nov 18 2021, 7:30 PM
cscott claimed this task.

This was fixed in 0955046ca59aae705fee8b098e4cfef394647b5e which was tagged with T303235, T295187, and T299973.