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 10:20 UTC
From: aashley at optimiser dot com Assigned: ieure
Status: Closed Package: Payment_Process
PHP Version: Irrelevant OS: Linux
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 24 + 3 = ?

 
 [2004-06-28 10: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 22: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-29 01: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-29 03:13 UTC] aashley at optimiser dot com
the patch at the above URL is now correct.
 [2004-07-01 21: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.