Skip to content

support mysqli_execute_query (php 8.2+ only) #425

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

Merged
merged 15 commits into from
Sep 25, 2022
Merged

Conversation

staabm
Copy link
Owner

@staabm staabm commented Aug 22, 2022

closes #409

requires #426

@staabm staabm force-pushed the mysqli_execute_query branch from f9aed1b to 1d30e2d Compare August 22, 2022 07:43
@staabm
Copy link
Owner Author

staabm commented Aug 22, 2022

the test fails because the rule involved runs into this early exit

$methodReflection = $scope->getMethodReflection($scope->getType($node->var), $node->name->toString());
if (null === $methodReflection) {
return [];
}

I guess it will work with the next phpstan version which has updated php8 stubs

@staabm staabm force-pushed the mysqli_execute_query branch from 501ad4f to 2fff777 Compare September 25, 2022 17:05
@staabm staabm force-pushed the mysqli_execute_query branch from 2fff777 to 5ec1dbc Compare September 25, 2022 17:29
@staabm staabm marked this pull request as ready for review September 25, 2022 17:58
@staabm staabm merged commit d73bbb7 into main Sep 25, 2022
@staabm staabm deleted the mysqli_execute_query branch September 25, 2022 17:59
marmichalski pushed a commit to marmichalski/phpstan-dba that referenced this pull request Oct 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support mysqli_execute_query()
1 participant