When parsing times, minutes should be less than 60

Issue #16 resolved
Brandon Nielsen repo owner created an issue

Currently, parsing times with 60 minutes is allowed:

>>> aniso8601.parse_time('01:60')
datetime.time(2, 0)
>>> aniso8601.parse_time('0160')
datetime.time(2, 0)

From section 4.2 of the spec: "minute is represented by two digits from [00] to [59]"

Comments (1)

  1. Brandon Nielsen reporter

    Fix minute range check in _parse_minute_time.

    Remove redundant range check in _parse_second_time, as minutes >= 60 are not allowed by the datetime.time constructor.

    This fixes issue #16.

    → <<cset c93610d406c0>>

  2. Log in to comment