Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prior to PHP 5.5, empty() only supports variables #207

Merged
merged 2 commits into from Jul 15, 2015

Conversation

bjork
Copy link
Contributor

@bjork bjork commented Jul 14, 2015

Test if the parameter of empty() is an arbitrary expression, and not
just a variable.

Test if the parameter of empty() is an arbitrary expression, and not
just a variable.
@@ -332,6 +332,9 @@ public function leaveNode(Node $node)

} elseif ($node instanceof Node\Scalar\MagicConst) {
$this->computeConstantVersions($node, $node->getName());

} else if ( $node instanceof Node\Expr\Empty_ ) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

elseif not else if

@llaville
Copy link
Owner

For documentation, here is the link http://www.php.net/manual/en/migration55.new-features.php#migration55.new-features.empty

I need to create a test unit for quality assurance. Will do it tomorrow

@llaville llaville added this to the 4.4.1 milestone Jul 14, 2015
@llaville llaville self-assigned this Jul 14, 2015
@llaville llaville added the bug label Jul 14, 2015
llaville added a commit that referenced this pull request Jul 15, 2015
llaville added a commit that referenced this pull request Jul 15, 2015
Prior to PHP 5.5, empty() only supports variables
@llaville llaville merged commit 91780eb into llaville:master Jul 15, 2015
@llaville
Copy link
Owner

Release 4.4.1 is planned for saturday morning (July 18).
Any feedback are welcome before deadline !

@llaville llaville mentioned this pull request Aug 18, 2020
24 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants