From f9d7058cd61858d35710ede475c51176eff429fa Mon Sep 17 00:00:00 2001
From: ninadsp <ninadsp16289@gmail.com>
Date: Wed, 30 Jun 2010 00:25:22 +0530
Subject: [PATCH] We need to close the dialog when reloading the database after
 renaming it.  The code did not do this, added it.

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

diff --git a/js/functions.js b/js/functions.js
index c46ae97da..c17d2883e 100755
--- a/js/functions.js
+++ b/js/functions.js
@@ -1878,7 +1878,6 @@ $(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 {
@@ -1893,7 +1892,8 @@ $(document).ready(function() {
         event.preventDefault();
 
         var curr_table_name = window.parent.table;
-        var curr_column_name = $(this).parents('tr').children('th').children('label').text();
+        var curr_row = $(this).parents('tr');
+        var curr_column_name = $(curr_row).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);
-                    $(this).remove();
+                    $(curr_row).hide("medium").remove();
                 }
                 else {
                     PMA_ajaxShowMessage(PMA_messages['strErrorProcessingRequest'] + " : " + data.error);
@@ -1927,6 +1927,7 @@ $(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 {