Re: Trouble with xmlrpc

On Tue, Feb 20, 2007 at 10:53:20PM +0900, James Edward Gray II wrote:
> Below is a patch for this issue.
> 
> CHANGELOG:
> 
> Adding DateTime support to xmlrpc.

Which DateTime are you trying to support?

> Index: lib/xmlrpc/create.rb
> ===================================================================
> --- lib/xmlrpc/create.rb	(revision 11805)
> +++ lib/xmlrpc/create.rb	(working copy)
> @@ -241,7 +241,7 @@
>  	    @writer.ele("data", *a)
>  	  )
>  
> -	when Time, Date
> +	when Time, Date, DateTime
                     ^^^^^^^^ Perhaps you mean ::DateTime?
>  	  @writer.tag("dateTime.iso8601", param.strftime("%Y%m%dT%H:%M:%S"))  
>  
>  	when XMLRPC::DateTime
         ^^^^^^^^^^^^^^^^ because XMLRPC::DateTime is handled already.

Btw, it is possible the existence of XMLRPC::DateTime pre-dates the
existence of ::DateTime? It doesn't appear to do anything that the more
commonly used DateTime doesn't do, maybe xmlrpc/datetime.rb should be
removed on 1.9 branch?

And I still think the maintainer should consider using ISO8601 for
dateTime.iso8601... :-)

Sam