Pronamic\WordPress\Pay\Gateways\Adyen\PaymentResultHelper::update_payment()

Complexity

Conditions	3
Paths	4

Size

Total Lines	30
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	12

Importance

Changes

0

<?php
	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();
	}