Category counter not updated on article deletion
Closed, ResolvedPublic
Actions

Assigned To

None

Authored By

	P.Copp
	Jan 25 2009, 9:20 PM

Description

In Article::doDeleteArticle(), the code for updating the category counters is executed after the delete on the categorylinks table, so the query

		$cats = array();
		$res = $dbw->select( 'categorylinks', 'cl_to', array( 'cl_from' => $id ), __METHOD__ );

will never return a result. Thus, the category counters aren't updated at all on article deletions.

Version: unspecified
Severity: minor
URL: http://en.wikipedia.org/wiki/Category:Articles_for_deletion

Status	Subtype	Assigned	Task
Open	Feature	None	T18660 Database table cleanup (tracking)
Resolved		None	T20488 Run refreshCategoryCounts.php on all WMF wikis
Resolved		None	T19155 Category counter not updated on article deletion

• bzimport raised the priority of this task from to Medium.Nov 21 2014, 10:27 PM

• bzimport set Reference to bz17155.

• bzimport added a subscriber: Unknown Object (MLST).

Proposed fix: Move updateCategoryCounts() up a few lines

Attached:

bug17155.patch1 KBDownload

ayg wrote:

Thanks for the report. This was inexplicably broken in r41018. I reverted the bad parts of that in r47318, backported to 1.14 in r47319.

ayg wrote:

(In reply to comment #2)

bad parts of that in r47318, backported to 1.14 in r47319.

That should be r47319 and r47320 respectively.

Note that r41018 only moved the step below the RC row deletion and Article::onArticleDelete( $this->mTitle ) steps. r47319 Just reverted this.

The *above* patch moves this up before the page row deletion and more importantly before:

$dbw->delete( 'categorylinks', array( 'cl_from' => $id ) );

This seems to be actually broken by r40912.

Restored in r47326

	F5356: bug17155.patch
	Nov 21 2014, 10:27 PM