Add admin.js to all admin pages using AssetController

[MAILPOET-4908]
mailpoet · Jan 6, 2023 · 4ff83a3 · 4ff83a3
1 parent 53cb253
commit 4ff83a3
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 9 deletions.
diff --git a/mailpoet/lib/AdminPages/PageRenderer.php b/mailpoet/lib/AdminPages/PageRenderer.php
@@ -11,6 +11,7 @@
 use MailPoet\Entities\SegmentEntity;
 use MailPoet\Entities\TagEntity;
 use MailPoet\Features\FeaturesController;
+use MailPoet\Form\AssetsController;
 use MailPoet\Referrals\ReferralDetector;
 use MailPoet\Segments\SegmentsRepository;
 use MailPoet\Services\Bridge;
@@ -69,6 +70,9 @@ class PageRenderer {
   /** @var WPFunctions */
   private $wp;
 
+  /*** @var AssetsController */
+  private $assetsController;
+
   public function __construct(
     Bridge $bridge,
     Renderer $renderer,
@@ -83,7 +87,8 @@ public function __construct(
     SubscribersFeature $subscribersFeature,
     TrackingConfig $trackingConfig,
     TransientCache $transientCache,
-    WPFunctions $wp
+    WPFunctions $wp,
+    AssetsController $assetsController
   ) {
     $this->bridge = $bridge;
     $this->renderer = $renderer;
@@ -99,6 +104,7 @@ public function __construct(
     $this->trackingConfig = $trackingConfig;
     $this->transientCache = $transientCache;
     $this->wp = $wp;
+    $this->assetsController = $assetsController;
   }
 
   /**
@@ -190,6 +196,7 @@ class_exists(Debugger::class)
         $this->subscribersCountCacheRecalculation->schedule();
       }
 
+      $this->assetsController->setupAdminPagesDependencies();
       // We are in control of the template and the data can be considered safe at this point
       // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped, WordPressDotOrg.sniffs.OutputEscaping.UnescapedOutputParameter
       echo $this->renderer->render($template, $data + $defaults);

diff --git a/mailpoet/lib/AdminPages/Pages/Newsletters.php b/mailpoet/lib/AdminPages/Pages/Newsletters.php
@@ -8,7 +8,6 @@
 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;
@@ -55,9 +54,6 @@ class Newsletters {
   /** @var AuthorizedSenderDomainController */
   private $senderDomainController;
 
-  /*** @var AssetsController */
-  private $assetsController;
-
   public function __construct(
     PageRenderer $pageRenderer,
     PageLimit $listingPageLimit,
@@ -69,8 +65,7 @@ public function __construct(
     SegmentsSimpleListRepository $segmentsListRepository,
     NewslettersRepository $newslettersRepository,
     Bridge $bridge,
-    AuthorizedSenderDomainController $senderDomainController,
-    AssetsController $assetsController
+    AuthorizedSenderDomainController $senderDomainController
   ) {
     $this->pageRenderer = $pageRenderer;
     $this->listingPageLimit = $listingPageLimit;
@@ -83,13 +78,11 @@ 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');