fix: correct priorityAscending/Descending order

jamiebrynes7 · Apr 6, 2024 · 269d2ed · 269d2ed
1 parent a671b85
commit 269d2ed
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/plugin/src/data/transformations/sorting.test.ts b/plugin/src/data/transformations/sorting.test.ts
@@ -63,7 +63,7 @@ describe("sortTasks", () => {
         makeTask("b", { priority: 1 }),
         makeTask("c", { priority: 4 }),
       ],
-      sortingOpts: [SortingVariant.PriorityDescending],
+      sortingOpts: [SortingVariant.PriorityAscending],
       expectedOutput: [
         makeTask("b", { priority: 1 }),
         makeTask("a", { priority: 2 }),

diff --git a/plugin/src/data/transformations/sorting.ts b/plugin/src/data/transformations/sorting.ts
@@ -26,7 +26,7 @@ function compareTask<T extends Task>(self: T, other: T, sorting: SortingVariant)
     case SortingVariant.Priority:
       // Note that priority in the API is reversed to that of in the app.
       return other.priority - self.priority;
-    case SortingVariant.PriorityDescending:
+    case SortingVariant.PriorityAscending:
       return self.priority - other.priority;
     case SortingVariant.Date:
       return compareTaskDate(self, other);

diff --git a/plugin/src/query/parser.ts b/plugin/src/query/parser.ts
@@ -195,8 +195,8 @@ function optionField<T>(
 
 const sortingLookup: Record<string, SortingVariant> = {
   priority: SortingVariant.Priority,
-  priorityAscending: SortingVariant.Priority,
-  priorityDescending: SortingVariant.PriorityDescending,
+  priorityAscending: SortingVariant.PriorityAscending,
+  priorityDescending: SortingVariant.Priority,
   date: SortingVariant.Date,
   dateAscending: SortingVariant.Date,
   dateDescending: SortingVariant.DateDescending,

diff --git a/plugin/src/query/query.ts b/plugin/src/query/query.ts
@@ -1,6 +1,6 @@
 export enum SortingVariant {
   Priority = 0,
-  PriorityDescending = 1,
+  PriorityAscending = 1,
   Date = 2,
   DateDescending = 3,
   Order = 4,