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 Cache Pools: SQLite3Cache constructor argument #6852

Merged
merged 1 commit into from
Aug 11, 2016

Conversation

wimme002
Copy link
Contributor

@wimme002 wimme002 commented Aug 5, 2016

The first argument for SQLite3Cache should be of type SQLite3 instead of a string

https://github.com/doctrine/cache/blob/master/lib/Doctrine/Common/Cache/SQLite3Cache.php#L68

The first argument for SQLite3Cache should be of type SQLite3 instead of a string
@wouterj
Copy link
Member

wouterj commented Aug 5, 2016

Agreed 👍

status: reviewed

@javiereguiluz
Copy link
Member

👍

@wouterj
Copy link
Member

wouterj commented Aug 11, 2016

Thank you for testing & fixing the code in the docs!

@wouterj wouterj merged commit 1a8c9ab into symfony:3.1 Aug 11, 2016
wouterj added a commit that referenced this pull request Aug 11, 2016
This PR was merged into the 3.1 branch.

Discussion
----------

Fix Cache Pools: SQLite3Cache constructor argument

The first argument for SQLite3Cache should be of type SQLite3 instead of a string

https://github.com/doctrine/cache/blob/master/lib/Doctrine/Common/Cache/SQLite3Cache.php#L68

Commits
-------

1a8c9ab Fix Cache Pools: SQLite3Cache constructor argument
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

4 participants