From abfb718c89c7e68ecb5e5c25c55d426c449d9883 Mon Sep 17 00:00:00 2001 From: Madhura Jayaratne Date: Wed, 23 Feb 2011 23:38:34 +0530 Subject: [PATCH] Inline edit extended to fields of type set. --- js/sql.js | 60 ++++++++++++++++++++++++++++------- libraries/display_tbl.lib.php | 7 +++- sql.php | 28 ++++++++++++++++ 3 files changed, 83 insertions(+), 12 deletions(-) diff --git a/js/sql.js b/js/sql.js index 34e1e74c0..1cbbca2f2 100644 --- a/js/sql.js +++ b/js/sql.js @@ -458,9 +458,9 @@ $(document).ready(function() { */ var relation_curr_value = $this_field.find('a').text(); /** - * @var enum_curr_value String current value of the field (for fields that are of type enum). + * @var curr_value String current value of the field (for fields that are of type enum or set). */ - var enum_curr_value = $this_field.text(); + var curr_value = $this_field.text(); if($this_field.is(':not(.not_null)')){ // add a checkbox to mark null for all the field that are nullable. @@ -490,8 +490,13 @@ $(document).ready(function() { // if 'chechbox_null__' is clicked empty the corresponding select/editor. $('.checkbox_null_' + field_name + '_' + this_row_index).bind('click', function(e) { - if ($this_field.is('.enum, .set')) { + if ($this_field.is('.enum')) { $this_field.find('select').attr('value', ''); + } else if ($this_field.is('.set')) { + $this_field.find('select').find('option').each(function() { + var $option = $(this); + $option.attr('selected', false); + }) } else if ($this_field.is('.relation')) { // if the dropdown is there to select the foreign value if ($this_field.find('select').length > 0) { @@ -511,7 +516,7 @@ $(document).ready(function() { // In each input sibling, wrap the current value in a textarea // and store the current value in a hidden span - if($this_field.is(':not(.truncated, .transformed, .relation, .enum, .null)')) { + if($this_field.is(':not(.truncated, .transformed, .relation, .enum, .set, .null)')) { // handle non-truncated, non-transformed, non-relation values // We don't need to get any more data, just wrap the value $this_field.append(''); @@ -582,7 +587,7 @@ $(document).ready(function() { 'table' : window.parent.table, 'column' : field_name, 'token' : window.parent.token, - 'curr_value' : enum_curr_value + 'curr_value' : curr_value } $.post('sql.php', post_params, function(data) { @@ -591,6 +596,29 @@ $(document).ready(function() { $(".original_data").hide(); }) // end $.post() } + else if($this_field.is('.set')) { + /** @lends jQuery */ + //handle set fields + + /** + * @var post_params Object containing parameters for the POST request + */ + var post_params = { + 'ajax_request' : true, + 'get_set_values' : true, + 'db' : window.parent.db, + 'table' : window.parent.table, + 'column' : field_name, + 'token' : window.parent.token, + 'curr_value' : curr_value + } + + $.post('sql.php', post_params, function(data) { + $this_field.append(data.select); + $this_field.append(''+data_value+''); + $(".original_data").hide(); + }) // end $.post() + } else if($this_field.is('.null')) { //handle null fields $this_field.append(''); @@ -701,13 +729,17 @@ $(document).ready(function() { if (is_null) { sql_query += ' ' + field_name + "=NULL , "; } else { - if($this_field.is(":not(.relation, .enum)")) { + if($this_field.is(":not(.relation, .enum, .set)")) { this_field_params[field_name] = $this_field.find('textarea').val(); if($this_field.is('.transformed')) { $.extend(transform_fields, this_field_params); } - } - else { + } else if ($this_field.is('.set')) { + $test_element = $this_field.find('select'); + this_field_params[field_name] = $test_element.map(function(){ + return $(this).val(); + }).get().join(","); + } else { // results from a drop-down $test_element = $this_field.find('select'); if ($test_element.length != 0) { @@ -777,7 +809,7 @@ $(document).ready(function() { $this_sibling.addClass('null'); } else { $this_sibling.removeClass('null'); - if($this_sibling.is(':not(.relation, .enum)')) { + if($this_sibling.is(':not(.relation, .enum, .set)')) { /** * @var new_html String containing value of the data field after edit */ @@ -821,9 +853,15 @@ $(document).ready(function() { return false; } }) - } - if($this_sibling.is('.enum')) { + } else if ($this_sibling.is('.enum')) { new_html = new_value; + } else if ($this_sibling.is('.set')) { + if (new_value != null) { + $.each(new_value, function(key, value) { + new_html = new_html + value + ','; + }) + new_html = new_html.substring(0, new_html.length-1); + } } } $this_sibling.html(new_html); diff --git a/libraries/display_tbl.lib.php b/libraries/display_tbl.lib.php index a57abb253..d47e94df7 100644 --- a/libraries/display_tbl.lib.php +++ b/libraries/display_tbl.lib.php @@ -2419,6 +2419,11 @@ function PMA_prepare_row_data($class, $condition_field, $analyzed_sql, $meta, $m $enum_class = ' enum'; } + $set_class = ''; + if(strpos($meta->flags, 'set') !== false) { + $set_class = ' set'; + } + $mime_type_class = ''; if(isset($meta->mimetype)) { $mime_type_class = ' ' . preg_replace('/\//', '_', $meta->mimetype); @@ -2428,7 +2433,7 @@ function PMA_prepare_row_data($class, $condition_field, $analyzed_sql, $meta, $m $result = ' class="' . $class . ($condition_field ? ' condition' : '') . $nowrap . ' ' . ($is_field_truncated ? ' truncated' : '') . ($transform_function != $default_function ? ' transformed' : '') - . $enum_class . $mime_type_class . '">'; + . $enum_class . $set_class . $mime_type_class . '">'; if (isset($analyzed_sql[0]['select_expr']) && is_array($analyzed_sql[0]['select_expr'])) { foreach ($analyzed_sql[0]['select_expr'] AS $select_expr_position => $select_expr) { diff --git a/sql.php b/sql.php index 42dff07f8..41f1ec6a8 100644 --- a/sql.php +++ b/sql.php @@ -115,6 +115,34 @@ if(isset($_REQUEST['get_enum_values']) && $_REQUEST['get_enum_values'] == true) PMA_ajaxResponse(NULL, true, $extra_data); } +/** + * Find possible values for set fields during inline edit. + */ +if(isset($_REQUEST['get_set_values']) && $_REQUEST['get_set_values'] == true) { + $field_info_query = 'SHOW FIELDS FROM `' . $db . '`.`' . $table . '` LIKE \'' . $_REQUEST['column'] . '\' ;'; + + $field_info_result = PMA_DBI_fetch_result($field_info_query, null, null, null, PMA_DBI_QUERY_STORE); + + $selected_values = explode(',', $_REQUEST['curr_value']); + + $search = array('set', '(', ')', "'"); + $values = explode(',', str_replace($search, '', $field_info_result[0]['Type'])); + + $select = ''; + foreach($values as $value) { + $select .= '