Fix: No longer crash when toggling a recurring task without any date (o…

…bsidian-tasks-group#597)

* Added a default case to handle when no due, scheduled, or start date is supplied.

* added test for default recurrence case

* Removed null check

* referenceDate is no longer nullable

src/Recurrence.ts

@@ -21,7 +21,7 @@ export class Recurrence {
      * same relative distance to the due date as the original task. For example
      * "starts one week before it is due".
      */
-    private readonly referenceDate: Moment | null;
+    private readonly referenceDate: Moment;
 
     constructor({
         rrule,
@@ -33,7 +33,7 @@ export class Recurrence {
     }: {
         rrule: RRule;
         baseOnToday: boolean;
-        referenceDate: Moment | null;
+        referenceDate: Moment;
         startDate: Moment | null;
         scheduledDate: Moment | null;
         dueDate: Moment | null;
@@ -80,9 +80,11 @@ export class Recurrence {
                     referenceDate = window.moment(scheduledDate);
                 } else if (startDate) {
                     referenceDate = window.moment(startDate);
+                } else {
+                    referenceDate = window.moment();
                 }
 
-                if (!baseOnToday && referenceDate !== null) {
+                if (!baseOnToday) {
                     options.dtstart = window
                         .moment(referenceDate)
                         .startOf('day')

tests/Task.test.ts

@@ -361,6 +361,9 @@ describe('toggle done', () => {
             nextScheduled: '2021-10-18',
             nextDue: '2021-10-20',
         },
+        {
+            interval: 'every week',
+        },
     ];
 
     test.concurrent.each<RecurrenceCase>(recurrenceCases)(