Remove useless $out parameter from SkinTemplate::prepareQuickTemplate()
authorAlexandre Emsenhuber <ialex.wiki@gmail.com>
Sun, 15 Dec 2013 14:53:20 +0000 (15:53 +0100)
committerAlexandre Emsenhuber <ialex.wiki@gmail.com>
Sun, 15 Dec 2013 14:53:20 +0000 (15:53 +0100)
commitf3f5b9d825179fe56bcd58e10fbadb76313bb50f
treebc6c07e2da61c46a7acb5a364504def23b288b2b
parentff4da357970f758c56f56c51390ab3a7e323c443
Remove useless $out parameter from SkinTemplate::prepareQuickTemplate()

Fix for I0a9a7f10ea (f5a0c23); use $this->getOutput() instead.

Passing different OutputPage object to Skin methods than the one set in
the context is deprecated, so there is no point having that parameter.

The method's signature is also misleading, since it allows null to be
passed, but if that value would be passed to it, the code would
crash since a such case is not handled in the method.

The $out parameter passed to the method from outputPage() is left since
it is required by the MobileFrontend extension. A similar commit will
be done in the extension, and the parameter's removal will happen once
this condition is lifted.

Change-Id: I4fac5b14afeab25f6f79f889c7ce56874827717d
includes/SkinTemplate.php