Skip Menu |

This queue is for tickets about the SOAP-Lite CPAN distribution.

Report information
The Basics
Id: 34514
Status: resolved
Priority: 0/
Queue: SOAP-Lite

People
Owner: Nobody in particular
Requestors: mkanat [...] cpan.org
Cc: nuabaranda [...] web.de
AdminCc:

Bug Information
Severity: Important
Broken in: 0.71
Fixed in: (no value)



Subject: Subclasses of XMLRPC::Serializer misbehave because of a "ref" call in SOAP::Lite
I have to monkey-patch XMLRPC::Serializer to make it work right with utf8, for Bugzilla. ( See https://bugzilla.mozilla.org/show_bug.cgi?id=415796 ) In doing this, I fixed the bug, but I found that I was getting *SOAP* arrays returned in my XML responses instead of XML-RPC arrays. The offending code is here, in SOAP::Serializer: elsif (UNIVERSAL::isa($object => 'ARRAY')) { # Added in SOAP::Lite 0.65_6 to fix an XMLRPC bug return $self->encodingStyle eq "" || ref $self eq 'XMLRPC::Serializer' ? $self->encode_array($object, $name, $type, $attr) : $self->encode_literal_array($object, $name, $type, $attr); } That "ref $self" should be "$self->isa" instead.
Hi Max, This has already been fixed in SOAP::Lite's SVN. you can get a fixed version from https://soaplite.svn.sourceforge.net/svnroot/soaplite/branches/0.71 Thanks for reporting, Martin On Fr. 28. Mär. 2008, 20:33:09, MKANAT wrote: Show quoted text
> I have to monkey-patch XMLRPC::Serializer to make it work right with > utf8, for Bugzilla. ( See > https://bugzilla.mozilla.org/show_bug.cgi?id=415796 ) > > In doing this, I fixed the bug, but I found that I was getting *SOAP* > arrays returned in my XML responses instead of XML-RPC arrays. The > offending code is here, in SOAP::Serializer: > > elsif (UNIVERSAL::isa($object => 'ARRAY')) { > # Added in SOAP::Lite 0.65_6 to fix an XMLRPC bug > return $self->encodingStyle eq "" > || ref $self eq 'XMLRPC::Serializer' > ? $self->encode_array($object, $name, $type, $attr) > : $self->encode_literal_array($object, $name, $type,
$attr); Show quoted text
> } > > That "ref $self" should be "$self->isa" instead.
Fixed in 0.71.01 and .02 Thanks for reporting, Martin