From 45a19b3f3ccfbbc71e22cb00afd8cee210135c17 Mon Sep 17 00:00:00 2001
From: Bruce Weirdan <weirdan@gmail.com>
Date: Sun, 13 Feb 2022 00:22:55 +0200
Subject: [PATCH 1/2] Honour global function suppressions for UndefinedFunction

Fixes vimeo/psalm#7610
---
 src/Psalm/Config.php        | 14 ++++++++++++--
 tests/Config/ConfigTest.php | 30 ++++++++++++++++++++++++++++++
 2 files changed, 42 insertions(+), 2 deletions(-)

diff --git a/src/Psalm/Config.php b/src/Psalm/Config.php
index da10ef51091..282499e1548 100644
--- a/src/Psalm/Config.php
+++ b/src/Psalm/Config.php
@@ -1790,11 +1790,21 @@ public function getReportingLevelForMethod(string $issue_type, string $method_id
 
     public function getReportingLevelForFunction(string $issue_type, string $function_id): ?string
     {
+        $level = null;
         if (isset($this->issue_handlers[$issue_type])) {
-            return $this->issue_handlers[$issue_type]->getReportingLevelForFunction($function_id);
+            $level = $this->issue_handlers[$issue_type]->getReportingLevelForFunction($function_id);
+
+            if ($level === null && $issue_type === 'UndefinedFunction') {
+                // undefined functions trigger global namespace fallback
+                // so we should also check reporting levels for the symbol in global scope
+                $root_function_id = preg_replace('/.*\\\/', '', $function_id);
+                if ($root_function_id !== $function_id) {
+                    $level = $this->issue_handlers[$issue_type]->getReportingLevelForFunction($root_function_id);
+                }
+            }
         }
 
-        return null;
+        return $level;
     }
 
     public function getReportingLevelForArgument(string $issue_type, string $function_id): ?string
diff --git a/tests/Config/ConfigTest.php b/tests/Config/ConfigTest.php
index 6a15f41c9ac..867099ff754 100644
--- a/tests/Config/ConfigTest.php
+++ b/tests/Config/ConfigTest.php
@@ -355,6 +355,36 @@ public function testIssueHandler(): void
         $this->assertFalse($config->reportIssueInFile('MissingReturnType', realpath('src/Psalm/Type.php')));
     }
 
+    public function testGlobalUndefinedFunctionSuppression(): void
+    {
+        $this->project_analyzer = $this->getProjectAnalyzerWithConfig(
+            Config::loadFromXML(
+                dirname(__DIR__, 2),
+                '<?xml version="1.0"?>
+                <psalm>
+                    <projectFiles>
+                        <directory name="src" />
+                        <directory name="tests" />
+                    </projectFiles>
+
+                    <issueHandlers>
+                        <UndefinedFunction>
+                            <errorLevel type="suppress">
+                                <referencedFunction name="zzz"/>
+                            </errorLevel>
+                        </UndefinedFunction>
+                    </issueHandlers>
+                </psalm>'
+            )
+        );
+
+        $config = $this->project_analyzer->getConfig();
+        $this->assertSame(
+            Config::REPORT_SUPPRESS,
+            $config->getReportingLevelForFunction('UndefinedFunction', 'Some\Namespace\zzz')
+        );
+    }
+
     public function testMultipleIssueHandlers(): void
     {
         $this->project_analyzer = $this->getProjectAnalyzerWithConfig(

From dc776d3ada23e412e3f857f902bab530b472bb80 Mon Sep 17 00:00:00 2001
From: Bruce Weirdan <weirdan@gmail.com>
Date: Sun, 13 Feb 2022 00:48:58 +0200
Subject: [PATCH 2/2] Suppress false-positive

---
 src/Psalm/Config.php | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/Psalm/Config.php b/src/Psalm/Config.php
index 282499e1548..ac4355b557c 100644
--- a/src/Psalm/Config.php
+++ b/src/Psalm/Config.php
@@ -1799,6 +1799,7 @@ public function getReportingLevelForFunction(string $issue_type, string $functio
                 // so we should also check reporting levels for the symbol in global scope
                 $root_function_id = preg_replace('/.*\\\/', '', $function_id);
                 if ($root_function_id !== $function_id) {
+                    /** @psalm-suppress PossiblyUndefinedStringArrayOffset https://github.com/vimeo/psalm/issues/7656 */
                     $level = $this->issue_handlers[$issue_type]->getReportingLevelForFunction($root_function_id);
                 }
             }