Merge "Set 'skin' template parameter rather than override parent method"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Wed, 17 Apr 2013 17:11:17 +0000 (17:11 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Wed, 17 Apr 2013 17:11:17 +0000 (17:11 +0000)
includes/specials/SpecialUserlogin.php
includes/templates/UserloginVForm.php

index e0ddc96..c3a954f 100644 (file)
@@ -1131,6 +1131,7 @@ class LoginForm extends SpecialPage {
                        : is_array( $wgPasswordResetRoutes ) && in_array( true, array_values( $wgPasswordResetRoutes ) );
 
                $template->set( 'header', '' );
+               $template->set( 'skin', $this->getSkin() );
                $template->set( 'name', $this->mUsername );
                $template->set( 'password', $this->mPassword );
                $template->set( 'retype', $this->mRetype );
index 132a7c1..2e90c09 100644 (file)
  */
 class UserloginTemplateVForm extends BaseTemplate {
 
-       /**
-        * Get the Skin object related to this object, so that BaseTemplate msg 31
-        * methods work.
-        *
-        * @return Skin object
-        */
-       public function getSkin() {
-               global $wgOut;
-               return $wgOut->getSkin();
-       }
-
        function execute() {
                global $wgCookieExpiration;
                $expirationDays = ceil( $wgCookieExpiration / ( 3600 * 24 ) );