DACCESS-360 - DRY up simple search param handling logic

[chrisrlc]

Summary: Reuses search param handling logic in SearchBuilder from advanced search for simple search, cleans up unused search param handling logic from CornellCatalog#index, fixes a couple adjacent bugs. More details:

In simple search:

• Uses boolean operators (AND, OR) instead of prefix operator (-)
• Wraps query strings in quotation marks
• Consistent handling of special characters across both simple and advanced search (replaces “ with ", removes ()[], escapes : + -)
• Fixes edge case bug where simple searches that started with a parentheses and contains a quotation mark searches the author solr field, e.g. https://catalog-int.library.cornell.edu/catalog?q=%28%22Uprooted%22%29&search_field=title
• Fixes bug where author-title "Total Works" search links displays "All Fields" as its search field constraint, instead of "Author (sorted by title)", e.g. https://catalog-int.library.cornell.edu/?q=%22Beethoven,%20Kaspar%20Karl%20van,%201774-1815.%20|%20Minuets,%20orchestra%22&search_field=authortitle_browse
• Fixes DACCESS-471, where user receives 500 when exceeding search limits
• Fixes DACCESS-474, where links to worldcat sometimes go nowhere

TODO:

• Maybe consider moving code out of SearchBuilder into separate QueryBuilder object? Will wait until removing prefix operators from bento search, since then it'll be more necessary to have reusable logic outside SearchBuilder.

[JeremyDuncan]

Wow lots of cleanup going on here! I pulled this and everything seems to work on my side.