diff --git a/facebook-commerce.php b/facebook-commerce.php index fbace4002..e8a95ce38 100644 --- a/facebook-commerce.php +++ b/facebook-commerce.php @@ -2855,10 +2855,18 @@ public function maybe_display_facebook_api_messages() { // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped echo $this->get_message_html( $message ); delete_transient( 'facebook_plugin_api_error' ); - WC_Facebookcommerce_Utils::fblog( - $error_msg, - [], - true + Logger::log( + 'Error message displayed to Admins', + array( + 'flow_name' => 'display_admin_message', + 'flow_step' => 'display_admin_error_message', + 'extra_data' => [ + 'displayed_message' => $error_msg, + ], + ), + array( + 'should_send_log_to_meta' => true, + ) ); } $warning_msg = get_transient( 'facebook_plugin_api_warning' ); diff --git a/includes/Logger/Logger.php b/includes/Logger/Logger.php index 0d6a7e49b..2aba5d198 100644 --- a/includes/Logger/Logger.php +++ b/includes/Logger/Logger.php @@ -41,12 +41,12 @@ public static function log($message, $context = [], $log_options = [ 'woocommerce_log_level' => \WC_Log_Levels::DEBUG, ]) { $is_debug_mode_enabled = 'yes' === get_option( self::SETTING_ENABLE_META_DIAGNOSIS ); - if ( $is_debug_mode_enabled && $log_options['should_save_log_in_woocommerce'] ) { + if ( $is_debug_mode_enabled && array_key_exists( 'should_save_log_in_woocommerce', $log_options ) && $log_options['should_save_log_in_woocommerce'] ) { facebook_for_woocommerce()->log( $message . ' : ' . wp_json_encode( $context ), null, $log_options['woocommerce_log_level'] ); } $is_meta_diagnosis_enabled = (bool) ( 'yes' === get_option( self::SETTING_ENABLE_META_DIAGNOSIS ) ); - if ( $log_options['should_send_log_to_meta'] && $is_meta_diagnosis_enabled ) { + if ( $is_meta_diagnosis_enabled && array_key_exists( 'should_send_log_to_meta', $log_options ) && $log_options['should_send_log_to_meta'] ) { $extra_data = $context['extra_data'] ?? []; $extra_data['message'] = $message;