Bug 15654 – SysTime.toISOString formats the time zones incorrectly

The ISO Extended format for time zones other than UTC is +hh:mm, -hh:mm, +hh, or -hh. ISO non-Extended is the same but without the colons. However, currently SysTime uses colons for non-Extended (it correctly doesn't put the dashes or colons in the date or time portion, but it still puts the colons in the time zone portion). So, we need to fix it so that toISOString does not use colons in the time zone, and fromISOString should not accept them either, though I think that we're going to have to accept them for a while in case existing code has been writing them out with toISOString and saving them to later read in with fromISOString.

Commits pushed to master at https://github.com/D-Programming-Language/phobos https://github.com/D-Programming-Language/phobos/commit/7ded5fcf686c988c4d26869ac0e16a1fd0e5a446 Rename SimpleTimeZone.to/fromISOString to to/fromISOExtString. This is per issue# 15654. https://github.com/D-Programming-Language/phobos/commit/2567a2493a168343be6be3050cd4bef90b320f58 Fix issue# 15654: SysTime.toISOString formats time zones incorrectly. Previously, it formatted the time zone the same as toISOExtString and put a colon between the hours and minutes - which is correct for the ISO extended format, but not the non-extended format. Now, it formats it correctly per the ISO 9601 standard. fromISOString temporarily accepts both the extended and non-extended formats to avoid breaking any programs that have been writing out ISO strings to read back in later, but eventually, we'll make it so that it only accepts the non-extended format like it's supposed to.

Commits pushed to stable at https://github.com/D-Programming-Language/phobos https://github.com/D-Programming-Language/phobos/commit/7ded5fcf686c988c4d26869ac0e16a1fd0e5a446 Rename SimpleTimeZone.to/fromISOString to to/fromISOExtString. https://github.com/D-Programming-Language/phobos/commit/2567a2493a168343be6be3050cd4bef90b320f58 Fix issue# 15654: SysTime.toISOString formats time zones incorrectly.

It looks like I screwed up and only fixed the fromISOString portion of this so that it accepted the correct type without actually fixing what was emitted.

Could you provide an example? I'm currently not sure if this is fixed or still a bug or not an issue anymore, because when I use toISOString, no timezone is printed at all...

THIS ISSUE HAS BEEN MOVED TO GITHUB https://github.com/dlang/phobos/issues/9674 DO NOT COMMENT HERE ANYMORE, NOBODY WILL SEE IT, THIS ISSUE HAS BEEN MOVED TO GITHUB