acquisitions: sort lines in order

Line sorting is now done on the PID to keep the same order as the command. * Closes #2653. Co-Authored-by: Bertrand Zuchuat <[email protected]>
rero · Sep 20, 2023 · e7e2f87 · e7e2f87
1 parent 770dc5e
commit e7e2f87
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/rero_ils/modules/acquisition/acq_orders/api.py b/rero_ils/modules/acquisition/acq_orders/api.py
@@ -357,9 +357,14 @@ def get_order_lines(self, output=None, includes=None):
         if output == 'count':
             return query.count()
         elif output == 'query':
-            return query
+            return query \
+                .params(preserve_order=True) \
+                .sort({'pid': {"order": "asc"}})
         else:
-            return get_objects(AcqOrderLine, query)
+            return get_objects(AcqOrderLine, query
+                               .params(preserve_order=True)
+                               .sort({'pid': {"order": "asc"}})
+                               )
 
     def get_order_provisional_total_amount(self):
         """Get provisional total amount of this order."""