nettika/phpmyadmin
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Revert "We need to close the dialog when reloading the database after renaming"

Browse Source 
This reverts commit f9d7058cd6.
This commit is contained in:
ninadsp
2010-06-30 00:28:15 +05:30
parent f9d7058cd6
commit b48033fbd6
1 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
js/functions.js
View File

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