feat: Apply timezone for supported allday-events

For allday events, the timezone is not used if defined. This works for moment-timezone objects and Luxon DateTime objects with a zone set. Close #592
sebbo2002 · Mar 31, 2024 · 176352c · 176352c
1 parent e65bbd3
commit 176352c
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/src/tools.ts b/src/tools.ts
@@ -54,13 +54,19 @@ export function formatDate (timezone: string | null, d: ICalDateTimeValue, dateo
     }
     else if(isMoment(d)) {
         // @see https://momentjs.com/timezone/docs/#/using-timezones/parsing-in-zone/
-        const m = timezone ? (isMomentTZ(d) && !d.tz() ? d.clone().tz(timezone) : d) : (floating ? d : d.utc());
+        const m = timezone
+            ? (isMomentTZ(d) && !d.tz() ? d.clone().tz(timezone) : d)
+            : (floating || (dateonly && isMomentTZ(d) && d.tz()) ? d : d.utc());
+
         return m.format('YYYYMMDD') + (!dateonly ? (
             'T' + m.format('HHmmss') + (floating || timezone ? '' : 'Z')
         ) : '');
     }
     else if(isLuxonDate(d)) {
-        const m = timezone ? d.setZone(timezone) : (floating ? d : d.setZone('utc'));
+        const m = timezone
+            ? d.setZone(timezone)
+            : (floating || (dateonly && d.zone.type !== 'system') ? d : d.setZone('utc'));
+
         return m.toFormat('yyyyLLdd') + (!dateonly ? (
             'T' + m.toFormat('HHmmss') + (floating || timezone ? '' : 'Z')
         ) : '');

diff --git a/src/types.ts b/src/types.ts
@@ -76,6 +76,7 @@ export interface ICalMomentDurationStub {
 
 export interface ICalLuxonDateTimeStub {
     setZone(zone?: string): ICalLuxonDateTimeStub;
+    zone: { type: string; };
     toFormat(fmt: string): string;
     toJSDate(): Date;
     get isValid(): boolean;