[Goals]: fix repeating spend templates (#4066)

* fix repeating spend templates * work with all repeat types * note
actualbudget · Jan 1, 2025 · 7060e4b · 7060e4b
1 parent da613ab
commit 7060e4b
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 2 deletions.
diff --git a/packages/loot-core/src/server/budget/categoryTemplate.ts b/packages/loot-core/src/server/budget/categoryTemplate.ts
@@ -444,11 +444,24 @@ export class CategoryTemplate {
   }
 
   private async runSpend(template): Promise<number> {
-    const fromMonth = `${template.from}`;
-    const toMonth = `${template.month}`;
+    let fromMonth = `${template.from}`;
+    let toMonth = `${template.month}`;
     let alreadyBudgeted = this.fromLastMonth;
     let firstMonth = true;
 
+    //update months if needed
+    const repeat = template.annual
+      ? (template.repeat || 1) * 12
+      : template.repeat;
+    let m = monthUtils.differenceInCalendarMonths(toMonth, this.month);
+    if (repeat && m < 0) {
+      while (m < 0) {
+        toMonth = monthUtils.addMonths(toMonth, repeat);
+        fromMonth = monthUtils.addMonths(fromMonth, repeat);
+        m = monthUtils.differenceInCalendarMonths(toMonth, this.month);
+      }
+    }
+
     for (
       let m = fromMonth;
       monthUtils.differenceInCalendarMonths(this.month, m) > 0;

diff --git a/upcoming-release-notes/4066.md b/upcoming-release-notes/4066.md
@@ -0,0 +1,6 @@
+---
+category: Bugfix
+authors: [youngcw]
+---
+
+Fix spend template not repeating