Skip to content

Commit ffc279e

Browse files
weirdanmuglug
authored andcommitted
Converted MissingPropertyType to property issue (#4099)
Fixes #2200
1 parent 127b480 commit ffc279e

File tree

4 files changed

+6
-4
lines changed

4 files changed

+6
-4
lines changed

config.xsd

+1-1
Original file line numberDiff line numberDiff line change
@@ -265,7 +265,7 @@
265265
<xs:element name="MissingFile" type="IssueHandlerType" minOccurs="0" />
266266
<xs:element name="MissingImmutableAnnotation" type="IssueHandlerType" minOccurs="0" />
267267
<xs:element name="MissingParamType" type="IssueHandlerType" minOccurs="0" />
268-
<xs:element name="MissingPropertyType" type="IssueHandlerType" minOccurs="0" />
268+
<xs:element name="MissingPropertyType" type="PropertyIssueHandlerType" minOccurs="0" />
269269
<xs:element name="MissingReturnType" type="IssueHandlerType" minOccurs="0" />
270270
<xs:element name="MissingTemplateParam" type="IssueHandlerType" minOccurs="0" />
271271
<xs:element name="MissingThrowsDocblock" type="IssueHandlerType" minOccurs="0" />

src/Psalm/Internal/Analyzer/ClassAnalyzer.php

+2-1
Original file line numberDiff line numberDiff line change
@@ -1710,7 +1710,8 @@ private function checkForMissingPropertyType(
17101710
if (IssueBuffer::accepts(
17111711
new MissingPropertyType(
17121712
$message,
1713-
new CodeLocation($source, $stmt->props[0]->name)
1713+
new CodeLocation($source, $stmt->props[0]->name),
1714+
$property_id
17141715
),
17151716
$this->source->getSuppressedIssues()
17161717
)) {

src/Psalm/Internal/Analyzer/Statements/Expression/Fetch/InstancePropertyFetchAnalyzer.php

+2-1
Original file line numberDiff line numberDiff line change
@@ -965,7 +965,8 @@ public static function analyze(
965965
new MissingPropertyType(
966966
'Property ' . $fq_class_name . '::$' . $prop_name
967967
. ' does not have a declared type',
968-
new CodeLocation($statements_analyzer->getSource(), $stmt)
968+
new CodeLocation($statements_analyzer->getSource(), $stmt),
969+
$property_id
969970
),
970971
$statements_analyzer->getSuppressedIssues()
971972
)) {

src/Psalm/Issue/MissingPropertyType.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<?php
22
namespace Psalm\Issue;
33

4-
class MissingPropertyType extends CodeIssue
4+
class MissingPropertyType extends PropertyIssue
55
{
66
const ERROR_LEVEL = 2;
77
const SHORTCODE = 45;

0 commit comments

Comments
 (0)