Skip Menu |

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

Report information
The Basics
Id: 22765
Status: resolved
Priority: 0/
Queue: Business-OnlinePayment-AuthorizeNet
People
Owner: Nobody in particular
Requestors: WMCKEE [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in:
  • 3.15
  • 3.00
  • 3.01
  • 3.10
  • 3.11
  • 3.12
  • 3.13
  • 3.14
Fixed in:
  • 3.16
  • 3.17

History Show all quoted text
  Thu Nov 02 15:00:07 2006 WMCKEE [...] cpan.org - Ticket created
Subject: Clarify source of error_message and result_code
Ivan, It'd be helpful if you would clarify in the POD where the values for the error_message() and result_code() methods come from. I discovered that the value of the result_code() method is actually the response reason code and that the value of the error_message() method is the response reason text. I had to do some digging to figure out that the module wasn't just using the response code and response reason text as these have similar values to the response reason code and response reason text fields. Thanks, William
  Thu Nov 02 15:34:09 2006 ivan-pause [...] 420.am - Correspondence added
Patch appriciated; planning on releasing 3.16 soon. On Thu Nov 02 15:00:07 2006, WMCKEE wrote: Show quoted text
> Ivan, > > It'd be helpful if you would clarify in the POD where the values for
the Show quoted text
> error_message() and result_code() methods come from. I discovered
that Show quoted text
> the value of the result_code() method is actually the response
reason Show quoted text
> code and that the value of the error_message() method is the
response Show quoted text
> reason text. > > I had to do some digging to figure out that the module wasn't just
using Show quoted text
> the response code and response reason text as these have similar
values Show quoted text
> to the response reason code and response reason text fields. > > > Thanks, > William
  Thu Nov 02 15:34:10 2006 The RT System itself - Status changed from 'new' to 'open'
  Fri Nov 03 11:39:43 2006 WMCKEE [...] cpan.org - Correspondence added
From: WMCKEE [...] cpan.org
On Thu Nov 02 15:34:09 2006, IVAN wrote: Show quoted text
> Patch appriciated; planning on releasing 3.16 soon.
Attached. I followed the format from Business::OnlinePayment for the patch. There are probably some other custom methods I overlooked as I haven't really studied the module in-depth. I hope you'll include this patch in some form as it's nice to have the methods listed in the POD. Thanks, William
--- tmp/Business-OnlinePayment-AuthorizeNet-3.15/AuthorizeNet.pm 2005-03-13 06:40:39.000000000 -0500 +++ AuthorizeNet.pm 2006-11-03 11:38:41.000000000 -0500 @@ -360,6 +360,22 @@ For detailed information see L<Business::OnlinePayment>. +=head1 METHODS AND FUNCTIONS + +See L<Business::OnlinePayment> for the complete list. The following methods either override the methods in L<Business::OnlinePayment> or provide additional functions. + +=head2 result_code + +Returns the response reason code (this is different than the response code). + +=head2 error_message + +Returns the response reason text. + +=head2 server_response + +Returns the complete response from the server. + =head1 NOTE Unlike Business::OnlinePayment or pre-3.0 verisons of
  Sun Sep 23 20:37:25 2007 ivan-pause [...] 420.am - Status changed from 'open' to 'resolved'
  Sun Sep 23 20:37:26 2007 ivan-pause [...] 420.am - Fixed in 3.16 added
  Sun Sep 23 20:37:26 2007 ivan-pause [...] 420.am - Fixed in 3.17 added
  Sun Sep 23 20:37:26 2007 ivan-pause [...] 420.am - Broken in 3.00 added
  Sun Sep 23 20:37:26 2007 ivan-pause [...] 420.am - Broken in 3.01 added
  Sun Sep 23 20:37:26 2007 ivan-pause [...] 420.am - Broken in 3.10 added
  Sun Sep 23 20:37:26 2007 ivan-pause [...] 420.am - Broken in 3.11 added
  Sun Sep 23 20:37:26 2007 ivan-pause [...] 420.am - Broken in 3.12 added
  Sun Sep 23 20:37:26 2007 ivan-pause [...] 420.am - Broken in 3.13 added
  Sun Sep 23 20:37:27 2007 ivan-pause [...] 420.am - Broken in 3.14 added