Add check for const with reserved word class

vimeo · Oct 7, 2022 · d476baa · d476baa
1 parent 028ac7f
commit d476baa
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 0 deletions.
diff --git a/src/Psalm/Internal/Analyzer/ClassAnalyzer.php b/src/Psalm/Internal/Analyzer/ClassAnalyzer.php
@@ -505,6 +505,16 @@ public function analyze(
                 $member_stmts[] = $stmt;
 
                 foreach ($stmt->consts as $const) {
+                    if ($const->name->toLowerString() === 'class') {
+                        IssueBuffer::maybeAdd(
+                            new ReservedWord(
+                                'A class constant cannot be named \'class\'',
+                                new CodeLocation($this, $this->class),
+                                $this->fq_class_name
+                            )
+                        );
+                    }
+
                     $const_id = strtolower($this->fq_class_name) . '::' . $const->name;
 
                     foreach ($codebase->class_constants_to_rename as $original_const_id => $new_const_name) {

diff --git a/tests/ClassTest.php b/tests/ClassTest.php
@@ -1004,6 +1004,18 @@ public function valid(): bool {
                 ',
                 'error_message' => 'MissingTemplateParam',
             ],
+            'cannotNameClassConstantClass' => [
+                '<?php
+                class Foo
+                {
+                    /** @var class-string<Bar> */
+                    protected const CLASS = Bar::class;
+                }
+                
+                class Bar {}
+                ',
+                'error_message' => 'ReservedWord',
+            ]
         ];
     }
 }