Merge pull request #281 from simPod/getTimeZoneOffset

marnusw · web-flow · commit 3c5b9ab82541 · 2024-04-16T13:26:50.000+01:00
fix: make getTimeZoneOffset's 2nd arg optional
diff --git a/src/format/formatters/index.ts b/src/format/formatters/index.ts
@@ -97,10 +97,10 @@ export const formatters: Record<
   },
 }
 
-function getTimeZoneOffset(timeZone: string | undefined, originalDate: Date) {
+function getTimeZoneOffset(timeZone: string | undefined, originalDate?: Date) {
   const timeZoneOffset = timeZone
     ? tzParseTimezone(timeZone, originalDate, true) / MILLISECONDS_IN_MINUTE
-    : originalDate.getTimezoneOffset()
+    : (originalDate?.getTimezoneOffset() ?? 0)
   if (Number.isNaN(timeZoneOffset)) {
     throw new RangeError('Invalid time zone specified: ' + timeZone)
   }

Original file line number	Diff line number	Diff line change
`@@ -97,10 +97,10 @@ export const formatters: Record<`
`97`	`97`	`},`
`98`	`98`	`}`
`99`	`99`
`100`		`-function getTimeZoneOffset(timeZone: string \| undefined, originalDate: Date) {`
	`100`	`+function getTimeZoneOffset(timeZone: string \| undefined, originalDate?: Date) {`
`101`	`101`	`const timeZoneOffset = timeZone`
`102`	`102`	`? tzParseTimezone(timeZone, originalDate, true) / MILLISECONDS_IN_MINUTE`
`103`		`- : originalDate.getTimezoneOffset()`
	`103`	`+ : (originalDate?.getTimezoneOffset() ?? 0)`
`104`	`104`	`if (Number.isNaN(timeZoneOffset)) {`
`105`	`105`	`throw new RangeError('Invalid time zone specified: ' + timeZone)`
`106`	`106`	`}`