Skip to content

[Cache] Make sure PdoAdapter::prune() always returns a bool #42904

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

Conversation

derrabus
Copy link
Member

@derrabus derrabus commented Sep 6, 2021

Q A
Branch? 4.4
Bug fix? yes
New feature? no
Deprecations? no
Tickets N/A
License MIT
Doc PR N/A

This fixes a failing test on the 6.0 branch. I wasn't sure if I should backport this change to 4.4/5.3 since the behavior of returning a DBAL result has always been wrong, kinda. Backported!

@nicolas-grekas
Copy link
Member

I think this should be fine for 4.4

@nicolas-grekas nicolas-grekas added this to the 4.4 milestone Sep 6, 2021
@fabpot
Copy link
Member

fabpot commented Sep 7, 2021

Can you rebase on 4.4 then?

@derrabus derrabus force-pushed the bugfix/pdo-adapter-return-type branch from 8b3a1df to abad7cf Compare September 7, 2021 06:54
@derrabus derrabus changed the base branch from 5.4 to 4.4 September 7, 2021 06:54
@derrabus derrabus closed this Sep 7, 2021
@derrabus derrabus reopened this Sep 7, 2021
Signed-off-by: Alexander M. Turek <me@derrabus.de>
@derrabus derrabus force-pushed the bugfix/pdo-adapter-return-type branch from abad7cf to b8ddb65 Compare September 7, 2021 06:55
@derrabus
Copy link
Member Author

derrabus commented Sep 7, 2021

Done! I've also adjusted AdapterTestCase to check the return value of prune() calls.

@nicolas-grekas
Copy link
Member

Thank you @derrabus.

@nicolas-grekas nicolas-grekas merged commit ba46a07 into symfony:4.4 Sep 7, 2021
@derrabus derrabus deleted the bugfix/pdo-adapter-return-type branch September 7, 2021 07:29
This was referenced Sep 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants