From: S Page Date: Tue, 12 Feb 2013 03:23:37 +0000 (-0800) Subject: Avoid Jenkins' CodeSniffer warnings for Userlogin X-Git-Tag: 1.31.0-rc.0~20718 X-Git-Url: http://git.cyclocoop.org/%24href?a=commitdiff_plain;h=343382e4b783d528d7b1a3153681463c4c6a4597;p=lhc%2Fweb%2Fwiklou.git Avoid Jenkins' CodeSniffer warnings for Userlogin Change property declarations from "var" to "public". This eliminates the 46 | ERROR | The var keyword must not be used to declare a property warnings from Jenkins CI, e.g. https://integration.mediawiki.org/ci/job/mediawiki-core-phpcs-HEAD/1565/console http://php.net/manual/en/language.oop5.properties.php says "If you declare a property using var instead of one of public, protected, or private, then PHP 5 will treat the property as if it had been declared as public." , so this should be a no-op. Change-Id: I754ccea018f95421bfe3f7b04e3ab3a6d9b407e3 --- diff --git a/includes/specials/SpecialUserlogin.php b/includes/specials/SpecialUserlogin.php index 1223713aa7..48dfb19086 100644 --- a/includes/specials/SpecialUserlogin.php +++ b/includes/specials/SpecialUserlogin.php @@ -43,12 +43,12 @@ class LoginForm extends SpecialPage { const NEED_TOKEN = 12; const WRONG_TOKEN = 13; - var $mUsername, $mPassword, $mRetype, $mReturnTo, $mCookieCheck, $mPosted; - var $mAction, $mCreateaccount, $mCreateaccountMail; - var $mLoginattempt, $mRemember, $mEmail, $mDomain, $mLanguage; - var $mSkipCookieCheck, $mReturnToQuery, $mToken, $mStickHTTPS; - var $mType, $mReason, $mRealName; - var $mAbortLoginErrorMsg = 'login-abort-generic'; + public $mUsername, $mPassword, $mRetype, $mReturnTo, $mCookieCheck, $mPosted; + public $mAction, $mCreateaccount, $mCreateaccountMail; + public $mLoginattempt, $mRemember, $mEmail, $mDomain, $mLanguage; + public $mSkipCookieCheck, $mReturnToQuery, $mToken, $mStickHTTPS; + public $mType, $mReason, $mRealName; + public $mAbortLoginErrorMsg = 'login-abort-generic'; private $mLoaded = false; /**