Squiz/LowercasePHPFunctions: bug fix for class names in attributes

Attributes cannot contain function calls and in most cases, `T_STRING`s in an attribute will be a class name, not a function name.

As things were, `T_STRING`'s in attributes which _looked_ like function calls (but in actual fact are class instantiations) would lead to false positives.

Fixed now by ignoring all code within attributes.

Includes unit test.

Fixes 3778

Author: jrfnl

src/Standards/Squiz/Sniffs/PHP/LowercasePHPFunctionsSniff.php

@@ -82,6 +82,11 @@ public function process(File $phpcsFile, $stackPtr)
         }
 
         // Make sure this is a function call or a use statement.
+        if (empty($tokens[$stackPtr]['nested_attributes']) === false) {
+            // Class instantiation in attribute, not function call.
+            return;
+        }
+
         $next = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true);
         if ($next === false) {
             // Not a function call.

src/Standards/Squiz/Tests/PHP/LowercasePHPFunctionsUnitTest.inc

@@ -41,3 +41,10 @@ $callToNamespacedFunction = namespace\STR_REPEAT($a, 2); // Could potentially be
 $filePath = new \File($path);
 
 $count = $object?->Count();
+
+class AttributesShouldBeIgnored
+{
+    #[Putenv('FOO', 'foo')]
+    public function foo(): void
+    {}
+}

src/Standards/Squiz/Tests/PHP/LowercasePHPFunctionsUnitTest.inc.fixed

@@ -41,3 +41,10 @@ $callToNamespacedFunction = namespace\STR_REPEAT($a, 2); // Could potentially be
 $filePath = new \File($path);
 
 $count = $object?->Count();
+
+class AttributesShouldBeIgnored
+{
+    #[Putenv('FOO', 'foo')]
+    public function foo(): void
+    {}
+}