Refactor renderHTML method from static to instance method
[MAILPOET-2665]
This commit is contained in:
committed by
Jack Kitterhing
parent
b8dd243ec1
commit
087a610be5
@ -28,6 +28,9 @@ class Forms extends APIEndpoint {
|
|||||||
/** @var UserFlagsController */
|
/** @var UserFlagsController */
|
||||||
private $userFlags;
|
private $userFlags;
|
||||||
|
|
||||||
|
/** @var FormRenderer */
|
||||||
|
private $formRenderer;
|
||||||
|
|
||||||
public $permissions = [
|
public $permissions = [
|
||||||
'global' => AccessControl::PERMISSION_MANAGE_FORMS,
|
'global' => AccessControl::PERMISSION_MANAGE_FORMS,
|
||||||
];
|
];
|
||||||
@ -36,12 +39,14 @@ class Forms extends APIEndpoint {
|
|||||||
Listing\BulkActionController $bulkAction,
|
Listing\BulkActionController $bulkAction,
|
||||||
Listing\Handler $listingHandler,
|
Listing\Handler $listingHandler,
|
||||||
Util\Styles $formStylesUtils,
|
Util\Styles $formStylesUtils,
|
||||||
UserFlagsController $userFlags
|
UserFlagsController $userFlags,
|
||||||
|
FormRenderer $formRenderer
|
||||||
) {
|
) {
|
||||||
$this->bulkAction = $bulkAction;
|
$this->bulkAction = $bulkAction;
|
||||||
$this->listingHandler = $listingHandler;
|
$this->listingHandler = $listingHandler;
|
||||||
$this->formStylesUtils = $formStylesUtils;
|
$this->formStylesUtils = $formStylesUtils;
|
||||||
$this->userFlags = $userFlags;
|
$this->userFlags = $userFlags;
|
||||||
|
$this->formRenderer = $formRenderer;
|
||||||
}
|
}
|
||||||
|
|
||||||
public function get($data = []) {
|
public function get($data = []) {
|
||||||
@ -130,7 +135,7 @@ class Forms extends APIEndpoint {
|
|||||||
|
|
||||||
public function previewEditor($data = []) {
|
public function previewEditor($data = []) {
|
||||||
// html
|
// html
|
||||||
$html = FormRenderer::renderHTML($data);
|
$html = $this->formRenderer->renderHTML($data);
|
||||||
|
|
||||||
// convert shortcodes
|
// convert shortcodes
|
||||||
$html = WPFunctions::get()->doShortcode($html);
|
$html = WPFunctions::get()->doShortcode($html);
|
||||||
|
@ -18,7 +18,7 @@ class Renderer {
|
|||||||
return $html;
|
return $html;
|
||||||
}
|
}
|
||||||
|
|
||||||
public static function renderHTML($form = []) {
|
public function renderHTML($form = []) {
|
||||||
if (isset($form['body']) && !empty($form['body'])) {
|
if (isset($form['body']) && !empty($form['body'])) {
|
||||||
return static::renderBlocks($form['body']);
|
return static::renderBlocks($form['body']);
|
||||||
}
|
}
|
||||||
|
@ -219,7 +219,7 @@ class Widget extends \WP_Widget {
|
|||||||
'form' => $form,
|
'form' => $form,
|
||||||
'title' => $title,
|
'title' => $title,
|
||||||
'styles' => $this->formRenderer->renderStyles($form, '#' . $formId),
|
'styles' => $this->formRenderer->renderStyles($form, '#' . $formId),
|
||||||
'html' => FormRenderer::renderHTML($form),
|
'html' => $this->formRenderer->renderHTML($form),
|
||||||
'before_widget' => $beforeWidget,
|
'before_widget' => $beforeWidget,
|
||||||
'after_widget' => $afterWidget,
|
'after_widget' => $afterWidget,
|
||||||
'before_title' => $beforeTitle,
|
'before_title' => $beforeTitle,
|
||||||
|
Reference in New Issue
Block a user