fix empty string in keyed array offset

vimeo · Jan 5, 2022 · 7e2b9d0 · 7e2b9d0
1 parent e41fc67
commit 7e2b9d0
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 1 deletion.
diff --git a/src/Psalm/Type/Atomic/TKeyedArray.php b/src/Psalm/Type/Atomic/TKeyedArray.php
@@ -430,7 +430,7 @@ public function getList(): TNonEmptyList
      */
     private function escapeAndQuote($name)
     {
-        if (is_string($name) && preg_match('/[^a-zA-Z0-9_]/', $name)) {
+        if (is_string($name) && ($name === '' || preg_match('/[^a-zA-Z0-9_]/', $name))) {
             $name = '\'' . str_replace("\n", '\n', addslashes($name)) . '\'';
         }
 

diff --git a/tests/AssertAnnotationTest.php b/tests/AssertAnnotationTest.php
@@ -1921,6 +1921,32 @@ function assertString(A $arg): bool {return $arg->b !== null;}
                     function requiresString(string $_str): void {}
                 ',
             ],
+            'assertWithEmptyStringOnKeyedArray' => [
+                '<?php
+                    class A
+                    {
+                        function test(): void
+                        {
+                            $a = ["" => ""];
+
+                            /** @var array<string, mixed> $b */
+                            $b = [];
+
+                            $this->assertSame($a, $b);
+                        }
+
+                        /**
+                         * @template T
+                         * @param T      $expected
+                         * @psalm-assert =T $actual
+                         */
+                        public function assertSame(mixed $expected, mixed $actual): void
+                        {
+                            return;
+                        }
+                    }
+                ',
+            ],
         ];
     }