From 49ced0204791fbcf05425eafa00b6cea6090c0ae Mon Sep 17 00:00:00 2001 From: Remco Tolsma <869674+remcotolsma@users.noreply.github.com> Date: Mon, 6 Nov 2023 12:04:02 +0100 Subject: [PATCH] Reduce `get_pronamic_payment()` calls. Related to https://github.com/pronamic/wp-pay-core/issues/162. --- src/Extension.php | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/src/Extension.php b/src/Extension.php index 7232223..6e2e2f8 100644 --- a/src/Extension.php +++ b/src/Extension.php @@ -1226,17 +1226,20 @@ public function replace_merge_tags( $text, $form, $entry, $url_encode, $esc_html $subscription_renew_url = $subscription->get_renewal_url(); } - $payment_id = gform_get_meta( rgar( $entry, 'id' ), 'pronamic_payment_id' ); - $subscription_payment_id = gform_get_meta( rgar( $entry, 'id' ), 'pronamic_subscription_payment_id' ); + $payment_id = (string) gform_get_meta( rgar( $entry, 'id' ), 'pronamic_payment_id' ); + $subscription_payment_id = (string) gform_get_meta( rgar( $entry, 'id' ), 'pronamic_subscription_payment_id' ); /** * Bank transfer recipient details. + * + * Use bank transfer details from last subscription payment if available. */ - // Use bank transfer details from last subscription payment if available. - $payment = \get_pronamic_payment( $subscription_payment_id ); + $payment = null; + + $payment = ( '' === $subscription_payment_id ) ? $payment : \get_pronamic_payment( $subscription_payment_id ); if ( null === $payment ) { - $payment = \get_pronamic_payment( $payment_id ); + $payment = ( '' === $payment_id ) ? $payment : \get_pronamic_payment( $payment_id ); } $bank_transfer_recipient_reference = '';