Split multi variable declaration

Each variable should have a documentation block

Change-Id: I1f9f892fc29c6fa3ef0197ebafa05c89fd07ca37
This commit is contained in:
Umherirrender 2017-06-29 19:40:05 +02:00 committed by Bartosz Dziewoński
parent 26024ac399
commit cfb327ea6a
2 changed files with 34 additions and 9 deletions

View file

@ -15,7 +15,11 @@
* Implements a title blacklist for MediaWiki
*/
class TitleBlacklist {
private $mBlacklist = null, $mWhitelist = null;
/** @var array */
private $mBlacklist = null;
/** @var array */
private $mWhitelist = null;
/** @var TitleBlacklist */
protected static $instance = null;
@ -340,12 +344,35 @@ class TitleBlacklist {
* Represents a title blacklist entry
*/
class TitleBlacklistEntry {
private
$mRaw, ///< Raw line
$mRegex, ///< Regular expression to match
$mParams, ///< Parameters for this entry
$mFormatVersion, ///< Entry format version
$mSource; ///< Source of this entry
/**
* Raw line
* @var string
*/
private $mRaw;
/**
* Regular expression to match
* @var string
*/
private $mRegex;
/**
* Parameters for this entry
* @var array
*/
private $mParams;
/**
* Entry format version
* @var string
*/
private $mFormatVersion;
/**
* Source of this entry
* @var string
*/
private $mSource;
/**
* Construct a new TitleBlacklistEntry.

View file

@ -2,8 +2,6 @@
<ruleset>
<rule ref="./vendor/mediawiki/mediawiki-codesniffer/MediaWiki">
<exclude name="Squiz.Classes.ValidClassName.NotCamelCaps"/>
<exclude name="PSR2.Classes.PropertyDeclaration.ScopeMissing"/><!-- T166381 -->
<exclude name="Squiz.WhiteSpace.ScopeKeywordSpacing.Incorrect"/><!-- T166381 -->
</rule>
<rule ref="MediaWiki.NamingConventions.ValidGlobalName">
<properties>