JiBX

Make JodaConvert.serializeZonedDateTime consistent

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Not A Bug
  • Affects Version/s: JiBX 1.2.2
  • Fix Version/s: None
  • Component/s: core
  • Labels:
    None
  • Number of attachments :
    0

Description

I would suggest that serializeZonedDateTime needs to be made consistent for all offsets.
Currently, an offset of 0 will result in exactly the same output as serializeUTCDateTime

DateTime dt = new DateTime().withZone(DateTimeZone.forOffsetHours(0));
System.out.println(JodaConvert.serializeUTCDateTime(dt));
System.out.println(JodaConvert.serializeZonedDateTime(dt));

this will result in

2010-04-16T09:26:40.42Z
2010-04-16T09:26:40.42Z

where I would have thought it should result in.

2010-04-16T09:26:40.42Z
2010-04-16T09:26:40.42+00:00

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Ben Davies added a comment -

Sorry, raised as "Major" by accident, but it is obviously not.

Show
Ben Davies added a comment - Sorry, raised as "Major" by accident, but it is obviously not.
Hide
Permalink
Dennis Sosnoski added a comment -

I started to implement this change, but after looking it over I'm not sure it's a good idea. The 'Z' offset is exactly equivalent to +/-00:00 in schema terms, and the 'Z' offset is both more compact and perhaps less confusing. I'm afraid that if I change to using +00:00 (or -00:00) I'll get complaints that input with 'Z' gets roundtripped with a different result.

Is there some reason you prefer to see the +00:00 offset instead of 'Z'?

Show
Dennis Sosnoski added a comment - I started to implement this change, but after looking it over I'm not sure it's a good idea. The 'Z' offset is exactly equivalent to +/-00:00 in schema terms, and the 'Z' offset is both more compact and perhaps less confusing. I'm afraid that if I change to using +00:00 (or -00:00) I'll get complaints that input with 'Z' gets roundtripped with a different result. Is there some reason you prefer to see the +00:00 offset instead of 'Z'?
Hide
Permalink
Ben Davies added a comment -

Dennis,

Fair enough.
Your way is actually how Joda does it too, so maybe I was to hasty with this suggestion.
Feel free to close.

Cheers

Show
Ben Davies added a comment - Dennis, Fair enough. Your way is actually how Joda does it too, so maybe I was to hasty with this suggestion. Feel free to close. Cheers

People

  • Assignee:
    Dennis Sosnoski
    Reporter:
    Ben Davies
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: