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

Adding the constant PARAM_ASCII_ARRAY to Doctrine\DBAL\Connection #5120

Merged
merged 1 commit into from Jan 9, 2022

Conversation

santoja
Copy link
Contributor

@santoja santoja commented Dec 15, 2021

Adding the constant PARAM_ASCII_ARRAY to Doctrine\DBAL\Connection

This fix enable to use ASCII_STRING as type for Arrays of strings.
This new parameter was added in version 2.11.x but unfortunally we don't have a way to use in a array.

Also added it in the check for Param array in ExpandArrayParameters.php so we can handle it there too

Q A
Type improvement
BC Break no
Fixed issues

Summary

@santoja
Copy link
Contributor Author

santoja commented Dec 15, 2021

Since I change it to be merged to 3.x version instead of 2.x, I closed the other PR #5115 and opened a new one

@derrabus
Copy link
Member

Can you look into the failing tests?

@santoja
Copy link
Contributor Author

santoja commented Dec 22, 2021

Sure., I'm out for holidays now, and will check when I get back

@derrabus
Copy link
Member

Sure, no pressure. Have a wonderful Christmas! 🎄

@derrabus derrabus changed the base branch from 3.2.x to 3.3.x January 6, 2022 21:20
@santoja
Copy link
Contributor Author

santoja commented Jan 7, 2022

I hope that you folks had great holidays, the test are passing now

@derrabus derrabus changed the title Adding the constant PARAM_ASCII_ARRAY to Doctrine\DBAL\Connection 3.x Adding the constant PARAM_ASCII_ARRAY to Doctrine\DBAL\Connection Jan 8, 2022
 This fix enable to use ASCII_STRING as type for Arrays of strings.
 This new parameter was added in version 2.11.x but unfortunally we don't have a way to use in a array.

 Also added it in the check for Param array in ExpandArrayParameters.php so we can handle it there too
@derrabus derrabus force-pushed the adding_ascii_parameter_array_3.x branch from a20b97b to f1037f4 Compare January 9, 2022 22:24
@derrabus derrabus added this to the 3.3.0 milestone Jan 9, 2022
@derrabus derrabus merged commit ca22115 into doctrine:3.3.x Jan 9, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants