Add LengthAwarePaginator to QueryBuilder

[francoism90]

When using ->paginate() or ->jsonPaginate(), it seems it doesn't inject the appends when needed.
This should fix #531.

This does work, but please let me know if the code can improved. :)

[dominikb]

Looks good to me. Does it make sense to add a test for this?

[francoism90]

@dominikb Thanks for your review. :)

It would be great if this could be merged, this is the only showstopper for upgrading to L8.

[francoism90]

@freekmurze Sorry, but will this ever be merged/fixed? It seems I'm not the only one having this issue and I'm only waiting for this fix or I need to downgrade the package to v2.

[freekmurze]

@francoism90 please have a little patience. We'll handle this we have some time. There are a lot of other issues and client projects that require our attention as well.

@francoism90 can you add a test that is fixed by this PR. This will ensure that we don't break this behaviour again.

[francoism90]

@freekmurze Sorry, I do understand client project have priority and I'm glad Spatie is maintaining so many open-source projects. :)

@freekmurze I have added tests for collections and paginates, they do work (and fail when needed), but maybe this should be done in a better way. Please let me know!

[freekmurze]

Thanks for your work on this! 👍

[francoism90]

@freekmurze Thanks for the merge! 👍