Missing timezone in description of ISO 8601 date

Description

Observed

The `valid_iso8601_date` function of the `Time_Definitions` class in the `BASE-foundation_types.time` package doesn't mention that a timezone may optionally part of an ISO 8601 date value.

And the `Iso8601_date` class in the same package doesn't include the syntax of the optional timezone part of the date value in its description.

Expected

The description of the `valid_iso8601_date` function and the `Iso8601_date` class should include the optional timezone part of ISO 8601 date values.

References

- org.openehr.base.foundation_types.time_definitions.adoc#L139-L153
- org.openehr.base.foundation_types.iso8601_date.adoc#L8-L15
- https://www.joda.org/joda-time/apidocs/org/joda/time/format/ISODateTimeFormat.html#dateParser--
- Timezone of ISO 8601 Date | discourse.openehr.org

Activity

Sebastian Iancu April 5, 2024 at 9:06 PM
Edited

for references, the specs link for that function is

in order to analyse this further, the discourse discussion mentioned above is important, as well as this answer on

if we would choose to do this and allow tz with dates, I added fix-version BASE 1.3.0

Details

Reporter

Components

Affects versions

Priority

Created August 16, 2023 at 8:54 PM
Updated January 10, 2025 at 9:24 AM