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

Dont store order failures when verbose is enabled to save memory.

This commit is contained in:
Vedanshu Jain 2023-04-06 15:23:19 +05:30
parent 6eca1ed561
commit dbb4bd5e27
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
plugins/woocommerce/src/Database/Migrations/CustomOrderTable/CLIRunner.php
View File

@ -363,7 +363,7 @@ class CLIRunner {
$batch_start_time = microtime( true ); $batch_start_time = microtime( true );
$failed_ids_in_current_batch = $this->post_to_cot_migrator->verify_migrated_orders( $order_ids ); $failed_ids_in_current_batch = $this->post_to_cot_migrator->verify_migrated_orders( $order_ids );
$failed_ids_in_current_batch = $this->verify_meta_data( $order_ids, $failed_ids_in_current_batch ); $failed_ids_in_current_batch = $this->verify_meta_data( $order_ids, $failed_ids_in_current_batch );
$failed_ids = $failed_ids + $failed_ids_in_current_batch; $failed_ids = $verbose ? array() : $failed_ids + $failed_ids_in_current_batch;
$processed += count( $order_ids ); $processed += count( $order_ids );
$batch_total_time = microtime( true ) - $batch_start_time; $batch_total_time = microtime( true ) - $batch_start_time;
$batch_count ++; $batch_count ++;
@ -410,6 +410,10 @@ class CLIRunner {
$progress->finish(); $progress->finish();
WP_CLI::log( __( 'Verification completed.', 'woocommerce' ) ); WP_CLI::log( __( 'Verification completed.', 'woocommerce' ) );
if ( $verbose ) {
return;
}
if ( 0 === count( $failed_ids ) ) { if ( 0 === count( $failed_ids ) ) {
return WP_CLI::success( return WP_CLI::success(
sprintf( sprintf(
@ -425,9 +429,6 @@ class CLIRunner {
) )
); );
} else { } else {
if ( $verbose ) {
return;
}
$errors = print_r( $failed_ids, true ); $errors = print_r( $failed_ids, true );
return WP_CLI::error( return WP_CLI::error(