Changed "starts with" browsing execution from OnChange to OnKeyUp (sounds more logical this way). Maybe we dont need "alphabet filter" above anymore. Or maybe we do (?).

Fixed songs starting with "All" never searched

1 files changed, 10 insertions, 8 deletions

diff --git a/server/browse.ajax.php b/server/browse.ajax.php
index ef176d88..6da77e66 100644
--- a/server/browse.ajax.php
+++ b/server/browse.ajax.php
@@ -29,14 +29,16 @@ switch ($_REQUEST['action']) {
 
 		$object_ids = array(); 
 
-                if ($_REQUEST['key'] && $_REQUEST['value']) {
-                        // Set any new filters we've just added
-			Browse::set_filter($_REQUEST['key'],$_REQUEST['value']); 
-                }
-                if ($_REQUEST['sort']) {
-                        // Set the new sort value
-                        Browse::set_sort($_REQUEST['sort']);
-                }
+      // Check 'value' with isset because it can null
+      //(user type a "start with" word and deletes it)
+      if ($_REQUEST['key'] && isset($_REQUEST['value'])) {
+        // Set any new filters we've just added
+			  Browse::set_filter($_REQUEST['key'],$_REQUEST['value']); 
+      }
+      if ($_REQUEST['sort']) {
+        // Set the new sort value
+        Browse::set_sort($_REQUEST['sort']);
+      }
 
 		// Refresh the browse div with our new filter options if we're not static
 		if (!Browse::$static_content) {