While fixing the simpletest.module I have encountered a strange behaviour of the xmlrpc system (breaking the validator1 compability).
In the function xmlrpc_date() there has recently [1][2][3] been added a iso8601 member variable (in a blogapi patch). However setting this does not consider the possibility of $time beeing a PHP timestamp.

Attached is a patch that makes the behaviour of ->iso8601 consistent (for HEAD).

[1] http://cvs.drupal.org/viewcvs/drupal/drupal/includes/xmlrpc.inc?r1=1.27&...
[2] http://drupal.org/files/issues/blogapi_HEAD.patch
[3] http://drupal.org/node/36079

CommentFileSizeAuthor
xmlrpc.iso8601.patch937 bytesThomas Ilsche
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drewish’s picture

drewish CreditAttribution: drewish commented

http://drupal.org/node/36079 is now marked as fixed...

Thomas Ilsche’s picture

Thomas Ilsche CreditAttribution: Thomas Ilsche commented

This problem is not with blogapi but with changes introduced to the xmlrpc libary while patching the blogapi.
Maybe I should have posted this as reply to this issue but I didn't want to complicate matters there.

chx’s picture

chx CreditAttribution: chx commented
Assigned: Unassigned » chx
Status: Needs review » Reviewed & tested by the community
Dries’s picture

Dries CreditAttribution: Dries commented
Status: Reviewed & tested by the community » Fixed

Committed to HEAD. Thank god we have some tests. ;-)

Anonymous’s picture

(not verified) CreditAttribution: commented
Status: Fixed » Closed (fixed)