Skip Menu |

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

Report information
The Basics
Id: 55162
Status: resolved
Priority: 0/
Queue: XML-XSLT
People
Owner: Nobody in particular
Requestors: tammer [...] ilukacg.com
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Tue Mar 02 23:01:45 2010 tammer [...] ilukacg.com - Ticket created
Subject: bug w/solution, V0.48
Date: Tue, 02 Mar 2010 22:59:49 -0500
To: undisclosed-recipients:;
From: Tammer Kamel <tammer [...] ilukacg.com>
Hi: I think line 3215 needs to be: elsif ( $lhs =~ /^([\w\.\:\-\/]+)$/ ) NOT elsif ( $lhs =~ /^([\w\.\:\-]+)$/ ) in other words, routine needs to accept "/" character as well as what is currently there. without this fix, an xsl:test will only accept an element as a lhv, but not a path. attached is the original version 0.48 (prefixed with "o") and the modified version. best Tammer.

Message body is not shown because sender requested not to inline it.

Message body is not shown because sender requested not to inline it.

  Sun Oct 27 10:59:45 2013 jns [...] gellyfish.co.uk - Correspondence added
On Tue Mar 02 23:01:45 2010, tammer@ilukacg.com wrote: Show quoted text
> Hi: > > I think line 3215 needs to be: > > elsif ( $lhs =~ /^([\w\.\:\-\/]+)$/ ) > > NOT > > elsif ( $lhs =~ /^([\w\.\:\-]+)$/ ) > > > in other words, routine needs to accept "/" character as well as what is > currently there. > > without this fix, an xsl:test will only accept an element as a lhv, but > not a path. > > attached is the original version 0.48 (prefixed with "o") and the > modified version. > > best > Tammer. >
Hi, I've added that as https://github.com/jonathanstowe/XML-XSLT/commit/133f1b96236e61fed4fbae3917e891fee23fdf6e it should be in the next release. If it doesn't work for you please submit the appropriate failing test. Thanks.
  Sun Oct 27 10:59:45 2013 The RT System itself - Status changed from 'new' to 'open'
  Sun Oct 27 10:59:45 2013 jns [...] gellyfish.co.uk - Status changed from 'open' to 'resolved'