Closed
Bug 550618
Opened 15 years ago
Closed 15 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•15 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•15 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•15 years ago
|
||
dkl: This is currently my only blocker on the 3.6rc1 release, so a speedy review would be appreciated.
Comment 5•15 years ago
|
||
Is this bug responsible for http://landfill.bugzilla.org/bugzillaqa/show_bug.cgi?id=3171 ?
Assignee | ||
Comment 6•15 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•15 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•15 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•15 years ago
|
Flags: approval?
Flags: approval3.6?
Flags: approval3.6+
Flags: approval+
Assignee | ||
Comment 10•15 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: 15 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•