modules: fix get_all_pids

* Corrects `get_alll_pids` function limit offset search.

Co-Authored-by: Peter Weber <[email protected]>

rero_ils/modules/api.py

@@ -282,10 +282,10 @@ def get_all_pids(cls, with_deleted=False, limit=100000):
         """Get all records pids. Return a generator iterator."""
         query = cls._get_all(with_deleted=with_deleted)
         if limit:
+            count = query.count()
             # slower, less memory
             query = query.order_by(text('pid_value')).limit(limit)
             offset = 0
-            count = cls.count(with_deleted=with_deleted)
             while offset < count:
                 for identifier in query.offset(offset):
                     yield identifier.pid_value

rero_ils/modules/monitoring/api.py

@@ -155,10 +155,10 @@ def get_all_pids(cls, doc_type, with_deleted=False, limit=100000,
         if date:
             query = query.filter(PersistentIdentifier.created < date)
         if limit:
+            count = query.count()
             # slower, less memory
             query = query.order_by(text('pid_value')).limit(limit)
             offset = 0
-            count = query.count()
             while offset < count:
                 for identifier in query.offset(offset):
                     yield identifier.pid_value