From 589d918ee7e4e747ce55f2f0c0905bc8d6fe1dc9 Mon Sep 17 00:00:00 2001 From: Marc Delisle Date: Tue, 8 Feb 2011 17:36:05 -0500 Subject: [PATCH] Errors from the Ajax create table dialog where not displayed --- js/functions.js | 3 ++- libraries/tbl_properties.inc.php | 2 +- tbl_create.php | 16 ++++++++++------ 3 files changed, 13 insertions(+), 8 deletions(-) diff --git a/js/functions.js b/js/functions.js index 4fcfb1d72..57b4644e5 100644 --- a/js/functions.js +++ b/js/functions.js @@ -1743,6 +1743,7 @@ $(document).ready(function() { //User wants to submit the form $.post($form.attr('action'), $form.serialize() + "&do_save_data=" + $(this).val(), function(data) { if(data.success == true) { + $('#properties_message').html(''); PMA_ajaxShowMessage(data.message); $("#create_table_dialog").dialog("close").remove(); @@ -1794,7 +1795,7 @@ $(document).ready(function() { } } else { - PMA_ajaxShowMessage(data.error); + $('#properties_message').html(data.error); } }) // end $.post() }) // end create table form (save) diff --git a/libraries/tbl_properties.inc.php b/libraries/tbl_properties.inc.php index 4bc1029af..14e493651 100644 --- a/libraries/tbl_properties.inc.php +++ b/libraries/tbl_properties.inc.php @@ -786,7 +786,7 @@ if ($action == 'tbl_create.php') { /> - +
diff --git a/tbl_create.php b/tbl_create.php index 6d569b50d..4d3171ad9 100644 --- a/tbl_create.php +++ b/tbl_create.php @@ -350,12 +350,16 @@ if (isset($_REQUEST['do_save_data'])) { } exit; } else { - PMA_mysqlDie('', '', '', $err_url, false); - // An error happened while inserting/updating a table definition. - // to prevent total loss of that data, we embed the form once again. - // The variable $regenerate will be used to restore data in libraries/tbl_properties.inc.php - $num_fields = $_REQUEST['orig_num_fields']; - $regenerate = true; + if ($GLOBALS['is_ajax_request'] == true) { + PMA_ajaxResponse(PMA_DBI_getError(), false); + } else { + PMA_mysqlDie('', '', '', $err_url, false); + // An error happened while inserting/updating a table definition. + // to prevent total loss of that data, we embed the form once again. + // The variable $regenerate will be used to restore data in libraries/tbl_properties.inc.php + $num_fields = $_REQUEST['orig_num_fields']; + $regenerate = true; + } } } // end do create table