Hi Amir!
On a first glance, it seems that you are right: it would be nicer if
SpecialUnconnectedPages derived from QueryPage. I would not call the current
implementation "incorrect" though. As far as I know, it works as designed.
When rewriting the special page, please use proper dependency injection. Since a
few weeks ago, it is now possible to register special pages using a factory
method, so we have full controll over the constructor parameters.
-- daniel
Am 25.09.2014 16:01, schrieb Amir Ladsgroup:
Hello,
If you check the php file of SpecilUnconnectedPages (in
Wikibase/client/includes/specials/) it extends SpecialPage instead of QueryPage.
I want to know is it intentional? if it's not Do you agree it's better to
rewrite the file to use the QueryPage class?
--
Daniel Kinzler
Senior Software Developer
Wikimedia Deutschland
Gesellschaft zur Förderung Freien Wissens e.V.