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

Assigned To

Authored By

	Dereckson
	Aug 24 2016, 10:24 PM

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.

Details

	Subject	Repo	Branch	Lines +/-
	Use wikipage instead of article	mediawiki/extensions/ProofreadPage	master	+7 -7

Customize query in gerrit

Related Objects

Mentioned Here: rEPRP8f0054967238: Fix ProofreadPage::updatePrIndex signature

Event Timeline

Dereckson created this task.Aug 24 2016, 10:24 PM

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

gerritbot added a project: Patch-For-Review.Oct 17 2016, 12:11 PM

Paladox subscribed.Oct 17 2016, 12:13 PM

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

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

ReleaseTaggerBot added a project: MW-1.28-release (WMF-deploy-2016-10-25_(1.28.0-wmf.23)).Oct 24 2016, 5:01 PM

Seems to be done.

Pass WikiPage instances to updatePrIndex() in ProofreadPageClosed, ResolvedPublicActions

Description

Details

Related Objects

Event Timeline

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