Fix for pagination ‘last’ link to give the real last page (#353)

* Fix for pagination ‘last’ link to give the real last page

* Updated test to support new pagination ‘last’ link change

* Added myself to AUTHORS

Author: yanivp

AUTHORS

@@ -4,4 +4,4 @@ Greg Aker <greg@gregaker.net>
 Jerel Unruh <mail@unruhdesigns.com>
 Matt Layman <http://www.mattlayman.com>
 Oliver Sauder <os@esite.ch>
-
+Yaniv Peer <yanivpeer@gmail.com>

example/tests/unit/test_pagination.py

@@ -41,7 +41,7 @@ def test_valid_offset_limit(self):
         offset = 10
         limit = 5
         count = len(self.queryset)
-        last_offset = count - limit
+        last_offset = (count // limit) * limit
         next_offset = 15
         prev_offset = 5
 

rest_framework_json_api/pagination.py

@@ -64,7 +64,7 @@ def get_last_link(self):
         url = self.request.build_absolute_uri()
         url = replace_query_param(url, self.limit_query_param, self.limit)
 
-        offset = self.count - self.limit
+        offset = (self.count // self.limit) * self.limit
 
         if offset <= 0:
             return remove_query_param(url, self.offset_query_param)