Cavemanon/piratepoet
8
Fork 0
Code Pull Requests Actions Releases 16 Activity

Refactor SQL for segment in country to use join instead subselect

Browse Source 
[MAILPOET-3226]
This commit is contained in:
Rostislav Wolny
2021-05-10 15:16:26 +02:00
committed by Veljko V
parent 1ec4300153
commit d4fbe95aef
1 changed files with 8 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
lib/Segments/DynamicSegments/Filters/WooCommerceCountry.php
View File

@@ -27,10 +27,14 @@ class WooCommerceCountry implements Filter {
$subscribersTable, $subscribersTable,
$wpdb->postmeta, $wpdb->postmeta,
'postmeta', 'postmeta',
"postmeta.meta_key = '_customer_user' "postmeta.meta_key = '_customer_user' AND $subscribersTable.wp_user_id=postmeta.meta_value"
AND $subscribersTable.wp_user_id=postmeta.meta_value )->innerJoin(
AND postmeta.post_id NOT IN ( SELECT id FROM {$wpdb->posts} as p WHERE p.post_status IN ('wc-cancelled', 'wc-failed'))" 'postmeta',
)->innerJoin('postmeta', $wpdb->posts,
'posts',
"postmeta.post_id = posts.id AND posts.post_status NOT IN ('wc-cancelled', 'wc-failed')"
)->innerJoin(
'postmeta',
$wpdb->postmeta, $wpdb->postmeta,
'postmetaCountry', 'postmetaCountry',
"postmeta.post_id = postmetaCountry.post_id AND postmetaCountry.meta_key = '_billing_country' AND postmetaCountry.meta_value = :$countryFilterParam" "postmeta.post_id = postmetaCountry.post_id AND postmetaCountry.meta_key = '_billing_country' AND postmetaCountry.meta_value = :$countryFilterParam"