From 72832f3d5a1ca66545f4fcd4e94595bdbeb9d3be Mon Sep 17 00:00:00 2001
From: Piyush Vijay <piyushvijay.1997@gmail.com>
Date: Sat, 24 Mar 2018 02:40:27 +0530
Subject: [PATCH] Revert "Designer: "Add options" dialog doesn't reset when
 canceling: #14110" This reverts commit
 b11ff54572acbe8a71d6554e8f03b5f95590fc04. Signed-off-by: Piyush Vijay
 <piyushvijay.1997@mail.com>

---
 js/designer/move.js                           | 21 ++++++++++---------
 .../database/designer/options_panel.twig      |  6 ++++--
 2 files changed, 15 insertions(+), 12 deletions(-)

diff --git a/js/designer/move.js b/js/designer/move.js
index 639dda8603ad..c2226ce27a6b 100644
--- a/js/designer/move.js
+++ b/js/designer/move.js
@@ -1741,15 +1741,6 @@ function Click_option (id_this, column_name, table_name) {
 
 function Close_option () {
     document.getElementById('designer_optionse').style.display = 'none';
-    document.getElementById('rel_opt').value = '--';
-    document.getElementById('Query').value = '';
-    document.getElementById('new_name').value = '';
-    document.getElementById('operator').value = '---';
-    document.getElementById('groupby').checked = false;
-    document.getElementById('h_rel_opt').value = '--';
-    document.getElementById('h_operator').value = '---';
-    document.getElementById('having').value = '';
-    document.getElementById('orderby').value = '---';
 }
 
 function Select_all (id_this, owner) {
@@ -1854,21 +1845,26 @@ function add_object () {
         where_obj = new where(rel.value, p.value);// make where object
         history_array.push(new history_obj(col_name, where_obj, tab_name, h_tabs[downer + '.' + tab_name], 'Where'));
         sum = sum + 1;
+        rel.value = '--';
+        p.value = '';
     }
     if (document.getElementById('new_name').value !== '') {
         var rename_obj = new rename(document.getElementById('new_name').value);// make Rename object
         history_array.push(new history_obj(col_name, rename_obj, tab_name, h_tabs[downer + '.' + tab_name], 'Rename'));
         sum = sum + 1;
+        document.getElementById('new_name').value = '';
     }
     if (document.getElementById('operator').value !== '---') {
         var aggregate_obj = new aggregate(document.getElementById('operator').value);
         history_array.push(new history_obj(col_name, aggregate_obj, tab_name, h_tabs[downer + '.' + tab_name], 'Aggregate'));
         sum = sum + 1;
+        document.getElementById('operator').value = '---';
         // make aggregate operator
     }
     if (document.getElementById('groupby').checked === true) {
         history_array.push(new history_obj(col_name, 'GroupBy', tab_name, h_tabs[downer + '.' + tab_name], 'GroupBy'));
         sum = sum + 1;
+        document.getElementById('groupby').checked = false;
         // make groupby
     }
     if (document.getElementById('h_rel_opt').value !== '--') {
@@ -1877,18 +1873,23 @@ function add_object () {
             document.getElementById('designer_hint').style.display = 'block';
             return;
         }
+        p = document.getElementById('having');
         where_obj = new having(
             document.getElementById('h_rel_opt').value,
-            document.getElementById('having').value,
+            p.value,
             document.getElementById('h_operator').value
         );// make where object
         history_array.push(new history_obj(col_name, where_obj, tab_name, h_tabs[downer + '.' + tab_name], 'Having'));
         sum = sum + 1;
+        document.getElementById('h_rel_opt').value = '--';
+        document.getElementById('h_operator').value = '---';
+        p.value = ''; // make having
     }
     if (document.getElementById('orderby').value !== '---') {
         var oderby_obj = new orderby(document.getElementById('orderby').value);
         history_array.push(new history_obj(col_name, oderby_obj, tab_name, h_tabs[downer + '.' + tab_name], 'OrderBy'));
         sum = sum + 1;
+        document.getElementById('orderby').value = '---';
         // make orderby
     }
     PMA_ajaxShowMessage(PMA_sprintf(PMA_messages.strObjectsCreated, sum));
diff --git a/templates/database/designer/options_panel.twig b/templates/database/designer/options_panel.twig
index 86b1599a8582..d9b8e86ff213 100644
--- a/templates/database/designer/options_panel.twig
+++ b/templates/database/designer/options_panel.twig
@@ -75,7 +75,8 @@
                                 {% trans 'subquery' %}
                             </td>
                             <td>
-                                <textarea id="Query" cols="18"></textarea>
+                                <textarea id="Query" cols="18">
+                                </textarea>
                             </td>
                         </tr>
                         <tr>
@@ -240,7 +241,8 @@
                                 {% trans 'subquery' %}
                             </td>
                             <td width="102">
-                                <textarea id="having" cols="18"></textarea>
+                                <textarea id="having" cols="18">
+                                </textarea>
                             </td>
                         </tr>
                     </tbody>