| 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) | ||
[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] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!
[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-07-01 21:53 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!