Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 0.6.8

Bug #1733 Accessing nonexistant property and constant in Process.php
Submitted: 2004-06-28 05:20 UTC Modified: 2006-12-26 23:21 UTC
From: aashley at optimiser dot com Assigned: ieure
Status: Closed Package: Payment_Process
PHP Version: Irrelevant OS: Linux
Roadmaps: (Not assigned)    
Subscription  


 [2004-06-28 05:20 UTC] aashley at optimiser dot com
Description: ------------ In Process.php there is an access to $this->_request->_payment->_driver; _driver does not exist, this is called _type on all the Type objects. Also there is usage of PAYMENT_PROCESS_TYPE_CREDITCARD this constant is not defined anywhere. Due to these two problems CVV checking will never be done. patch at: http://frood.cernun.net/stuff/Process.php-validate.patch

Comments

 [2004-06-28 17:05 UTC] ieure
What version are you using? I don't see this in 0.5.7, and your patch doesn't apply. Perhaps you reversed diff's arguments?
 [2004-06-28 20:38 UTC] aashley at optimiser dot com
yeah i did reverse the patch. the two changes in the patch happen at lines 602 and 604 of Process.php. I'll post an updated version of the patch when i get into the office today.
 [2004-06-28 22:13 UTC] aashley at optimiser dot com
the patch at the above URL is now correct.
 [2004-07-01 16:53 UTC] ieure
This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better.