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

Fix function-map for PDOStatement pass-by-reference functions #480

Merged
merged 1 commit into from
Mar 20, 2021
Merged

Fix function-map for PDOStatement pass-by-reference functions #480

merged 1 commit into from
Mar 20, 2021

Conversation

jaylinski
Copy link
Contributor

$param = 1;

$statement->execute();
$statement->bindColumn('col', $col);
Copy link
Member

Choose a reason for hiding this comment

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

Have you first made sure that the test fails? It won't because it's not referenced from anywhere. See NodeScopeResolver::testFileAsserts. The test needs to contain assertVariableCertainty call.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nope... took me 2 hours to figure out how to run a single test-method in isolation. I now made sure the test fails.

Copy link
Member

Choose a reason for hiding this comment

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

What was the problem? vendor/bin/phing tests-fast (which is mentioned in the README) runs all essential tests in around 14 seconds on my computer.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, that was probably a problem on my end. tests-fast took about 20 minutes in my PHP 7.4-cli docker container. Probably something with xdebug...

Thanks for your help and this nice peace of software! :octocat:

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, Xdebug will slow it down, I enable it only when I need to debug something. Thank you!

@ondrejmirtes ondrejmirtes merged commit 1991bc7 into phpstan:master Mar 20, 2021
@ondrejmirtes
Copy link
Member

Thank you.

@jaylinski jaylinski deleted the bug-4714 branch March 20, 2021 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants