Add admin.js to all admin pages using AssetController

[MAILPOET-4908]

mailpoet/lib/AdminPages/PageRenderer.php

@@ -12,6 +12,7 @@ use MailPoet\Cron\Workers\SubscribersCountCacheRecalculation;
 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;
@@ -70,6 +71,9 @@ class PageRenderer {
   /** @var WPFunctions */
   private $wp;
 
+  /*** @var AssetsController */
+  private $assetsController;
+
   public function __construct(
     Bridge $bridge,
     Renderer $renderer,
@@ -84,7 +88,8 @@ class PageRenderer {
     SubscribersFeature $subscribersFeature,
     TrackingConfig $trackingConfig,
     TransientCache $transientCache,
-    WPFunctions $wp
+    WPFunctions $wp,
+    AssetsController $assetsController
   ) {
     $this->bridge = $bridge;
     $this->renderer = $renderer;
@@ -100,6 +105,7 @@ class PageRenderer {
     $this->trackingConfig = $trackingConfig;
     $this->transientCache = $transientCache;
     $this->wp = $wp;
+    $this->assetsController = $assetsController;
   }
 
   /**
@@ -193,6 +199,7 @@ class PageRenderer {
         $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);

mailpoet/lib/AdminPages/Pages/Newsletters.php

@@ -8,7 +8,6 @@ use MailPoet\Config\Env;
 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 @@ class Newsletters {
     SegmentsSimpleListRepository $segmentsListRepository,
     NewslettersRepository $newslettersRepository,
     Bridge $bridge,
-    AuthorizedSenderDomainController $senderDomainController,
-    AssetsController $assetsController
+    AuthorizedSenderDomainController $senderDomainController
   ) {
     $this->pageRenderer = $pageRenderer;
     $this->listingPageLimit = $listingPageLimit;
@@ -83,13 +78,11 @@ class Newsletters {
     $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');