LocalDate.ToString does not provide calendar information about nodatime

[GoogleCodeExporter]

Original comment by jonathan.skeet on 17 Mar 2012 at 9:28

• Added labels: V1-OutOfScope

[GoogleCodeExporter]

Original comment by malcolm.rowe on 30 Jul 2012 at 7:47

• Added labels: Milestone-1.1-consider
• Removed labels: V1-OutOfScope

[GoogleCodeExporter]

We added the 'r' format specifier to LocalDateTime in revision a7722264b63c, 
which includes the calendar system.  From what I can see, there's no 'o' or 'r' 
format for LocalDate.

LocalDateTime.ToString() currently uses 'G' to format by default, which is 
culture-sensitive; LocalDate currently uses 'D'.  I'm not entirely sure what 
the expectations are for ToString() in .NET: whether it's intended to be a 
debugging output, or a 'reasonable' human-readable output; if the latter, 
changing to the roundtrip format (for LocalDateTime) would probably be 
unreasonable, but perhaps suffixing " (c)" for non-ISO calendars would be okay.

[ToString()'s documentation is "the value of the current instance in the 
standard format pattern"; I don't know what "standard" format pattern refers to 
in this case, or whether it would be considered a breaking change to modify it 
in 1.1.]

Original comment by malcolm.rowe on 9 Nov 2012 at 7:49

[GoogleCodeExporter]

Per discussion: we don't really want to change the default (parameterless) 
format used by ToString(), just to provide a roundtrip standard format.

So the only thing left to do is to create a roundtrip pattern for LocalDate (we 
don't currently have one), and expose it/them via the 'o'/'O'/'r' standard 
pattern.

This probably partly depends upon issue 138: consistent naming for standard 
patterns.

Original comment by malcolm.rowe on 10 Nov 2012 at 4:57

• Now blocked on: #138

[GoogleCodeExporter]

Original comment by malcolm.rowe on 10 Nov 2012 at 4:58

• Changed title: LocalDate.ToString does provide calendar information
• Added labels: Type-Enhancement

[GoogleCodeExporter]

Original comment by malcolm.rowe on 10 Nov 2012 at 4:58

• Changed title: LocalDate.ToString does not provide calendar information

[GoogleCodeExporter]

Original comment by malcolm.rowe on 25 Jan 2013 at 8:52

• Added labels: Milestone-1.2.0
• Removed labels: Milestone-1.1-consider

[GoogleCodeExporter]

I suspect that all of our types will want a "roundtrip assuming I'm using the 
right calendar" format, and a "roundtrip by specifying the calendar" format. 
Use of non-Gregorian calendars is going to be pretty rare, I suspect.

Original comment by jonathan.skeet on 9 Apr 2013 at 7:35

[GoogleCodeExporter]

We now have the c specifier for all appropriate types.

Original comment by jonathan.skeet on 26 Jul 2013 at 6:33

• Changed state: Fixed