A/T SMS Sending Results In Incorrectly Validated Events For Upgraded Sites

Bug #1897146 reported by Chris Sharp
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Undecided
Unassigned

Bug Description

Branching from bug 1096209. Sites who upgraded at some point from 2.1 -> 2.2 (very long ago) probably missed adding a new parameter to Action Trigger event definitions that send SMS messages.

Jeff Davis developed an upgrade script in 2015 that should correct the problem if present, but is harmless if not:

Working branch user/jeffdavis/lp1096209_check_sms_notify has an upgrade script that adds the check_sms_notify to the appropriate event defs, as suggested in my last comment:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/jeffdavis/lp1096209_check_sms_notify.

Evergreen 2.2 forward (for sites upgraded from 2.1)

Revision history for this message
Chris Sharp (chrissharp123) wrote :

Having applied this years ago in PINES, I'm certain adding the parameter fixes this issue. Pushed the fix to master. Thanks, Jeff, for keeping the flame alive on this one!

Changed in evergreen:
status: New → Fix Committed
tags: added: actiontrigger bitesize
Changed in evergreen:
milestone: none → 3.7-beta
milestone: 3.7-beta → 3.6-beta
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.