From a42b8476156f072ef16a7a4f9ff9047d00de3970 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 95f6998208..4c299334b4 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 @@ -1911,8 +1911,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