Hi,
On Nov 15, 2:59 am, Moo <
mi...@redinnovation.com> wrote:
> Hi,
>
> I just went through Brandon's work. Looks nice to me, though there are
> some show stopping issues.
>
> The offsite payment processor interface was empty. It is very
> difficult for third parties to implement payment processor unless
> interfaces are documented with the required precision.
>
> If I read the core correctly there can be just one on-site payment
> processor. Also, the wizard step to choose payment processor is
> missing.
The wizard step is not needed here.
The checkout process should be done with as few click from the buyer
as possible. This is important point for the shop owners.
With paypal standard, you get a checkout button from the payment
provider and put it on your site anywhere, allowing the customer the
start the payment immediately. Later the payment processor can be
asked and/or will notify the shop about the transaction.
With paypal express checkout, you have to ask the paypal server for a
token, before you can generate the checkout button, so the extra step
in checkout process after selecting paypal is needed.
The current paypal module in trunk works with paypal standard.
> As is, this work is useless for many use cases, like
>
> * Having a wire payment processor
>
> * Choose between credit card and wire payment
>
> * Choose between different on-sites processors (imagine different
> processor for different credit cards)
>
In Brandon's branch you can use offsite processors and onsite
processor together, the buyer can choose between them during the
checkout.
However the offsite button rendering is done with black magic, a
refactoring would be nice.
> What's the status of the future of Brandon's work? In
> multiplepaymentprocessor support for these use cases exist.
I'm just trying out multiplepaymentprocessors branch, which offsite
payment processors should I test with? Currently only see the onsite
nullpayment.
> I am willing to help to merge multiplepaymentprocessor branch with
> brandon and brandon branch with trunk if
>
> 1) Community "decision making process" sees this as a necessary
> requirement for the future
>
> 2) There is someone else willing to put hours on the issue
>
+1
> -Mikko
Cheers,
Marton