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

Abstracts WP's remove_all_filters function and fixes export test

Browse Source
This commit is contained in:
Vlad
2018-04-13 14:20:53 -04:00
parent 8f6ae91924
commit 6c64ba90c7
2 changed files with 20 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
lib/WP/Hooks.php
View File

@ -26,6 +26,10 @@ class Hooks {
return self::callWithFallback('remove_action', func_get_args());
}
static function removeAllFilters() {
return self::callWithFallback('remove_all_filters', func_get_args());
}
private static function callWithFallback($func, $args) {
$local_func = __NAMESPACE__ . '\\' . $func;
if(function_exists($local_func)) {

31
tests/unit/Subscribers/ImportExport/Export/DynamicSubscribersGetterTest.php
View File

@ -86,6 +86,7 @@ class DynamicSubscribersGetterTest extends \MailPoetTest {
$entity->value = $this->subscribers_data[1][1];
$entity->save();
Hooks::removeAllFilters('mailpoet_get_segment_filters');
Hooks::addAction(
'mailpoet_get_segment_filters',
function($segment_id) {
@ -104,7 +105,7 @@ class DynamicSubscribersGetterTest extends \MailPoetTest {
$data = array();
foreach($subscriber as $key => $value) {
if(in_array($key, array(
'first_name', 'last_name', 'email', 'global_status',
'first_name', 'last_name', 'email', 'global_status',
'status', 'list_status', 'segment_name', 1
)))
$data[$key] = $value;
@ -122,7 +123,7 @@ class DynamicSubscribersGetterTest extends \MailPoetTest {
'last_name' => 'Smith',
'email' => 'adam@smith.com',
'status' => Subscriber::STATUS_UNCONFIRMED,
'global_status' => Subscriber::STATUS_UNCONFIRMED,
'global_status' => Subscriber::STATUS_UNCONFIRMED,
'list_status' => Subscriber::STATUS_UNCONFIRMED,
'segment_name' => 'Newspapers',
1 => null
@ -132,7 +133,7 @@ class DynamicSubscribersGetterTest extends \MailPoetTest {
'last_name' => 'Jane',
'email' => 'mary@jane.com',
'status' => Subscriber::STATUS_SUBSCRIBED,
'global_status' => Subscriber::STATUS_SUBSCRIBED,
'global_status' => Subscriber::STATUS_SUBSCRIBED,
'list_status' => Subscriber::STATUS_SUBSCRIBED,
'segment_name' => 'Newspapers',
1 => 'Brazil',
@ -150,7 +151,7 @@ class DynamicSubscribersGetterTest extends \MailPoetTest {
'last_name' => 'Smith',
'email' => 'adam@smith.com',
'status' => Subscriber::STATUS_UNCONFIRMED,
'global_status' => Subscriber::STATUS_UNCONFIRMED,
'global_status' => Subscriber::STATUS_UNCONFIRMED,
'list_status' => Subscriber::STATUS_UNCONFIRMED,
'segment_name' => 'Newspapers',
1 => null
@ -160,20 +161,20 @@ class DynamicSubscribersGetterTest extends \MailPoetTest {
'last_name' => 'Jane',
'email' => 'mary@jane.com',
'status' => Subscriber::STATUS_SUBSCRIBED,
'global_status' => Subscriber::STATUS_SUBSCRIBED,
'global_status' => Subscriber::STATUS_SUBSCRIBED,
'list_status' => Subscriber::STATUS_SUBSCRIBED,
'segment_name' => 'Newspapers',
1 => 'Brazil',
)
]);
expect($this->filterSubscribersData($getter->get()))->equals([
array(
'first_name' => 'Adam',
'last_name' => 'Smith',
'email' => 'adam@smith.com',
'status' => Subscriber::STATUS_UNCONFIRMED,
'global_status' => Subscriber::STATUS_UNCONFIRMED,
'global_status' => Subscriber::STATUS_UNCONFIRMED,
'list_status' => Subscriber::STATUS_UNCONFIRMED,
'segment_name' => 'Journals',
1 => null
@ -183,7 +184,7 @@ class DynamicSubscribersGetterTest extends \MailPoetTest {
'last_name' => 'Kookoo',
'email' => 'john@kookoo.com',
'status' => Subscriber::STATUS_UNCONFIRMED,
'global_status' => Subscriber::STATUS_UNCONFIRMED,
'global_status' => Subscriber::STATUS_UNCONFIRMED,
'list_status' => Subscriber::STATUS_UNCONFIRMED,
'segment_name' => 'Journals',
1 => null,
@ -193,7 +194,7 @@ class DynamicSubscribersGetterTest extends \MailPoetTest {
'last_name' => 'Newman',
'email' => 'paul@newman.com',
'status' => Subscriber::STATUS_UNCONFIRMED,
'global_status' => Subscriber::STATUS_UNCONFIRMED,
'global_status' => Subscriber::STATUS_UNCONFIRMED,
'list_status' => Subscriber::STATUS_UNCONFIRMED,
'segment_name' => 'Journals',
1 => null,
@ -211,7 +212,7 @@ class DynamicSubscribersGetterTest extends \MailPoetTest {
'last_name' => 'Smith',
'email' => 'adam@smith.com',
'status' => Subscriber::STATUS_UNCONFIRMED,
'global_status' => Subscriber::STATUS_UNCONFIRMED,
'global_status' => Subscriber::STATUS_UNCONFIRMED,
'list_status' => Subscriber::STATUS_UNCONFIRMED,
'segment_name' => 'Newspapers',
1 => null
@ -221,13 +222,13 @@ class DynamicSubscribersGetterTest extends \MailPoetTest {
'last_name' => 'Jane',
'email' => 'mary@jane.com',
'status' => Subscriber::STATUS_SUBSCRIBED,
'global_status' => Subscriber::STATUS_SUBSCRIBED,
'global_status' => Subscriber::STATUS_SUBSCRIBED,
'list_status' => Subscriber::STATUS_SUBSCRIBED,
'segment_name' => 'Newspapers',
1 => 'Brazil',
)
]);
expect($this->filterSubscribersData($getter->get()))->equals([]);
expect($this->filterSubscribersData($getter->get()))->equals([
@ -236,7 +237,7 @@ class DynamicSubscribersGetterTest extends \MailPoetTest {
'last_name' => 'Smith',
'email' => 'adam@smith.com',
'status' => Subscriber::STATUS_UNCONFIRMED,
'global_status' => Subscriber::STATUS_UNCONFIRMED,
'global_status' => Subscriber::STATUS_UNCONFIRMED,
'list_status' => Subscriber::STATUS_UNCONFIRMED,
'segment_name' => 'Journals',
1 => null
@ -246,7 +247,7 @@ class DynamicSubscribersGetterTest extends \MailPoetTest {
'last_name' => 'Kookoo',
'email' => 'john@kookoo.com',
'status' => Subscriber::STATUS_UNCONFIRMED,
'global_status' => Subscriber::STATUS_UNCONFIRMED,
'global_status' => Subscriber::STATUS_UNCONFIRMED,
'list_status' => Subscriber::STATUS_UNCONFIRMED,
'segment_name' => 'Journals',
1 => null,
@ -259,7 +260,7 @@ class DynamicSubscribersGetterTest extends \MailPoetTest {
'last_name' => 'Newman',
'email' => 'paul@newman.com',
'status' => Subscriber::STATUS_UNCONFIRMED,
'global_status' => Subscriber::STATUS_UNCONFIRMED,
'global_status' => Subscriber::STATUS_UNCONFIRMED,
'list_status' => Subscriber::STATUS_UNCONFIRMED,
'segment_name' => 'Journals',
1 => null,