]> 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:30:42 +0000 (15:30 -0400)
commit33c341fb3ed474fc628f832c571d6e53edc91669
tree09b366d0df251312965d7f7a73ecf985f9a29c0f
parentb72f90c81372b082565b25d106099ada97251827
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