Skip Menu |
 

This queue is for tickets about the RPC-XML CPAN distribution.

Report information
The Basics
Id: 71452
Status: resolved
Priority: 0/
Queue: RPC-XML

People
Owner: rjray [...] blackperl.com
Requestors: dougf [...] ferguson.cx
Cc:
AdminCc:

Bug Information
Severity: Critical
Broken in: 0.76
Fixed in: 0.77



Subject: RPC/XML.pm v1.56 line 577
Date: Tue, 4 Oct 2011 15:58:48 -0400 (EDT)
To: bug-RPC-XML [...] rt.cpan.org
From: Doug Ferguson <dougf [...] ferguson.cx>
Download (untitled) / with headers
text/plain 193b
now contains sub type { return 'dateTime.iso8601'; }; should probably contain sub type { return 'datetime_iso8601'; }; ...because RPC::XML::Procedure no longer recognizes the first form.
Download (untitled) / with headers
text/plain 338b
I must not have any tests that use dateTime.iso8601 in their signature, I'll have to check on that. The type() accessor cannot be changed, because it is used for XML generation. I'll have to handle this another way. First, I have to craft a test that triggers this issue. Randy -- Randy J. Ray rjray@blackperl.com randy.j.ray@gmail.com
Download (untitled) / with headers
text/plain 512b
Changing the title to reflect the actual problem here. The set-up of %VALID_TYPES in RPC/XML/Procedure.pm and the code in the call() method did not allow for proper handling of parameters or return values that were of the dateTime.iso8601 type. Additionally, there were no tests that exercised this. Tests have been added (to t/30_method.t, which is being renamed to t/30_procedure.t), and the code in Procedure.pm now handles these things correctly. -- Randy J. Ray rjray@blackperl.com randy.j.ray@gmail.com


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.