Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Feb 24, 2023. It is now read-only.

Remove the @Template owner after kernel.view#404

Merged
fabpot merged 1 commit into
sensiolabs:masterfrom
linaori:fix/template-owner
Mar 1, 2016
Merged

Remove the @Template owner after kernel.view#404
fabpot merged 1 commit into
sensiolabs:masterfrom
linaori:fix/template-owner

Conversation

@linaori
Copy link
Copy Markdown
Contributor

@linaori linaori commented Mar 1, 2016

The issue is most likely that the owner is somehow cached or still referred at a point where it cannot be cleared properly. This will fix the reference but is not a permanent fix. This is something I can properly fix with symfony/symfony#17933.

Q A
Branch master
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #402
License MIT
Doc PR ~

@craue
Copy link
Copy Markdown
Contributor

craue commented Mar 1, 2016

Fixes #402 for me. 👍

@fabpot
Copy link
Copy Markdown
Member

fabpot commented Mar 1, 2016

Thank you @iltar.

@fabpot fabpot merged commit 0da6408 into sensiolabs:master Mar 1, 2016
fabpot added a commit that referenced this pull request Mar 1, 2016
This PR was merged into the 3.0.x-dev branch.

Discussion
----------

Remove the @template owner after kernel.view

The issue is most likely that the owner is somehow cached or still referred at a point where it cannot be cleared properly. This will fix the reference but is not a permanent fix. This is something I can properly fix with symfony/symfony#17933.

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

Commits
-------

0da6408 Remove the @template owner after kernel.view
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants