"Nikerabbit" changed the status of MediaWiki.r101286 to "resolved"
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/101286

Old status:  new
> New status: resolved

Commit summary for MediaWiki.r101286:

* Cleanups to updateRecentChanges():
** Don't assume rc_this_oldid is chronologically ordered; use rc_timestamp.
** Removed broken and redundant code around updating rc_patrolled for $rcId. 
Only one caller passed in $rcId and it was for $rev, which gets its RC row 
updated already.
** Mark all pending revs "unpatrolled" on "un-accept" as it should. This aligns 
with the "accept" behavior. However, if $wgUseRCPatrol is off, only update new 
page RC rows as the other ones won't be used.
** Add function documentation.
* Added sanity exception to RevisionReviewForm::approveRevision().
* Removed unused $wgFlaggedRevsPatrolNamespaces var and obsolete comments.

_______________________________________________
MediaWiki-CodeReview mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview

Reply via email to