From 1c2b74e6440da6a986db64d606fd05b4371199a7 Mon Sep 17 00:00:00 2001 From: vinkmeta Date: Wed, 11 Jun 2025 11:45:26 +0100 Subject: [PATCH 1/2] Removed duplicate logging --- includes/Framework/BatchLogHandler.php | 22 ++++++++++++++++++---- 1 file changed, 18 insertions(+), 4 deletions(-) diff --git a/includes/Framework/BatchLogHandler.php b/includes/Framework/BatchLogHandler.php index 786d8e712..03ead0560 100644 --- a/includes/Framework/BatchLogHandler.php +++ b/includes/Framework/BatchLogHandler.php @@ -68,14 +68,28 @@ function ( $log ) { try { $response = facebook_for_woocommerce()->get_api()->log_to_meta( $context ); if ( $response->success ) { - WC_Facebookcommerce_Utils::log_with_debug_mode_enabled( 'Meta logs: ' . wp_json_encode( $context ) ); return []; } else { - WC_Facebookcommerce_Utils::log_with_debug_mode_enabled( 'Bad response from log_to_meta request' ); - return $logs_chunk; + Logger::log( + 'Bad response from Meta logging APIs', + [], + array( + 'should_send_log_to_meta' => false, + 'should_save_log_in_woocommerce' => true, + 'woocommerce_log_level' => \WC_Log_Levels::ERROR, + ) + ); } } catch ( \Exception $e ) { - WC_Facebookcommerce_Utils::log_with_debug_mode_enabled( 'Error persisting meta logs: ' . $e->getMessage() ); + Logger::log( + 'Error persisting Meta logs: ' . $e->getMessage(), + [], + array( + 'should_send_log_to_meta' => false, + 'should_save_log_in_woocommerce' => true, + 'woocommerce_log_level' => \WC_Log_Levels::ERROR, + ) + ); return $logs_chunk; } }, From c4b57b4d311d35a5d992f555871041acab4ee0bb Mon Sep 17 00:00:00 2001 From: vinkmeta Date: Wed, 11 Jun 2025 11:48:37 +0100 Subject: [PATCH 2/2] Fixed typo --- includes/Framework/BatchLogHandler.php | 1 + 1 file changed, 1 insertion(+) diff --git a/includes/Framework/BatchLogHandler.php b/includes/Framework/BatchLogHandler.php index 03ead0560..5e0d890fc 100644 --- a/includes/Framework/BatchLogHandler.php +++ b/includes/Framework/BatchLogHandler.php @@ -79,6 +79,7 @@ function ( $log ) { 'woocommerce_log_level' => \WC_Log_Levels::ERROR, ) ); + return $logs_chunk; } } catch ( \Exception $e ) { Logger::log(