Drop scheduled dates as an option. #2751

Author: chmac

src/Config/Settings.ts

@@ -76,6 +76,7 @@ export interface Settings {
     filenameAsScheduledDateFormat: string;
     filenameAsDateFolders: string[];
     recurrenceOnNextLine: boolean;
+    dropScheduledDateOnRecurrence: boolean;
 
     // The custom status states.
     statusSettings: StatusSettings;
@@ -110,6 +111,7 @@ const defaultSettings: Settings = {
     filenameAsScheduledDateFormat: '',
     filenameAsDateFolders: [],
     recurrenceOnNextLine: false,
+    dropScheduledDateOnRecurrence: false,
     statusSettings: new StatusSettings(),
     features: Feature.settingsFlags,
     generalSettings: {

src/Config/SettingsTab.ts

@@ -376,6 +376,19 @@ export class SettingsTab extends PluginSettingTab {
                 });
             });
 
+        new Setting(containerEl)
+            .setName(i18n.t('settings.recurringTasks.dropScheduledDate.name'))
+            .setDesc(
+                SettingsTab.createFragmentWithHTML(i18n.t('settings.recurringTasks.dropScheduledDate.description')),
+            )
+            .addToggle((toggle) => {
+                const { dropScheduledDateOnRecurrence } = getSettings();
+                toggle.setValue(dropScheduledDateOnRecurrence).onChange(async (value) => {
+                    updateSettings({ dropScheduledDateOnRecurrence: value });
+                    await this.plugin.saveSettings();
+                });
+            });
+
         // ---------------------------------------------------------------------------
         new Setting(containerEl).setName(i18n.t('settings.autoSuggest.heading')).setHeading();
         // ---------------------------------------------------------------------------

src/Task/Occurrence.ts

@@ -88,7 +88,7 @@ export class Occurrence {
      *
      * @param nextReferenceDate
      */
-    public next(nextReferenceDate: Date): Occurrence {
+    public next(nextReferenceDate: Date, dropScheduledDate: boolean = false): Occurrence {
         // Only if a reference date is given. A reference date will exist if at
         // least one of the other dates is set.
         if (this.referenceDate === null) {
@@ -99,10 +99,14 @@ export class Occurrence {
             });
         }
 
+        const startDate = this.nextOccurrenceDate(this.startDate, nextReferenceDate);
+        const scheduledDate = dropScheduledDate ? null : this.nextOccurrenceDate(this.scheduledDate, nextReferenceDate);
+        const dueDate = this.nextOccurrenceDate(this.dueDate, nextReferenceDate);
+
         return new Occurrence({
-            startDate: this.nextOccurrenceDate(this.startDate, nextReferenceDate),
-            scheduledDate: this.nextOccurrenceDate(this.scheduledDate, nextReferenceDate),
-            dueDate: this.nextOccurrenceDate(this.dueDate, nextReferenceDate),
+            startDate,
+            scheduledDate,
+            dueDate,
         });
     }
 

src/Task/Recurrence.ts

@@ -73,14 +73,14 @@ export class Recurrence {
      *
      * @param today - Optional date representing the completion date. Defaults to today.
      */
-    public next(today = window.moment()): Occurrence | null {
+    public next(today = window.moment(), dropScheduledDate: boolean = false): Occurrence | null {
         const nextReferenceDate = this.nextReferenceDate(today);
 
         if (nextReferenceDate === null) {
             return null;
         }
 
-        return this.occurrence.next(nextReferenceDate);
+        return this.occurrence.next(nextReferenceDate, dropScheduledDate);
     }
 
     public identicalTo(other: Recurrence) {

src/Task/Task.ts

@@ -377,7 +377,8 @@ export class Task extends ListItem {
             return [toggledTask];
         }
 
-        const nextOccurrence = this.recurrence.next(today);
+        const { dropScheduledDateOnRecurrence } = getSettings();
+        const nextOccurrence = this.recurrence.next(today, dropScheduledDateOnRecurrence);
         if (nextOccurrence === null) {
             return [toggledTask];
         }

src/i18n/locales/en.json

@@ -180,6 +180,10 @@
       "nextLine": {
         "description": "Enabling this will make the next recurrence of a task appear on the line below the completed task. Otherwise the next recurrence will appear before the completed one.",
         "name": "Next recurrence appears on the line below"
+      },
+      "dropScheduledDate": {
+        "description": "Enabling this will make the next recurrence of a task have no scheduled date.",
+        "name": "Drop scheduled date on recurrence"
       }
     },
     "seeTheDocumentation": "See the documentation",

tests/Task/Recurrence.test.ts

@@ -228,3 +228,23 @@ describe('identicalTo', () => {
         expect(date1Recurrence?.identicalTo(date2Recurrence)).toBe(false);
     });
 });
+
+describe('Recurrence - with dropScheduledDateOnRecurrence', () => {
+    it('drops the scheduledDate when dropScheduledDate is true', () => {
+        // Arrange
+        const recurrence = Recurrence.fromText({
+            recurrenceRuleText: 'every month',
+            occurrence: new Occurrence({
+                startDate: moment('2022-01-01').startOf('day'),
+                scheduledDate: moment('2022-01-04').startOf('day'),
+                dueDate: moment('2022-01-10').startOf('day'),
+            }),
+        });
+
+        // Act
+        const next = recurrence!.next(undefined, true);
+
+        // Assert
+        expect(next!.scheduledDate).toBeNull();
+    });
+});