Bug #68932 for XML-XPath: /usr/bin/xpath outputs unwanted text when quiet mode ist set

Sun Jun 19 08:28:52 2011 1983-01-06 [...] gmx.net - Ticket created

Subject:	/usr/bin/xpath outputs unwanted text when quiet mode ist set
Date:	Sun, 19 Jun 2011 14:28:41 +0200
To:	bug-XML-XPath [...] rt.cpan.org
From:	Michael-O <1983-01-06 [...] gmx.net>

If I perform a non nodset action it simply outputs: Query didn't return a nodeset. Value: <value> Even if I set -q and still prints that line on stderr. This is annoying. In quiet mode it should spit out the value only. A small patch fixes the issue: --- /usr/bin/xpath 2004-10-29 00:51:18.000000000 +0200 +++ /usr/bin/xpath.new 2011-06-19 14:23:07.000000000 +0200 @@ -98,7 +98,7 @@ } } else { - print STDERR "Query didn't return a nodeset. Value: "; + print STDERR "Query didn't return a nodeset. Value: " unless $quiet; print $nodes->value, "\n"; } Mike

Fri Nov 22 07:46:57 2013 bwagner [...] cpan.org - Correspondence added

I had to add a few lines to make Mike's patch work. It's a trivial change that requires the -q option to come first if at all. On Sun Jun 19 08:28:52 2011, 1983-01-06@gmx.net wrote: Show quoted text

> If I perform a non nodset action it simply outputs: > Query didn't return a nodeset. Value: <value> > > Even if I set -q and still prints that line on stderr. This is annoying. > In quiet mode it should spit out the value only. > > A small patch fixes the issue: > --- /usr/bin/xpath 2004-10-29 00:51:18.000000000 +0200 > +++ /usr/bin/xpath.new 2011-06-19 14:23:07.000000000 +0200 > @@ -98,7 +98,7 @@ > } > } > else { > - print STDERR "Query didn't return a nodeset. Value: "; > + print STDERR "Query didn't return a nodeset. Value: " > unless $quiet; > print $nodes->value, "\n"; > } > > > Mike > >

Subject:

xpath.patch

--- xpath 2013-11-22 13:37:31.000000000 +0100 +++ xpath.new 2013-11-22 13:37:21.000000000 +0100 @@ -20,12 +20,18 @@ my $xpath; my $pipeline; +my $quiet; +if ($ARGV[0] eq '-q') { + # quiet mode + $quiet = 1; + shift @ARGV; +} if ($ARGV[0] eq '-p') { # pipeline mode $pipeline = 1; shift @ARGV; } if (@ARGV >= 2) { $xpath = XML::XPath->new(filename => shift(@ARGV)); } @@ -37,7 +43,8 @@ unless ($nodes->isa('XML::XPath::NodeSet')) { NOTNODES: - print STDERR "Query didn't return a nodeset. Value: "; + print STDERR "Query didn't return a nodeset. Value: " + unless $quiet; print $nodes->value, "\n"; exit; }

Fri Nov 22 07:46:58 2013 The RT System itself - Status changed from 'new' to 'open'

Fri Nov 22 07:46:59 2013 bwagner [...] cpan.org - Fixed in 1.13 added

Fri Nov 22 07:49:11 2013 bwagner [...] cpan.org - Correspondence added

On Fri Nov 22 07:46:57 2013, BWAGNER wrote: Show quoted text

> I had to add a few lines to make Mike's patch work. > It's a trivial change that requires the -q option to come first if at all. > > On Sun Jun 19 08:28:52 2011, 1983-01-06@gmx.net wrote:

> > If I perform a non nodset action it simply outputs: > > Query didn't return a nodeset. Value: <value> > > > > Even if I set -q and still prints that line on stderr. This is annoying. > > In quiet mode it should spit out the value only. > > > > A small patch fixes the issue: > > --- /usr/bin/xpath 2004-10-29 00:51:18.000000000 +0200 > > +++ /usr/bin/xpath.new 2011-06-19 14:23:07.000000000 +0200 > > @@ -98,7 +98,7 @@ > > } > > } > > else { > > - print STDERR "Query didn't return a nodeset. Value: "; > > + print STDERR "Query didn't return a nodeset. Value: " > > unless $quiet; > > print $nodes->value, "\n"; > > } > > > > > > Mike > > > >

> >

Fri Nov 22 07:49:12 2013 bwagner [...] cpan.org - Fixed in 1.13 deleted

Fri Mar 04 23:33:02 2016 MANWAR [...] cpan.org - Correspondence added

Hi, Thanks for reporting the issue and patch. Much appreciated. I will try my best to patch it asap. Best Regards, Mohammad S Anwar

Fri Mar 04 23:33:03 2016 MANWAR [...] cpan.org - Taken

Fri Mar 04 23:33:30 2016 MANWAR [...] cpan.org - Severity Normal added

Fri Mar 04 23:33:31 2016 MANWAR [...] cpan.org - Broken in 1.33 added

Thu Apr 14 12:10:32 2016 MANWAR [...] cpan.org - Correspondence added

Resolved.

Thu Apr 14 12:10:34 2016 MANWAR [...] cpan.org - Status changed from 'open' to 'resolved'

Thu Apr 14 12:10:34 2016 MANWAR [...] cpan.org - Fixed in 1.36 added