Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Business-PayPal-API CPAN distribution.

Report information
The Basics
Id: 70424
Status: resolved
Priority: 0/
Queue: Business-PayPal-API

People
Owner: Nobody in particular
Requestors: rsimoes [...] cpan.org
Cc:
AdminCc:

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



Subject: Syntax deprecated in perl v5.14
rsimoes@desktop:~$ perl -v This is perl 5, version 14, subversion 1 (v5.14.1) built for x86_64-linux ... rsimoes@desktop:~$ perl -MBusiness::PayPal::API -e 1 Use of qw(...) as parentheses is deprecated at /home/rsimoes/opt/perlbrew/perls/perl- 5.14.1/lib/site_perl/5.14.1/Business/PayPal/API.pm line 246.
From: olaf [...] wundersolutions.com
On Mon Aug 22 16:27:42 2011, rsimoes wrote: Show quoted text
> > rsimoes@desktop:~$ perl -v > > This is perl 5, version 14, subversion 1 (v5.14.1) built for x86_64-linux > > ... > > rsimoes@desktop:~$ perl -MBusiness::PayPal::API -e 1 > Use of qw(...) as parentheses is deprecated at > /home/rsimoes/opt/perlbrew/perls/perl- > 5.14.1/lib/site_perl/5.14.1/Business/PayPal/API.pm line 246.
I'd also love to see this corrected. From perldiag: Use of qw(...) as parentheses is deprecated (D deprecated) You have something like foreach $x qw(a b c) {...} , using a qw(...) list literal where a parenthesised expression is expected. Historically the parser fooled itself into thinking that qw(...) literals were always enclosed in parentheses, and as a result you could sometimes omit parentheses around them. (You could never do the foreach qw(a b c) {...} that you might have expected, though.) The parser no longer lies to itself in this way. Wrap the list literal in parentheses, like foreach $x (qw(a b c)) {...} .
On Wed Sep 28 23:56:45 2011, OALDERS wrote: Show quoted text
> On Mon Aug 22 16:27:42 2011, rsimoes wrote:
> > > > rsimoes@desktop:~$ perl -v > > > > This is perl 5, version 14, subversion 1 (v5.14.1) built for x86_64-
> linux
> > > > ... > > > > rsimoes@desktop:~$ perl -MBusiness::PayPal::API -e 1 > > Use of qw(...) as parentheses is deprecated at > > /home/rsimoes/opt/perlbrew/perls/perl- > > 5.14.1/lib/site_perl/5.14.1/Business/PayPal/API.pm line 246.
> > I'd also love to see this corrected. From perldiag: > > Use of qw(...) as parentheses is deprecated > (D deprecated) You have something like foreach $x qw(a b c) {...} , > using a qw(...) list literal > where a parenthesised expression is expected. Historically the parser > fooled itself into > thinking that qw(...) literals were always enclosed in parentheses, > and as a result you could > sometimes omit parentheses around them. (You could never do the > foreach qw(a b c) {...} that > you might have expected, though.) The parser no longer lies to itself > in this way. Wrap the list > literal in parentheses, like foreach $x (qw(a b c)) {...} . >
A patched fork with the warning silenced can be found here: https://github.com/oalders/business-paypal-api
Subject: Re: [rt.cpan.org #70424] Syntax deprecated in perl v5.14
Date: Sun, 20 Nov 2011 18:20:46 -0800
To: bug-Business-PayPal-API [...] rt.cpan.org
From: danny hembree <danny-hembree [...] dynamical.org>
thanks On Sun, 2011-11-20 at 16:29 -0500, Olaf Alders via RT wrote: Show quoted text
> Queue: Business-PayPal-API > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=70424 > > > On Wed Sep 28 23:56:45 2011, OALDERS wrote:
> > On Mon Aug 22 16:27:42 2011, rsimoes wrote:
> > > > > > rsimoes@desktop:~$ perl -v > > > > > > This is perl 5, version 14, subversion 1 (v5.14.1) built for x86_64-
> > linux
> > > > > > ... > > > > > > rsimoes@desktop:~$ perl -MBusiness::PayPal::API -e 1 > > > Use of qw(...) as parentheses is deprecated at > > > /home/rsimoes/opt/perlbrew/perls/perl- > > > 5.14.1/lib/site_perl/5.14.1/Business/PayPal/API.pm line 246.
> > > > I'd also love to see this corrected. From perldiag: > > > > Use of qw(...) as parentheses is deprecated > > (D deprecated) You have something like foreach $x qw(a b c) {...} , > > using a qw(...) list literal > > where a parenthesised expression is expected. Historically the parser > > fooled itself into > > thinking that qw(...) literals were always enclosed in parentheses, > > and as a result you could > > sometimes omit parentheses around them. (You could never do the > > foreach qw(a b c) {...} that > > you might have expected, though.) The parser no longer lies to itself > > in this way. Wrap the list > > literal in parentheses, like foreach $x (qw(a b c)) {...} . > >
> > A patched fork with the warning silenced can be found here: > > https://github.com/oalders/business-paypal-api > >
Hi, I just wanted to give you a nudge about this. :) Would it be possible to get a new release with this patch? I find myself forced to use the patched module in some projects right now and that's less than optimal. If you have time constraints (which I *totally* respect), you could give me co-maint and I'll can do the release. I appreciate all of your work on this. Thanks! Olaf
Another polite nudge on this - we're using this code, and this warning is really annoying, yet really easy to fix with just another pair of parenthesis (as per the fix already shared by OALDERS), so a release with that fix in would be really handy...
Fixed in 0.70, which I've just uploaded. Forked repo is at https://github.com/oalders/business- paypal-api