Patch from Jeff Godin to make holds placed by staff respect the patron's notification...
authordbs <dbs@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Tue, 3 Nov 2009 04:46:26 +0000 (04:46 +0000)
committerdbs <dbs@dcc99617-32d9-48b4-a31d-7c20da2025e4>
Tue, 3 Nov 2009 04:46:26 +0000 (04:46 +0000)
commitf452446ce6a64a887e3f5af80fa66133bbd47fce
tree4399643b106bb296be77edf63d15cd0ecfaa4993
parent910d4dfc56699857b03d90d4da3889af93f8dbca
Patch from Jeff Godin to make holds placed by staff respect the patron's notification preferences.

The attached patch should fix a bug which resulted in patron notification
preferences (phone/email/both) being ignored in favor of the staff user's
notification preferences when staff place a hold on behalf of another
user.

BUG: When staff place a hold for another user, the staff member's
notification preferences (not the recipient user's) are consulted.

This patch fixes that bug, as well as a bug where notification selections
were not reset between canceled attempts to place holds for different
recipients.

There is also a minor change to grabUserPrefs() regarding which session
variable to send as an auth token (now uses G.user.session, which seems to
be an acceptable convention used elsewhere. This was needed, as the passed
user does not always have a valid session.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Signed-off-by: Jeff Godin <jeff@tcnet.org>
git-svn-id: svn://svn.open-ils.org/ILS/trunk@14735 dcc99617-32d9-48b4-a31d-7c20da2025e4
Open-ILS/web/opac/common/js/opac_utils.js
Open-ILS/web/opac/skin/default/js/holds.js