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

Add filter form manage subscription form fields

Browse Source 
[MAILPOET-1635]
This commit is contained in:
Rostislav Wolny
2020-07-30 14:18:07 +02:00
committed by Veljko V
parent 7c979c9b56
commit 7c2e9162a1
2 changed files with 50 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
lib/Subscription/ManageSubscriptionFormRenderer.php
View File

@ -74,6 +74,8 @@ class ManageSubscriptionFormRenderer {
]
);
$form = $this->wp->applyFilters('mailpoet_manage_subscription_page_form_fields', $form);
$templateData = [
'actionUrl' => admin_url('admin-post.php'),
'redirectUrl' => $this->urlHelper->getCurrentUrl(),

66
tests/integration/Subscription/ManageSubscriptionFormRendererTest.php
View File

@ -6,6 +6,7 @@ use MailPoet\Entities\SegmentEntity;
use MailPoet\Entities\SubscriberEntity;
use MailPoet\Entities\SubscriberSegmentEntity;
use MailPoet\Models\Subscriber;
use MailPoet\WP\Functions as WPFunctions;
class ManageSubscriptionFormRendererTest extends \MailPoetTest {
/** @var ManageSubscriptionFormRenderer */
@ -18,24 +19,7 @@ class ManageSubscriptionFormRendererTest extends \MailPoetTest {
}
public function testItGeneratesForm() {
$segment = new SegmentEntity();
$segment->setName('Test segment');
$segment->setDescription('Description');
$segment->setType(SegmentEntity::TYPE_DEFAULT);
$this->entityManager->persist($segment);
$subscriber = new SubscriberEntity();
$subscriber->setStatus(SubscriberEntity::STATUS_SUBSCRIBED);
$subscriber->setFirstName('Fname');
$subscriber->setLastName('Lname');
$subscriber->setEmail('subscriber@test.com');
$subscriberSegment = new SubscriberSegmentEntity($segment, $subscriber, SubscriberEntity::STATUS_SUBSCRIBED);
$this->entityManager->persist($subscriberSegment);
$subscriber->getSegments()->add($subscriberSegment);
$this->entityManager->persist($subscriber);
$this->entityManager->flush();
$subscriber = $this->getSubscriber($this->getSegment());
$form = $this->formRenderer->renderForm(Subscriber::findOne($subscriber->getId()));
expect($form)->regExp('/<form class="mailpoet-manage-subscription" method="post" action="[a-z0-9:\/\.]+wp-admin\/admin-post.php" novalidate>/');
expect($form)->contains('<input type="hidden" name="data[email]" value="subscriber@test.com" />');
@ -45,6 +29,52 @@ class ManageSubscriptionFormRendererTest extends \MailPoetTest {
expect($form)->contains('Need to change your email address? Unsubscribe here, then simply sign up again.');
}
public function testItAppliesFieldsFilter() {
$subscriber = $this->getSubscriber($this->getSegment());
$wp = $this->diContainer->get(WPFunctions::class);
$wp->addFilter('mailpoet_manage_subscription_page_form_fields', function($fields) {
$fields[] = [
'type' => 'text',
'name' => 'Additional info',
'id' => 'additional_info',
'params' => [
'label' => 'Additional info',
],
];
return $fields;
});
$form = $this->formRenderer->renderForm(Subscriber::findOne($subscriber->getId()));
expect($form)->regExp('/<input type="text" class="mailpoet_text" name="data\[[a-zA-Z0-9=_]+\]" title="Additional info" value="" \/>/');
}
private function getSegment(): SegmentEntity {
$segment = new SegmentEntity();
$segment->setName('Test segment');
$segment->setDescription('Description');
$segment->setType(SegmentEntity::TYPE_DEFAULT);
$this->entityManager->persist($segment);
$this->entityManager->flush();
return $segment;
}
private function getSubscriber(SegmentEntity $segment = null): SubscriberEntity {
$subscriber = new SubscriberEntity();
$subscriber->setStatus(SubscriberEntity::STATUS_SUBSCRIBED);
$subscriber->setFirstName('Fname');
$subscriber->setLastName('Lname');
$subscriber->setEmail('subscriber@test.com');
if ($segment) {
$subscriberSegment = new SubscriberSegmentEntity($segment, $subscriber, SubscriberEntity::STATUS_SUBSCRIBED);
$this->entityManager->persist($subscriberSegment);
$subscriber->getSegments()->add($subscriberSegment);
}
$this->entityManager->persist($subscriber);
$this->entityManager->flush();
return $subscriber;
}
private function cleanup() {
$this->truncateEntity(SubscriberEntity::class);
$this->truncateEntity(SubscriberSegmentEntity::class);