From 58b931258646c8c7ef50db4134e102d075efc80b Mon Sep 17 00:00:00 2001 From: Suzanne Paterno Date: Tue, 6 Aug 2019 11:29:57 -0400 Subject: [PATCH] LP 1774268: Default Hold Notification checkboxes On patron edit screen set the email and phone notification to true when patron doesn't have any prefernces. This mimics the behavior in the xul client. Signed-off-by: Michele Morgan Signed-off-by: Jason Boyer --- Open-ILS/web/js/ui/default/staff/circ/patron/regctl.js | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/Open-ILS/web/js/ui/default/staff/circ/patron/regctl.js b/Open-ILS/web/js/ui/default/staff/circ/patron/regctl.js index 72a9ebd863..f8c5ca7106 100644 --- a/Open-ILS/web/js/ui/default/staff/circ/patron/regctl.js +++ b/Open-ILS/web/js/ui/default/staff/circ/patron/regctl.js @@ -1896,8 +1896,12 @@ function($scope , $routeParams , $q , $uibModal , $window , egCore , function extract_hold_notify() { var p = $scope.patron; - var notify = $scope.user_settings['opac.hold_notify'] || ''; - + var notify = $scope.user_settings['opac.hold_notify']; + if (!notify) { + $scope.hold_notify_type.phone = true; + $scope.hold_notify_type.email = true; + return; + } $scope.hold_notify_type.phone = Boolean(notify.match(/phone/)); $scope.hold_notify_type.email = Boolean(notify.match(/email/)); $scope.hold_notify_type.sms = Boolean(notify.match(/sms/)); -- 2.11.0