Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Query didn't use property without arguments to sort. #12

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Contributor

kkamkou commented May 17, 2011

not sure that this solution is correct, but for me it works.
Situation: I have the comments table and need to sort entries by date created.
Please, review this solution.

Owner

a-musing-moose commented May 23, 2011

Hey kkamkou,

Thanks for the fix and additional test, really appreciated. Looks like you forked from the PHP 5.2 branch - this makes it a little trickier to merge. I may have to make the amendments manually but I'll see what I can do.

Thanks,
Jon

Owner

a-musing-moose commented May 26, 2011

This has now been merge into the php5.2 branch and the same ideas into master.

Thanks for the fix and extra points for the unit test update!

Contributor

kkamkou commented May 26, 2011

Thank you, Jonathan.

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