IMPROVEMENT Add existing query params to pagination links (#30)

* Add existing query params to pagingation links * Add appends method to mock object for passing tests
ellipsesynergie · Jul 14, 2016 · 74d0641 · 74d0641
1 parent 3b16343
commit 74d0641
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/src/Laravel/Response.php b/src/Laravel/Response.php
@@ -41,6 +41,9 @@ public function withArray(array $array, array $headers = [])
      */
     public function withPaginator(LengthAwarePaginator $paginator, $transformer, $resourceKey = null, $meta = [])
     {
+        $queryParams = array_diff_key($_GET, array_flip(['page']));
+        $paginator->appends($queryParams);
+
         $resource = new Collection($paginator->items(), $transformer, $resourceKey);
         $resource->setPaginator(new IlluminatePaginatorAdapter($paginator));
 

diff --git a/tests/Laravel/ResponseTest.php b/tests/Laravel/ResponseTest.php
@@ -35,6 +35,7 @@ public function testWithPaginatorWorkProperly()
         $paginator->shouldReceive('perPage')->andReturn(1);
         $paginator->shouldReceive('url')->andReturn('localhost');
         $paginator->shouldReceive('count')->andReturn(3);
+        $paginator->shouldReceive('appends')->andReturn([['foo' => 'bar']]);
 
         $response = new ResponseFake(new Manager());