Reverted to using s/// instead of quotemeta for title/author/illustrator,
to avoid quoting spaces (which results in most titles/authors etc. becoming unsearchable) (Bug 197)
This commit is contained in:
parent
35f3408db4
commit
f1ae358bcd
1 changed files with 8 additions and 1 deletions
|
@ -858,9 +858,16 @@ sub CatSearch {
|
|||
my $dbh = C4::Context->dbh;
|
||||
my $query = '';
|
||||
my @results;
|
||||
|
||||
# Why not just use quotemeta to escape all questionable characters,
|
||||
# not just single-quotes? Because that would also escape spaces,
|
||||
# which would cause titles/authors/illustrators with a space to
|
||||
# become unsearchable (Bug 197)
|
||||
|
||||
for my $field ('title', 'author', 'illustrator') {
|
||||
$search->{$field} = quotemeta($search->{$field});
|
||||
$search->{$field} =~ s/['"]/\\\1/g;
|
||||
}
|
||||
|
||||
my $title = lc($search->{'title'});
|
||||
if ($type eq 'loose') {
|
||||
if ($search->{'author'} ne ''){
|
||||
|
|
Loading…
Reference in a new issue