Skip to content

Commit 7483f4d

Browse files
authored
Merge pull request #9740 from edsrzf/better-property-error
Fix OverriddenPropertyAccess error message
2 parents f8895ed + d2f0f6c commit 7483f4d

2 files changed

Lines changed: 4 additions & 2 deletions

File tree

src/Psalm/Internal/Analyzer/ClassAnalyzer.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -702,7 +702,7 @@ public static function addContextProperties(
702702
new OverriddenPropertyAccess(
703703
'Property ' . $fq_class_name . '::$' . $property_name
704704
. ' has different access level than '
705-
. $storage->name . '::$' . $property_name,
705+
. $guide_class_name . '::$' . $property_name,
706706
$property_storage->location,
707707
),
708708
);

tests/ClassTest.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,8 @@
55
use Psalm\Tests\Traits\InvalidCodeAnalysisTestTrait;
66
use Psalm\Tests\Traits\ValidCodeAnalysisTestTrait;
77

8+
use const DIRECTORY_SEPARATOR;
9+
810
class ClassTest extends TestCase
911
{
1012
use InvalidCodeAnalysisTestTrait;
@@ -936,7 +938,7 @@ class B extends A {
936938
/** @var string|null */
937939
private $foo;
938940
}',
939-
'error_message' => 'OverriddenPropertyAccess',
941+
'error_message' => 'OverriddenPropertyAccess - src' . DIRECTORY_SEPARATOR . 'somefile.php:9:33 - Property B::$foo has different access level than A::$foo',
940942
],
941943
'overridePublicPropertyAccessLevelToProtected' => [
942944
'code' => '<?php

0 commit comments

Comments
 (0)