Ticket #520 (new Bug)

Opened 4 months ago

Last modified 2 months ago

Critical Bug in Paypalpro payment driver

Reported by: atomless Assigned to: zombor
Priority: critical Milestone: 2.2
Component: Libraries Version: SVN HEAD
Keywords: Cc:
SVN Revision (if applicable):

Description

after code tidy-up for svn inclusion the name-value-pair response array was wrongly named in return statement of process method. This has been fixed in the attached file.

Attachments

Paypalpro.php (5.7 kB) - added by atomless on 03/26/08 15:45:05.
Payment driver for paypal website payments pro
line226.php (43 bytes) - added by atomless on 03/29/08 11:40:05.
patch for line 226 of paypalpro payment driver - the return statement of the process function

Change History

03/26/08 15:45:05 changed by atomless

  • attachment Paypalpro.php added.

Payment driver for paypal website payments pro

03/26/08 16:05:51 changed by atomless

  • type changed from Patch to Bug.

03/27/08 07:26:01 changed by atomless

  • priority changed from minor to critical.

03/27/08 07:26:08 changed by atomless

  • owner changed from Shadowhand to - No owner -.
  • component changed from Core to Libraries.

03/27/08 07:41:43 changed by atomless

  • summary changed from Fixed bug in Paypalpro payment driver to Critical Bug in Paypalpro payment driver.

03/27/08 07:48:06 changed by atomless

  • owner changed from - No owner - to Shadowhand.

03/28/08 05:11:36 changed by atomless

  • owner changed from Shadowhand to - No owner -.

The bug prevents the driver from working completely so can anyone authorized to replace the Paypalpro driver in the current svn with the one attached to this ticket please take a second to replace it? (the attached file differs only in that the bug is fixed) - see the naming of the nvp response array in the return statement of the process method, the array currently returned does not exist!

(follow-up: ↓ 9 ) 03/29/08 11:23:23 changed by zombor

  • owner changed from - No owner - to zombor.

Please provide a patch instead of a whole file replacement.

03/29/08 11:26:15 changed by zombor

  • milestone changed from 2.1.2 to 2.2.

(in reply to: ↑ 7 ) 03/29/08 11:35:12 changed by atomless

ok, it's just the return statement in the process method that needs replacing, line 266

return ($nvp_res_arrayACK? == TRUE);

03/29/08 11:35:49 changed by atomless

<pre>return ($nvp_res_arrayACK? == TRUE);</pre>

03/29/08 11:36:26 changed by atomless

<code>return ($nvp_res_arrayACK? == TRUE);</code>

03/29/08 11:37:36 changed by atomless

I'm sorry! can't get trac to not mess with the code - will attach a file.

03/29/08 11:40:05 changed by atomless

  • attachment line226.php added.

patch for line 226 of paypalpro payment driver - the return statement of the process function

03/31/08 05:49:22 changed by atomless

Zombor, was a file conatining a replacement for the line at fault not what you had in mind?

06/08/08 22:44:33 changed by sell

http://www.wroughtironhouse.cn/