Skip to content

[FrameworkBundle] Do not activate the cache if Doctrine's cache is not present #24743

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

sroze
Copy link
Contributor

@sroze sroze commented Oct 29, 2017

Q A
Branch? 3.4
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #24683
License MIT
Doc PR ø

Annotations are automatically enabled when the doctrine/annotations package is around. These annotations are enabled with the php_array cache by default, which require doctrine/cache.

This PR allow this scenario to happen by disabling the annotation cache when doctrine/cache is not present.

@sroze
Copy link
Contributor Author

sroze commented Oct 29, 2017

Failures are un-related. But related with the timezone change and VarDumper (wink wink @nicolas-grekas)

@fabpot
Copy link
Member

fabpot commented Oct 29, 2017

Thank you @sroze.

@fabpot fabpot merged commit c5c7a23 into symfony:3.4 Oct 29, 2017
fabpot added a commit that referenced this pull request Oct 29, 2017
…cache is not present (sroze)

This PR was merged into the 3.4 branch.

Discussion
----------

[FrameworkBundle] Do not activate the cache if Doctrine's cache is not present

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #24683
| License       | MIT
| Doc PR        | ø

Annotations are automatically enabled when the `doctrine/annotations` package is around. These annotations are enabled with the `php_array` cache by default, which require `doctrine/cache`.

This PR allow this scenario to happen by disabling the annotation cache when `doctrine/cache` is not present.

Commits
-------

c5c7a23 Do not activate the cache if Doctrine's cache is not present
@sroze sroze deleted the issue-24683-cache-none-if-no-doctrine-cache branch October 29, 2017 23:15
This was referenced Oct 30, 2017
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.

5 participants