Feature/pdct 671 add order by for all gets for families and collections to be on last edited

app/repository/collection.py

@@ -4,7 +4,7 @@
 from datetime import datetime
 from typing import Optional, Tuple, Union, cast
 
-from sqlalchemy import Column, and_, or_
+from sqlalchemy import Column, and_, desc, or_
 from sqlalchemy import delete as db_delete
 from sqlalchemy import update as db_update
 from sqlalchemy.exc import NoResultFound, OperationalError
@@ -94,7 +94,7 @@ def all(db: Session) -> list[CollectionReadDTO]:
     :param db Session: the database connection
     :return Optional[CollectionResponse]: All of things
     """
-    collections = _get_query(db).all()
+    collections = _get_query(db).order_by(desc(Collection.last_modified)).all()
 
     if not collections:
         return []
@@ -145,7 +145,11 @@ def search(
     condition = and_(*search) if len(search) > 1 else search[0]
     try:
         found = (
-            _get_query(db).filter(condition).limit(query_params["max_results"]).all()
+            _get_query(db)
+            .filter(condition)
+            .order_by(desc(Collection.last_modified))
+            .limit(query_params["max_results"])
+            .all()
         )
     except OperationalError as e:
         if "canceling statement due to statement timeout" in str(e):

app/repository/document.py

@@ -137,6 +137,7 @@ def all(db: Session) -> list[DocumentReadDTO]:
     :param db Session: the database connection
     :return Optional[DocumentResponse]: All of things
     """
+    # TODO: PDCT-672 .Add ordering e.g., order_by(desc(FamilyDocument.last_modified))
     result = _get_query(db).all()
 
     if not result:
@@ -183,6 +184,7 @@ def search(
 
     condition = and_(*search) if len(search) > 1 else search[0]
     try:
+        # TODO: Fix order by on search PDCT-672
         result = (
             _get_query(db).filter(condition).limit(query_params["max_results"]).all()
         )

app/repository/family.py

@@ -4,7 +4,7 @@
 from datetime import datetime
 from typing import Optional, Tuple, Union, cast
 
-from sqlalchemy import Column, and_, or_
+from sqlalchemy import Column, and_, desc, or_
 from sqlalchemy import delete as db_delete
 from sqlalchemy import update as db_update
 from sqlalchemy.exc import NoResultFound, OperationalError
@@ -133,7 +133,7 @@ def all(db: Session) -> list[FamilyReadDTO]:
     :param db Session: the database connection
     :return Optional[FamilyResponse]: All of things
     """
-    family_geo_metas = _get_query(db).all()
+    family_geo_metas = _get_query(db).order_by(desc(Family.last_modified)).all()
 
     if not family_geo_metas:
         return []
@@ -203,7 +203,11 @@ def search(
     condition = and_(*search) if len(search) > 1 else search[0]
     try:
         found = (
-            _get_query(db).filter(condition).limit(query_params["max_results"]).all()
+            _get_query(db)
+            .filter(condition)
+            .order_by(desc(Family.last_modified))
+            .limit(query_params["max_results"])
+            .all()
         )
     except OperationalError as e:
         if "canceling statement due to statement timeout" in str(e):