Hold emails go out with email_notify disabled

Bug #927934 reported by Thomas Berezansky
18
This bug affects 5 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Bill Erickson

Bug Description

Due to a lack of A/T validation checking the hold notification flags hold available emails go out to anyone with an email address on file.

The branch below seeks to correct that by adding new optional checks to the HoldIsAvailable and HoldIsCanceled validators, plus adding a new HoldNotifyCheck validator.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/tsbere/hold_available_email_notify

Tags: pullrequest
Changed in evergreen:
status: New → In Progress
Changed in evergreen:
status: In Progress → New
milestone: 2.2.0alpha2 → 2.2.0beta1
Bill Erickson (berick)
Changed in evergreen:
assignee: nobody → Bill Erickson (erickson-esilibrary)
status: New → In Progress
Revision history for this message
Bill Erickson (berick) wrote :

Tested and pushed to master. Gracias.

Changed in evergreen:
status: In Progress → Fix Committed
Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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