Fixed bug #3167 : Generic.WhiteSpace.ScopeIndent false positive when using PHP 8.0 constructor property promotion

Author: gsherwood

src/Standards/Generic/Sniffs/WhiteSpace/ScopeIndentSniff.php

@@ -740,15 +740,29 @@ public function process(File $phpcsFile, $stackPtr)
                     && $tokens[$next]['code'] !== T_VARIABLE
                     && $tokens[$next]['code'] !== T_FN)
                 ) {
-                    if ($this->debug === true) {
-                        $line = $tokens[$checkToken]['line'];
-                        $type = $tokens[$checkToken]['type'];
-                        Common::printStatusMessage("* method prefix ($type) found on line $line; indent set to exact *", 1);
+                    $isMethodPrefix = true;
+                    if (isset($tokens[$checkToken]['nested_parenthesis']) === true) {
+                        $parenthesis = array_keys($tokens[$checkToken]['nested_parenthesis']);
+                        $deepestOpen = array_pop($parenthesis);
+                        if (isset($tokens[$deepestOpen]['parenthesis_owner']) === true
+                            && $tokens[$tokens[$deepestOpen]['parenthesis_owner']]['code'] === T_FUNCTION
+                        ) {
+                            // This is constructor property promotion and not a method prefix.
+                            $isMethodPrefix = false;
+                        }
                     }
 
-                    $exact = true;
-                }
-            }
+                    if ($isMethodPrefix === true) {
+                        if ($this->debug === true) {
+                            $line = $tokens[$checkToken]['line'];
+                            $type = $tokens[$checkToken]['type'];
+                            Common::printStatusMessage("* method prefix ($type) found on line $line; indent set to exact *", 1);
+                        }
+
+                        $exact = true;
+                    }
+                }//end if
+            }//end if
 
             // Open PHP tags needs to be indented to exact column positions
             // so they don't cause problems with indent checks for the code

src/Standards/Generic/Tests/WhiteSpace/ScopeIndentUnitTest.1.inc

@@ -1468,6 +1468,14 @@ if (true) {
     )?><?php
 }
 
+class Foo
+{
+    public function __construct(
+        public int $intArg,
+    ) {
+    }
+}
+
 /* ADD NEW TESTS ABOVE THIS LINE AND MAKE SURE THAT THE 1 (space-based) AND 2 (tab-based) FILES ARE IN SYNC! */
 ?>
 

src/Standards/Generic/Tests/WhiteSpace/ScopeIndentUnitTest.1.inc.fixed

@@ -1468,6 +1468,14 @@ if (true) {
     )?><?php
 }
 
+class Foo
+{
+    public function __construct(
+        public int $intArg,
+    ) {
+    }
+}
+
 /* ADD NEW TESTS ABOVE THIS LINE AND MAKE SURE THAT THE 1 (space-based) AND 2 (tab-based) FILES ARE IN SYNC! */
 ?>
 

src/Standards/Generic/Tests/WhiteSpace/ScopeIndentUnitTest.2.inc

@@ -1468,6 +1468,14 @@ if (true) {
 	)?><?php
 }
 
+class Foo
+{
+	public function __construct(
+		public int $intArg,
+	) {
+	}
+}
+
 /* ADD NEW TESTS ABOVE THIS LINE AND MAKE SURE THAT THE 1 (space-based) AND 2 (tab-based) FILES ARE IN SYNC! */
 ?>
 

src/Standards/Generic/Tests/WhiteSpace/ScopeIndentUnitTest.2.inc.fixed

@@ -1468,6 +1468,14 @@ if (true) {
 	)?><?php
 }
 
+class Foo
+{
+	public function __construct(
+		public int $intArg,
+	) {
+	}
+}
+
 /* ADD NEW TESTS ABOVE THIS LINE AND MAKE SURE THAT THE 1 (space-based) AND 2 (tab-based) FILES ARE IN SYNC! */
 ?>
 

src/Standards/Generic/Tests/WhiteSpace/ScopeIndentUnitTest.php

@@ -151,10 +151,10 @@ public function getErrorList($testFile='ScopeIndentUnitTest.inc')
             1340 => 1,
             1342 => 1,
             1345 => 1,
-            1479 => 1,
-            1480 => 1,
-            1481 => 1,
-            1482 => 1,
+            1487 => 1,
+            1488 => 1,
+            1489 => 1,
+            1490 => 1,
         ];
 
     }//end getErrorList()