User "Hashar" posted a comment on MediaWiki.r102083.

Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/102083#c26697
Commit summary:

Made use of Pager::doBatchLookups() and Pager::getUser() in special page Pagers

Comment:

Both methods are exactly the same in StablePages_body.php 
ReviewedPages_body.php etc .. Those classes could be refactored to extend a 
common class that would host the getStartBody() / doBatchLookups() methods.

But I am not sure it is worth the time to do it, hence why I marked the change 
ok.

Is it clearer?

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

Reply via email to