wikimedia/mediawiki-extensions-DiscussionTools
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/DiscussionTools synced 2024-11-15 03:44:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge "Don't rely on ParserOutput::getTOCHTML() to determine if ToC is present"

Browse source
This commit is contained in:
jenkins-bot 2023-01-27 10:13:37 +00:00 committed by Gerrit Code Review
parent cf10fb1566 086ce6550f
commit e97d9c3dfa
2 changed files with 12 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
includes/CommentFormatter.php
View file

@ -62,17 +62,19 @@ class CommentFormatter {
// Enhance the table of contents in supporting skins (vector-2022)
// Only do the work if the HTML would be shown. It looks like we can only check this
// by checking whether the HTML for the normal TOC has been generated. Code in
// OutputPage::addParserOutputMetadata does the same.
if ( $pout->getTOCHTML() ) {
// If the TOC HTML has been generated, then the parser cache is already split by user
// language (because of the "Contents" header in the TOC), so we can render text in user
// language as well. If that behavior in core changes, then we'll have to change this to
// happen in a post-processing step (like all other transformations) to avoid splitting it.
// Only do the work if the ToC is present
// (Note that skins may decide to show the ToC even if
// "not recommended" (ie, the SHOW_TOC flag is false); see
// T315862.
if ( $pout->getTOCData() != null ) {
// The TOC generation no longer splits the parser cache
// by user language, but the call below to
// getUserLangObj() will do so.
// FIXME: change this to happen in a post-processing
// step (like all other transformations) to avoid
// splitting the cache.
$lang = $parser->getOptions()->getUserLangObj();
$sections = $pout->getTOCData() === null ? [] :
$pout->getTOCData()->getSections();
$sections = $pout->getTOCData()->getSections();
foreach ( $sections as $item ) {
$key = str_replace( '_', ' ', $item->anchor );
// Unset if we did not format this section as a topic container

2
tests/phpunit/integration/ApiParseDiscussionToolsTest.php
View file

@ -14,8 +14,6 @@ class ApiParseDiscussionToolsTest extends ApiTestCase {
* @covers \MediaWiki\Extension\DiscussionTools\CommentFormatter::addDiscussionTools
*/
public function testApiParseSections() {
$this->markTestSkipped( 'T328072' );
$params = [
'action' => 'parse',
'title' => 'Talk:Test',