-
Notifications
You must be signed in to change notification settings - Fork 351
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
Add DoctrinePaginatorAdapter to interface with Doctrine pagination #362
Add DoctrinePaginatorAdapter to interface with Doctrine pagination #362
Conversation
* @param callable $routeGenerator | ||
* | ||
*/ | ||
public function __construct(Paginator $paginator, $routeGenerator) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
type hint callable here?
* | ||
* @return callable | ||
*/ | ||
public function getRouteGenerator() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a need to get
this outside of this class?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great, a few changes requested
All fair points, agreed. Unfortunately it is required to bring in the 'doctrine/orm' package to allow the tests to run but it is only require-dev. EDIT: Ah looks like you removed the comment regarding the 'doctrine/orm' package. I seen it in email and didn't cross reference to here |
Yeah, I commented before realizing it was require-dev. You're all good! Thank you! |
No description provided.