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

Can't :order by field defined with :ext_method #87

Open
rpaterson opened this issue Mar 26, 2014 · 3 comments
Open

Can't :order by field defined with :ext_method #87

rpaterson opened this issue Mar 26, 2014 · 3 comments

Comments

@rpaterson
Copy link

AFAICT it's not possible to correctly :order by a field defined with the :ext_method feature. Would be very useful to me to add this.

@wvanbergen
Copy link
Owner

I think this should not be too hard to add/fix. I have never used ext_method myself though.

@rpaterson
Copy link
Author

So :ext_method gets passed a key, an operator, and a value. To build the ORDER BY clause you have none of those things. I think this is going to require defining a different method e.g. :ext_order_method or something...

@wvanbergen
Copy link
Owner

I am willing to look into this if you can provide me a small code snippet with the expected behavior, or even better, a failing test case :).

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

No branches or pull requests

2 participants