From 1059b4f0d9bb87886c6799b1be8c9ac21e35bf0d Mon Sep 17 00:00:00 2001 From: Jason Boyer Date: Wed, 1 Apr 2020 07:10:42 -0400 Subject: [PATCH] LP1774268: Stop treating undefined settings as special Rather than have to treat an undefined hold notification setting as a specificcombination of values, just make it explicit and then save them. Signed-off-by: Jason Boyer --- Open-ILS/web/js/ui/default/staff/circ/patron/regctl.js | 15 ++++++--------- 1 file changed, 6 insertions(+), 9 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 b3a5cda829..b017ed7e66 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 @@ -1897,16 +1897,13 @@ function($scope , $routeParams , $q , $uibModal , $window , egCore , function extract_hold_notify() { var p = $scope.patron; var notify = $scope.user_settings['opac.hold_notify']; - if (notify === '') { - $scope.hold_notify_type.phone = false; - $scope.hold_notify_type.email = false; - return; - } - if (!notify) { - $scope.hold_notify_type.phone = true; - $scope.hold_notify_type.email = true; - return; + + if (!notify && !(notify === '')) { + // This is where an OUS for default hold notification types could be used. + notify = "phone:email"; + $scope.user_settings['opac.hold_notify'] = notify; } + $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