fix(db): ledger updates sort order (#441)

* Fix ledger updates sort order * Allow more sort strings
iotaledger · Jul 15, 2022 · df0786d · df0786d
1 parent bf36dc4
commit df0786d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 8 deletions.
diff --git a/bin/inx-chronicle/src/api/stardust/history/extractors.rs b/bin/inx-chronicle/src/api/stardust/history/extractors.rs
@@ -29,8 +29,8 @@ impl FromStr for Sort {
 
     fn from_str(s: &str) -> Result<Self, Self::Err> {
         match s {
-            "asc" => Ok(Self::Ascending),
-            "desc" => Ok(Self::Descending),
+            "asc" | "oldest" => Ok(Self::Ascending),
+            "desc" | "newest" => Ok(Self::Descending),
             _ => Err(ParseError::BadSortDescriptor),
         }
     }

diff --git a/src/db/collections/ledger_update.rs b/src/db/collections/ledger_update.rs
@@ -187,20 +187,20 @@ impl MongoDb {
         if let Some(output_id) = start_output_id {
             match order {
                 SortOrder::Newest => {
-                    filter.insert("output_id", doc! { "$lte": output_id });
+                    filter.insert("output_id", doc! { "$gte": output_id });
                 }
                 SortOrder::Oldest => {
-                    filter.insert("output_id", doc! { "$gte": output_id });
+                    filter.insert("output_id", doc! { "$lte": output_id });
                 }
             }
         }
 
         let options = FindOptions::builder()
             .limit(page_size as i64)
             .sort(if order.is_newest() {
-                inverse_ledger_index()
-            } else {
                 ledger_index()
+            } else {
+                inverse_ledger_index()
             })
             .build();
 
@@ -251,9 +251,9 @@ impl MongoDb {
         let options = FindOptions::builder()
             .limit(page_size as i64)
             .sort(if order.is_newest() {
-                inverse_ledger_index()
-            } else {
                 ledger_index()
+            } else {
+                inverse_ledger_index()
             })
             .build();