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

Doctrine2 support #3

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Doctrine2 support #3

wants to merge 3 commits into from

Conversation

sndpl
Copy link

@sndpl sndpl commented Oct 19, 2011

Hi,

This is a doctrine2 plugin for doctrine 2.1 and some small fixes in the Database adapter.

Regards,

Sandy

@fuhrysteve
Copy link

Nice! This works for me.

I've added a little documentation for it and tweaked a few things that I'd like to try and merge in once this pull is accepted.

see: https://github.com/MontmereLimited/ZFDebug

@fuhrysteve
Copy link

Got tired of reading those long unindented doctrine queries, so i built in support for sqlparse (http://code.google.com/p/python-sqlparse/)

It is disabled by default since most people won't have it installed, but it can be can be enabled like this

ZFDebug_Controller_Plugin_Debug_Plugin_Doctrine2::$_sqlParseEnabled = true;

see: https://github.com/MontmereLimited/ZFDebug/tree/sqlparse

I'll do an additional pull request for that once this is accepted.

@dextervip
Copy link

+1
It would be nice to support Doctrine 2

@fuhrysteve
Copy link

@dextervip Checkout my fork, it supports Doctrine 2:

https://github.com/MontmereLimited/ZFDebug

@dextervip
Copy link

@fuhrysteve Does it work as well for ODM?

@fuhrysteve
Copy link

I haven't tested it, but it might. Never used ODM.

If it doesn't, just model it after this, shouldn't take much time.

https://github.com/MontmereLimited/ZFDebug/blob/master/library/ZFDebug/Controller/Plugin/Debug/Plugin/Doctrine2.php

@iurijacob
Copy link

Nice work, Steve. Awesome plugin!

@holtkamp
Copy link

+1 bump

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants