Skip to content

Commit

Permalink
Enqueue newletter assest using asset controller
Browse files Browse the repository at this point in the history
[MAILPOET-4908]
  • Loading branch information
samnajian committed Jan 4, 2023
1 parent 55dd032 commit dd2eea7
Show file tree
Hide file tree
Showing 3 changed files with 20 additions and 6 deletions.
9 changes: 8 additions & 1 deletion mailpoet/lib/AdminPages/Pages/Newsletters.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
use MailPoet\Config\Installer;
use MailPoet\Config\Menu;
use MailPoet\Entities\NewsletterEntity;
use MailPoet\Form\AssetsController;
use MailPoet\Listing\PageLimit;
use MailPoet\Newsletter\NewslettersRepository;
use MailPoet\NewsletterTemplates\NewsletterTemplatesRepository;
Expand Down Expand Up @@ -54,6 +55,9 @@ class Newsletters {
/** @var AuthorizedSenderDomainController */
private $senderDomainController;

/*** @var AssetsController */
private $assetsController;

public function __construct(
PageRenderer $pageRenderer,
PageLimit $listingPageLimit,
Expand All @@ -65,7 +69,8 @@ public function __construct(
SegmentsSimpleListRepository $segmentsListRepository,
NewslettersRepository $newslettersRepository,
Bridge $bridge,
AuthorizedSenderDomainController $senderDomainController
AuthorizedSenderDomainController $senderDomainController,
AssetsController $assetsController
) {
$this->pageRenderer = $pageRenderer;
$this->listingPageLimit = $listingPageLimit;
Expand All @@ -78,11 +83,13 @@ public function __construct(
$this->newslettersRepository = $newslettersRepository;
$this->bridge = $bridge;
$this->senderDomainController = $senderDomainController;
$this->assetsController = $assetsController;
}

public function render() {
global $wp_roles; // phpcs:ignore Squiz.NamingConventions.ValidVariableName.MemberNotCamelCaps

$this->assetsController->setupAdminPagesDependencies();
$data = [];

$data['items_per_page'] = $this->listingPageLimit->getLimitPerPage('newsletters');
Expand Down
12 changes: 12 additions & 0 deletions mailpoet/lib/Form/AssetsController.php
Original file line number Diff line number Diff line change
Expand Up @@ -180,4 +180,16 @@ public function setupNewsletterEditorDependencies(): void {
*/
\wp_scripts()->do_item('newsletter_editor');
}

public function setupAdminPagesDependencies(): void {

$this->wp->wpEnqueueScript(
'mailpoet_admin_pages',
Env::$assetsUrl . '/dist/js/' . $this->renderer->getJsAsset('admin.js'),
[],
Env::$version,
true
);
$this->wp->wpSetScriptTranslations('mailpoet_admin_pages', 'mailpoet');
}
}
5 changes: 0 additions & 5 deletions mailpoet/views/layout.html
Original file line number Diff line number Diff line change
Expand Up @@ -216,11 +216,6 @@
'admin_vendor.js'
)%>

<%= do_action('mailpoet_scripts_admin_before') %>

<%= javascript(
'admin.js'
)%>

<%if is_loading_3rd_party_enabled() %>
<%= javascript('lib/analytics.js') %>
Expand Down

0 comments on commit dd2eea7

Please sign in to comment.