Steve-Dee-Designs
/
woocommerce
mirror of https://github.com/woocommerce/woocommerce.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

pass the whole gateway instance into notify_admin_payment_gateway_enabled

This commit is contained in:
Leif Singer 2023-11-24 11:43:34 +01:00
parent e3c2c79389
commit ee04b60a83
1 changed files with 10 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
plugins/woocommerce/includes/class-wc-payment-gateways.php
View File

@ -137,8 +137,12 @@ class WC_Payment_Gateways {
private function on_payment_gateways_initialized( WC_Payment_Gateways $wc_payment_gateways ) { private function on_payment_gateways_initialized( WC_Payment_Gateways $wc_payment_gateways ) {
foreach ( $this->payment_gateways as $gateway ) { foreach ( $this->payment_gateways as $gateway ) {
$option_key = $gateway->get_option_key(); $option_key = $gateway->get_option_key();
self::add_action( 'add_option_' . $option_key, array( $this, 'payment_gateway_settings_option_changed' ), 10, 2 ); self::add_action( 'add_option_' . $option_key, function( $option, $value ) use ( $gateway) {
self::add_action( 'update_option_' . $option_key, array( $this, 'payment_gateway_settings_option_changed' ), 10, 3 ); $this->payment_gateway_settings_option_changed( $gateway, $value, $option );
}, 10, 2 );
self::add_action( 'update_option_' . $option_key, function( $old_value, $value, $option ) use ( $gateway) {
$this->payment_gateway_settings_option_changed( $gateway, $value, $option, $old_value );
}, 10, 3 );
} }
} }
@ -150,19 +154,14 @@ class WC_Payment_Gateways {
* @param string $option Option name. * @param string $option Option name.
* @since 8.5.0 * @since 8.5.0
*/ */
private function payment_gateway_settings_option_changed( $old_value, $value, $option = null ) { private function payment_gateway_settings_option_changed( $gateway, $value, $option, $old_value = null ) {
if ( null === $option ) {
// We're in the add_option_ hook so there's no old value and parameter order is different.
$option = $old_value;
$old_value = null;
}
if ( ! $this->was_gateway_enabled( $value, $old_value ) ) { if ( ! $this->was_gateway_enabled( $value, $old_value ) ) {
return; return;
} }
// This is a change to a payment gateway's settings and it was just enabled. Let's send an email to the admin. // This is a change to a payment gateway's settings and it was just enabled. Let's send an email to the admin.
// "untitled" shouldn't happen, but just in case. // "untitled" shouldn't happen, but just in case.
$this->notify_admin_payment_gateway_enabled( $value['title'] ?? 'untitled' ); $this->notify_admin_payment_gateway_enabled( $gateway );
} }
/** /**
@ -172,11 +171,12 @@ class WC_Payment_Gateways {
* @return bool Whether the email was sent or not. * @return bool Whether the email was sent or not.
* @since 8.5.0 * @since 8.5.0
*/ */
private function notify_admin_payment_gateway_enabled( $gateway_title ) { private function notify_admin_payment_gateway_enabled( $gateway ) {
$admin_email = get_option( 'admin_email' ); $admin_email = get_option( 'admin_email' );
$user = get_user_by( 'email', $admin_email ); $user = get_user_by( 'email', $admin_email );
$username = $user ? $user->user_login : $admin_email; $username = $user ? $user->user_login : $admin_email;
$gateway_settings_url = self_admin_url( 'admin.php?page=wc-settings&tab=checkout' ); $gateway_settings_url = self_admin_url( 'admin.php?page=wc-settings&tab=checkout' );
$gateway_title = $gateway->get_title();
$site_name = wp_specialchars_decode( get_option( 'blogname' ), ENT_QUOTES ); $site_name = wp_specialchars_decode( get_option( 'blogname' ), ENT_QUOTES );
$site_url = home_url(); $site_url = home_url();
/** /**