CommentParser.php
No OneTemporary
Actions

Size

49 KB

Referenced Files

None

Subscribers

None

CommentParser.php
View Options

	<?php

	namespace MediaWiki\Extension\DiscussionTools;

	use DateInterval;
	use DateTime;
	use DateTimeImmutable;
	use DateTimeZone;
	use InvalidArgumentException;
	use LogicException;
	use MediaWiki\Config\Config;
	use MediaWiki\Extension\DiscussionTools\ThreadItem\ContentCommentItem;
	use MediaWiki\Extension\DiscussionTools\ThreadItem\ContentHeadingItem;
	use MediaWiki\Extension\DiscussionTools\ThreadItem\ContentThreadItem;
	use MediaWiki\Language\Language;
	use MediaWiki\Languages\LanguageConverterFactory;
	use MediaWiki\Title\MalformedTitleException;
	use MediaWiki\Title\TitleParser;
	use MediaWiki\Title\TitleValue;
	use MediaWiki\Utils\MWTimestamp;
	use RuntimeException;
	use Wikimedia\Assert\Assert;
	use Wikimedia\IPUtils;
	use Wikimedia\Parsoid\DOM\Element;
	use Wikimedia\Parsoid\DOM\Node;
	use Wikimedia\Parsoid\DOM\Text;
	use Wikimedia\Parsoid\Utils\DOMCompat;
	use Wikimedia\Timestamp\TimestampException;

	// TODO consider making timestamp parsing not a returned function

	class CommentParser {

	/**
	* How far backwards we look for a signature associated with a timestamp before giving up.
	* Note that this is not a hard limit on the length of signatures we detect.
	*/
	private const SIGNATURE_SCAN_LIMIT = 100;

	private Config $config;
	private Language $language;
	private LanguageConverterFactory $languageConverterFactory;
	private TitleParser $titleParser;

	/** @var string[] */
	private array $dateFormat;
	/** @var string[][] */
	private array $digits;
	/** @var string[][] */
	private $contLangMessages;
	private string $localTimezone;
	/** @var string[][] */
	private array $timezones;
	private string $specialContributionsName;

	private Element $rootNode;
	private TitleValue $title;

	/**
	* @param Config $config
	* @param Language $language Content language
	* @param LanguageConverterFactory $languageConverterFactory
	* @param LanguageData $languageData
	* @param TitleParser $titleParser
	*/
	public function __construct(
	Config $config,
	Language $language,
	LanguageConverterFactory $languageConverterFactory,
	LanguageData $languageData,
	TitleParser $titleParser
	) {
	$this->config = $config;
	$this->language = $language;
	$this->languageConverterFactory = $languageConverterFactory;
	$this->titleParser = $titleParser;

	$data = $languageData->getLocalData();
	$this->dateFormat = $data['dateFormat'];
	$this->digits = $data['digits'];
	$this->contLangMessages = $data['contLangMessages'];
	$this->localTimezone = $data['localTimezone'];
	$this->timezones = $data['timezones'];
	$this->specialContributionsName = $data['specialContributionsName'];
	}

	/**
	* Parse a discussion page.
	*
	* @param Element $rootNode Root node of content to parse
	* @param TitleValue $title Title of the page being parsed
	* @return ContentThreadItemSet
	*/
	public function parse( Element $rootNode, TitleValue $title ): ContentThreadItemSet {
	$this->rootNode = $rootNode;
	$this->title = $title;

	$result = $this->buildThreadItems();
	$this->buildThreads( $result );
	$this->computeIdsAndNames( $result );

	return $result;
	}

	/**
	* Return the next leaf node in the tree order that is likely a part of a discussion comment,
	* rather than some boring "separator" element.
	*
	* Currently, this can return a Text node with content other than whitespace, or an Element node
	* that is a "void element" or "text element", except some special cases that we treat as comment
	* separators (isCommentSeparator()).
	*
	* @param ?Node $node Node after which to start searching
	* (if null, start at the beginning of the document).
	* @return Node
	*/
	private function nextInterestingLeafNode( ?Node $node ): Node {
	$rootNode = $this->rootNode;
	$treeWalker = new TreeWalker(
	$rootNode,
	NodeFilter::SHOW_ELEMENT \| NodeFilter::SHOW_TEXT,
	static function ( $n ) use ( $node, $rootNode ) {
	// Skip past the starting node and its descendants
	if ( $n === $node \|\| $n->parentNode === $node ) {
	return NodeFilter::FILTER_REJECT;
	}
	// Ignore some elements usually used as separators or headers (and their descendants)
	if ( CommentUtils::isCommentSeparator( $n ) ) {
	return NodeFilter::FILTER_REJECT;
	}
	// Ignore nodes with no rendering that mess up our indentation detection
	if ( CommentUtils::isRenderingTransparentNode( $n ) ) {
	return NodeFilter::FILTER_REJECT;
	}
	if ( CommentUtils::isCommentContent( $n ) ) {
	return NodeFilter::FILTER_ACCEPT;
	}
	return NodeFilter::FILTER_SKIP;
	}
	);
	if ( $node ) {
	$treeWalker->currentNode = $node;
	}
	$treeWalker->nextNode();
	if ( !$treeWalker->currentNode ) {
	throw new RuntimeException( 'nextInterestingLeafNode not found' );
	}
	return $treeWalker->currentNode;
	}

	/**
	* @param string[] $values Values to match
	* @return string Regular expression
	*/
	private static function regexpAlternateGroup( array $values ): string {
	return '(' . implode( '\|', array_map( static function ( string $x ) {
	return preg_quote( $x, '/' );
	}, $values ) ) . ')';
	}

	/**
	* Get text of localisation messages in content language.
	*
	* @param string $contLangVariant Content language variant
	* @param string[] $messages Message keys
	* @return string[] Message values
	*/
	private function getMessages( string $contLangVariant, array $messages ): array {
	return array_map( function ( string $key ) use ( $contLangVariant ) {
	return $this->contLangMessages[$contLangVariant][$key];
	}, $messages );
	}

	/**
	* Get a regexp that matches timestamps generated using the given date format.
	*
	* This only supports format characters that are used by the default date format in any of
	* MediaWiki's languages, namely: D, d, F, G, H, i, j, l, M, n, Y, xg, xkY (and escape characters),
	* and only dates when MediaWiki existed, let's say 2000 onwards (Thai dates before 1941 are
	* complicated).
	*
	* @param string $contLangVariant Content language variant
	* @param string $format Date format
	* @param string $digitsRegexp Regular expression matching a single localised digit, e.g. '[0-9]'
	* @param array $tzAbbrs Associative array mapping localised timezone abbreviations to
	* IANA abbreviations, for the local timezone, e.g. [ 'EDT' => 'EDT', 'EST' => 'EST' ]
	* @return string Regular expression
	*/
	private function getTimestampRegexp(
	string $contLangVariant, string $format, string $digitsRegexp, array $tzAbbrs
	): string {
	$formatLength = strlen( $format );
	$s = '';
	$raw = false;
	// Adapted from Language::sprintfDate()
	for ( $p = 0; $p < $formatLength; $p++ ) {
	$num = false;
	$code = $format[ $p ];
	if ( $code === 'x' && $p < $formatLength - 1 ) {
	$code .= $format[++$p];
	}
	if ( $code === 'xk' && $p < $formatLength - 1 ) {
	$code .= $format[++$p];
	}

	switch ( $code ) {
	case 'xx':
	$s .= 'x';
	break;
	case 'xg':
	$s .= static::regexpAlternateGroup(
	$this->getMessages( $contLangVariant, Language::MONTH_GENITIVE_MESSAGES )
	);
	break;
	case 'xn':
	$raw = true;
	break;
	case 'd':
	$num = '2';
	break;
	case 'D':
	$s .= static::regexpAlternateGroup(
	$this->getMessages( $contLangVariant, Language::WEEKDAY_ABBREVIATED_MESSAGES )
	);
	break;
	case 'j':
	$num = '1,2';
	break;
	case 'l':
	$s .= static::regexpAlternateGroup(
	$this->getMessages( $contLangVariant, Language::WEEKDAY_MESSAGES )
	);
	break;
	case 'F':
	$s .= static::regexpAlternateGroup(
	$this->getMessages( $contLangVariant, Language::MONTH_MESSAGES )
	);
	break;
	case 'M':
	$s .= static::regexpAlternateGroup(
	$this->getMessages( $contLangVariant, Language::MONTH_ABBREVIATED_MESSAGES )
	);
	break;
	case 'm':
	$num = '2';
	break;
	case 'n':
	$num = '1,2';
	break;
	case 'Y':
	$num = '4';
	break;
	case 'xkY':
	$num = '4';
	break;
	case 'G':
	$num = '1,2';
	break;
	case 'H':
	$num = '2';
	break;
	case 'i':
	$num = '2';
	break;
	case 's':
	$num = '2';
	break;
	case '\\':
	// Backslash escaping
	if ( $p < $formatLength - 1 ) {
	$s .= preg_quote( $format[++$p], '/' );
	} else {
	$s .= preg_quote( '\\', '/' );
	}
	break;
	case '"':
	// Quoted literal
	if ( $p < $formatLength - 1 ) {
	$endQuote = strpos( $format, '"', $p + 1 );
	if ( $endQuote === false ) {
	// No terminating quote, assume literal "
	$s .= '"';
	} else {
	$s .= preg_quote( substr( $format, $p + 1, $endQuote - $p - 1 ), '/' );
	$p = $endQuote;
	}
	} else {
	// Quote at end of string, assume literal "
	$s .= '"';
	}
	break;
	default:
	// Copy whole characters together, instead of single bytes
	$char = mb_substr( mb_strcut( $format, $p, 4 ), 0, 1 );
	$s .= preg_quote( $char, '/' );
	$p += strlen( $char ) - 1;
	}
	if ( $num !== false ) {
	if ( $raw ) {
	$s .= '([0-9]{' . $num . '})';
	$raw = false;
	} else {
	$s .= '(' . $digitsRegexp . '{' . $num . '})';
	}
	}
	// Ignore some invisible Unicode characters that often sneak into copy-pasted timestamps (T308448)
	$s .= '[\\x{200E}\\x{200F}]?';
	}

	$tzRegexp = static::regexpAlternateGroup( array_keys( $tzAbbrs ) );

	// Hard-coded parentheses and space like in Parser::pstPass2
	// Ignore some invisible Unicode characters that often sneak into copy-pasted timestamps (T245784)
	// \uNNNN syntax can only be used from PHP 7.3
	return '/' . $s . ' [\\x{200E}\\x{200F}]?\$' . $tzRegexp . '\$/u';
	}

	/**
	* Get a function that parses timestamps generated using the given date format, based on the result
	* of matching the regexp returned by getTimestampRegexp()
	*
	* @param string $contLangVariant Content language variant
	* @param string $format Date format, as used by MediaWiki
	* @param array<int,string>\|null $digits Localised digits from 0 to 9, e.g. `[ '0', '1', ..., '9' ]`
	* @param string $localTimezone Local timezone IANA name, e.g. `America/New_York`
	* @param array $tzAbbrs Map of localised timezone abbreviations to IANA abbreviations
	* for the local timezone, e.g. [ 'EDT' => 'EDT', 'EST' => 'EST' ]
	* @return callable Parser function
	*/
	private function getTimestampParser(
	string $contLangVariant, string $format, ?array $digits, string $localTimezone, array $tzAbbrs
	): callable {
	$untransformDigits = static function ( string $text ) use ( $digits ): int {
	return (int)( $digits ? strtr( $text, array_flip( $digits ) ) : $text );
	};

	$formatLength = strlen( $format );
	$matchingGroups = [];
	for ( $p = 0; $p < $formatLength; $p++ ) {
	$code = $format[$p];
	if ( $code === 'x' && $p < $formatLength - 1 ) {
	$code .= $format[++$p];
	}
	if ( $code === 'xk' && $p < $formatLength - 1 ) {
	$code .= $format[++$p];
	}

	switch ( $code ) {
	case 'xx':
	case 'xn':
	break;
	case 'xg':
	case 'd':
	case 'j':
	case 'D':
	case 'l':
	case 'F':
	case 'M':
	case 'm':
	case 'n':
	case 'Y':
	case 'xkY':
	case 'G':
	case 'H':
	case 'i':
	case 's':
	$matchingGroups[] = $code;
	break;
	case '\\':
	// Backslash escaping
	if ( $p < $formatLength - 1 ) {
	$p++;
	}
	break;
	case '"':
	// Quoted literal
	if ( $p < $formatLength - 1 ) {
	$endQuote = strpos( $format, '"', $p + 1 );
	if ( $endQuote !== false ) {
	$p = $endQuote;
	}
	}
	break;
	default:
	break;
	}
	}

	return function ( array $match ) use (
	$matchingGroups, $untransformDigits, $localTimezone, $tzAbbrs, $contLangVariant
	) {
	if ( is_array( $match[0] ) ) {
	// Strip PREG_OFFSET_CAPTURE data
	unset( $match['offset'] );
	$match = array_map( static function ( array $tuple ) {
	return $tuple[0];
	}, $match );
	}
	$year = 0;
	$monthIdx = 0;
	$day = 0;
	$hour = 0;
	$minute = 0;
	foreach ( $matchingGroups as $i => $code ) {
	$text = $match[$i + 1];
	switch ( $code ) {
	case 'xg':
	$monthIdx = array_search(
	$text,
	$this->getMessages( $contLangVariant, Language::MONTH_GENITIVE_MESSAGES ),
	true
	);
	break;
	case 'd':
	case 'j':
	$day = $untransformDigits( $text );
	break;
	case 'D':
	case 'l':
	// Day of the week - unused
	break;
	case 'F':
	$monthIdx = array_search(
	$text,
	$this->getMessages( $contLangVariant, Language::MONTH_MESSAGES ),
	true
	);
	break;
	case 'M':
	$monthIdx = array_search(
	$text,
	$this->getMessages( $contLangVariant, Language::MONTH_ABBREVIATED_MESSAGES ),
	true
	);
	break;
	case 'm':
	case 'n':
	$monthIdx = $untransformDigits( $text ) - 1;
	break;
	case 'Y':
	$year = $untransformDigits( $text );
	break;
	case 'xkY':
	// Thai year
	$year = $untransformDigits( $text ) - 543;
	break;
	case 'G':
	case 'H':
	$hour = $untransformDigits( $text );
	break;
	case 'i':
	$minute = $untransformDigits( $text );
	break;
	case 's':
	// Seconds - unused, because most timestamp formats omit them
	break;
	default:
	throw new LogicException( 'Not implemented' );
	}
	}

	// The last matching group is the timezone abbreviation
	$tzAbbr = $tzAbbrs[ end( $match ) ];

	// Most of the time, the timezone abbreviation is not necessary to parse the date, since we
	// can assume all times are in the wiki's local timezone.
	$date = new DateTime();
	// setTimezone must be called before setDate/setTime
	$date->setTimezone( new DateTimeZone( $localTimezone ) );
	$date->setDate( $year, $monthIdx + 1, $day );
	$date->setTime( $hour, $minute, 0 );

	// But during the "fall back" at the end of DST, some times will happen twice.
	// Since the timezone abbreviation disambiguates the DST/non-DST times, we can detect
	// when PHP chose the wrong one, and then try the other one. It appears that PHP always
	// uses the later (non-DST) hour, but that behavior isn't documented, so we account for both.
	$dateWarning = null;
	if ( $date->format( 'T' ) !== $tzAbbr ) {
	$altDate = clone $date;
	if ( $date->format( 'I' ) ) {
	// Parsed time is DST, try non-DST by advancing one hour
	$altDate->add( new DateInterval( 'PT1H' ) );
	} else {
	// Parsed time is non-DST, try DST by going back one hour
	$altDate->sub( new DateInterval( 'PT1H' ) );
	}
	if ( $altDate->format( 'T' ) === $tzAbbr ) {
	$date = $altDate;
	$dateWarning = 'Timestamp has timezone abbreviation for the wrong time';
	} else {
	$dateWarning = 'Ambiguous time at DST switchover was parsed';
	}
	}

	// Now set the timezone back to UTC for formatting
	$date->setTimezone( new DateTimeZone( 'UTC' ) );
	$date = DateTimeImmutable::createFromMutable( $date );

	// We require the date to be compatible with our libraries, for example zero or negative years (T352455)
	// In PHP we need to check with MWTimestamp.
	// In JS we need to check with Moment.
	try {
	// @phan-suppress-next-line PhanNoopNew
	new MWTimestamp( $date->format( 'c' ) );
	} catch ( TimestampException $ex ) {
	return null;
	}

	return [
	'date' => $date,
	'warning' => $dateWarning,
	];
	};
	}

	/**
	* Get a regexp that matches timestamps in the local date format, for each language variant.
	*
	* This calls getTimestampRegexp() with predefined data for the current wiki.
	*
	* @return string[] Regular expressions
	*/
	public function getLocalTimestampRegexps(): array {
	$langConv = $this->languageConverterFactory->getLanguageConverter( $this->language );
	return array_map( function ( $contLangVariant ) {
	return $this->getTimestampRegexp(
	$contLangVariant,
	$this->dateFormat[$contLangVariant],
	'[' . implode( '', $this->digits[$contLangVariant] ) . ']',
	$this->timezones[$contLangVariant]
	);
	}, $langConv->getVariants() );
	}

	/**
	* Get a function that parses timestamps in the local date format, for each language variant,
	* based on the result of matching the regexp returned by getLocalTimestampRegexp().
	*
	* This calls getTimestampParser() with predefined data for the current wiki.
	*
	* @return callable[] Parser functions
	*/
	private function getLocalTimestampParsers(): array {
	$langConv = $this->languageConverterFactory->getLanguageConverter( $this->language );
	return array_map( function ( $contLangVariant ) {
	return $this->getTimestampParser(
	$contLangVariant,
	$this->dateFormat[$contLangVariant],
	$this->digits[$contLangVariant],
	$this->localTimezone,
	$this->timezones[$contLangVariant]
	);
	}, $langConv->getVariants() );
	}

	/**
	* Given a link node (`<a>`), if it's a link to a user-related page, return their username.
	*
	* @param Element $link
	* @return array\|null Array, or null:
	* - string 'username' Username
	* - string\|null 'displayName' Display name (link text if link target was in the user namespace)
	*/
	private function getUsernameFromLink( Element $link ): ?array {
	// Selflink: use title of current page
	if ( DOMCompat::getClassList( $link )->contains( 'mw-selflink' ) ) {
	$title = $this->title;
	} else {
	$titleString = CommentUtils::getTitleFromUrl( $link->getAttribute( 'href' ) ?? '', $this->config ) ?? '';
	// Performance optimization, skip strings that obviously don't contain a namespace
	if ( $titleString === '' \|\| !str_contains( $titleString, ':' ) ) {
	return null;
	}
	$title = $this->parseTitle( $titleString );
	if ( !$title ) {
	return null;
	}
	}

	$username = null;
	$displayName = null;
	$mainText = $title->getText();

	if ( $title->inNamespace( NS_USER ) \|\| $title->inNamespace( NS_USER_TALK ) ) {
	$username = $mainText;
	if ( str_contains( $username, '/' ) ) {
	return null;
	}
	if ( $title->inNamespace( NS_USER ) ) {
	// Use regex trim for consistency with JS implementation
	$text = preg_replace( [ '/^[\s]+/u', '/[\s]+$/u' ], '', $link->textContent ?? '' );
	// Record the display name if it has been customised beyond changing case
	if ( $text && mb_strtolower( $text ) !== mb_strtolower( $username ) ) {
	$displayName = $text;
	}
	}
	} elseif ( $title->inNamespace( NS_SPECIAL ) ) {
	$parts = explode( '/', $mainText );
	if ( count( $parts ) === 2 && $parts[0] === $this->specialContributionsName ) {
	// Normalize the username: users may link to their contributions with an unnormalized name
	$userpage = $this->titleParser->makeTitleValueSafe( NS_USER, $parts[1] );
	if ( !$userpage ) {
	return null;
	}
	$username = $userpage->getText();
	}
	}
	if ( $username === null ) {
	return null;
	}
	if ( IPUtils::isIPv6( $username ) ) {
	// Bot-generated links "Preceding unsigned comment added by" have non-standard case
	$username = strtoupper( $username );
	}
	return [
	'username' => $username,
	'displayName' => $displayName,
	];
	}

	/**
	* Find a user signature preceding a timestamp.
	*
	* The signature includes the timestamp node.
	*
	* A signature must contain at least one link to the user's userpage, discussion page or
	* contributions (and may contain other links). The link may be nested in other elements.
	*
	* @param Text $timestampNode
	* @param Node\|null $until Node to stop searching at
	* @return array Result, an associative array with the following keys:
	* - Node[] `nodes` Sibling nodes comprising the signature, in reverse order (with
	* $timestampNode or its parent node as the first element)
	* - string\|null `username` Username, null for unsigned comments
	*/
	private function findSignature( Text $timestampNode, ?Node $until = null ): array {
	$sigUsername = null;
	$sigDisplayName = null;
	$length = 0;
	$lastLinkNode = $timestampNode;

	CommentUtils::linearWalkBackwards(
	$timestampNode,
	function ( string $event, Node $node ) use (
	&$sigUsername, &$sigDisplayName, &$lastLinkNode, &$length,
	$until, $timestampNode
	) {
	if ( $event === 'enter' && $node === $until ) {
	return true;
	}
	if ( $length >= static::SIGNATURE_SCAN_LIMIT ) {
	return true;
	}
	if ( CommentUtils::isBlockElement( $node ) ) {
	// Don't allow reaching into preceding paragraphs
	return true;
	}

	if ( $event === 'leave' && $node !== $timestampNode ) {
	$length += $node instanceof Text ?
	mb_strlen( CommentUtils::htmlTrim( $node->textContent ?? '' ) ) : 0;
	}

	// Find the closest link before timestamp that links to the user's user page.
	//
	// Support timestamps being linked to the diff introducing the comment:
	// if the timestamp node is the only child of a link node, use the link node instead
	//
	// Handle links nested in formatting elements.
	if ( $event === 'leave' && $node instanceof Element && strtolower( $node->tagName ) === 'a' ) {
	$classList = DOMCompat::getClassList( $node );
	// Generated timestamp links sometimes look like username links (e.g. on user talk pages)
	// so ignore these.
	if ( !$classList->contains( 'ext-discussiontools-init-timestamplink' ) ) {
	$user = $this->getUsernameFromLink( $node );
	if ( $user ) {
	// Accept the first link to the user namespace, then only accept links to that user
	if ( $sigUsername === null ) {
	$sigUsername = $user['username'];
	}
	if ( $user['username'] === $sigUsername ) {
	$lastLinkNode = $node;
	if ( $user['displayName'] ) {
	$sigDisplayName = $user['displayName'];
	}
	}
	}
	// Keep looking if a node with links wasn't a link to a user page
	// "Doc James (talk · contribs · email)"
	}
	}
	}
	);

	$range = new ImmutableRange(
	$lastLinkNode->parentNode,
	CommentUtils::childIndexOf( $lastLinkNode ),
	$timestampNode->parentNode,
	CommentUtils::childIndexOf( $timestampNode ) + 1
	);

	// Expand the range so that it covers sibling nodes.
	// This will include any wrapping formatting elements as part of the signature.
	//
	// Helpful accidental feature: users whose signature is not detected in full (due to
	// text formatting) can just wrap it in a <span> to fix that.
	// "Ten Pound Hammer • (What did I screw up now?)"
	// "« Saper // dyskusja »"
	//
	// TODO Not sure if this is actually good, might be better to just use the range...
	$sigNodes = array_reverse( CommentUtils::getCoveredSiblings( $range ) );

	return [
	'nodes' => $sigNodes,
	'username' => $sigUsername,
	'displayName' => $sigDisplayName,
	];
	}

	/**
	* Callback for TreeWalker that will skip over nodes where we don't want to detect
	* comments (or section headings).
	*
	* @param Node $node
	* @return int Appropriate NodeFilter constant
	*/
	public static function acceptOnlyNodesAllowingComments( Node $node ): int {
	if ( $node instanceof Element ) {
	$tagName = strtolower( $node->tagName );
	// The table of contents has a heading that gets erroneously detected as a section
	if ( $node->getAttribute( 'id' ) === 'toc' ) {
	return NodeFilter::FILTER_REJECT;
	}
	// Don't detect comments within quotes (T275881)
	if (
	$tagName === 'blockquote' \|\|
	$tagName === 'cite' \|\|
	$tagName === 'q'
	) {
	return NodeFilter::FILTER_REJECT;
	}
	$classList = DOMCompat::getClassList( $node );
	// Don't attempt to parse blocks marked 'mw-notalk'
	if ( $classList->contains( 'mw-notalk' ) ) {
	return NodeFilter::FILTER_REJECT;
	}
	// Don't detect comments within references. We can't add replies to them without bungling up
	// the structure in some cases (T301213), and you're not supposed to do that anyway…
	if (
	// <ol class="references"> is the only reliably consistent thing between the two parsers
	$tagName === 'ol' &&
	DOMCompat::getClassList( $node )->contains( 'references' )
	) {
	return NodeFilter::FILTER_REJECT;
	}
	}
	$parentNode = $node->parentNode;
	// Don't detect comments within headings (but don't reject the headings themselves)
	if ( $parentNode instanceof Element && preg_match( '/^h([1-6])$/i', $parentNode->tagName ) ) {
	return NodeFilter::FILTER_REJECT;
	}
	return NodeFilter::FILTER_ACCEPT;
	}

	/**
	* Convert a byte offset within a text node to a unicode codepoint offset
	*
	* @param Text $node Text node
	* @param int $byteOffset Byte offset
	* @return int Codepoint offset
	*/
	private static function getCodepointOffset( Text $node, int $byteOffset ): int {
	return mb_strlen( substr( $node->nodeValue ?? '', 0, $byteOffset ) );
	}

	/**
	* Find a timestamps in a given text node
	*
	* @param Text $node
	* @param string[] $timestampRegexps
	* @return array\|null Array with the following keys:
	* - int 'offset' Length of extra text preceding the node that was used for matching (in bytes)
	* - int 'parserIndex' Which of the regexps matched
	* - array 'matchData' Regexp match data, which specifies the location of the match,
	* and which can be parsed using getLocalTimestampParsers() (offsets are in bytes)
	* - ImmutableRange 'range' Range covering the timestamp
	*/
	public function findTimestamp( Text $node, array $timestampRegexps ): ?array {
	$nodeText = '';
	$offset = 0;
	// Searched nodes (reverse order)
	$nodes = [];

	while ( $node ) {
	$nodeText = $node->nodeValue . $nodeText;
	$nodes[] = $node;

	// In Parsoid HTML, entities are represented as a 'mw:Entity' node, rather than normal HTML
	// entities. On Arabic Wikipedia, the "UTC" timezone name contains some non-breaking spaces,
	// which apparently are often turned into   entities by buggy editing tools. To handle
	// this, we must piece together the text, so that our regexp can match those timestamps.
	if (
	( $previousSibling = $node->previousSibling ) &&
	$previousSibling instanceof Element &&
	$previousSibling->getAttribute( 'typeof' ) === 'mw:Entity'
	) {
	$nodeText = $previousSibling->firstChild->nodeValue . $nodeText;
	$offset += strlen( $previousSibling->firstChild->nodeValue ?? '' );
	$nodes[] = $previousSibling->firstChild;

	// If the entity is preceded by more text, do this again
	if (
	$previousSibling->previousSibling &&
	$previousSibling->previousSibling instanceof Text
	) {
	$offset += strlen( $previousSibling->previousSibling->nodeValue ?? '' );
	$node = $previousSibling->previousSibling;
	} else {
	$node = null;
	}
	} else {
	$node = null;
	}
	}

	foreach ( $timestampRegexps as $i => $timestampRegexp ) {
	$matchData = null;
	// Allows us to mimic match.index in #getComments
	if ( preg_match( $timestampRegexp, $nodeText, $matchData, PREG_OFFSET_CAPTURE ) ) {
	$timestampLength = strlen( $matchData[0][0] );
	// Bytes at the end of the last node which aren't part of the match
	$tailLength = strlen( $nodeText ) - $timestampLength - $matchData[0][1];
	// We are moving right to left, but we start to the right of the end of
	// the timestamp if there is trailing garbage, so that is a negative offset.
	$count = -$tailLength;
	$endNode = $nodes[0];
	$endOffset = strlen( $endNode->nodeValue ?? '' ) - $tailLength;

	foreach ( $nodes as $n ) {
	$count += strlen( $n->nodeValue ?? '' );
	// If we have counted to beyond the start of the timestamp, we are in the
	// start node of the timestamp
	if ( $count >= $timestampLength ) {
	$startNode = $n;
	// Offset is how much we overshot the start by
	$startOffset = $count - $timestampLength;
	break;
	}
	}
	Assert::precondition( $endNode instanceof Node, 'endNode of timestamp is a Node' );
	Assert::precondition( $startNode instanceof Node, 'startNode of timestamp range found' );
	Assert::precondition( is_int( $startOffset ), 'startOffset of timestamp range found' );

	$startOffset = static::getCodepointOffset( $startNode, $startOffset );
	$endOffset = static::getCodepointOffset( $endNode, $endOffset );

	$range = new ImmutableRange( $startNode, $startOffset, $endNode, $endOffset );

	return [
	'matchData' => $matchData,
	// Bytes at the start of the first node which aren't part of the match
	// TODO: Remove this and use 'range' instead
	'offset' => $offset,
	'range' => $range,
	'parserIndex' => $i,
	];
	}
	}
	return null;
	}

	/**
	* @param Node[] $sigNodes
	* @param array $match
	* @param Text $node
	* @return ImmutableRange
	*/
	private function adjustSigRange( array $sigNodes, array $match, Text $node ): ImmutableRange {
	$firstSigNode = end( $sigNodes );
	$lastSigNode = $sigNodes[0];

	// TODO Document why this needs to be so complicated
	$lastSigNodeOffsetByteOffset =
	$match['matchData'][0][1] + strlen( $match['matchData'][0][0] ) - $match['offset'];
	$lastSigNodeOffset = $lastSigNode === $node ?
	static::getCodepointOffset( $node, $lastSigNodeOffsetByteOffset ) :
	CommentUtils::childIndexOf( $lastSigNode ) + 1;
	$sigRange = new ImmutableRange(
	$firstSigNode->parentNode,
	CommentUtils::childIndexOf( $firstSigNode ),
	$lastSigNode === $node ? $node : $lastSigNode->parentNode,
	$lastSigNodeOffset
	);

	return $sigRange;
	}

	private function buildThreadItems(): ContentThreadItemSet {
	$result = new ContentThreadItemSet();

	$timestampRegexps = $this->getLocalTimestampRegexps();
	$dfParsers = $this->getLocalTimestampParsers();

	$curCommentEnd = null;

	$treeWalker = new TreeWalker(
	$this->rootNode,
	NodeFilter::SHOW_ELEMENT \| NodeFilter::SHOW_TEXT,
	[ static::class, 'acceptOnlyNodesAllowingComments' ]
	);
	while ( $node = $treeWalker->nextNode() ) {
	if ( $node instanceof Element && preg_match( '/^h([1-6])$/i', $node->tagName, $match ) ) {
	$headingNode = CommentUtils::getHeadlineNode( $node );
	$range = new ImmutableRange(
	$headingNode, 0, $headingNode, $headingNode->childNodes->length
	);
	$transcludedFrom = $this->computeTranscludedFrom( $range );
	$curComment = new ContentHeadingItem( $range, $transcludedFrom, (int)( $match[ 1 ] ) );
	$curComment->setRootNode( $this->rootNode );
	$result->addThreadItem( $curComment );
	$curCommentEnd = $node;
	} elseif ( $node instanceof Text && ( $match = $this->findTimestamp( $node, $timestampRegexps ) ) ) {
	$warnings = [];
	$foundSignature = $this->findSignature( $node, $curCommentEnd );
	$author = $foundSignature['username'];

	if ( $author === null ) {
	// Ignore timestamps for which we couldn't find a signature. It's probably not a real
	// comment, but just a false match due to a copypasted timestamp.
	continue;
	}

	$sigRanges = [];
	$timestampRanges = [];

	$sigRanges[] = $this->adjustSigRange( $foundSignature['nodes'], $match, $node );
	$timestampRanges[] = $match['range'];

	// Everything from the last comment up to here is the next comment
	$startNode = $this->nextInterestingLeafNode( $curCommentEnd );
	$endNode = $foundSignature['nodes'][0];

	// Skip to the end of the "paragraph". This only looks at tag names and can be fooled by CSS, but
	// avoiding that would be more difficult and slower.
	//
	// If this skips over another potential signature, also skip it in the main TreeWalker loop, to
	// avoid generating multiple comments when there is more than one signature on a single "line".
	// Often this is done when someone edits their comment later and wants to add a note about that.
	// (Or when another person corrects a typo, or strikes out a comment, etc.) Multiple comments
	// within one paragraph/list-item result in a confusing double "Reply" button, and we also have
	// no way to indicate which one you're replying to (this might matter in the future for
	// notifications or something).
	CommentUtils::linearWalk(
	$endNode,
	function ( string $event, Node $n ) use (
	&$endNode, &$sigRanges, &$timestampRanges,
	$treeWalker, $timestampRegexps, $node
	) {
	if ( CommentUtils::isBlockElement( $n ) \|\| CommentUtils::isCommentSeparator( $n ) ) {
	// Stop when entering or leaving a block node
	return true;
	}
	if (
	$event === 'leave' &&
	$n instanceof Text && $n !== $node &&
	( $match2 = $this->findTimestamp( $n, $timestampRegexps ) )
	) {
	// If this skips over another potential signature, also skip it in the main TreeWalker loop
	$treeWalker->currentNode = $n;
	// …and add it as another signature to this comment (regardless of the author and timestamp)
	$foundSignature2 = $this->findSignature( $n, $node );
	if ( $foundSignature2['username'] !== null ) {
	$sigRanges[] = $this->adjustSigRange( $foundSignature2['nodes'], $match2, $n );
	$timestampRanges[] = $match2['range'];
	}
	}
	if ( $event === 'leave' ) {
	// Take the last complete node which we skipped past
	$endNode = $n;
	}
	}
	);

	$length = ( $endNode instanceof Text ) ?
	mb_strlen( rtrim( $endNode->nodeValue ?? '', "\t\n\f\r " ) ) :
	// PHP bug: childNodes can be null for comment nodes
	// (it should always be a NodeList, even if the node can't have children)
	( $endNode->childNodes ? $endNode->childNodes->length : 0 );
	$range = new ImmutableRange(
	$startNode->parentNode,
	CommentUtils::childIndexOf( $startNode ),
	$endNode,
	$length
	);
	$transcludedFrom = $this->computeTranscludedFrom( $range );

	$startLevel = CommentUtils::getIndentLevel( $startNode, $this->rootNode ) + 1;
	$endLevel = CommentUtils::getIndentLevel( $node, $this->rootNode ) + 1;
	if ( $startLevel !== $endLevel ) {
	$warnings[] = 'Comment starts and ends with different indentation';
	}
	// Should this use the indent level of $startNode or $node?
	$level = min( $startLevel, $endLevel );

	$parserResult = $dfParsers[ $match['parserIndex'] ]( $match['matchData'] );
	if ( !$parserResult ) {
	continue;
	}
	[ 'date' => $dateTime, 'warning' => $dateWarning ] = $parserResult;

	if ( $dateWarning ) {
	$warnings[] = $dateWarning;
	}

	$curComment = new ContentCommentItem(
	$level,
	$range,
	$transcludedFrom,
	$sigRanges,
	$timestampRanges,
	$dateTime,
	$author,
	$foundSignature['displayName']
	);
	$curComment->setRootNode( $this->rootNode );
	if ( $warnings ) {
	$curComment->addWarnings( $warnings );
	}
	if ( $result->isEmpty() ) {
	// Add a fake placeholder heading if there are any comments in the 0th section
	// (before the first real heading)
	$range = new ImmutableRange( $this->rootNode, 0, $this->rootNode, 0 );
	$fakeHeading = new ContentHeadingItem( $range, false, null );
	$fakeHeading->setRootNode( $this->rootNode );
	$result->addThreadItem( $fakeHeading );
	}
	$result->addThreadItem( $curComment );
	$curCommentEnd = $curComment->getRange()->endContainer;
	}
	}

	return $result;
	}

	/**
	* Get the name of the page from which this thread item is transcluded (if any). Replies to
	* transcluded items must be posted on that page, instead of the current one.
	*
	* This is tricky, because we don't want to mark items as trancluded when they're just using a
	* template (e.g. {{ping\|…}} or a non-substituted signature template). Sometimes the whole comment
	* can be template-generated (e.g. when using some wrapper templates), but as long as a reply can
	* be added outside of that template, we should not treat it as transcluded.
	*
	* The start/end boundary points of comment ranges and Parsoid transclusion ranges don't line up
	* exactly, even when to a human it's obvious that they cover the same content, making this more
	* complicated.
	*
	* @return string\|bool `false` if this item is not transcluded. A string if it's transcluded
	* from a single page (the page title, in text form with spaces). `true` if it's transcluded, but
	* we can't determine the source.
	*/
	public function computeTranscludedFrom( ImmutableRange $commentRange ) {
	// Collapsed ranges should otherwise be impossible, but they're not (T299583)
	// TODO: See if we can fix the root cause, and remove this?
	if ( $commentRange->collapsed ) {
	return false;
	}

	// General approach:
	//
	// Compare the comment range to each transclusion range on the page, and if it overlaps any of
	// them, examine the overlap. There are a few cases:
	//
	// * Comment and transclusion do not overlap:
	// → Not transcluded.
	// * Comment contains the transclusion:
	// → Not transcluded (just a template).
	// * Comment is contained within the transclusion:
	// → Transcluded, we can determine the source page (unless it's a complex transclusion).
	// * Comment and transclusion overlap partially:
	// → Transcluded, but we can't determine the source page.
	// * Comment (almost) exactly matches the transclusion:
	// → Maybe transcluded (it could be that the source page only contains that single comment),
	// maybe not transcluded (it could be a wrapper template that covers a single comment).
	// This is very sad, and we decide based on the namespace.
	//
	// Most transclusion ranges on the page trivially fall in the "do not overlap" or "contains"
	// cases, and we only have to carefully examine the two transclusion ranges that contain the
	// first and last node of the comment range.
	//
	// To check for almost exact matches, we walk between the relevant boundary points, and if we
	// only find uninteresting nodes (that would be ignored when detecting comments), we treat them
	// like exact matches.

	$startTransclNode = CommentUtils::getTranscludedFromElement(
	CommentUtils::getRangeFirstNode( $commentRange )
	);
	$endTransclNode = CommentUtils::getTranscludedFromElement(
	CommentUtils::getRangeLastNode( $commentRange )
	);

	// We only have to examine the two transclusion ranges that contain the first/last node of the
	// comment range (if they exist). Ignore ranges outside the comment or in the middle of it.
	$transclNodes = [];
	if ( $startTransclNode ) {
	$transclNodes[] = $startTransclNode;
	}
	if ( $endTransclNode && $endTransclNode !== $startTransclNode ) {
	$transclNodes[] = $endTransclNode;
	}

	foreach ( $transclNodes as $transclNode ) {
	$transclRange = static::getTransclusionRange( $transclNode );
	$compared = CommentUtils::compareRanges( $commentRange, $transclRange );
	$transclTitles = $this->getTransclusionTitles( $transclNode );
	$simpleTransclTitle = count( $transclTitles ) === 1 && $transclTitles[0] !== null ?
	$this->parseTitle( $transclTitles[0] ) : null;

	switch ( $compared ) {
	case 'equal':
	// Comment (almost) exactly matches the transclusion
	if ( $simpleTransclTitle === null ) {
	// Allow replying to some accidental complex transclusions consisting of only templates
	// and wikitext (T313093)
	if ( count( $transclTitles ) > 1 ) {
	foreach ( $transclTitles as $transclTitleString ) {
	if ( $transclTitleString !== null ) {
	$transclTitle = $this->parseTitle( $transclTitleString );
	if ( $transclTitle && !$transclTitle->inNamespace( NS_TEMPLATE ) ) {
	return true;
	}
	}
	}
	// Continue examining the other ranges.
	break;
	}
	// Multi-template transclusion, or a parser function call, or template-affected wikitext outside
	// of a template call, or a mix of the above
	return true;

	} elseif ( $simpleTransclTitle->inNamespace( NS_TEMPLATE ) ) {
	// Is that a subpage transclusion with a single comment, or a wrapper template
	// transclusion on this page? We don't know, but let's guess based on the namespace.
	// (T289873)
	// Continue examining the other ranges.
	break;
	} elseif ( !$this->titleCanExist( $simpleTransclTitle ) ) {
	// Special page transclusion (T344622) or something else weird. Don't return the title,
	// since it's useless for replying, and can't be stored in the permalink database.
	return true;
	} else {
	Assert::precondition( $transclTitles[0] !== null, "Simple transclusion found" );
	return strtr( $transclTitles[0], '_', ' ' );
	}

	case 'contains':
	// Comment contains the transclusion

	// If the entire transclusion is contained within the comment range, that's just a
	// template. This is the same as a transclusion in the middle of the comment, which we
	// ignored earlier, it just takes us longer to get here in this case.

	// Continue examining the other ranges.
	break;

	case 'contained':
	// Comment is contained within the transclusion
	if ( $simpleTransclTitle === null ) {
	return true;
	} elseif ( !$this->titleCanExist( $simpleTransclTitle ) ) {
	// Special page transclusion (T344622) or something else weird. Don't return the title,
	// since it's useless for replying, and can't be stored in the permalink database.
	return true;
	} else {
	Assert::precondition( $transclTitles[0] !== null, "Simple transclusion found" );
	return strtr( $transclTitles[0], '_', ' ' );
	}

	case 'after':
	case 'before':
	// Comment and transclusion do not overlap

	// This should be impossible, because we ignored these ranges earlier.
	throw new LogicException( 'Unexpected transclusion or comment range' );

	case 'overlapstart':
	case 'overlapend':
	// Comment and transclusion overlap partially
	return true;

	default:
	throw new LogicException( 'Unexpected return value from compareRanges()' );
	}
	}

	// If we got here, the comment range was not contained by or overlapping any of the transclusion
	// ranges. Comment is not transcluded.
	return false;
	}

	private function titleCanExist( TitleValue $title ): bool {
	return $title->getNamespace() >= NS_MAIN &&
	!$title->isExternal() &&
	$title->getText() !== '';
	}

	private function parseTitle( string $titleString ): ?TitleValue {
	try {
	return $this->titleParser->parseTitle( $titleString );
	} catch ( MalformedTitleException $err ) {
	return null;
	}
	}

	/**
	* Return the page titles for each part of the transclusion, or nulls for each part that isn't
	* transcluded from another page.
	*
	* If the node represents a single-page transclusion, this will return an array containing a
	* single string.
	*
	* @param Element $node
	* @return array<string\|null>
	*/
	private function getTransclusionTitles( Element $node ): array {
	$dataMw = json_decode( $node->getAttribute( 'data-mw' ) ?? '', true );
	$out = [];

	foreach ( $dataMw['parts'] ?? [] as $part ) {
	if (
	!is_string( $part ) &&
	// 'href' will be unset if this is a parser function rather than a template
	isset( $part['template']['target']['href'] )
	) {
	$parsoidHref = $part['template']['target']['href'];
	Assert::precondition( substr( $parsoidHref, 0, 2 ) === './', "href has valid format" );
	$out[] = rawurldecode( substr( $parsoidHref, 2 ) );
	} else {
	$out[] = null;
	}
	}

	return $out;
	}

	/**
	* Given a transclusion's first node (e.g. returned by CommentUtils::getTranscludedFromElement()),
	* return a range starting before the node and ending after the transclusion's last node.
	*
	* @param Element $startNode
	* @return ImmutableRange
	*/
	private function getTransclusionRange( Element $startNode ): ImmutableRange {
	$endNode = $startNode;
	while (
	// Phan doesn't realize that the conditions on $nextSibling can terminate the loop
	// @phan-suppress-next-line PhanInfiniteLoop
	$endNode &&
	( $nextSibling = $endNode->nextSibling ) &&
	$nextSibling instanceof Element &&
	$nextSibling->getAttribute( 'about' ) === $endNode->getAttribute( 'about' )
	) {
	$endNode = $nextSibling;
	}

	$range = new ImmutableRange(
	$startNode->parentNode,
	CommentUtils::childIndexOf( $startNode ),
	$endNode->parentNode,
	CommentUtils::childIndexOf( $endNode ) + 1
	);

	return $range;
	}

	/**
	* Truncate user generated parts of IDs so full ID always fits within a database field of length 255
	*
	* nb: Text should already have had spaces replaced with underscores by this point.
	*
	* @param string $text Text
	* @param bool $legacy Generate legacy ID, not needed in JS implementation
	* @return string Truncated text
	*/
	private function truncateForId( string $text, bool $legacy = false ): string {
	$truncated = $this->language->truncateForDatabase( $text, 80, '' );
	if ( !$legacy ) {
	$truncated = trim( $truncated, '_' );
	}
	return $truncated;
	}

	/**
	* Given a thread item, return an identifier for it that is unique within the page.
	*
	* @param ContentThreadItem $threadItem
	* @param ContentThreadItemSet $previousItems
	* @param bool $legacy Generate legacy ID, not needed in JS implementation
	* @return string
	*/
	private function computeId(
	ContentThreadItem $threadItem, ContentThreadItemSet $previousItems, bool $legacy = false
	): string {
	$id = null;

	if ( $threadItem instanceof ContentHeadingItem && $threadItem->isPlaceholderHeading() ) {
	// The range points to the root note, using it like below results in silly values
	$id = 'h-';
	} elseif ( $threadItem instanceof ContentHeadingItem ) {
	$id = 'h-' . $this->truncateForId( $threadItem->getLinkableId(), $legacy );
	} elseif ( $threadItem instanceof ContentCommentItem ) {
	$id = 'c-' . $this->truncateForId( str_replace( ' ', '_', $threadItem->getAuthor() ), $legacy ) .
	'-' . $threadItem->getTimestampString();
	} else {
	throw new InvalidArgumentException( 'Unknown ThreadItem type' );
	}

	// If there would be multiple comments with the same ID (i.e. the user left multiple comments
	// in one edit, or within a minute), add the parent ID to disambiguate them.
	$threadItemParent = $threadItem->getParent();
	if ( $threadItemParent instanceof ContentHeadingItem && !$threadItemParent->isPlaceholderHeading() ) {
	$id .= '-' . $this->truncateForId( $threadItemParent->getLinkableId(), $legacy );
	} elseif ( $threadItemParent instanceof ContentCommentItem ) {
	$id .= '-' . $this->truncateForId( str_replace( ' ', '_', $threadItemParent->getAuthor() ), $legacy ) .
	'-' . $threadItemParent->getTimestampString();
	}

	if ( $threadItem instanceof ContentHeadingItem ) {
	// To avoid old threads re-appearing on popular pages when someone uses a vague title
	// (e.g. dozens of threads titled "question" on [[Wikipedia:Help desk]]: https://w.wiki/fbN),
	// include the oldest timestamp in the thread (i.e. date the thread was started) in the
	// heading ID.
	$oldestComment = $threadItem->getOldestReply();
	if ( $oldestComment ) {
	$id .= '-' . $oldestComment->getTimestampString();
	}
	}

	if ( $previousItems->findCommentById( $id ) ) {
	// Well, that's tough
	if ( !$legacy ) {
	$threadItem->addWarning( 'Duplicate comment ID' );
	}
	// Finally, disambiguate by adding sequential numbers, to allow replying to both comments
	$number = 1;
	while ( $previousItems->findCommentById( "$id-$number" ) ) {
	$number++;
	}
	$id = "$id-$number";
	}

	return $id;
	}

	/**
	* Given a thread item, return an identifier for it that is consistent across all pages and
	* revisions where this comment might appear.
	*
	* Multiple comments on a page can have the same name; use ID to distinguish them.
	*/
	private function computeName( ContentThreadItem $threadItem ): string {
	$name = null;

	if ( $threadItem instanceof ContentHeadingItem ) {
	$name = 'h-';
	$mainComment = $threadItem->getOldestReply();
	} elseif ( $threadItem instanceof ContentCommentItem ) {
	$name = 'c-';
	$mainComment = $threadItem;
	} else {
	throw new InvalidArgumentException( 'Unknown ThreadItem type' );
	}

	if ( $mainComment ) {
	$name .= $this->truncateForId( str_replace( ' ', '_', $mainComment->getAuthor() ) ) .
	'-' . $mainComment->getTimestampString();
	}

	return $name;
	}

	private function buildThreads( ContentThreadItemSet $result ): void {
	$lastHeading = null;
	$replies = [];

	foreach ( $result->getThreadItems() as $threadItem ) {
	if ( count( $replies ) < $threadItem->getLevel() ) {
	// Someone skipped an indentation level (or several). Pretend that the previous reply
	// covers multiple indentation levels, so that following comments get connected to it.
	$threadItem->addWarning( 'Comment skips indentation level' );
	while ( count( $replies ) < $threadItem->getLevel() ) {
	$replies[] = end( $replies );
	}
	}

	if ( $threadItem instanceof ContentHeadingItem ) {
	// New root (thread)
	// Attach as a sub-thread to preceding higher-level heading.
	// Any replies will appear in the tree twice, under the main-thread and the sub-thread.
	$maybeParent = $lastHeading;
	while ( $maybeParent && $maybeParent->getHeadingLevel() >= $threadItem->getHeadingLevel() ) {
	$maybeParent = $maybeParent->getParent();
	}
	if ( $maybeParent ) {
	$threadItem->setParent( $maybeParent );
	$maybeParent->addReply( $threadItem );
	}
	$lastHeading = $threadItem;
	} elseif ( isset( $replies[ $threadItem->getLevel() - 1 ] ) ) {
	// Add as a reply to the closest less-nested comment
	$threadItem->setParent( $replies[ $threadItem->getLevel() - 1 ] );
	$threadItem->getParent()->addReply( $threadItem );
	} else {
	$threadItem->addWarning( 'Comment could not be connected to a thread' );
	}

	$replies[ $threadItem->getLevel() ] = $threadItem;
	// Cut off more deeply nested replies
	array_splice( $replies, $threadItem->getLevel() + 1 );
	}
	}

	/**
	* Set the IDs and names used to refer to comments and headings.
	* This has to be a separate pass because we don't have the list of replies before
	* this point.
	*/
	private function computeIdsAndNames( ContentThreadItemSet $result ): void {
	foreach ( $result->getThreadItems() as $threadItem ) {
	$name = $this->computeName( $threadItem );
	$threadItem->setName( $name );

	$id = $this->computeId( $threadItem, $result );
	$threadItem->setId( $id );
	$legacyId = $this->computeId( $threadItem, $result, true );
	if ( $legacyId !== $id ) {
	$threadItem->setLegacyId( $legacyId );
	}

	$result->updateIdAndNameMaps( $threadItem );
	}
	}
	}

File Metadata

Mime Type: text/x-php
Expires: Sat, Oct 3, 04:57 (2 w, 6 d ago)
Storage Engine: local-disk
Storage Format: Raw Data
Storage Handle: 9d/53/d23ae31606cd7d1b4f4bcf56388e
Default Alt Text: CommentParser.php (49 KB)

CommentParser.phpNo OneTemporaryActions

CommentParser.phpView Options

File Metadata

Event Timeline

CommentParser.php
No OneTemporary
Actions

CommentParser.php
View Options