fix(fields): fix issue with nil params in date-related functions [CLK-509330]

package.json

@@ -1,6 +1,6 @@
 {
     "name": "@time-loop/hot-formula-parser",
-    "version": "4.1.2",
+    "version": "4.2.0",
     "description": "Formula parser",
     "type": "commonjs",
     "main": "dist/index.js",

src/clickup/formulajsProxy.ts

@@ -0,0 +1,52 @@
+import * as formulajs from '@formulajs/formulajs';
+
+const isNil = (...args: unknown[]) =>
+    args.some((arg) => arg === null || arg === undefined || arg === '' || arg === false);
+const nullToZero = (arg: unknown) => (isNil(arg) ? 0 : arg);
+
+const overrides = {
+    DATE: (year: unknown, month: unknown, day: unknown) =>
+        isNil(year, month, day) ? Number.NaN : formulajs.DATE(year, month, day),
+    DATEVALUE: (dateText: unknown) => (isNil(dateText) ? Number.NaN : formulajs.DATEVALUE(dateText)),
+    DAY: (date: unknown) => (isNil(date) ? Number.NaN : formulajs.DAY(date)),
+    DAYS: (startDate: unknown, endDate: unknown) =>
+        isNil(startDate, endDate) ? Number.NaN : formulajs.DAYS(startDate, endDate),
+    DAYS360: (startDate: unknown, endDate: unknown, method: unknown) =>
+        isNil(startDate, endDate) ? Number.NaN : formulajs.DAYS360(startDate, endDate, method),
+    EDATE: (startDate: unknown, months: unknown) =>
+        isNil(startDate) ? Number.NaN : formulajs.EDATE(startDate, nullToZero(months)),
+    EOMONTH: (startDate: unknown, months: unknown) =>
+        isNil(startDate) ? Number.NaN : formulajs.EOMONTH(startDate, nullToZero(months)),
+    HOUR: (date: unknown) => (isNil(date) ? Number.NaN : formulajs.HOUR(date)),
+    INTERVAL: (seconds: unknown) => formulajs.INTERVAL(nullToZero(seconds)),
+    ISOWEEKNUM: (date: unknown) => (isNil(date) ? Number.NaN : formulajs.ISOWEEKNUM(date)),
+    MINUTE: (serialNumber: unknown) => (isNil(serialNumber) ? Number.NaN : formulajs.MINUTE(serialNumber)),
+    MONTH: (date: unknown) => (isNil(date) ? Number.NaN : formulajs.MONTH(date)),
+    SECOND: (serialNumber: unknown) => (isNil(serialNumber) ? Number.NaN : formulajs.SECOND(serialNumber)),
+    TIME: (hour: unknown, minute: unknown, second: unknown) =>
+        isNil(hour, minute, second) ? Number.NaN : formulajs.TIME(hour, minute, second),
+    TIMEVALUE: (timeText: unknown) => (isNil(timeText) ? Number.NaN : formulajs.TIMEVALUE(timeText)),
+    WEEKDAY: (serialNumber: unknown, returnType: unknown) =>
+        isNil(serialNumber) ? Number.NaN : formulajs.WEEKDAY(serialNumber, returnType),
+    WEEKNUM: (serialNumber: unknown, returnType: unknown) =>
+        isNil(serialNumber) ? Number.NaN : formulajs.WEEKNUM(serialNumber, returnType),
+    YEAR: (date: unknown) => (isNil(date) ? Number.NaN : formulajs.YEAR(date)),
+    YEARFRAC: (startDate: unknown, endDate: unknown, basis: unknown) =>
+        isNil(startDate, endDate) ? Number.NaN : formulajs.YEARFRAC(startDate, endDate, basis),
+    WORKDAY: (startDate: unknown, days: unknown, holidays: unknown) =>
+        isNil(startDate) ? Number.NaN : formulajs.WORKDAY(startDate, nullToZero(days), holidays),
+    NETWORKDAYS: (startDate: unknown, endDate: unknown, holidays: unknown) =>
+        isNil(startDate, endDate) ? Number.NaN : formulajs.NETWORKDAYS(startDate, endDate, holidays),
+};
+
+export const formulajsProxy = new Proxy(formulajs, {
+    get: (target, prop) => {
+        if (prop in overrides) {
+            return overrides[prop as keyof typeof overrides];
+        }
+        if (prop in target) {
+            return target[prop];
+        }
+        return null;
+    },
+});

src/evaluate-by-operator/operator/formula-function.js

@@ -1,4 +1,4 @@
-import * as formulajs from '@formulajs/formulajs';
+import { formulajsProxy as formulajs } from '../../clickup/formulajsProxy';
 import SUPPORTED_FORMULAS from '../../supported-formulas';
 import { ERROR_NAME } from '../../error';
 

src/helper/number.js

@@ -1,4 +1,4 @@
-import * as formulajs from '@formulajs/formulajs';
+import { formulajsProxy as formulajs } from '../clickup/formulajsProxy';
 import splitFormula from './formula';
 import { getNumberOfDaysSinceEpoch, isDate } from './date';