Vote Details for "Payment_PayPalSoap" by izi

» Details
  • Voter: David Jean Louis 
  • Vote: +1 (not conditional)
  • Reviews: Cursory source review
» Comment
I'm not sure the Payment_PayPal_Soap abstract class is a good idea.
I would just move Payment_PayPal_Soap_Client in a Payment_PayPal_Soap non abstract class containing the ERROR_* constants.

Good and well documented code otherwise !