Page MenuHomePhabricator

Pass WikiPage instances to updatePrIndex() in ProofreadPage
Closed, ResolvedPublic

Description

We've currently some instructions to build Article instances:

// Update index data.
$article = new Article( $nt );
if ( $article ) {
    self::updatePrIndex( $article );
}

We should create WikiPage instances instead.

See also 8f005496.

Details

Related Gerrit Patches:
mediawiki/extensions/ProofreadPage : masterUse wikipage instead of article

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 24 2016, 10:24 PM
Legoktm renamed this task from Replace Article instances by WikiPage to Pass WikiPage instances to updatePrIndex() in ProofreadPage.Aug 24 2016, 10:27 PM
Legoktm added a project: good first task.
Restricted Application added a subscriber: TerraCodes. · View Herald TranscriptAug 24 2016, 10:27 PM

WikiPage::factory( $nt ) should work.

Change 316331 had a related patch set (by Paladox) published:
Use wikipage instead of article

https://gerrit.wikimedia.org/r/316331

Change 316331 merged by jenkins-bot:
Use wikipage instead of article

https://gerrit.wikimedia.org/r/316331

Tpt closed this task as Resolved.Mar 27 2017, 4:11 PM
Tpt claimed this task.

Seems to be done.