I can confirm that adding a new entry in Trigger Event Parameters (Parameter Name = check_sms_notify, Parameter Value = 1) results in exactly what Thomas describes in comment #1. This should have been done during/after an upgrade (per the comment in Open-ILS/src/sql/PG/upgrade/0683.hold_available_email_notify.sql:
-- NOT COVERED: Adding check_sms_notify to the proper trigger. It doesn't have a static id.
So anyone who ran the 2.1-2.2-upgrade-db.sql script will face this same issue. We should really RAISE NOTICEs about this kind of thing in upgrade scripts!
I can confirm that adding a new entry in Trigger Event Parameters (Parameter Name = check_sms_notify, Parameter Value = 1) results in exactly what Thomas describes in comment #1. This should have been done during/after an upgrade (per the comment in Open-ILS/ src/sql/ PG/upgrade/ 0683.hold_ available_ email_notify. sql:
-- NOT COVERED: Adding check_sms_notify to the proper trigger. It doesn't have a static id.
So anyone who ran the 2.1-2.2- upgrade- db.sql script will face this same issue. We should really RAISE NOTICEs about this kind of thing in upgrade scripts!