Ticket #15481: list-table.dev.js-x7815.diff

Index: wp-admin/js/list-table.dev.js
===================================================================
--- wp-admin/js/list-table.dev.js       (revision 16472)
+++ wp-admin/js/list-table.dev.js       (working copy)
@@ -214,17 +214,23 @@
 
                return false;
        });
+       
+       // Keep track of how many times setTimeout has been called.
+       var refresh_results_counter = 0;
 
-       // searchbox
-       function change_search(ev) {
-               if ( 'keypress' == ev.type && 13 != ev.keyCode )
+       function refresh_results(search_text)
+       {
+               // Decrease the timer only if it is above 0. Otherwise race condition: type 4 characters quickly then press enter...
+               if (refresh_results_counter > 0)
+                       refresh_results_counter --;
+
+               // If setTimeout has been called more than once, just ignore the first ones.
+               if (refresh_results_counter > 0)
                        return;
+               
+               // NOW we can search 
+               var data = $(search_text).parent('.search-box').find(':input').serializeObject();
 
-               ev.preventDefault();
-               ev.stopImmediatePropagation();
-
-               var data = $(this).parent('.search-box').find(':input').serializeObject();
-
                listTable.update_rows(data, true, function() {
                        $('h2 .subtitle').remove();
 
@@ -234,6 +240,34 @@
                                ));
                });
        }
+
+       // searchbox
+       function change_search(ev) {
+               if ( 'keypress' != ev.type )
+                       return;
+                       
+               if ( 13 == ev.keyCode )
+               {
+                       ev.preventDefault();
+                       ev.stopImmediatePropagation();
+                       // Get the results immediately.
+                       refresh_results_counter = 1;
+                       refresh_results(this);
+               }
+               else
+               {
+                       // Set a timeout for one second before fetching the new results.
+                       // Increase the counter so that the search always waits one second after the LAST keypress.
+                       refresh_results_counter++;
+                       // This is a "closure".
+                       setTimeout(function(that) {
+                               return function() {
+                                       refresh_results(that);
+                               };
+                       }(this), 1000);
+               }
+
+       }
        $('.search-box :submit').click(change_search);
        $('.search-box :text').keypress(change_search);