Fix: duplicated entries returned by API if pagination offset > total …

…number of elements
philipflohr · Jan 14, 2021 · aa801a7 · aa801a7
1 parent 9de540f
commit aa801a7
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/personio_py/client.py b/src/personio_py/client.py
@@ -169,7 +169,10 @@ def request_paginated(
             resp_data = response['data']
             if resp_data:
                 data_acc.extend(resp_data)
-                params['offset'] += len(resp_data)
+                if response['metadata']['current_page'] == response['metadata']['total_pages']:
+                    break
+                else:
+                    params['offset'] += len(resp_data)
             else:
                 break
         # return the accumulated data