Paginator class is missing (L5)

Level 6

You're in the wrong namespace i guess, it should be

\Illuminate\Pagination\Paginator::make(....

orosznyet

Level 1

Ops, I forgot to mention that I tried that as well but then I got the following problem:

Call to undefined method Illuminate\Pagination\Paginator::make()

Moe

Level 6

public function products(Paginator $paginator)
{
    $products = [];
    return $paginator->make($products, count($products), '10');
}

adnan

Level 1

Pagination class Laravel 5 not have make method.

What laravel version is used?

Level 56

L5 :

$paginator = new Illuminate\Pagination\LengthAwarePaginator($items, $total, $per_page);

Best Answer

Level 88

A more advanced sample:

// Get all items you want
$items = Item::all();

// Get the current page from the url if it's not set default to 1
$page = Input::get('page', 1); 

// Number of items per page
$perPage = 10;

// Start displaying items from this number;
$offSet = ($page * $perPage) - $perPage; // Start displaying items from this number

// Get only the items you need using array_slice (only get 10 items since that's what you need)
$itemsForCurrentPage = array_slice($items->toArray(), $offSet, $perPage, true);

// Return the paginator with only 10 items but with the count of all items and set the it on the correct page
return new LengthAwarePaginator($itemsForCurrentPage, count($items), $perPage, $page);

If you have something like filters on your site or other parameters in your url you will lose them when you start paginating, you fix this you by adding this:

return new LengthAwarePaginator($items, count($items), $perPage, $page, ['path' => Request::url(), 'query' => Request::query()]);

And you can just render the the pagination like this

{!! $items->render() !!}

2 likes

Level 56

@bobbybouwmann It think it is wrong the purpose of paginator class is to handle this logic. If you pass it the total number of items, the items per page, and the current page all this will be calculated by the paginator.

Level 88

@pmall Mmmh you might be right but I just wanted to show a way where you only pass the items you will be displaying anyway instead of passing everything!

Level 56

@bobbybouwmann I think the point is to pass everything and let the paginator handle the logic. But I may be wrong your post makes me doubt about it now lol

Level 88

Hehe :P Well I had some problems with getting the correct values from the pagination object since I was creating the paginator by myself.. I came up with this and it's works fine for me :P

The paginate function from the query builder is doing the same thing except for the array_slice part

public function paginate($perPage = null, $columns = ['*'])
{
    $total = $this->query->getCountForPagination();

    $this->query->forPage(
        $page = Pagination::resolveCurrentPage(),
        $perPage = $perPage ?: $this->model->getPerPage()
    );

    return new LengthAwarePaginator($this->get($columns), $total, $perPage, $page, [
        'path' => Paginator::resolveCurrentPath(),
    ]);
}

Level 56

Yes it is normal it only query the requested items but I was thinking if the number of items was higher than the per page value the paginator would display the correct ones by itself.

Level 88

You have to specify that..

Level 56

@bobbybouwmann ok I guess you are right. I'm disappointed by the paginator :)

Level 88

@pmall Hehe, I already accepted the paginator as it is ;)

orosznyet