-
Notifications
You must be signed in to change notification settings - Fork 101
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
Use Paginator in QueryObject::fetchOne() #118
Comments
👍 need this :-) |
@enumag a pullrequest would be nice :) |
@fprochazka I know, I'd love to send it but am currently too busy. :-( |
@fprochazka I wanted to write a failing test at least but it seems there are currently no tests for the QueryObject class. |
Please remove setMaxResults(1) in next release :) |
I think we've learned that this was a bad idea. |
@fprochazka Can you elaborate? I think I missed that lesson. ;-) |
@enumag if it's not obviouse from #174 (comment) tell me and I'll elaborate |
@fprochazka Well we never actually tried using Paginator for this. |
@fprochazka :-D |
and get rid of
setMaxResults(1)
The text was updated successfully, but these errors were encountered: