Closed
Bug 550618
Opened 14 years ago
Closed 14 years ago
XML-RPC WebService returning wrong dateTime format
Categories
(Bugzilla :: WebService, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 3.6
People
(Reporter: mkanat, Assigned: mkanat)
References
Details
(Keywords: regression)
Attachments
(1 file)
2.27 KB,
patch
|
dkl
:
review+
|
Details | Diff | Splinter Review |
I forgot that the XML-RPC Specification requires that we send back something that really has nothing to do with ISO-8601: 19980717T14:08:55.
Flags: blocking3.6+
Assignee | ||
Comment 1•14 years ago
|
||
Okay, well, it's not *nothing to do* with ISO-8601, but it's the less-commonly-used format. In any case, we have to fix it. :-|
Assignee | ||
Comment 2•14 years ago
|
||
Okay, this fixes it. It's mostly just some re-architecture and then replacing the hyphens in the base implementation with nothing.
Assignee | ||
Comment 4•14 years ago
|
||
dkl: This is currently my only blocker on the 3.6rc1 release, so a speedy review would be appreciated.
Comment 5•14 years ago
|
||
Is this bug responsible for http://landfill.bugzilla.org/bugzillaqa/show_bug.cgi?id=3171 ?
Assignee | ||
Comment 6•14 years ago
|
||
(In reply to comment #5) > Is this bug responsible for > http://landfill.bugzilla.org/bugzillaqa/show_bug.cgi?id=3171 ? Yes.
Comment 7•14 years ago
|
||
Comment on attachment 430764 [details] [diff] [review] v1 Code looks good and returns proper date formats in my testing. r=dkl
Attachment #430764 -
Flags: review?(dkl) → review+
Comment 9•14 years ago
|
||
(In reply to comment #8) > Should this also be approved for 3.4? No, this is a regression from a patch I landed on 3.6, 3.4 should be good.
Assignee | ||
Updated•14 years ago
|
Flags: approval?
Flags: approval3.6?
Flags: approval3.6+
Flags: approval+
Assignee | ||
Comment 10•14 years ago
|
||
Thank you for the fast review, dkl! I know that it's the weekend, so I really appreciate it. Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/ modified Bugzilla/WebService.pm modified Bugzilla/WebService/Server.pm Committed revision 7057. modified Bugzilla/WebService.pm modified Bugzilla/WebService/Server.pm Committed revision 7024.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•