wp-pay-gateways / adyen

src/PaymentResultHelper.php

<?php
/**
 * Payment result helper
 *
 * @author    Pronamic <info@pronamic.eu>
 * @copyright 2005-2019 Pronamic
 * @license   GPL-3.0-or-later
 * @package   Pronamic\WordPress\Pay\Gateways\Adyen
 */

namespace Pronamic\WordPress\Pay\Gateways\Adyen;

use Pronamic\WordPress\Pay\Payments\Payment;

/**
 * Payment result helper
 *
 * @author  Remco Tolsma
 * @version 1.0.0
 * @since   1.0.0
 */
class PaymentResultHelper {
	/**
	 * Update payment to the payment result response.
	 *
	 * @param Payment               $payment  Payment.
	 * @param PaymentResultResponse $response Response.
	 */
	public static function update_payment( Payment $payment, PaymentResultResponse $response ) {
		// Add note.
		$note = sprintf(
			'<p>%1$s</p><pre>%2$s</pre>',
			sprintf(
				/* translators: %s: payment provider name */
				__( 'Verified payment result.', 'pronamic_ideal' ),
				__( 'Adyen', 'pronamic_ideal' )
			),
			wp_json_encode( $response->get_json(), JSON_PRETTY_PRINT )

It seems like wp_json_encode($response->get_json(), Pronamic\WordPress\Pay\Gateways\Adyen\JSON_PRETTY_PRINT) can also be of type false; however, parameter $args of sprintf() does only seem to accept string, maybe add an additional type check?

		);

		$payment->add_note( $note );

		// PSP reference.
		$psp_reference = $response->get_psp_reference();

		if ( null !== $psp_reference ) {
			$payment->set_transaction_id( $psp_reference );
		}

		// Result code.
		$result_code = $response->get_result_code();

		if ( null !== $result_code ) {
			$payment->set_status( ResultCode::transform( $result_code ) );
		}

		// Payment save.
		$payment->save();
	}
}