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.

Sign up for GitHub

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

Jump to bottom

Fix profiler query explaining without the templating component #709

Merged
merged 1 commit into from
Oct 4, 2017
Merged

Fix profiler query explaining without the templating component #709

merged 1 commit into from
Oct 4, 2017

Conversation

stof
Copy link
Member

@stof stof commented Oct 2, 2017

Fixes #707

Should be backported to 1.6.x too.

@stof stof mentioned this pull request Oct 2, 2017
Closed
@stof stof requested a review from fabpot October 2, 2017 14:20
@fabpot fabpot merged commit 1085c1b into doctrine:master Oct 4, 2017
@ArrowStrike
Copy link

ArrowStrike commented Mar 14, 2018 •
edited
Loading

We have the same problem in "doctrine/doctrine-bundle": "^1.6".
And this guy fix it perfectly
https://github.com/ssaki/DoctrineBundle/commit/8e91477358a3814c7b3c979e645adc88014c6ed2
using Symfony\Component\VarDumper\Cloner\Data;
Please, can you add this to Doctrine?

@goetas
Copy link
Member

goetas commented Mar 29, 2018 •
edited
Loading

Same issue with doctrine/doctrine-bundle 1.8.x and the latest sf 3.4, the solution by ssaki/DoctrineBundle@8e91477 worked pretty well.

@kimhemsoe
Copy link
Member

@goetas Can you open a new issue and how to reproduce this. I want to spam this issue more and this sounds like a different issue.

@kimhemsoe
Copy link
Member

Reported here #757

@stof stof deleted the fix_profiler branch April 20, 2018 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jwage jwage jwage approved these changes

@fabpot fabpot fabpot approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@stof @ArrowStrike @goetas @kimhemsoe @fabpot @jwage