Prevent schedules with null amounts from crashing the app (#3958)

* test * note * add comment
actualbudget · Dec 18, 2024 · 4ce5e2f · 4ce5e2f
1 parent 11bde73
commit 4ce5e2f
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/packages/loot-core/src/shared/schedules.ts b/packages/loot-core/src/shared/schedules.ts
@@ -276,6 +276,10 @@ export function getScheduledAmount(
   amount: number | { num1: number; num2: number },
   inverse: boolean = false,
 ): number {
+  // this check is temporary, and required at the moment as a schedule rule
+  // allows the amount condition to be deleted which causes a crash
+  if (amount == null) return 0;
+
   if (typeof amount === 'number') {
     return inverse ? -amount : amount;
   }

diff --git a/upcoming-release-notes/3958.md b/upcoming-release-notes/3958.md
@@ -0,0 +1,6 @@
+---
+category: Bugfix
+authors: [matt-fidd]
+---
+
+Prevent schedules with null amounts from crashing the app