Inline edit links were not displayed in search results

Doap · Dec 30, 2010 · 818965f · 818965f
1 parent 0c97997
commit 818965f
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 5 deletions.
diff --git a/js/sql.js b/js/sql.js
@@ -54,10 +54,11 @@ function getFieldName($this_field, disp_mode) {
  * The function that iterates over each row in the table_results and appends a
  * new inline edit anchor to each table row.
  *
- * @param   disp_mode   string
  */
-function appendInlineAnchor(disp_mode) {
-    if(disp_mode == 'vertical') {
+function appendInlineAnchor() {
+    var disp_mode = $("#top_direction_dropdown").val();
+
+    if (disp_mode == 'vertical') {
         // there can be one or two tr containing this class, depending
         // on the ModifyDeleteAtLeft and ModifyDeleteAtRight cfg parameters 
         $('#table_results tr')
@@ -163,10 +164,9 @@ $(document).ready(function() {
      * Attach the {@link appendInlineAnchor} function to a custom event, which
      * will be triggered manually everytime the table of results is reloaded
      * @memberOf    jQuery
-     * @name        sqlqueryresults_live
      */
     $("#sqlqueryresults").live('appendAnchor',function() {
-        appendInlineAnchor(disp_mode);
+        appendInlineAnchor();
     })
 
     /**

diff --git a/js/tbl_select.js b/js/tbl_select.js
@@ -22,6 +22,25 @@ $(document).ready(function() {
         cache: 'false'
     });
 
+    /**
+     * Attach the {@link appendInlineAnchor} function to a custom event, which
+     * will be triggered manually everytime the table of results is reloaded;
+     *
+     * Needs appendInlineAnchor() which is defined in sql.js
+     * @memberOf    jQuery
+     */
+    $("#searchresults").live('appendAnchor',function() {
+        appendInlineAnchor();
+    })
+
+    /**
+     * Trigger the appendAnchor event to prepare the first table for inline edit
+     *
+     * @memberOf    jQuery
+     */
+    $("#searchresults").trigger('appendAnchor');
+
+
     $('<a id="togglesearchform"></a>')
      .html(PMA_messages['strShowSearchCriteria'])
      .insertAfter('#tbl_search_form')
@@ -63,6 +82,7 @@ $(document).ready(function() {
             if (typeof response == 'string') {
                 // found results
                 $("#searchresults").html(response);
+                $("#searchresults").trigger('appendAnchor');
                 $('#tbl_search_form').hide();
                 $('#togglesearchform')
                  // always start with the Show message

diff --git a/tbl_select.php b/tbl_select.php
@@ -16,6 +16,7 @@
 require_once './libraries/common.inc.php';
 require_once './libraries/mysql_charsets.lib.php';
 
+$GLOBALS['js_include'][] = 'sql.js';
 $GLOBALS['js_include'][] = 'tbl_select.js';
 $GLOBALS['js_include'][] = 'jquery/jquery-ui-1.8.custom.js';
 $GLOBALS['js_include'][] = 'jquery/timepicker.js';