]> git.evergreen-ils.org Git - Evergreen.git/commit
LP 791546: advanced search ISBN/ISSN in .staff mode
authorDan Scott <dan@coffeecode.net>
Mon, 6 Jun 2011 16:11:58 +0000 (12:11 -0400)
committerDan Wells <dbw2@calvin.edu>
Mon, 6 Jun 2011 19:25:06 +0000 (15:25 -0400)
commitd30e80f798d1f868f96ed2ac637a9a0de012291c
treeb938e3aacc473638512092870199fefabd55ec2b
parentb772570df3b9e98f48226fe4ad7d156fc259aad6
LP 791546: advanced search ISBN/ISSN in .staff mode

In 1.6, advanced search ISBN/ISSN searches always operated in .staff
mode, returning results whether the results should have been visible to
the user or not. This confused patrons who saw records to which they had
no access.

In 2.0, this behaviour changed so that advanced search ISBN/ISSN
searches never operated in .staff mode. This confused staff who were
used to retrieving records via the ISBN/ISSN search when they wanted to
add holdings for their own library to the records.

The pattern for addressing this problem and satisfying both use cases is
the same - use the multiclass.query.staff method if we invoke the ISBN
or ISSN searches with the .staff method name.

One could easily refactor many of the search method bodies in this
module to use the exact same logic, keying off the method name to
identify the identifier field and the .staff portion of the method name
to determine whether to invoke .staff mode or not. For now we just
address the ISBN and ISSN entry points.

Signed-off-by: Dan Scott <dscott@laurentian.ca>
Signed-off-by: Dan Wells <dbw2@calvin.edu>
Open-ILS/src/perlmods/OpenILS/Application/Search/Biblio.pm
Open-ILS/web/opac/common/js/config.js