> The patch is supposed to support both formats, and indeed, it
> does seem to. The test also tests both formats.
>
> The problem is that another, apparenly unrelated test, now fails
> (with details below). I'm really not sure why this separate test
> fails now. If anyone can suggest anything to try after looking
> at the failure below, I'd love to hear it.
I read the ISO8601 3rd edition standard specification[1], from 2004-12-01, and added support for these features:
- reduce date format
- week format
- day of year format
- hours/minutes fractions
- optional date/time separators
- optional seconds/minutes
- timezones with only one digit for hour
Attached is the patch with tests for each feature. Additionally,
it fixes the failing test introduced by the initial work on this
issue.
Please review it and give me a '+1', so I can commit my first
Zope code contribution!
Status: Pending => Accepted
Supporters added: dtremea
Uploaded: DateTime- ISO8601. patch
Hey folks,
> The patch is supposed to support both formats, and indeed, it
> does seem to. The test also tests both formats.
>
> The problem is that another, apparenly unrelated test, now fails
> (with details below). I'm really not sure why this separate test
> fails now. If anyone can suggest anything to try after looking
> at the failure below, I'd love to hear it.
I read the ISO8601 3rd edition standard specification[1], from 2004-12-01, and added support for these features:
- reduce date format
- week format
- day of year format
- hours/minutes fractions
- optional date/time separators
- optional seconds/minutes
- timezones with only one digit for hour
Attached is the patch with tests for each feature. Additionally,
it fixes the failing test introduced by the initial work on this
issue.
Please review it and give me a '+1', so I can commit my first
Zope code contribution!
Any feedback is welcome.
[1] http:// www.omg. org/docs/ ISO-stds/ 06-08-01. pdf
PS: Don't be afraid of that 45 lines regex, it's just for the
sake of readability and understanding... :-)