danog/psalm
1
0
Fork 0
mirror of https://github.com/danog/psalm.git synced 2024-12-02 17:52:45 +01:00
Code Issues Projects Releases Wiki Activity

Make UndefinedProperty issues more specific

Browse Source
This commit is contained in:
Matthew Brown 2016-10-10 19:29:38 -04:00
parent a30c82e5d7
commit 64b4adfb22
7 changed files with 37 additions and 21 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/Psalm/Checker/StatementsChecker.php
View File

@ -38,8 +38,10 @@ use Psalm\Issue\TypeCoercion;
use Psalm\Issue\UndefinedClass;
use Psalm\Issue\UndefinedConstant;
use Psalm\Issue\UndefinedFunction;
use Psalm\Issue\UndefinedProperty;
use Psalm\Issue\UndefinedThisProperty;
use Psalm\Issue\UndefinedPropertyAssignment;
use Psalm\Issue\UndefinedPropertyFetch;
use Psalm\Issue\UndefinedThisPropertyAssignment;
use Psalm\Issue\UndefinedThisPropertyFetch;
use Psalm\Issue\UndefinedVariable;
use Psalm\Type;
@ -1505,7 +1507,7 @@ class StatementsChecker
if (!$class_properties || !isset($class_properties[$stmt->name])) {
if ($stmt_var_id === 'this') {
if (IssueBuffer::accepts(
new UndefinedThisProperty(
new UndefinedThisPropertyFetch(
'Property ' . $lhs_type_part->value .'::$' . $stmt->name . ' is not defined',
$this->checked_file_name,
$stmt->getLine()
@ -1517,7 +1519,7 @@ class StatementsChecker
}
else {
if (IssueBuffer::accepts(
new UndefinedProperty(
new UndefinedPropertyFetch(
'Property ' . $lhs_type_part->value .'::$' . $stmt->name . ' is not defined',
$this->checked_file_name,
$stmt->getLine()
@ -1724,7 +1726,7 @@ class StatementsChecker
if (!isset($class_properties[$prop_name])) {
if ($stmt->var->name === 'this') {
if (IssueBuffer::accepts(
new UndefinedThisProperty(
new UndefinedThisPropertyAssignment(
'Instance property ' . $lhs_type_part->value . '::' . $prop_name . ' is not defined',
$this->checked_file_name,
$stmt->getLine()
@ -1736,7 +1738,7 @@ class StatementsChecker
}
else {
if (IssueBuffer::accepts(
new UndefinedProperty(
new UndefinedPropertyAssignment(
'Instance property ' . $lhs_type_part->value . '::' . $prop_name . ' is not defined',
$this->checked_file_name,
$stmt->getLine()
@ -3558,7 +3560,7 @@ class StatementsChecker
}
else {
IssueBuffer::add(
new UndefinedProperty('Static property ' . $var_id . ' does not exist', $this->checked_file_name, $stmt->getLine())
new UndefinedPropertyFetch('Static property ' . $var_id . ' does not exist', $this->checked_file_name, $stmt->getLine())
);
}

7
src/Psalm/Issue/UndefinedProperty.php
View File

@ -1,7 +0,0 @@
<?php
namespace Psalm\Issue;
class UndefinedProperty extends CodeError
{
}

7
src/Psalm/Issue/UndefinedPropertyAssignment.php Normal file
View File

@ -0,0 +1,7 @@
<?php
namespace Psalm\Issue;
class UndefinedPropertyAssignment extends CodeError
{
}

7
src/Psalm/Issue/UndefinedPropertyFetch.php Normal file
View File

@ -0,0 +1,7 @@
<?php
namespace Psalm\Issue;
class UndefinedPropertyFetch extends CodeError
{
}

7
src/Psalm/Issue/UndefinedThisProperty.php
View File

@ -1,7 +0,0 @@
<?php
namespace Psalm\Issue;
class UndefinedThisProperty extends UndefinedProperty
{
}

7
src/Psalm/Issue/UndefinedThisPropertyAssignment.php Normal file
View File

@ -0,0 +1,7 @@
<?php
namespace Psalm\Issue;
class UndefinedThisPropertyAssignment extends UndefinedPropertyAssignment
{
}

7
src/Psalm/Issue/UndefinedThisPropertyFetch.php Normal file
View File

@ -0,0 +1,7 @@
<?php
namespace Psalm\Issue;
class UndefinedThisPropertyFetch extends UndefinedPropertyFetch
{
}