The branch, master has been updated via 036d1a8397ac9da6c166f1d28d3fe9d3898c4213 (commit) from e81b6c1cfd5f95ebd4809b54d6aaeed80beea52f (commit)
- Log ----------------------------------------------------------------- commit 036d1a8397ac9da6c166f1d28d3fe9d3898c4213 Author: Aris Feryanto aris_feryanto@yahoo.com Date: Mon Aug 29 15:00:06 2011 +0800
Correctly show error message when unable to save UI property
-----------------------------------------------------------------------
Summary of changes: libraries/Table.class.php | 3 ++- sql.php | 6 +++--- 2 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/libraries/Table.class.php b/libraries/Table.class.php index 3816b9b..34b144a 100644 --- a/libraries/Table.class.php +++ b/libraries/Table.class.php @@ -1509,7 +1509,8 @@ class PMA_Table // there is no $table_create_time, or // supplied $table_create_time is older than current create time, // so don't save - return false; + return PMA_Message::error(sprintf( + __('Cannot save UI property "%s". The changes made will not be persistent after you refresh this page. Please check if the table structure has been changed.'), $property)); } } // save the value diff --git a/sql.php b/sql.php index b233400..8b8cf12 100644 --- a/sql.php +++ b/sql.php @@ -185,17 +185,17 @@ if (isset($_REQUEST['set_col_prefs']) && $_REQUEST['set_col_prefs'] == true) { if (isset($_REQUEST['col_order'])) { $col_order = explode(',', $_REQUEST['col_order']); $retval = $pmatable->setUiProp(PMA_Table::PROP_COLUMN_ORDER, $col_order, $_REQUEST['table_create_time']); - if ($retval !== true) { + if (gettype($retval) != 'boolean') { PMA_ajaxResponse($retval->getString(), false); } }
// set column visibility - if (isset($_REQUEST['col_visib'])) { + if ($retval === true && isset($_REQUEST['col_visib'])) { $col_visib = explode(',', $_REQUEST['col_visib']); $retval = $pmatable->setUiProp(PMA_Table::PROP_COLUMN_VISIB, $col_visib, $_REQUEST['table_create_time']); - if ($retval !== true) { + if (gettype($retval) != 'boolean') { PMA_ajaxResponse($retval->getString(), false); } }
hooks/post-receive