Bug 12443 - Initial re-factoring of buildQuery
authorDavid Cook <dcook@prosentient.com.au>
Wed, 18 Jun 2014 06:26:31 +0000 (16:26 +1000)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Fri, 27 Jun 2014 11:52:03 +0000 (08:52 -0300)
commitc4c0cb329dd5a7a095cdf0a37ef93c4aecca0e07
tree5ad71f368e8a7423b54cd99d7ab6111069975311
parent37ec008df79d36f065b1138912cbfd2b3d37565f
Bug 12443 - Initial re-factoring of buildQuery

This patch reduces three repeated code fragments into a single
internal subroutine, which is easier to read, has comments,
and should make it easier to refactor more buildQuery code
in the future.

_TEST PLAN_

Before applying

1) Run a bunch of different searches in the staff client and OPAC
in separate tabs

2) Apply the patch

3) Run the same searches again (maybe in yet more tabs) and notice
that the results are exactly the same.

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
Same results, no errors.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
C4/Search.pm