Merge pull request #1187 from PHPCSStandards/php-8.4/squiz-membervarspacing-support-abstract-properties

PHP 8.4 | Squiz/MemberVarSpacing: add support for abstract properties

Author: jrfnl

src/Standards/Squiz/Sniffs/WhiteSpace/MemberVarSpacingSniff.php

@@ -58,6 +58,7 @@ protected function processMemberVar(File $phpcsFile, $stackPtr)
         $validPrefixes[] = T_FINAL;
         $validPrefixes[] = T_VAR;
         $validPrefixes[] = T_READONLY;
+        $validPrefixes[] = T_ABSTRACT;
 
         $startOfStatement = $phpcsFile->findNext($validPrefixes, ($endOfPreviousStatement + 1), $stackPtr, false, null, true);
         if ($startOfStatement === false) {

src/Standards/Squiz/Tests/WhiteSpace/MemberVarSpacingUnitTest.1.inc

@@ -459,3 +459,16 @@ class AsymVisibility {
 
     private(set) private bool $asymPrivate;
 }
+
+abstract class PHP84AbstractProperties {
+    abstract int $abstractA {get;}
+
+    /**
+     * Docblock
+     */
+    public abstract string $publicAbstract { set; }
+    #[AnAttribute]
+    abstract bool $abstractB {get;}
+
+    abstract protected bool $abstractProtected { set; }
+}

src/Standards/Squiz/Tests/WhiteSpace/MemberVarSpacingUnitTest.1.inc.fixed

@@ -446,3 +446,18 @@ class AsymVisibility {
 
     private(set) private bool $asymPrivate;
 }
+
+abstract class PHP84AbstractProperties {
+
+    abstract int $abstractA {get;}
+
+    /**
+     * Docblock
+     */
+    public abstract string $publicAbstract { set; }
+
+    #[AnAttribute]
+    abstract bool $abstractB {get;}
+
+    abstract protected bool $abstractProtected { set; }
+}

src/Standards/Squiz/Tests/WhiteSpace/MemberVarSpacingUnitTest.php

@@ -91,6 +91,8 @@ public function getErrorList($testFile='')
                 456 => 1,
                 457 => 1,
                 460 => 1,
+                464 => 1,
+                471 => 1,
             ];
 
         default: