pronamic / wp-pronamic-pay-buckaroo Goto Github PK

View Code? Open in Web Editor NEW

3.0 5.0 2.0 521 KB

Buckaroo driver for the WordPress payment processing library.

Home Page: http://www.wp-pay.org/gateways/buckaroo/

JavaScript 1.70% PHP 98.30%

buckaroo php ideal wordpress pay wp-pronamic-pay-gateway buckaroo-gateway pronamic pronamic-pay

wp-pronamic-pay-buckaroo's Introduction

WordPress Pay Gateway: Buckaroo

Buckaroo driver for the WordPress payment processing library.

Documentation

Production Environment

Payment Server URL: https://checkout.buckaroo.nl/html/

Test Environment

Payment Server URL: https://testcheckout.buckaroo.nl/html/

wp-pronamic-pay-buckaroo's People

Contributors

Stargazers

Watchers

Forkers

nielsdeblaauw

wp-pronamic-pay-buckaroo's Issues

Psalm warns for syntax errors

In our automated workflows, Psalm warns for issues regarding syntax errors in some recent changes to this gateway.

Specifically:

ERROR: ParseError - plugins/pronamic-ideal/vendor/wp-pay-gateways/buckaroo/src/Gateway.php:470:5 - Syntax error, unexpected T_NS_SEPARATOR, expecting ',' or ']' or ')' on line 470 (see https://psalm.dev/173)
				\

and

ERROR: ParseError - plugins/pronamic-ideal/vendor/wp-pay-gateways/buckaroo/src/Gateway.php:472:75 - Syntax error, unexpected ',' on line 472 (see https://psalm.dev/173)
				null === $data ? '' : \base64_encode( \md5( $request_content, true ) ),

The issues are also noted in the commit that introduced them: db53d8e by phpstan.

PHP warning `Invalid argument supplied for foreach()` when enriching payment methods

A customer noticed the following error after updating a Buckaroo configuration:

Warning: Invalid argument supplied for foreach() in /wp-content/plugins/pronamic-ideal2/vendor/wp-pay-gateways/buckaroo/src/Gateway.php on line 141

Internal Help Scout ticket: https://secure.helpscout.net/conversation/2023552014/24521

Add `Software` header for partnership

https://docs.buckaroo.io/docs/authentication

Additional Headers (not mandatory)
Culture: The user culture if applicable (for example: nl-NL). If not provided, a default of en-US will be used. This will be used in determining the language of both the checkout and the email templates.

Channel: The used channel, mostly 'Web' is used (which is default), but in some cases 'Backoffice' is required. Please see the payment method and services sections for the supported channel per service action.
Software: A definition of the software used to create the request, this needs be a JSON string in the following format:
{"PlatformName":"X","PlatformVersion":"1.0","ModuleSupplier":"X","ModuleName":"X","ModuleVersion":"1.0"}

Error "No valid subscription found for service `amex`"

I just came across the following errors with a Buckaroo account for which the American Express card brand is not active, when using the credit card payment method:

Message
	Multiple primary actions specified. Please specify 1 primary action.
Message
	No valid subscription found for service 'amex'.

This is caused by the fact that we're always including amex as a service for the credit card payment method on payment start:

wp-pronamic-pay-buckaroo/src/Gateway.php

Lines 313 to 339 in 4967b4c

    
           			/** 
        
           			 * Payment method creditcard. 
        
           			 * 
        
           			 * @link https://dev.buckaroo.nl/PaymentMethods/Description/creditcards#pay 
        
           			 */ 
        
           			case Core_PaymentMethods::CREDIT_CARD: 
        
           				$data->Services->ServiceList[] = (object) array( 
        
           					'Action' => 'Pay', 
        
           					'Name'   => PaymentMethods::AMERICAN_EXPRESS, 
        
           				); 
        
           				$data->Services->ServiceList[] = (object) array( 
        
           					'Action' => 'Pay', 
        
           					'Name'   => PaymentMethods::MAESTRO, 
        
           				); 
        
           				$data->Services->ServiceList[] = (object) array( 
        
           					'Action' => 'Pay', 
        
           					'Name'   => PaymentMethods::MASTERCARD, 
        
           				); 
        
           				$data->Services->ServiceList[] = (object) array( 
        
           					'Action' => 'Pay', 
        
           					'Name'   => PaymentMethods::VISA, 
        
           				); 
        
           				break;

To check if a brand is available for a Buckaroo account, we could use the following endpoints:

In case a brand is not available, the response will be something like:

{
	"SupportedCurrencies": null,
	"Actions": null,
	"Name": "Service Not Found!",
	"Version": 0,
	"Description": null
}

Internal Help Scout ticket: https://secure.helpscout.net/conversation/2006986721/24458

pronamic / wp-pronamic-pay-buckaroo Goto Github PK

wp-pronamic-pay-buckaroo's Introduction

WordPress Pay Gateway: Buckaroo

Links

Documentation

Production Environment

Test Environment

wp-pronamic-pay-buckaroo's People

Contributors

Stargazers

Watchers

Forkers

wp-pronamic-pay-buckaroo's Issues

Psalm warns for syntax errors

PHP warning `Invalid argument supplied for foreach()` when enriching payment methods

Add `Software` header for partnership

Additional Headers (not mandatory)

Error "No valid subscription found for service `amex`"

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

	/**
	* Payment method creditcard.
	*
	* @link https://dev.buckaroo.nl/PaymentMethods/Description/creditcards#pay
	*/
	case Core_PaymentMethods::CREDIT_CARD:
	$data->Services->ServiceList[] = (object) array(
	'Action' => 'Pay',
	'Name' => PaymentMethods::AMERICAN_EXPRESS,
	);

	$data->Services->ServiceList[] = (object) array(
	'Action' => 'Pay',
	'Name' => PaymentMethods::MAESTRO,
	);

	$data->Services->ServiceList[] = (object) array(
	'Action' => 'Pay',
	'Name' => PaymentMethods::MASTERCARD,
	);

	$data->Services->ServiceList[] = (object) array(
	'Action' => 'Pay',
	'Name' => PaymentMethods::VISA,
	);

	break;