Skip Menu |

This queue is for tickets about the Business-OnlinePayment-PPIPayMover CPAN distribution.

Report information
The Basics
Id: 105720
Status: patched
Priority: 0/
Queue: Business-OnlinePayment-PPIPayMover
People
Owner: ivan-pause [...] 420.am
Requestors: ether [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Tue Jul 07 13:48:17 2015 ether [...] cpan.org - Ticket created
Subject: incorrect VERSION sub
sub VERSION { "Perl Plug v1.8.0" } This is wrong. See perldoc UNIVERSAL. A VERSION sub should check the arguments that are passed in. This sub can just be removed, to make it behave properly.
  Tue Jul 07 14:20:38 2015 ivan-pause [...] 420.am - Correspondence added
On Tue Jul 07 13:48:17 2015, ETHER wrote: Show quoted text
> sub VERSION { "Perl Plug v1.8.0" } > > This is wrong. See perldoc UNIVERSAL. A VERSION sub should check the > arguments that are passed in. > > This sub can just be removed, to make it behave properly.
Well, its not quite that simple. It was converted over from some non-CPAN vendor code that used that VERSION() call for something else entirely (see TransactionClient.pm), so that needed to be renamed. Done in git, will be in 0.02. Thanks for the report! -- Ivan Kohler, President and Head Geek, Freeside Internet Services, Inc. Open-source billing, ticketing and provisioning - http://www.freeside.biz/
  Tue Jul 07 14:20:38 2015 The RT System itself - Status changed from 'new' to 'open'
  Tue Jul 07 14:20:45 2015 ivan-pause [...] 420.am - Taken
  Fri Sep 11 12:00:04 2015 ivan-pause [...] 420.am - Status changed from 'open' to 'patched'