[Goals] Negate schedule amount to budget if income (#2125)

* Negate schedule amount to budget if income

* Release notes

* Determine sign at initial calc

To ensure no unintended impact to remainder calcs etc.

* Lint fixes

packages/loot-core/src/server/budget/goals/goalsSchedule.ts

@@ -17,6 +17,7 @@ export async function goalsSchedule(
   last_month_balance,
   to_budget,
   errors,
+  category,
 ) {
   if (!scheduleFlag) {
     scheduleFlag = true;
@@ -34,12 +35,15 @@ export async function goalsSchedule(
       const conditions = rule.serialize().conditions;
       const { date: dateConditions, amount: amountCondition } =
         extractScheduleConds(conditions);
+      const sign = category.is_income ? 1 : -1;
       const target =
         amountCondition.op === 'isbetween'
-          ? -Math.round(
-              amountCondition.value.num1 + amountCondition.value.num2,
-            ) / 2
-          : -amountCondition.value;
+          ? (sign *
+              Math.round(
+                amountCondition.value.num1 + amountCondition.value.num2,
+              )) /
+            2
+          : sign * amountCondition.value;
       const next_date_string = getNextDate(
         dateConditions,
         monthUtils._parse(current_month),

packages/loot-core/src/server/budget/goaltemplates.ts

@@ -590,6 +590,7 @@ async function applyCategoryTemplate(
           last_month_balance,
           to_budget,
           errors,
+          category,
         );
         to_budget = goalsReturn.to_budget;
         errors = goalsReturn.errors;

upcoming-release-notes/2125.md

+---
+category: Bugfix
+authors: [mk-french]
+---
+
+Goals: Negate schedule amount to budget if income