From 1395cdc1d0376f867078a346366992f97b4e815a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Maur=C3=ADcio=20Meneghini=20Fauth?= <mauricio@fauth.dev>
Date: Mon, 2 Mar 2020 22:01:16 -0300
Subject: [PATCH] Extract HTML from MultSubmits::getHtmlForCopyMultipleTables
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Maurício Meneghini Fauth <mauricio@fauth.dev>
---
 libraries/classes/MultSubmits.php             | 37 ++++---------
 .../mult_submits/copy_multiple_tables.twig    | 54 +++++++++++++++++++
 2 files changed, 64 insertions(+), 27 deletions(-)
 create mode 100644 templates/mult_submits/copy_multiple_tables.twig

diff --git a/libraries/classes/MultSubmits.php b/libraries/classes/MultSubmits.php
index 3801701be5..ea700a0819 100644
--- a/libraries/classes/MultSubmits.php
+++ b/libraries/classes/MultSubmits.php
@@ -31,12 +31,16 @@ class MultSubmits
     /** @var Operations */
     private $operations;
 
+    /** @var Template */
+    private $template;
+
     public function __construct()
     {
         $this->transformations = new Transformations();
         $relation = new Relation($GLOBALS['dbi']);
         $this->relationCleanup = new RelationCleanup($GLOBALS['dbi'], $relation);
         $this->operations = new Operations($GLOBALS['dbi'], $relation);
+        $this->template = new Template();
     }
 
     /**
@@ -393,9 +397,6 @@ class MultSubmits
      */
     public function getHtmlForCopyMultipleTables($action, array $urlParams)
     {
-        $html = '<form id="ajax_form" action="' . $action . '" method="post">';
-        $html .= Url::getHiddenInputs($urlParams);
-        $html .= '<fieldset class = "input">';
         $databasesList = $GLOBALS['dblist']->databases;
         foreach ($databasesList as $key => $databaseName) {
             if ($databaseName == $GLOBALS['db']) {
@@ -403,30 +404,12 @@ class MultSubmits
                 break;
             }
         }
-        $html .= '<strong><label for="db_name_dropdown">' . __('Database') . ':</label></strong>';
-        $html .= '<select id="db_name_dropdown" class="halfWidth" name="target_db" >'
-            . $databasesList->getHtmlOptions(true, false)
-            . '</select>';
-        $html .= '<br><br>';
-        $html .= '<strong><label>' . __('Options') . ':</label></strong><br>';
-        $html .= '<input type="radio" id ="what_structure" value="structure" name="what">';
-        $html .= '<label for="what_structure">' . __('Structure only') . '</label><br>';
-        $html .= '<input type="radio" id ="what_data" value="data" name="what" checked="checked">';
-        $html .= '<label for="what_data">' . __('Structure and data') . '</label><br>';
-        $html .= '<input type="radio" id ="what_dataonly" value="dataonly" name="what">';
-        $html .= '<label for="what_dataonly">' . __('Data only') . '</label><br><br>';
-        $html .= '<input type="checkbox" id="checkbox_drop" value="true" name="drop_if_exists">';
-        $html .= '<label for="checkbox_drop">' . __('Add DROP TABLE') . '</label><br>';
-        $html .= '<input type="checkbox" id="checkbox_auto_increment_cp" value="1" name="sql_auto_increment">';
-        $html .= '<label for="checkbox_auto_increment_cp">' . __('Add AUTO INCREMENT value') . '</label><br>';
-        $html .= '<input type="checkbox" id="checkbox_constraints" value="1" name="sql_auto_increment" checked="checked">';
-        $html .= '<label for="checkbox_constraints">' . __('Add constraints') . '</label><br><br>';
-        $html .= '<input name="adjust_privileges" value="1" id="checkbox_adjust_privileges" checked="checked" type="checkbox">';
-        $html .= '<label for="checkbox_adjust_privileges">' . __('Adjust privileges') . '<a href="./doc/html/faq.html#faq6-39" target="documentation"><img src="themes/dot.gif" title="Documentation" alt="Documentation" class="icon ic_b_help"></a></label>';
-        $html .= '</fieldset>';
-        $html .= '<input type="hidden" name="mult_btn" value="' . __('Yes') . '">';
-        $html .= '</form>';
-        return $html;
+
+        return $this->template->render('mult_submits/copy_multiple_tables', [
+            'action' => $action,
+            'url_params' => $urlParams,
+            'options' => $databasesList->getHtmlOptions(true, false),
+        ]);
     }
 
     /**
diff --git a/templates/mult_submits/copy_multiple_tables.twig b/templates/mult_submits/copy_multiple_tables.twig
new file mode 100644
index 0000000000..2f6ce211dc
--- /dev/null
+++ b/templates/mult_submits/copy_multiple_tables.twig
@@ -0,0 +1,54 @@
+<form id="ajax_form" action="{{ action|raw }}" method="post">
+  {{ get_hidden_inputs(url_params) }}
+
+  <fieldset>
+    <strong><label for="db_name_dropdown">{% trans 'Database:' %}</label></strong>
+    <select id="db_name_dropdown" name="target_db">
+      {{ options|raw }}
+    </select>
+
+    <br><br>
+
+    <strong><label>{% trans 'Options:' %}</label></strong>
+
+    <br>
+
+    <input type="radio" id="what_structure" value="structure" name="what">
+    <label for="what_structure">{% trans 'Structure only' %}</label>
+
+    <br>
+
+    <input type="radio" id="what_data" value="data" name="what" checked>
+    <label for="what_data">{% trans 'Structure and data' %}</label>
+
+    <br>
+
+    <input type="radio" id="what_dataonly" value="dataonly" name="what">
+    <label for="what_dataonly">{% trans 'Data only' %}</label>
+
+    <br><br>
+
+    <input type="checkbox" id="checkbox_drop" value="true" name="drop_if_exists">
+    <label for="checkbox_drop">{% trans 'Add DROP TABLE' %}</label>
+
+    <br>
+
+    <input type="checkbox" id="checkbox_auto_increment_cp" value="1" name="sql_auto_increment">
+    <label for="checkbox_auto_increment_cp">{% trans 'Add AUTO INCREMENT value' %}</label>
+
+    <br>
+
+    <input type="checkbox" id="checkbox_constraints" value="1" name="sql_auto_increment" checked>
+    <label for="checkbox_constraints">{% trans 'Add constraints' %}</label>
+
+    <br><br>
+
+    <input type="checkbox" name="adjust_privileges" value="1" id="checkbox_adjust_privileges" checked>
+    <label for="checkbox_adjust_privileges">
+      {% trans 'Adjust privileges' %}
+      {{ show_docu('faq', 'faq6-39') }}
+    </label>
+  </fieldset>
+
+  <input type="hidden" name="mult_btn" value="{% trans 'Yes' %}">
+</form>