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

Handle fetch mode deprecation of DBAL 2.11. #36987

Merged

Conversation

derrabus
Copy link
Member

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

See doctrine/dbal#4019

DBAL has deprecated PDO-style fetch modes in favor of more explicit methods.

@derrabus derrabus force-pushed the improvement/3.4-fetchmode-deprecation branch 3 times, most recently from d761aec to 812f2ba Compare May 27, 2020 23:14
@nicolas-grekas nicolas-grekas added this to the 3.4 milestone May 28, 2020
@nicolas-grekas
Copy link
Member

There's a related failure on appveyor.

@derrabus
Copy link
Member Author

There's a related failure on appveyor.

We don't have ext-pdo_sqlite on AppVeyor? 😱

@nicolas-grekas
Copy link
Member

appveyor runs without any, then with many extensions.

@stof
Copy link
Member

stof commented May 28, 2020

add @requires extension pdo_sqlite on the test so that it is skipped in that case.

@derrabus derrabus force-pushed the improvement/3.4-fetchmode-deprecation branch from 812f2ba to 0c1e254 Compare May 28, 2020 07:54
@derrabus derrabus force-pushed the improvement/3.4-fetchmode-deprecation branch from 0c1e254 to ed51855 Compare May 28, 2020 07:55
@derrabus
Copy link
Member Author

AppVeyor is happy now. The segfaulting php 8 Cache tests on Travis is hopefully unrealated. 😅

@nicolas-grekas
Copy link
Member

Thank you @derrabus.

@nicolas-grekas nicolas-grekas merged commit e1e314d into symfony:3.4 May 28, 2020
@derrabus derrabus deleted the improvement/3.4-fetchmode-deprecation branch May 28, 2020 08:25
nicolas-grekas added a commit that referenced this pull request May 28, 2020
This PR was merged into the 4.4 branch.

Discussion
----------

Handle fetch mode deprecation of DBAL 2.11

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

Continuation of #36987 on the 4.4 branch.

Commits
-------

1385213 Handle fetch mode deprecation of DBAL 2.11.
@nicolas-grekas
Copy link
Member

Things moved in DBAL v3 and tests are now failing after doctrine/dbal#4049 and/or doctrine/dbal#4045
Could you please have a look if you have some time for that?

@xabbuh
Copy link
Member

xabbuh commented Jun 9, 2020

Looks like part of the fix might be as simple as backporting #37141.

nicolas-grekas added a commit that referenced this pull request Jun 9, 2020
…bbuh)

This PR was merged into the 3.4 branch.

Discussion
----------

[Cache] fix forward compatibility with Doctrine DBAL 3

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | Fix #36987 (comment)
| License       | MIT
| Doc PR        |

Commits
-------

316efef fix forward compatibility with Doctrine DBAL 3
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.

None yet

7 participants