Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Net-XMPP CPAN distribution.

Report information
The Basics
Id: 58949
Status: patched
Priority: 4/
Queue: Net-XMPP

People
Owner: dapatrick [...] cpan.org
Requestors: tlawton [...] gmx.de
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 1.02
Fixed in: (no value)



Subject: Bug in Net::XMPP::Stanza with fix
Date: Tue, 29 Jun 2010 18:25:24 +0100
To: bug-Net-XMPP [...] rt.cpan.org
From: Tom Lawton <tlawton [...] gmx.de>
Hi, Spent ages trying to work out why my jabber bot couldn't spot delay tags, until I found the bug in Stanza.pm _parse_tree. I've enclosed the fixed version of the function below (only the one line indicated has changed). thanks, tom sub _parse_tree { my $self = shift; my @xTrees = $self->{TREE}->XPath('*[@xmlns]'); if ($#xTrees > -1) { foreach my $xTree (@xTrees) { if( exists($Net::XMPP::Namespaces::NS{$xTree- Show quoted text
>get_attrib("xmlns")}))
### TL *** Changed above line to say xTree rather than xTrees[0] { $self->AddChild($xTree); $self->{TREE}->remove_child($xTree); } } } }
Hi Tom, Could you elaborate on the symptoms of your problem and also give steps to reproduce? I'd like to have a full idea of the issue and how do document it in the changelog once it is fixed. Thanks, Darian
Stalling until I hear back from tlawton, or replicate this issue myself.
This has been fixed in https://github.com/dap/Net-XMPP/commit/a0224e2b2c4a46b53a2591c0f369d63ae580d592 and will be included in dev release 1.05_01. Darian