Bug #3765 for Business-OnlinePayment-LinkPoint: Staging server should not be forced

Wed Sep 10 15:27:22 2003 Guest - Ticket created

Subject:

Staging server should not be forced

When using the testmode of Business::OnlinePayment ($tx->test_transaction(1)), the Linkpoint plugin forces the server to be set to staging.linkpt.net. If you do not have a test account setup with LinkPoint, you will receive the following error: Card was rejected: Error (Merchant config file is missing, empty or cannot be read). When I spoke with LinkPoint's tech support, I was told to use the live server and simply send in the RESULT => good parameter. The module should not force the use of the staging server at line 123; if a user has a test account, they can set the server using the $tx->server() function.

Wed Sep 10 16:54:17 2003 ivan-pause [...] 420.am - Status changed from 'new' to 'open'

Wed Sep 10 16:54:18 2003 ivan-pause [...] 420.am - Severity Important changed to Normal

Wed Sep 10 16:56:53 2003 ivan-pause [...] 420.am - Correspondence added

I'm sorry, I fail to understand your logic. The test server is not "forced", it is requested explicity with $tx->test_transaction(1). The default is the live server. Unless you can convince me otherwise, I intend to keep the current behaviour.

Sat Sep 27 12:16:20 2003 Guest - Correspondence added

[IVAN - Wed Sep 10 16:56:53 2003]: Show quoted text

> I'm sorry, I fail to understand your logic. The test server > is not "forced", it is requested explicity with > $tx->test_transaction(1). The default is the live server. > > Unless you can convince me otherwise, I intend to keep > the current behaviour.

Hi Ivan, I thought that I had replied to this message but I don't see the response here so I'm writing again. Your code forces a user to use the staging server when putting the module into test mode (line 123). Since my client does not have an account on the staging server, I receive errors when trying to use the module in test mode. Simply commenting out the line in your module where you set the server to staging is all that is required for it to work. William

Thu Oct 30 13:50:14 2003 Guest - Correspondence added

[IVAN - Wed Sep 10 16:56:53 2003]: Show quoted text

> I'm sorry, I fail to understand your logic. The test server > is not "forced", it is requested explicity with > $tx->test_transaction(1). The default is the live server. > > Unless you can convince me otherwise, I intend to keep > the current behaviour.

Hi Ivan, Have none of my replies convinced you to apply the patch I sent almost 2 months ago? It's cool if you want the module to sets this to a different server when test mode is invoked, but it shouldn't be changed if a user manually sets it to a different host. Thanks, William

Thu Oct 30 15:28:22 2003 ivan [...] 420.am - Correspondence added

From:	ivan [...] 420.am
Date:	Thu, 30 Oct 2003 12:27:45 -0800
To:	Guest via RT <bug-Business-OnlinePayment-LinkPoint [...] rt.cpan.org>
Subject:	Re: [cpan #3765] Staging server should not be forced
RT-Send-Cc:

On Thu, Oct 30, 2003 at 01:50:15PM -0500, Guest via RT wrote: Show quoted text

> > This message about Business-OnlinePayment-LinkPoint was sent to you by guest <> via rt.cpan.org > > Full context and any attached attachments can be found at: > <URL: https://rt.cpan.org/Ticket/Display.html?id=3765 > > > [IVAN - Wed Sep 10 16:56:53 2003]: >

> > I'm sorry, I fail to understand your logic. The test server > > is not "forced", it is requested explicity with > > $tx->test_transaction(1). The default is the live server. > > > > Unless you can convince me otherwise, I intend to keep > > the current behaviour.

> > Hi Ivan, > > Have none of my replies convinced you to apply the patch I sent almost 2 > months ago?

I haven't had time to consider the issue fully. I do see that the issue is more complicated than I had thought initially. -- _ivan

Fri Mar 25 16:54:21 2005 ivan-pause [...] 420.am - Fixed in 0.05 added

Fri Mar 25 16:54:21 2005 ivan-pause [...] 420.am - Fixed in 0.04 added

Fri Mar 25 16:54:28 2005 ivan-pause [...] 420.am - Status changed from 'open' to 'resolved'