-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Fix default value for page in Model::paginate() #3194
Conversation
@michalsn pagination not working now. |
Apparently there is another bug with |
Thank you @michalsn . Pagination links() method also have a problem.
It gives me a error, because links first Argument must be a string null given. Is it possible to give null value on links method both Arguments ? |
@mmrtonmoybd instead of |
@michalsn do you fixed paginate show results with segment bug ? |
@mmrtonmoybd Yes, I hope it's fixed. The tests fail because of a random test failure. |
Well done @michalsn |
@mmrtonmoybd I was only waiting for you to confirm that it's working fine. Thanks for your feedback! |
@michalsn array(3) { [0]=> array(3) { ["uri"]=> string(42) "http://localhost:8080/category/exclusive/3" ["title"]=> int(3) ["active"]=> bool(false) } [1]=> array(3) { ["uri"]=> string(42) "http://localhost:8080/category/exclusive/4" ["title"]=> int(4) ["active"]=> bool(false) } [2]=> array(3) { ["uri"]=> string(42) "http://localhost:8080/category/exclusive/5" ["title"]=> int(5) ["active"]=> bool(false) } } |
Description
This PR fixes the default value for the
$page
parameter in Model::paginate().Ref: #3188
Checklist: