Fix deprecated hooks not having a non-deprecated alternative
authorThiemo Mättig <thiemo.maettig@wikimedia.de>
Mon, 30 Jun 2014 14:48:08 +0000 (16:48 +0200)
committerThiemo Mättig <thiemo.maettig@wikimedia.de>
Mon, 30 Jun 2014 15:08:15 +0000 (17:08 +0200)
commit08fee4ce2f1bbf362c8422a3e306a905c8775094
tree7ff16df71347d19c0bcf44e3a53186812bf6436f
parent7b488f64ee6f0ed3ce1c5ede62c25e756a8840e0
Fix deprecated hooks not having a non-deprecated alternative

The implementation in ChangesListSpecialPage::doMainQuery() is
dead code. It's never executed because the relevant subclasses
override the method completely without ever calling
parent::doMainQuery().

This means that the deprecation of the hooks in the subclasses
leaves us with no alternative. Which is a real problem in
Wikibase, see I27716275e966147ee26d81d8ce3f14951937e718.

The same deprecation in ChangesListSpecialPage::getCustomFilters()
works because all suclasses I'm aware of call the parent method.
But please note that extensions may still break if they do this
different. I can not checked them all.

If you want to be sure you should just revert patch
I9cceda5d2dcfc53c852c5682c466b48ad8f31202 that introduced this
non-functional hook. I'm sorry to say that but I wonder how this
passed review.

Change-Id: I7ba3ea64cb145c06011a856e5b56399da4f42339
includes/specialpage/ChangesListSpecialPage.php
includes/specials/SpecialRecentchanges.php
includes/specials/SpecialRecentchangeslinked.php
includes/specials/SpecialWatchlist.php