preserveTimezones has no effect

[theonlypwner]

On version 10.20.0, preserveTimezones doesn't seem to work at all.

const { Liquid } = require('liquidjs')

new Liquid({ preserveTimezones: true, dateFormat: '%Y-%m-%d %H:%M:%S %z' }).parseAndRender(
  `{{ "2025-01-02 03:04:05 -0100" | date }}`
).then(console.log)

Actual output: (machine local timezone is UTC-7)
2025-01-01 21:04:05 -0700

Expected output:
2025-01-02 03:04:05 -0100

[harttle]

Please try 10.20.1, let me know if there's still any issue.

[theonlypwner]

It is working in 10.20.1 for my particular use case, but I noticed that TIMEZONE_PATTERN requires the minutes to be present, and I've seen some other people use formats that include only the timezone offset hours and imply 0 for minutes, such as this example using the same test script from my original post:

• 2025-01-02 03:04:05 -01 input
• 2025-01-01 21:04:05 -0700 actual output
• 2025-01-02 03:04:05 -0100 expected output

Also, the parsing doesn't seem to work if T is present, as in this example:

• 2025-01-02T03:04:05 -01:00 input
• 2025-01-02T03:04:05 -01:00 actual output
• 2025-01-02 03:04:05 -0100 expected output

However, it works if the space is removed, as with the input 2025-01-02T03:04:05-01:00 (actual output 2025-01-02 03:04:05 -0100 is expected).

[harttle]

We can support time zones without minutes. But not for cases when it’s not a valid init string for JavaScript Date. As T is required for ISO9601.

You can check whether it’s a LiquidJS issue or JavaScript issue by passing the string to ‘new Date’.