Skip Menu |

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

Report information
The Basics
Id: 7872
Status: resolved
Priority: 0/
Queue: XML-XSLT

People
Owner: Nobody in particular
Requestors: andy_bach [...] wiwb.usourts.gov
Cc:
AdminCc:

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



Subject: XML-XSLT 0.48 examples/agenda warnings
Running the included examples/agenda project w/ the included xslt-parser throw out the errors/warnings: Use of uninitialized value in concatenation (.) or string at /usr/local/lib/perl5/site_perl/5.8.0/XML/XSLT.pm line 3212. The problem is sub __evaluate_test__ is matching against undef vars as: if ($test =~ /^\s*(\S+?)\s*(<=|>=|!=|<|>|=)\s*['"]?([^'"]*?)['"]?\s*$/) { $lhs = $1; $test_cond = $2; $expval = $3; } $self->debug("Test LHS: $lhs"); the match against $test fails. Added an else (standard diff): 3208c3208,3211 < } --- Show quoted text
> } else { > $self->debug("no match for test"); > return ""; > }
Which is what happens anyways, after 2 failed matches against the undefined $lhs a one if/elsif/else later. a
[guest - Mon Oct 4 22:17:30 2004]: Show quoted text
> Running the included examples/agenda project w/ the included xslt- > parser throw out the errors/warnings: > Use of uninitialized value in concatenation (.) or string at > /usr/local/lib/perl5/site_perl/5.8.0/XML/XSLT.pm line 3212. > > > The problem is sub __evaluate_test__ is matching against undef vars > as: > if ($test =~ > /^\s*(\S+?)\s*(<=|>=|!=|<|>|=)\s*['"]?([^'"]*?)['"]?\s*$/) > { > $lhs = $1; > $test_cond = $2; > $expval = $3; > } > $self->debug("Test LHS: $lhs"); > > the match against $test fails. Added an else (standard diff): > 3208c3208,3211 > < } > ---
> > } else { > > $self->debug("no match for test"); > > return ""; > > }
> > Which is what happens anyways, after 2 failed matches against the > undefined $lhs a one if/elsif/else later.
Thanks for this, I have applied the change manually (The CVS version has changed since the last release) and the new version can be found in the CVS at sourceforge. There are still some outstanding bugs that require fixing before I can make a new release. /J\