Soft deprecate QuickTemplate::msgWiki()
authorThiemo Kreuz <thiemo.kreuz@wikimedia.de>
Mon, 14 Jan 2019 13:43:29 +0000 (14:43 +0100)
committerBartosz Dziewoński <matma.rex@gmail.com>
Fri, 1 Feb 2019 21:25:18 +0000 (13:25 -0800)
commit8ec90547df35a1f3386d8b12f3c22d6c3d8da33f
tree2ddda52a6af9e680accbf3d90b9535ebdccb350c
parent080edfd840deed635d6f43420229b8465217c9b5
Soft deprecate QuickTemplate::msgWiki()

This method just should not exist. It does have a bad name that does not
say much about what it does and how it is supposed to be used. And it
does horrible things: it accesses the global $wgOut, which currently
results in test failures, see
https://integration.wikimedia.org/ci/job/quibble-vendor-mysql-php70-docker/15200/console

Note how it was marked as @private, which is honorable, but doesn't
work for multiple reasons. First, an overload exists in the BaseTemplate
subclass. Second, calls from external code exist. Third, if it would be
really private, it could as well be deleted, because there is no private
caller.

Luckily the number of callers is extremely close to zero, see
https://codesearch.wmflabs.org/search/?q=>msgWiki%5C(

A patch for the Collection extension already exists.

Change-Id: I8d2c14f10fbf314735d1aa82bdc8edfb4fa9a0dd
RELEASE-NOTES-1.33
includes/skins/BaseTemplate.php
includes/skins/QuickTemplate.php