From b48033fbd6b53fe48144fe7a137a013205f5ae1c Mon Sep 17 00:00:00 2001
From: ninadsp <ninadsp16289@gmail.com>
Date: Wed, 30 Jun 2010 00:28:15 +0530
Subject: [PATCH] Revert "We need to close the dialog when reloading the
 database after renaming"

This reverts commit f9d7058cd61858d35710ede475c51176eff429fa.
---
 js/functions.js | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/js/functions.js b/js/functions.js
index c17d2883e..c46ae97da 100755
--- a/js/functions.js
+++ b/js/functions.js
@@ -1878,6 +1878,7 @@ $(document).ready(function() {
             $.get(url, {'is_js_confirmed' : 1, 'ajax_request' : true}, function(data) {
                 if(data.success == true) {
                     PMA_ajaxShowMessage(data.message);
+                    //need to find a better solution here.  The icon should be replaced
                     $(curr_row).hide("medium").remove();
                 }
                 else {
@@ -1892,8 +1893,7 @@ $(document).ready(function() {
         event.preventDefault();
 
         var curr_table_name = window.parent.table;
-        var curr_row = $(this).parents('tr');
-        var curr_column_name = $(curr_row).children('th').children('label').text();
+        var curr_column_name = $(this).parents('tr').children('th').children('label').text();
         var question = PMA_messages['strDoYouReally'] + ' :\n ALTER TABLE `' + curr_table_name + '` DROP `' + curr_column_name + '`';
 
         $(this).PMA_confirm(question, $(this).attr('href'), function(url) {
@@ -1903,7 +1903,7 @@ $(document).ready(function() {
             $.get(url, {'is_js_confirmed' : 1, 'ajax_request' : true}, function(data) {
                 if(data.success == true) {
                     PMA_ajaxShowMessage(data.message);
-                    $(curr_row).hide("medium").remove();
+                    $(this).remove();
                 }
                 else {
                     PMA_ajaxShowMessage(PMA_messages['strErrorProcessingRequest'] + " : " + data.error);
@@ -1927,7 +1927,6 @@ $(document).ready(function() {
             $.get(url, {'is_js_confirmed' : 1, 'ajax_request' : true}, function(data) {
                 if(data.success == true) {
                     PMA_ajaxShowMessage(data.message);
-                    // @todo Need a better response here. Just removing the button shouldn't be the best solution.
                     $(this).remove();
                 }
                 else {