add offset parameter to report.get

Author: dakra

CHANGES.txt

@@ -3,6 +3,7 @@
 
 - fix: setup.py not working under some environments (`open` used wrong codec)
 - add SellerProfile model
+- add `offset` parameter to report.get
 
 
 1.0

atomx/__init__.py

@@ -216,7 +216,7 @@ def report_status(self, report):
             raise APIError(r.json()['error'])
         return r.json()['report']
 
-    def report_get(self, report, limit=None, sort=None):
+    def report_get(self, report, sort=None, limit=None, offset=None):
         """Get the content (csv) of a :class:`.models.Report`
 
         Typically used by calling :meth:`.models.Report.content` or
@@ -235,6 +235,8 @@ def report_get(self, report, limit=None, sort=None):
         params = {}
         if limit:
             params['limit'] = int(limit)
+        if offset:
+            params['offset'] = int(offset)
         if sort:
             params['sort'] = sort
 

atomx/models.py

@@ -207,18 +207,19 @@ def status(self):
         self.__init__(session=self.session, **status)
         return self
 
-    def get(self, limit=None, sort=None):
+    def get(self, sort=None, limit=None, offset=None):
         """Get the first ``limit`` lines of the report ``content``
         and in the specified ``sort`` order.
 
-        :param int limit: limit the amount of lines to return (defaults to no limit)
         :param str sort: defines the sort order of the report content.
             ``sort`` can be `column_name`[.asc|.desc][,column_name[.asc|.desc]]`...
+        :param int limit: limit the amount of lines to return (defaults to no limit)
+        :param int offset: Skip the first `offset` number of lines (defaults to none)
         :return: report content
         """
         if not self.is_ready:
             raise ReportNotReadyError()
-        return self.session.report_get(self, limit=limit, sort=sort)
+        return self.session.report_get(self, sort=sort, limit=limit, offset=offset)
 
     @property
     def content(self):