[ic] Verifone IPCharge
richard.siddall at elirion.net
Fri Mar 20 20:59:16 UTC 2009
Mike Heins wrote:
> Obviously you don't know the Interchange way. Bloat 'R us. 8-)
> You may be right, but maybe not. It could be that the current
> way of doing it is what should be removed.
Possibly. Presumably, you'd want to retain the MV_PAYMENT_* variable
> There is no way to remap or account for conflicting key names the way
> that is provided. If we are going to use this as a main payment method I
> think a little flexibility is required. All it would take is for one of
> the payment modules to require "gateway" as a parameter, and it is
> broken. Doesn't sound like a good solution to me.
I think your patch would be better passing the extra parameters to the
constructor. As I understand it, that's the Business::OnlinePayment way.
More information about the interchange-users