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

Fix webhook delivery log deletion 

Use wp_delete_comment() instead of SQL so comment meta is deleted as
well.
This commit is contained in:
Max Rice 2014-07-31 11:52:16 -04:00
parent 3e6c4dfac2
commit 3c566e3419
1 changed files with 6 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
includes/class-wc-webhook.php
View File

@ -387,12 +387,11 @@ class WC_Webhook {
if ( $log->total_comments > apply_filters( 'woocommerce_max_webhook_delivery_logs', 25 ) ) {
global $wpdb;
$wpdb->query(
$wpdb->prepare(
"DELETE FROM {$wpdb->comments} WHERE comment_post_ID = %d ORDER BY comment_date_gmt ASC LIMIT 1",
$this->id
)
);
$comment_id = $wpdb->get_var( $wpdb->prepare( "SELECT comment_ID FROM {$wpdb->comments} WHERE comment_post_ID = %d ORDER BY comment_date_gmt ASC LIMIT 1", $this->id ) );
if ( $comment_id ) {
wp_delete_comment( $comment_id, true );
}
}
}
@ -590,7 +589,7 @@ class WC_Webhook {
$args = array(
'user-agent' => sprintf( 'WooCommerce/%s Hookshot (WordPress/%s)', WC_VERSION, $GLOBALS['wp_version'] ),
'body' => "webhook_id={$this->id}",
'body' => "webhook_id={$this->id}",
);
wp_remote_post( $this->get_delivery_url(), $args );