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

Request #3875 Misnomer: Payment_Process_Result::validate()
Submitted: 2005-03-18 19:29 UTC
From: jausions Assigned:
Status: Open Package: Payment_Process
PHP Version: Irrelevant OS: Irrelevant
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 : 25 - 13 = ?

 
 [2005-03-18 19:29 UTC] jausions
Description: ------------ At best the Payment_Process_Result::validate() method is misnamed. The validate() method should tell is the result is valid, not if the transaction was approved. This is a totally different thing. A declined transaction is a valid result. Garbage received from the payment gateway is not. Furthermore, someone may want to accept a payment even if AVS didn't pass. A mismatch in AVS (or CVV) doesn't mean the transaction was declined. Depending on your settings in accepting transactions vis-a-vis AVS and CVV, a mismatched AVS and/or CVV could be discarded. So testing for AVS and or CVV mismatch before the overall transaction result, could result in accepted orders to retur a PEAR_Error instance. -Philippe

Comments

 [2005-07-28 03:08 UTC] jstump
Not sure what we can do here. I'm open to suggestions. You have some code you had in mind?