darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-02-03 01:41:05 -05:00
Code Issues Packages Projects Releases Wiki Activity

fix first paginate method

Browse Source
This commit is contained in:
Kenneth Kehl
2024-10-18 09:05:03 -07:00
parent 78aab59d15
commit 3b651a2716
1 changed files with 2 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
app/dao/inbound_sms_dao.py
View File

@@ -48,20 +48,13 @@ def dao_get_paginated_inbound_sms_for_service_for_public_api(
older_than_created_at = ( older_than_created_at = (
db.session.query(InboundSms.created_at) db.session.query(InboundSms.created_at)
.filter(InboundSms.id == older_than) .filter(InboundSms.id == older_than)
.as_scalar() .scalar_subquery()
) )
filters.append(InboundSms.created_at < older_than_created_at) filters.append(InboundSms.created_at < older_than_created_at)
# As part of the move to sqlalchemy 2.0, we do this manual pagination # As part of the move to sqlalchemy 2.0, we do this manual pagination
# 1.4 had a paginate() method which assumed 'page' was 1 if it was not specified,
# so we set page to 1 here to mimic that.
page = 1
query = db.session.query(InboundSms).filter(*filters) query = db.session.query(InboundSms).filter(*filters)
paginated_items = ( paginated_items = query.order_by(desc(InboundSms.created_at)).limit(page_size).all()
query.order_by(desc(InboundSms.created_at))
.offset((page - 1) * page_size)
.limit(page_size)
)
return paginated_items return paginated_items