]> git.evergreen-ils.org Git - Evergreen.git/commit
lp1902302 Opac. SMS and Carriers appear in Edit Holds Regardless of OU Setting.
authorGarry Collum <gcollum@gmail.com>
Fri, 30 Oct 2020 19:55:35 +0000 (15:55 -0400)
committerGalen Charlton <gmc@equinoxOLI.org>
Wed, 26 May 2021 19:32:20 +0000 (15:32 -0400)
commitba719715c577ce04ef3057268d02695efb29a692
tree460cc85fec03fbfe49910ef16815a9ee225d00d8
parentbabd3305fa2cec7e1d2ff6e6b90d29244f896629
lp1902302 Opac. SMS and Carriers appear in Edit Holds Regardless of OU Setting.

The SMS Number field and Default Carrier field display when editing an
individual hold in the opac, regardless of the 'Enable features that send SMS text messages' setting.
This patch rectifies that behavior.

To test.
Set the 'Enable features that send SMS text messages' to False.
Place a hold for a patron.
Login to the opac as that patron and view the holds.
Edit the hold, and notice that the SMS and Carrier fields are displayed.

Apply the patch.
Refresh the edit form.  The fields no longer display.

Set the 'Enable features that send SMS text messages' to True.
Refresh the edit form.  The fields now display.

Signed-off-by: Garry Collum <gcollum@gmail.com>
Signed-off-by: Kyle Huckins <khuckins@catalyte.io>
Signed-off-by: Galen Charlton <gmc@equinoxOLI.org>
Open-ILS/src/templates/opac/myopac/holds/edit.tt2