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

Merge "Don't leave unclosed <li> behind"

Browse source
This commit is contained in:
jenkins-bot 2019-12-01 10:36:56 +00:00 committed by Gerrit Code Review
parent 76ef4af51f 3cf1e99cc2
commit 469a1e6364
1 changed files with 22 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

30
src/FootnoteBodyFormatter.php
View file

@ -67,20 +67,21 @@ class FootnoteBodyFormatter {
$indented = false;
foreach ( $groupRefs as $key => $value ) {
// FIXME: This assumes extended references appear immediately after their parent in the
// array. Reorder the refs according to their stored numbering.
// data structure. Reorder the refs according to their stored numbering.
if ( !$indented && isset( $value['extends'] ) ) {
// Hack: The nested <ol> needs to go inside of the <li>.
$parserInput = preg_replace( '/<\/li>\s*$/', '', $parserInput );
// The nested <ol> must be inside the parent's <li>
if ( preg_match( '#</li>\s*$#D', $parserInput, $matches, PREG_OFFSET_CAPTURE ) ) {
$parserInput = substr( $parserInput, 0, $matches[0][1] );
}
$parserInput .= Html::openElement( 'ol', [ 'class' => 'mw-extended-references' ] );
$indented = true;
$indented = $matches[0][0] ?? true;
} elseif ( $indented && !isset( $value['extends'] ) ) {
// FIXME: This is't closed if there is no unindented element at the end
$parserInput .= Html::closeElement( 'ol' );
$parserInput .= $this->closeIndention( $indented );
$indented = false;
}
$parserInput .= $this->referencesFormatEntry(
$key, $value, $isSectionPreview ) . "\n";
$parserInput .= $this->referencesFormatEntry( $key, $value, $isSectionPreview ) . "\n";
}
$parserInput .= $this->closeIndention( $indented );
$parserInput = Html::rawElement( 'ol', [ 'class' => [ 'references' ] ], $parserInput );
// Live hack: parse() adds two newlines on WM, can't reproduce it locally -ævar
$ret = rtrim( $this->parser->recursiveTagParse( $parserInput ), "\n" );
@ -98,6 +99,19 @@ class FootnoteBodyFormatter {
}
}
/**
* @param string|bool $closingLi
*
* @return string
*/
private function closeIndention( $closingLi ) : string {
if ( !$closingLi ) {
return '';
}
return Html::closeElement( 'ol' ) . ( is_string( $closingLi ) ? $closingLi : '' );
}
/**
* Format a single entry for the referencesFormat() function
*