Mohamedsabil83\LaravelLoadmore\Loadmore

Complexity

Total Complexity

4

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	loadmore()	0	16	3
A	scopeLoadmore()	0	4	1

<?php

namespace Mohamedsabil83\LaravelLoadmore;

use Illuminate\Pagination\Paginator;
use Illuminate\Pagination\LengthAwarePaginator;

trait Loadmore
{
    /**
     * Prepare a loadmore pagination.
     *
     * @param integer $initial
     * @param integer $loadMore
     * @return \Illuminate\Pagination\LengthAwarePaginator
     */
    public function loadmore($initial = 4, $loadMore = 16)
    {
        $page = (int) Paginator::resolveCurrentPage();
        $perPage = ($page == 1) ? $initial : $loadMore;
        $skip = ($page == 1) ? 0 : ($initial + ($loadMore * ($page - 2)));
        $items = $this->skip($skip)->take($perPage)->get();

It seems like skip() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $total = $this->count() + $loadMore - $initial;

It seems like count() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return new LengthAwarePaginator(
            $items,
            $total,
            $loadMore,
            Paginator::resolveCurrentPage(),
            ['path' => Paginator::resolveCurrentPath()]
        );
    }

    /**
     * Scope a query to match loadmore style.
     *
     * @param \Illuminate\Database\Eloquent\Builder $query
     * @param integer $initial
     * @param integer $loadMore
     * @return \Illuminate\Database\Eloquent\Builder
     */
    public function scopeLoadmore($query, $initial = 4, $loadMore = 16)

The parameter $query is not used and could be removed.

    {
        return $this->loadmore($initial, $loadMore);
    }
}