$unpatrolled is unused in the method, so make it optional,
and then uses of it can be removed.
I realize it would do no harm to remove it completely, even
if a few (2 that I know of) extensions still pass the variable
to the method, but think this is nicer way.
Change-Id: Idbe6f00e9eb40db6a28de76fca0aea7c17b75656
/**
* @param string $s HTML to update
* @param RecentChange $rc
- * @param bool $unpatrolled
+ * @param bool|null $unpatrolled Unused variable, since 1.27.
*/
- public function insertDiffHist( &$s, &$rc, $unpatrolled ) {
+ public function insertDiffHist( &$s, &$rc, $unpatrolled = null ) {
# Diff link
if (
$rc->mAttribs['rc_type'] == RC_NEW ||
}
// Regular entries
} else {
- $this->insertDiffHist( $html, $rc, $unpatrolled );
+ $this->insertDiffHist( $html, $rc );
# M, N, b and ! (minor, new, bot and unpatrolled)
$html .= $this->recentChangesFlags(
array(