Comment 1 for bug 677122

Revision history for this message
Mike Rylander (mrylander) wrote :

The problem is that the OPAC code in Open-ILS/web/opac/skin/default/js/adv_global.js (adjust for skin) is blinding splitting on spaces and prepending - to each "word". This is fine, but we need to watch out for empty "words". The simplest thing to do is avoid that completely by removing doubled, leading and trailing spaces when dealing with a notcontains term. See: http://svn.open-ils.org/trac/ILS/changeset/18901

To be backported pending independent confirmation of the fix.

Note: while it wouldn't hurt to invest in avoiding this at the query parser level as well (bug #605921), that is a different, broader angle of attack.