Closed Bug 550618 Opened 14 years ago Closed 14 years ago

XML-RPC WebService returning wrong dateTime format

Categories

(Bugzilla :: WebService, defect)

3.5.3
defect
Not set
critical

Tracking

()

RESOLVED FIXED
Bugzilla 3.6

People

(Reporter: mkanat, Assigned: mkanat)

References

Details

(Keywords: regression)

Attachments

(1 file)

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+
  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. :-|
Attached patch v1Splinter Review
Okay, this fixes it. It's mostly just some re-architecture and then replacing the hyphens in the base implementation with nothing.
Assignee: webservice → mkanat
Status: NEW → ASSIGNED
Attachment #430764 - Flags: review?(dkl)
...sorry :(
Depends on: 545299
Keywords: regression
  dkl: This is currently my only blocker on the 3.6rc1 release, so a speedy review would be appreciated.
Blocks: 550732
(In reply to comment #5)
> Is this bug responsible for
> http://landfill.bugzilla.org/bugzillaqa/show_bug.cgi?id=3171 ?

  Yes.
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+
Should this also be approved for 3.4?
Flags: approval?
Flags: approval3.6?
(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.
Flags: approval?
Flags: approval3.6?
Flags: approval3.6+
Flags: approval+
  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.

Attachment

General

Created:
Updated:
Size: