AutoSuggest: prevent building a bad tsquery when user types leading spaces
authorLebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Mon, 13 Feb 2012 19:57:39 +0000 (14:57 -0500)
committerMike Rylander <mrylander@gmail.com>
Mon, 20 Feb 2012 19:22:27 +0000 (14:22 -0500)
The problem used to bubble up and manifest as an Apache 500 error for
the user.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>

Open-ILS/src/perlmods/lib/OpenILS/WWW/AutoSuggest.pm

index 3e4d182..cf8644b 100644 (file)
@@ -58,7 +58,7 @@ sub prepare_for_tsquery {
     $str =~ s/[^\w\s]/ /ig;
     $str .= ":*" unless $str =~ /\s$/;
 
-    return join(" & ", split(/\s+/, $str));
+    return join(" & ", grep(length, split(/\s+/, $str)));
 }
 
 # The third argument to our stored procedure, metabib.suggest_browse_entries(),