diff --git a/src/Standards/PSR2/Sniffs/Classes/PropertyDeclarationSniff.php b/src/Standards/PSR2/Sniffs/Classes/PropertyDeclarationSniff.php index 29d7023ea9..baac59acf7 100644 --- a/src/Standards/PSR2/Sniffs/Classes/PropertyDeclarationSniff.php +++ b/src/Standards/PSR2/Sniffs/Classes/PropertyDeclarationSniff.php @@ -53,7 +53,10 @@ protected function processMemberVar(File $phpcsFile, $stackPtr) if ($tokens[$prev]['code'] === T_VAR) { $error = 'The var keyword must not be used to declare a property'; - $phpcsFile->addError($error, $stackPtr, 'VarUsed'); + $fix = $phpcsFile->addFixableError($error, $stackPtr, 'VarUsed'); + if ($fix === true) { + $phpcsFile->fixer->replaceToken($prev, 'public'); + } } $next = $phpcsFile->findNext([T_VARIABLE, T_SEMICOLON], ($stackPtr + 1)); diff --git a/src/Standards/PSR2/Tests/Classes/PropertyDeclarationUnitTest.inc.fixed b/src/Standards/PSR2/Tests/Classes/PropertyDeclarationUnitTest.inc.fixed index c4e22fc18b..b7fee08869 100644 --- a/src/Standards/PSR2/Tests/Classes/PropertyDeclarationUnitTest.inc.fixed +++ b/src/Standards/PSR2/Tests/Classes/PropertyDeclarationUnitTest.inc.fixed @@ -6,9 +6,9 @@ class MyClass private $var = null; $var = null; - var $var = null; + public $var = null; static $var = null; - public var $var = null; // Parse error. + public public $var = null; // Parse error. public $_var = null; protected $_var = null; @@ -46,7 +46,7 @@ class MyClass public string $var = null; protected ?Folder\ClassName $var = null; - var int $var = null; + public int $var = null; static int $var = null; private int $_var = null;