time_from_isodate should not require seconds to be specified

[Tollef Fog Heen]

From the description of time_from_isodate:

 * time_from_isodate:
 * @str: Date/time value in ISO 8601 format.
 *
 * Converts an ISO 8601 UTC time string into a time_t value.

However, it doesn't handle times, only dates:

With
#include <stdio.h>
#include <libecal/e-cal-time-util.h>

int main(int argc, char **argv) {
    printf("%d %s\n", time_from_isodate(argv[1]), argv[1]);
}

as our sample program, compiled using:

$ gcc $(pkg-config --libs --cflags libecal-1.2) test.c -o testt

We then get:

: tfheen@thosu /tmp > ./testt 20050601T010
-1 20050601T010
: tfheen@thosu /tmp > ./testt 20050601T010Z
-1 20050601T010Z
: tfheen@thosu /tmp >

[Tollef Fog Heen (tfheen)]

Actually, this bug is bogus, since I forgot to include the seconds.

I think that time_from_isodate should be better at parsing the input, though and not require seconds.

[Sebastien Bacher (seb128)]

I've forwarded that upstream as a feature request, I would not be surprised if they reject it though since they have already a lot of work on real bugs for the moment and that doesn't look at really feature since it's easy enough to use the full format: http://bugzilla.gnome.org/show_bug.cgi?id=349770