[FrameworkBundle] Fix annotations cache folder path #21552

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
@akeeman
Contributor

akeeman commented Feb 7, 2017

Q A
Branch? 3.2
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? n/a
Fixed tickets n/a
License MIT
Doc PR n/a

The argument at index 2 sets a cache path as it extends the file system adapter by default since e59f0e0, where it did set the version before that. This change now results in annotation cache folders showing up in the project root (because of the overwritten cache root folder).
Because cache.annotations has a parent now, this line is not needed anymore and fixes the problem.

Fix annotations cache folder path
Argument 2 sets a cache path since e59f0e0

@akeeman akeeman changed the title from Fix annotations cache folder path to [FrameworkBundle] Fix annotations cache folder path Feb 7, 2017

@nicolas-grekas nicolas-grekas added this to the 3.2 milestone Feb 7, 2017

@nicolas-grekas

This comment has been minimized.

Show comment
Hide comment
@nicolas-grekas

nicolas-grekas Feb 7, 2017

Member

Thank you @akeeman.

Member

nicolas-grekas commented Feb 7, 2017

Thank you @akeeman.

nicolas-grekas added a commit that referenced this pull request Feb 7, 2017

bug #21552 [FrameworkBundle] Fix annotations cache folder path (akeeman)
This PR was submitted for the master branch but it was merged into the 3.2 branch instead (closes #21552).

Discussion
----------

[FrameworkBundle] Fix annotations cache folder path

| Q             | A
| ------------- | ---
| Branch?       |  3.2
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | n/a
| Fixed tickets | n/a
| License       | MIT
| Doc PR        | n/a

The argument at index 2 sets a cache path as it extends the file system adapter by default since e59f0e0, where it did set the version before that. This change now results in annotation cache folders showing up in the project root (because of the overwritten cache root folder).
Because cache.annotations has a parent now, this line is not needed anymore and fixes the problem.

Commits
-------

8e5cfa7 Fix annotations cache folder path

@fabpot fabpot referenced this pull request Feb 17, 2017

Merged

Release v3.2.4 #21640

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment