2016-10-22 19:23:18 +02:00
|
|
|
<?php
|
|
|
|
namespace Psalm\Checker\Statements\Block;
|
|
|
|
|
|
|
|
use PhpParser;
|
|
|
|
use Psalm\Checker\ClassChecker;
|
|
|
|
use Psalm\Checker\ClassLikeChecker;
|
|
|
|
use Psalm\Checker\CommentChecker;
|
2017-11-24 18:57:00 +01:00
|
|
|
use Psalm\Checker\InterfaceChecker;
|
2016-10-22 19:23:18 +02:00
|
|
|
use Psalm\Checker\MethodChecker;
|
2016-12-04 20:14:00 +01:00
|
|
|
use Psalm\Checker\Statements\Expression\AssignmentChecker;
|
2017-05-19 06:48:26 +02:00
|
|
|
use Psalm\Checker\Statements\ExpressionChecker;
|
2016-11-02 07:29:00 +01:00
|
|
|
use Psalm\Checker\StatementsChecker;
|
2017-05-19 06:48:26 +02:00
|
|
|
use Psalm\CodeLocation;
|
|
|
|
use Psalm\Context;
|
2017-11-15 03:43:31 +01:00
|
|
|
use Psalm\Exception\DocblockParseException;
|
|
|
|
use Psalm\Issue\InvalidDocblock;
|
2016-10-22 19:23:18 +02:00
|
|
|
use Psalm\Issue\InvalidIterator;
|
2017-05-22 17:59:58 +02:00
|
|
|
use Psalm\Issue\NullIterator;
|
|
|
|
use Psalm\Issue\PossiblyNullIterator;
|
2017-11-15 04:55:48 +01:00
|
|
|
use Psalm\Issue\RawObjectIteration;
|
2017-05-25 04:07:49 +02:00
|
|
|
use Psalm\IssueBuffer;
|
2017-12-03 00:28:18 +01:00
|
|
|
use Psalm\Scope\LoopScope;
|
2016-10-22 19:23:18 +02:00
|
|
|
use Psalm\Type;
|
2017-12-04 14:50:59 +01:00
|
|
|
use Psalm\VarDocblockComment;
|
2016-10-22 19:23:18 +02:00
|
|
|
|
|
|
|
class ForeachChecker
|
|
|
|
{
|
|
|
|
/**
|
2016-11-02 07:29:00 +01:00
|
|
|
* @param StatementsChecker $statements_checker
|
|
|
|
* @param PhpParser\Node\Stmt\Foreach_ $stmt
|
|
|
|
* @param Context $context
|
2017-05-27 02:16:18 +02:00
|
|
|
*
|
2016-11-02 07:29:00 +01:00
|
|
|
* @return false|null
|
2016-10-22 19:23:18 +02:00
|
|
|
*/
|
2017-01-07 21:09:47 +01:00
|
|
|
public static function analyze(
|
2016-11-02 07:29:00 +01:00
|
|
|
StatementsChecker $statements_checker,
|
|
|
|
PhpParser\Node\Stmt\Foreach_ $stmt,
|
|
|
|
Context $context
|
|
|
|
) {
|
2017-01-07 21:09:47 +01:00
|
|
|
if (ExpressionChecker::analyze($statements_checker, $stmt->expr, $context) === false) {
|
2016-10-22 19:23:18 +02:00
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
$foreach_context = clone $context;
|
2017-03-13 16:23:26 +01:00
|
|
|
$foreach_context->inside_loop = true;
|
2016-10-22 19:23:18 +02:00
|
|
|
|
|
|
|
$key_type = null;
|
|
|
|
$value_type = null;
|
|
|
|
|
|
|
|
$var_id = ExpressionChecker::getVarId(
|
|
|
|
$stmt->expr,
|
2016-11-08 01:16:51 +01:00
|
|
|
$statements_checker->getFQCLN(),
|
2017-01-07 20:35:07 +01:00
|
|
|
$statements_checker
|
2016-10-22 19:23:18 +02:00
|
|
|
);
|
|
|
|
|
|
|
|
if (isset($stmt->expr->inferredType)) {
|
|
|
|
/** @var Type\Union */
|
|
|
|
$iterator_type = $stmt->expr->inferredType;
|
2017-11-16 02:45:53 +01:00
|
|
|
} elseif ($var_id && $foreach_context->hasVariable($var_id)) {
|
2016-10-22 19:23:18 +02:00
|
|
|
$iterator_type = $foreach_context->vars_in_scope[$var_id];
|
2016-11-02 07:29:00 +01:00
|
|
|
} else {
|
2016-10-22 19:23:18 +02:00
|
|
|
$iterator_type = null;
|
|
|
|
}
|
|
|
|
|
2017-10-07 16:22:52 +02:00
|
|
|
$project_checker = $statements_checker->getFileChecker()->project_checker;
|
|
|
|
|
2016-10-22 19:23:18 +02:00
|
|
|
if ($iterator_type) {
|
2017-05-22 17:59:58 +02:00
|
|
|
if ($iterator_type->isNull()) {
|
|
|
|
if (IssueBuffer::accepts(
|
|
|
|
new NullIterator(
|
|
|
|
'Cannot iterate over null',
|
|
|
|
new CodeLocation($statements_checker->getSource(), $stmt->expr)
|
|
|
|
),
|
|
|
|
$statements_checker->getSuppressedIssues()
|
|
|
|
)) {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
} elseif ($iterator_type->isNullable() && !$iterator_type->ignore_nullable_issues) {
|
|
|
|
if (IssueBuffer::accepts(
|
|
|
|
new PossiblyNullIterator(
|
|
|
|
'Cannot iterate over nullable var ' . $iterator_type,
|
|
|
|
new CodeLocation($statements_checker->getSource(), $stmt->expr)
|
|
|
|
),
|
|
|
|
$statements_checker->getSuppressedIssues()
|
|
|
|
)) {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2017-02-11 00:12:59 +01:00
|
|
|
foreach ($iterator_type->types as $iterator_type) {
|
2016-10-22 19:23:18 +02:00
|
|
|
// if it's an empty array, we cannot iterate over it
|
2017-12-10 22:17:27 +01:00
|
|
|
if ($iterator_type instanceof Type\Atomic\TArray
|
|
|
|
&& $iterator_type->type_params[1]->isEmpty()
|
|
|
|
) {
|
2016-10-22 19:23:18 +02:00
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
2017-05-22 17:59:58 +02:00
|
|
|
if ($iterator_type instanceof Type\Atomic\TNull) {
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
2017-02-11 00:12:59 +01:00
|
|
|
if ($iterator_type instanceof Type\Atomic\TArray) {
|
2016-10-22 19:23:18 +02:00
|
|
|
if (!$value_type) {
|
2017-02-11 00:12:59 +01:00
|
|
|
$value_type = $iterator_type->type_params[1];
|
2016-11-02 07:29:00 +01:00
|
|
|
} else {
|
2017-02-11 00:12:59 +01:00
|
|
|
$value_type = Type::combineUnionTypes($value_type, $iterator_type->type_params[1]);
|
2016-10-22 19:23:18 +02:00
|
|
|
}
|
|
|
|
|
2017-02-11 00:12:59 +01:00
|
|
|
$key_type_part = $iterator_type->type_params[0];
|
2016-10-22 19:23:18 +02:00
|
|
|
|
2017-02-11 00:12:59 +01:00
|
|
|
if (!$key_type) {
|
|
|
|
$key_type = $key_type_part;
|
|
|
|
} else {
|
|
|
|
$key_type = Type::combineUnionTypes($key_type, $key_type_part);
|
2016-10-22 19:23:18 +02:00
|
|
|
}
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
2017-02-11 00:12:59 +01:00
|
|
|
if ($iterator_type instanceof Type\Atomic\Scalar ||
|
|
|
|
$iterator_type instanceof Type\Atomic\TVoid
|
2017-01-15 01:06:58 +01:00
|
|
|
) {
|
|
|
|
if (IssueBuffer::accepts(
|
|
|
|
new InvalidIterator(
|
2017-02-11 00:12:59 +01:00
|
|
|
'Cannot iterate over ' . $iterator_type->getKey(),
|
2017-01-15 01:06:58 +01:00
|
|
|
new CodeLocation($statements_checker->getSource(), $stmt->expr)
|
|
|
|
),
|
|
|
|
$statements_checker->getSuppressedIssues()
|
|
|
|
)) {
|
|
|
|
return false;
|
|
|
|
}
|
2016-10-22 19:23:18 +02:00
|
|
|
|
2017-01-15 01:06:58 +01:00
|
|
|
$value_type = Type::getMixed();
|
2017-08-08 00:38:38 +02:00
|
|
|
} elseif ($iterator_type instanceof Type\Atomic\TObject ||
|
2017-02-11 00:12:59 +01:00
|
|
|
$iterator_type instanceof Type\Atomic\TMixed ||
|
|
|
|
$iterator_type instanceof Type\Atomic\TEmpty
|
2017-01-15 01:06:58 +01:00
|
|
|
) {
|
|
|
|
$value_type = Type::getMixed();
|
2017-02-11 00:12:59 +01:00
|
|
|
} elseif ($iterator_type instanceof Type\Atomic\TNamedObject) {
|
|
|
|
if ($iterator_type->value !== 'Traversable' &&
|
|
|
|
$iterator_type->value !== $statements_checker->getClassName()
|
2017-01-15 01:06:58 +01:00
|
|
|
) {
|
|
|
|
if (ClassLikeChecker::checkFullyQualifiedClassLikeName(
|
2017-07-29 21:05:06 +02:00
|
|
|
$project_checker,
|
2017-02-11 00:12:59 +01:00
|
|
|
$iterator_type->value,
|
2017-01-15 01:06:58 +01:00
|
|
|
new CodeLocation($statements_checker->getSource(), $stmt->expr),
|
2016-10-22 19:23:18 +02:00
|
|
|
$statements_checker->getSuppressedIssues()
|
2017-01-15 01:06:58 +01:00
|
|
|
) === false) {
|
2016-10-22 19:23:18 +02:00
|
|
|
return false;
|
|
|
|
}
|
2017-01-15 01:06:58 +01:00
|
|
|
}
|
2016-11-02 07:29:00 +01:00
|
|
|
|
2017-02-11 00:12:59 +01:00
|
|
|
if ($iterator_type instanceof Type\Atomic\TGenericObject &&
|
|
|
|
(strtolower($iterator_type->value) === 'iterable' ||
|
|
|
|
strtolower($iterator_type->value) === 'traversable' ||
|
|
|
|
ClassChecker::classImplements(
|
2017-07-29 21:05:06 +02:00
|
|
|
$project_checker,
|
2017-02-11 00:12:59 +01:00
|
|
|
$iterator_type->value,
|
|
|
|
'Traversable'
|
|
|
|
))
|
|
|
|
) {
|
|
|
|
$value_index = count($iterator_type->type_params) - 1;
|
|
|
|
$value_type_part = $iterator_type->type_params[$value_index];
|
|
|
|
|
|
|
|
if (!$value_type) {
|
|
|
|
$value_type = $value_type_part;
|
|
|
|
} else {
|
|
|
|
$value_type = Type::combineUnionTypes($value_type, $value_type_part);
|
|
|
|
}
|
|
|
|
|
|
|
|
if ($value_index) {
|
|
|
|
$key_type_part = $iterator_type->type_params[0];
|
|
|
|
|
|
|
|
if (!$key_type) {
|
|
|
|
$key_type = $key_type_part;
|
|
|
|
} else {
|
|
|
|
$key_type = Type::combineUnionTypes($key_type, $key_type_part);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
2017-01-15 01:06:58 +01:00
|
|
|
if (ClassChecker::classImplements(
|
2017-07-29 21:05:06 +02:00
|
|
|
$project_checker,
|
2017-02-11 00:12:59 +01:00
|
|
|
$iterator_type->value,
|
2017-01-15 01:06:58 +01:00
|
|
|
'Iterator'
|
2017-11-24 18:57:00 +01:00
|
|
|
) ||
|
2017-12-01 01:00:09 +01:00
|
|
|
(
|
|
|
|
InterfaceChecker::interfaceExists($project_checker, $iterator_type->value)
|
2017-11-24 18:57:00 +01:00
|
|
|
&& InterfaceChecker::interfaceExtends(
|
|
|
|
$project_checker,
|
|
|
|
$iterator_type->value,
|
|
|
|
'Iterator'
|
|
|
|
)
|
|
|
|
)
|
|
|
|
) {
|
2017-02-11 00:12:59 +01:00
|
|
|
$iterator_method = $iterator_type->value . '::current';
|
2017-07-29 21:05:06 +02:00
|
|
|
$iterator_class_type = MethodChecker::getMethodReturnType($project_checker, $iterator_method);
|
2017-01-15 01:06:58 +01:00
|
|
|
|
|
|
|
if ($iterator_class_type) {
|
2017-07-29 21:05:06 +02:00
|
|
|
$value_type_part = ExpressionChecker::fleshOutType(
|
|
|
|
$project_checker,
|
|
|
|
$iterator_class_type,
|
2017-02-11 00:12:59 +01:00
|
|
|
$iterator_type->value,
|
2017-01-15 01:06:58 +01:00
|
|
|
$iterator_method
|
|
|
|
);
|
2017-01-09 06:56:51 +01:00
|
|
|
|
2017-01-15 01:06:58 +01:00
|
|
|
if (!$value_type) {
|
|
|
|
$value_type = $value_type_part;
|
2016-11-02 07:29:00 +01:00
|
|
|
} else {
|
2017-01-15 01:06:58 +01:00
|
|
|
$value_type = Type::combineUnionTypes($value_type, $value_type_part);
|
2016-10-22 19:23:18 +02:00
|
|
|
}
|
2017-01-15 01:06:58 +01:00
|
|
|
} else {
|
|
|
|
$value_type = Type::getMixed();
|
2016-10-22 19:23:18 +02:00
|
|
|
}
|
2017-11-15 17:58:46 +01:00
|
|
|
} elseif (ClassChecker::classImplements(
|
|
|
|
$project_checker,
|
|
|
|
$iterator_type->value,
|
|
|
|
'Traversable'
|
2017-11-24 18:57:00 +01:00
|
|
|
) ||
|
2017-12-01 01:00:09 +01:00
|
|
|
(
|
|
|
|
InterfaceChecker::interfaceExists($project_checker, $iterator_type->value)
|
2017-11-24 18:57:00 +01:00
|
|
|
&& InterfaceChecker::interfaceExtends(
|
|
|
|
$project_checker,
|
|
|
|
$iterator_type->value,
|
|
|
|
'Traversable'
|
|
|
|
)
|
|
|
|
)
|
|
|
|
) {
|
2017-11-15 17:58:46 +01:00
|
|
|
// @todo try and get value type
|
2017-11-15 18:09:13 +01:00
|
|
|
} elseif (!in_array(
|
|
|
|
strtolower($iterator_type->value),
|
2017-11-24 18:57:00 +01:00
|
|
|
['iterator', 'iterable', 'traversable'],
|
|
|
|
true
|
2017-11-15 18:09:13 +01:00
|
|
|
)) {
|
2017-11-15 04:55:48 +01:00
|
|
|
if (IssueBuffer::accepts(
|
|
|
|
new RawObjectIteration(
|
|
|
|
'Possibly undesired iteration over regular object ' . $iterator_type->value,
|
|
|
|
new CodeLocation($statements_checker->getSource(), $stmt->expr)
|
|
|
|
),
|
|
|
|
$statements_checker->getSuppressedIssues()
|
|
|
|
)) {
|
|
|
|
return false;
|
|
|
|
}
|
2017-01-15 01:06:58 +01:00
|
|
|
}
|
2016-10-22 19:23:18 +02:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2017-03-15 01:14:25 +01:00
|
|
|
$before_context = clone $foreach_context;
|
|
|
|
|
2016-12-29 06:32:12 +01:00
|
|
|
if ($stmt->keyVar && $stmt->keyVar instanceof PhpParser\Node\Expr\Variable && is_string($stmt->keyVar->name)) {
|
2017-02-08 00:09:12 +01:00
|
|
|
$key_var_id = '$' . $stmt->keyVar->name;
|
|
|
|
$foreach_context->vars_in_scope[$key_var_id] = $key_type ?: Type::getMixed();
|
|
|
|
$foreach_context->vars_possibly_in_scope[$key_var_id] = true;
|
|
|
|
|
|
|
|
if (!$statements_checker->hasVariable($key_var_id)) {
|
|
|
|
$statements_checker->registerVariable(
|
|
|
|
$key_var_id,
|
2017-02-08 00:18:33 +01:00
|
|
|
new CodeLocation($statements_checker, $stmt->keyVar)
|
2017-02-08 00:09:12 +01:00
|
|
|
);
|
|
|
|
}
|
2016-10-22 19:23:18 +02:00
|
|
|
}
|
|
|
|
|
2017-01-07 21:09:47 +01:00
|
|
|
AssignmentChecker::analyze(
|
2016-12-04 20:14:00 +01:00
|
|
|
$statements_checker,
|
|
|
|
$stmt->valueVar,
|
|
|
|
null,
|
|
|
|
$value_type ?: Type::getMixed(),
|
|
|
|
$foreach_context,
|
|
|
|
(string)$stmt->getDocComment()
|
|
|
|
);
|
2016-10-22 19:23:18 +02:00
|
|
|
|
2017-03-02 04:27:52 +01:00
|
|
|
$doc_comment_text = (string)$stmt->getDocComment();
|
|
|
|
|
|
|
|
if ($doc_comment_text) {
|
2017-12-04 14:50:59 +01:00
|
|
|
/** @var VarDocblockComment|null $var_comment */
|
|
|
|
$var_comment = null;
|
|
|
|
|
2017-11-15 03:43:31 +01:00
|
|
|
try {
|
|
|
|
$var_comment = CommentChecker::getTypeFromComment(
|
|
|
|
$doc_comment_text,
|
|
|
|
$statements_checker->getSource(),
|
|
|
|
$statements_checker->getSource()->getAliases()
|
|
|
|
);
|
|
|
|
} catch (DocblockParseException $e) {
|
|
|
|
if (IssueBuffer::accepts(
|
|
|
|
new InvalidDocblock(
|
|
|
|
(string)$e->getMessage(),
|
|
|
|
new CodeLocation($statements_checker, $stmt)
|
|
|
|
)
|
|
|
|
)) {
|
|
|
|
// fall through
|
|
|
|
}
|
|
|
|
}
|
2017-05-25 07:32:34 +02:00
|
|
|
|
|
|
|
if ($var_comment && $var_comment->var_id) {
|
2017-10-07 16:22:52 +02:00
|
|
|
$comment_type = ExpressionChecker::fleshOutType(
|
|
|
|
$project_checker,
|
|
|
|
Type::parseString($var_comment->type),
|
|
|
|
$context->self
|
|
|
|
);
|
|
|
|
|
|
|
|
$foreach_context->vars_in_scope[$var_comment->var_id] = $comment_type;
|
2017-05-25 07:32:34 +02:00
|
|
|
}
|
2017-03-02 04:27:52 +01:00
|
|
|
}
|
2016-10-22 19:23:18 +02:00
|
|
|
|
2017-12-03 00:28:18 +01:00
|
|
|
$loop_scope = new LoopScope($foreach_context, $context);
|
|
|
|
|
|
|
|
LoopChecker::analyze($statements_checker, $stmt->stmts, [], [], $loop_scope);
|
2016-10-22 19:23:18 +02:00
|
|
|
|
2016-11-02 07:29:00 +01:00
|
|
|
$context->vars_possibly_in_scope = array_merge(
|
|
|
|
$foreach_context->vars_possibly_in_scope,
|
|
|
|
$context->vars_possibly_in_scope
|
|
|
|
);
|
|
|
|
|
2017-11-24 18:17:28 +01:00
|
|
|
$context->referenced_var_ids = array_merge(
|
|
|
|
$foreach_context->referenced_var_ids,
|
|
|
|
$context->referenced_var_ids
|
|
|
|
);
|
2017-02-01 05:24:33 +01:00
|
|
|
|
2016-11-02 07:29:00 +01:00
|
|
|
return null;
|
2016-10-22 19:23:18 +02:00
|
|
|
}
|
|
|
|
}
|