Level 50
get method returns a collection. I think you should paginate it before calling get.
$files = $files->latest('created_at')->paginate();
May 15, 2019
7
Level 5
how to add pagination to this code
$files = File::where('campaign_id', $id);
if ($month = request('month')) {
$files->whereMonth('created_at', Carbon::parse($month)->month);
}
if ($year = request('year')) {
$files->whereYear('created_at', $year);
}
$files = $files->get()->sortByDesc('created_at');
I tried adding it on the first line but get
Too few arguments to function Illuminate\Support\Collection::get(), 0 passed, at least 1 expected
I tried adding it on the last line and get
Method Illuminate\Database\Eloquent\Collection::paginate does not exist.
Level 5
@ANDREICH1980 - if i do this
$files = $files->paginate(15)->get()->sortByDesc('created_at');
I get this error
Too few arguments to function Illuminate\Support\Collection::get(), 0 passed in and at least 1 expected
Level 47
@MICHALIS - you should copy @andreich1980 's code , don't add get() in it and don't change a thing , then magic
2 likes
Level 11
You should use @andreich1980's code. You cannot combine paginate() with get() as they both do something similar.
Level 50
@michalis there is an entire page on the Laravel documentation about pagination. See https://laravel.com/docs/5.8/pagination
Level 5
@ROBSTAR - I have, it did not mention that get() and paginate() do the same thing, or maybe I didnt understand
Level 50
@MICHALIS - Fair enough, but an example titled 'basic example' is at the top of the page and states exactly what you need to do - https://laravel.com/docs/5.8/pagination#basic-usage :)
Please or to participate in this conversation.