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

Fix removal of WP segment subscribers without wp_user_id [MAILPOET-1102]

Browse Source
This commit is contained in:
stoletniy
2017-09-13 16:46:50 +03:00
parent 97d157192a
commit 483dfbe1ec
2 changed files with 6 additions and 47 deletions
Download Patch File Download Diff File Expand all files Collapse all files

40
lib/Segments/WP.php
View File

@@ -88,8 +88,6 @@ class WP {
self::updateFristNameIfMissing();
self::insertUsersToSegment();
self::removeOrphanedSubscribers();
self::removeWPInconsistentubscribers();
self::removeOrphanedLinks();
return true;
}
@@ -159,7 +157,7 @@ class WP {
Subscriber::raw_execute(sprintf('
INSERT IGNORE INTO %s(subscriber_id, segment_id, created_at)
SELECT mps.id, "%s", CURRENT_TIMESTAMP() FROM %s mps
WHERE mps.wp_user_id IS NOT NULL
WHERE mps.wp_user_id > 0
', $wp_mailpoet_subscriber_segment_table, $wp_segment->id, $subscribers_table));
}
@@ -176,42 +174,14 @@ class WP {
// remove orphaned wp segment subscribers (not having a matching wp user id),
// e.g. if wp users were deleted directly from the database
global $wpdb;
$wp_segment = Segment::getWPSegment();
Subscriber::table_alias('wpms')
->leftOuterJoin($wpdb->prefix . 'users', array('wpms.wp_user_id', '=', 'wu.id'), 'wu')
$wp_segment->subscribers()
->leftOuterJoin($wpdb->prefix . 'users', array(MP_SUBSCRIBERS_TABLE . '.wp_user_id', '=', 'wu.id'), 'wu')
->whereNull('wu.id')
->whereGt('wpms.wp_user_id', 0)
->findResultSet()
->set('wp_user_id', null)
->delete();
}
private static function removeWPInconsistentubscribers() {
// remove subscribers subscribed to WP list, but not having wp_user_id
$wp_segment = Segment::getWPSegment();
$subscribers_table = Subscriber::$_table;
$wp_mailpoet_subscriber_segment_table = SubscriberSegment::$_table;
Subscriber::raw_execute(sprintf('
DELETE wpms FROM
%s as wpms
JOIN %s as wuss ON wpms.id = wuss.subscriber_id
WHERE
wpms.wp_user_id IS NULL
and wuss.segment_id=%d;
', $subscribers_table, $wp_mailpoet_subscriber_segment_table, $wp_segment->id));
}
private static function removeOrphanedLinks() {
$subscribers_table = Subscriber::$_table;
$wp_mailpoet_subscriber_segment_table = SubscriberSegment::$_table;
Subscriber::raw_execute(sprintf('
DELETE wuss FROM
%s as wuss
LEFT JOIN %s as wpms ON wpms.id = wuss.subscriber_id
WHERE wpms.id IS NULL
', $wp_mailpoet_subscriber_segment_table, $subscribers_table));
}
}

13
tests/unit/Segments/WPTest.php
View File

@@ -118,18 +118,6 @@ class WPTest extends \MailPoetTest {
expect($subscribers->count())->equals(1);
}
function testItRemovesOrphanedLinks() {
$count1 = SubscriberSegment::count();
$wp_segment = Segment::getWPSegment();
$association = SubscriberSegment::create();
$association->subscriber_id = rand() * 4;
$association->segment_id = $wp_segment->id;
$association->save();
WP::synchronizeUsers();
$count2 = SubscriberSegment::count();
expect($count1)->equals($count2);
}
function testItDoesntDeleteNonWPData() {
$this->insertUser();
// wp_user_id is null
@@ -188,6 +176,7 @@ class WPTest extends \MailPoetTest {
private function cleanData() {
\ORM::raw_execute('TRUNCATE ' . Segment::$_table);
\ORM::raw_execute('TRUNCATE ' . SubscriberSegment::$_table);
global $wpdb;
$db = \ORM::getDb();
$db->exec(sprintf('