From c11b1a78f1c31b25fe17761b5143f285bb111641 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ja=CC=81n=20Mikla=CC=81s=CC=8C?= <neosinner@gmail.com>
Date: Tue, 1 Oct 2019 10:44:50 +0200
Subject: [PATCH] Use Twig to render WooCommerce settings overlay

[MAILPOET-2283]
---
 lib/WooCommerce/Settings.php            | 51 +++++--------------------
 views/woocommerce/settings_overlay.html | 30 +++++++++++++++
 2 files changed, 40 insertions(+), 41 deletions(-)
 create mode 100644 views/woocommerce/settings_overlay.html

diff --git a/lib/WooCommerce/Settings.php b/lib/WooCommerce/Settings.php
index 566804924d..2bb26acb2c 100644
--- a/lib/WooCommerce/Settings.php
+++ b/lib/WooCommerce/Settings.php
@@ -2,29 +2,29 @@
 
 namespace MailPoet\WooCommerce;
 
+use MailPoet\Config\Renderer;
 use MailPoet\Features\FeaturesController;
 use MailPoet\Settings\SettingsController;
-use MailPoet\WP\Functions as WPFunctions;
 
 class Settings {
 
   /** @var FeaturesController */
   private $features_controller;
 
+  /** @var Renderer */
+  private $renderer;
+
   /** @var SettingsController */
   private $settings;
 
-  /** @var WPFunctions */
-  private $wp;
-
   function __construct(
     FeaturesController $features_controller,
-    SettingsController $settings,
-    WPFunctions $wp
+    Renderer $renderer,
+    SettingsController $settings
   ) {
     $this->features_controller = $features_controller;
+    $this->renderer = $renderer;
     $this->settings = $settings;
-    $this->wp = $wp;
   }
 
   function disableWooCommerceSettings() {
@@ -37,40 +37,9 @@ class Settings {
     if (!(bool)$this->settings->get('woocommerce.use_mailpoet_editor')) {
       return;
     }
-    $woocommerce_template_id = $this->settings->get('woocommerce.transactional_email_id');
-    ?>
 
-    <style>
-      /* Hide WooCommerce section with template styling */
-      #email_template_options-description + .form-table {
-        opacity: 0.2;
-        pointer-events: none;
-      }
-
-      /* Position MailPoet buttons over hidden table */
-      .mailpoet-woocommerce-email-overlay {
-        bottom: 320px;
-        left: 0;
-        max-width: 100%;
-        text-align: left;
-        position: absolute;
-        text-align: center;
-        width: 640px;
-        z-index: 1;
-      }
-    </style>
-
-    <div class="mailpoet-woocommerce-email-overlay">
-      <a class="button button-primary" href="?page=mailpoet-newsletter-editor&id=<?php echo $woocommerce_template_id; ?>">
-        <?php echo $this->wp->_x('Customize with MailPoet', 'Button in WooCommerce settings page'); ?>
-      </a>
-      <br>
-      <br>
-      <a href="?page=mailpoet-settings#woocommerce">
-        <?php echo $this->wp->_x('Disable MailPoet customizer', 'Link from WooCommerce plugin to MailPoet'); ?>
-      </a>
-    </div>
-
-    <?php
+    echo $this->renderer->render('woocommerce/settings_overlay.html', [
+      'woocommerce_template_id' => $this->settings->get('woocommerce.transactional_email_id'),
+    ]);
   }
 }
diff --git a/views/woocommerce/settings_overlay.html b/views/woocommerce/settings_overlay.html
new file mode 100644
index 0000000000..bcb1230372
--- /dev/null
+++ b/views/woocommerce/settings_overlay.html
@@ -0,0 +1,30 @@
+<style>
+  /* Hide WooCommerce section with template styling */
+  #email_template_options-description + .form-table {
+    opacity: 0.2;
+    pointer-events: none;
+  }
+
+  /* Position MailPoet buttons over hidden table */
+  .mailpoet-woocommerce-email-overlay {
+    bottom: 320px;
+    left: 0;
+    max-width: 100%;
+    text-align: left;
+    position: absolute;
+    text-align: center;
+    width: 640px;
+    z-index: 1;
+  }
+</style>
+
+<div class="mailpoet-woocommerce-email-overlay">
+  <a class="button button-primary" href="?page=mailpoet-newsletter-editor&id=<%= woocommerce_template_id %>">
+  	<%= _x('Customize with MailPoet', 'Button in WooCommerce settings page') %>
+  </a>
+  <br>
+  <br>
+  <a href="?page=mailpoet-settings#woocommerce">
+    <%= _x('Disable MailPoet customizer', 'Link from WooCommerce plugin to MailPoet') %>
+  </a>
+</div>