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

Add emoji sanitisation to forms save API endpoint

Browse Source 
[MAILPOET-3196]
This commit is contained in:
Rostislav Wolny
2020-10-06 11:53:32 +02:00
committed by Veljko V
parent 96f2bfaa20
commit 3eb71f8012
1 changed files with 11 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
lib/API/JSON/v1/Forms.php
View File

@ -16,6 +16,7 @@ use MailPoet\Listing;
use MailPoet\Models\Form; use MailPoet\Models\Form;
use MailPoet\Models\StatisticsForms; use MailPoet\Models\StatisticsForms;
use MailPoet\Settings\UserFlagsController; use MailPoet\Settings\UserFlagsController;
use MailPoet\WP\Emoji;
use MailPoet\WP\Functions as WPFunctions; use MailPoet\WP\Functions as WPFunctions;
class Forms extends APIEndpoint { class Forms extends APIEndpoint {
@ -46,6 +47,9 @@ class Forms extends APIEndpoint {
/** @var FormsRepository */ /** @var FormsRepository */
private $formsRepository; private $formsRepository;
/** @var Emoji */
private $emoji;
public function __construct( public function __construct(
Listing\BulkActionController $bulkAction, Listing\BulkActionController $bulkAction,
Listing\Handler $listingHandler, Listing\Handler $listingHandler,
@ -53,7 +57,8 @@ class Forms extends APIEndpoint {
FormFactory $formFactory, FormFactory $formFactory,
FormsRepository $formsRepository, FormsRepository $formsRepository,
FormsResponseBuilder $formsResponseBuilder, FormsResponseBuilder $formsResponseBuilder,
WPFunctions $wp WPFunctions $wp,
Emoji $emoji
) { ) {
$this->bulkAction = $bulkAction; $this->bulkAction = $bulkAction;
$this->listingHandler = $listingHandler; $this->listingHandler = $listingHandler;
@ -62,6 +67,7 @@ class Forms extends APIEndpoint {
$this->wp = $wp; $this->wp = $wp;
$this->formsRepository = $formsRepository; $this->formsRepository = $formsRepository;
$this->formsResponseBuilder = $formsResponseBuilder; $this->formsResponseBuilder = $formsResponseBuilder;
$this->emoji = $emoji;
} }
public function get($data = []) { public function get($data = []) {
@ -225,6 +231,10 @@ class Forms extends APIEndpoint {
$settings['segments_selected_by'] = 'admin'; $settings['segments_selected_by'] = 'admin';
} }
if ($body !== null) {
$body = $this->emoji->sanitizeEmojisInFormBody($body);
}
$form = Form::createOrUpdate([ $form = Form::createOrUpdate([
'id' => $formId, 'id' => $formId,
'name' => $name, 'name' => $name,