[ic] Verifone IPCharge

[Richard Siddall]

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 
functionality.

> 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.
> 

True.

I think your patch would be better passing the extra parameters to the
constructor.  As I understand it, that's the Business::OnlinePayment way.

Regards,

	Richard.