Skip Menu |

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

Report information
The Basics
Id: 112470
Status: resolved
Priority: 0/
Queue: XML-LibXML
People
Owner: Nobody in particular
Requestors: the.rob.dixon [...] gmail.com
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Sat Feb 27 03:58:41 2016 the.rob.dixon [...] gmail.com - Ticket created
Subject: The attributes method of an XML::LibXML::Text node in list context wrongly returns (undef)
Date: Sat, 27 Feb 2016 08:58:32 +0000
To: bug-XML-LibXML [...] rt.cpan.org
From: Rob Dixon <the.rob.dixon [...] gmail.com>
A call like my @attr = $node->attributes will set `@attr` to contain a single element with the value `undef`. This should be an empty list. The fix is simple. The method, on line 1778 of LibXML.pm, should be changed from sub attributes { return undef; } to sub attributes { return } Thanks, Rob Dixon Norfolk England
  Sat Feb 27 05:54:13 2016 SHLOMIF [...] cpan.org - Correspondence added
On Sat Feb 27 03:58:41 2016, the.rob.dixon@gmail.com wrote: Show quoted text
> A call like > > my @attr = $node->attributes > > will set `@attr` to contain a single element with the value `undef`. This > should be an empty list. > > The fix is simple. The method, on line 1778 of LibXML.pm, should be changed > from > > sub attributes { return undef; } > > to > > sub attributes { return } > > Thanks, > > Rob Dixon > Norfolk > England
Thanks for the report! I'll look into it.
  Sat Feb 27 05:54:14 2016 The RT System itself - Status changed from 'new' to 'open'
  Sat Feb 27 06:24:18 2016 SHLOMIF [...] cpan.org - Correspondence added
On Sat Feb 27 05:54:13 2016, SHLOMIF wrote: Show quoted text
> On Sat Feb 27 03:58:41 2016, the.rob.dixon@gmail.com wrote:
> > A call like > > > > my @attr = $node->attributes > > > > will set `@attr` to contain a single element with the value `undef`. > > This > > should be an empty list. > > > > The fix is simple. The method, on line 1778 of LibXML.pm, should be > > changed > > from > > > > sub attributes { return undef; } > > > > to > > > > sub attributes { return } > > > > Thanks, > > > > Rob Dixon > > Norfolk > > England
> > Thanks for the report! I'll look into it.
Fixed in 2.0124 (with a test). Thanks!
  Sat Feb 27 06:24:26 2016 SHLOMIF [...] cpan.org - Status changed from 'open' to 'resolved'