thephpleague / omnipay-buckaroo

Buckaroo driver for the Omnipay PHP payment processing library
MIT License
13 stars 30 forks source link

Buckaroo All gateway #21

Closed yasasn86 closed 8 years ago

yasasn86 commented 8 years ago

buckaroo payments methods select page

silvano-a commented 8 years ago

Is this pull request complete? what does it do?

yasasn86 commented 8 years ago

It helps to go to buckaroo payments methods select page, so that we can select any payment method supported by buckaroo there. we can use the gateway Buckaroo_All to do that if you merged this code. I don't know whether there is another way to do that currently.

silvano-a commented 8 years ago

there is, yes :)

it's called the 'PurchaseRequest'.

usage: $gateway = new BuckarooGateway(); $gateway->purchase();

yasasn86 commented 8 years ago

ok. I was using omnipay with payum bundle. here is my current configuration buckaroo_via_omnipay:     factory: omnipay     type: Buckaroo_All     websiteKey: %buckaroo_website_key%     secretKey: %buckaroo_secret_key%     testMode: %test_mode%

I don't know which name should be used for type in this configuration as you suggest. If I use Buckaroo it does not work and throws an error. but if I use Buckaroo_All it works well (with the code in this pull request) .

here is a screenshot for the error I get when I use Buckaroo as the gateway. screenshot from 2016-09-26 16-12-51

silvano-a commented 8 years ago

Could you try replacing type: Buckaroo_All by type: Buckaroo_Buckaroo and post the results here? https://github.com/thephpleague/omnipay-buckaroo/issues/22

yasasn86 commented 8 years ago

type: BuckarooGateway screenshot from 2016-09-26 16-53-48

yasasn86 commented 8 years ago

ok. my first sreenshot is for type: Buckaroo https://cloud.githubusercontent.com/assets/3061748/18831647/7516c214-8404-11e6-8237-22e8e1bc6d77.png

silvano-a commented 8 years ago

i've created an issue for this. it seems that there is something going wrong in the 'BuckarooGateway' lets move the discussion over there https://github.com/thephpleague/omnipay-buckaroo/issues/22

yasasn86 commented 8 years ago

ok. I think if we change the class name of BuckarooGateway to Gateway, it will fix the issue class BuckarooGateway extends AbstractGateway { public function getName() { return 'Buckaroo Gateway'; } ... to class Gateway extends AbstractGateway { public function getName() { return 'Buckaroo'; } ...

as in the omnipay-worldpay https://github.com/thephpleague/omnipay-worldpay/blob/master/src/Gateway.php

silvano-a commented 8 years ago

can you confirm if this works, and if so, create a pull request including unit test?

yasasn86 commented 8 years ago

I just checked it and it worked with class name Gateway

class Gateway extends AbstractGateway { public function getName() { return 'Buckaroo'; } ....

silvano-a commented 8 years ago

nice!

Could you create a pull request that fixes this?

yasasn86 commented 8 years ago

ok. I will create a pull request :)

silvano-a commented 8 years ago

awesome! then this PR can be closed as an issue is opened for it, and this PR is not the solution to the problem