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 default value for $params argument in Doctrine_Connection_Statement::execute method #36

Conversation

obukhov
Copy link

@obukhov obukhov commented Nov 28, 2017

That's obviously how it supposed to be.

In the method, there are two foreach statement iterating through $params.

@j0k3r j0k3r merged commit d435a81 into FriendsOfSymfony1:master Nov 28, 2017
@j0k3r
Copy link

j0k3r commented Nov 28, 2017

Looking good, thanks.

@obukhov obukhov deleted the feature/fix-doctrine-connection-statement-params-default branch November 29, 2017 16:32
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.

2 participants