ReferenceUsedNamesOnlySniff: fixed "allowFullyQualifiedNameForCollidingFunctions" and "allowFullyQualifiedNameForCollidingConstants"

Author: kukulich

SlevomatCodingStandard/Sniffs/Namespaces/ReferenceUsedNamesOnlySniff.php

@@ -218,19 +218,21 @@ function (ReferencedName $referencedName): string {
 			$isGlobalFunctionFallback = $referencedName->isFunction() && $isGlobalFallback;
 			$isGlobalConstantFallback = $referencedName->isConstant() && $isGlobalFallback;
 
-			if ($referencedName->isClass() && $this->allowFullyQualifiedNameForCollidingClasses) {
-				$lowerCasedUnqualifiedClassName = strtolower($unqualifiedName);
-				if (isset($classReferencesIndex[$lowerCasedUnqualifiedClassName]) || array_key_exists($lowerCasedUnqualifiedClassName, $definedClassesIndex)) {
-					continue;
-				}
-			} elseif ($referencedName->isFunction() && $this->allowFullyQualifiedNameForCollidingFunctions) {
-				$lowerCasedUnqualifiedFunctionName = strtolower($unqualifiedName);
-				if (isset($functionReferencesIndex[$lowerCasedUnqualifiedFunctionName]) || array_key_exists($lowerCasedUnqualifiedFunctionName, $definedFunctionsIndex)) {
-					continue;
-				}
-			} elseif ($referencedName->isConstant() && $this->allowFullyQualifiedNameForCollidingConstants) {
-				if (isset($constantReferencesIndex[$unqualifiedName]) || array_key_exists($unqualifiedName, $definedConstantsIndex)) {
-					continue;
+			if ($isFullyQualified) {
+				if ($referencedName->isClass() && $this->allowFullyQualifiedNameForCollidingClasses) {
+					$lowerCasedUnqualifiedClassName = strtolower($unqualifiedName);
+					if (isset($classReferencesIndex[$lowerCasedUnqualifiedClassName]) || array_key_exists($lowerCasedUnqualifiedClassName, $definedClassesIndex)) {
+						continue;
+					}
+				} elseif ($referencedName->isFunction() && $this->allowFullyQualifiedNameForCollidingFunctions) {
+					$lowerCasedUnqualifiedFunctionName = strtolower($unqualifiedName);
+					if (isset($functionReferencesIndex[$lowerCasedUnqualifiedFunctionName]) || array_key_exists($lowerCasedUnqualifiedFunctionName, $definedFunctionsIndex)) {
+						continue;
+					}
+				} elseif ($referencedName->isConstant() && $this->allowFullyQualifiedNameForCollidingConstants) {
+					if (isset($constantReferencesIndex[$unqualifiedName]) || array_key_exists($unqualifiedName, $definedConstantsIndex)) {
+						continue;
+					}
 				}
 			}
 

tests/Sniffs/Namespaces/ReferenceUsedNamesOnlySniffTest.php

@@ -775,4 +775,19 @@ public function testCollidingFullyQualifiedConstantNameDisallowed(): void
 		$this->assertSniffError($report, 12, ReferenceUsedNamesOnlySniff::CODE_REFERENCE_VIA_FULLY_QUALIFIED_NAME);
 	}
 
+	public function testReferencingGlobalFunctionViaFallbackErrorsWithMoreComplexSettings(): void
+	{
+		$report = $this->checkFile(
+			__DIR__ . '/data/referencingGlobalFunctionViaFallbackErrorsWithMoreComplexSettings.php',
+			[
+				'allowFallbackGlobalFunctions' => false,
+				'allowFullyQualifiedNameForCollidingFunctions' => true,
+				'allowFullyQualifiedGlobalFunctions' => true,
+			]
+		);
+
+		$this->assertSame(1, $report->getErrorCount());
+		$this->assertSniffError($report, 17, ReferenceUsedNamesOnlySniff::CODE_REFERENCE_VIA_FALLBACK_GLOBAL_NAME);
+	}
+
 }

tests/Sniffs/Namespaces/data/referencingGlobalFunctionViaFallbackErrorsWithMoreComplexSettings.php

@@ -0,0 +1,20 @@
+<?php
+
+namespace FooNamespace;
+
+function php_version()
+{
+	return '6.0.0';
+}
+
+class Foo
+{
+
+	public function boo()
+	{
+		\php_version();
+		\max(1, 3);
+		min(1, 3);
+	}
+
+}