Model best practices in our core processors
Subissue of #130
Subtasks for this issue
Throw exceptions, do not return error object
#131 (closed)
Implement doPayment, do not implement doTransferPayment, doDirectPayment #135 (closed)
- this is currently blocked on clarifying return parameters #141 (closed)
Cast $params to a PropertyBag at the start of the function
- note this is currently blocked on PropertyBag support for the full range of properties - see #130 (comment 42912)
Use Guzzle, not Curl #143
Have tests So far the following processors have tests
- Paypal (all versions)
- Authorize.net
- Eway
- Payflow Pro
- Dummy