On Wed, 15 Jul 2020 at 14:45:54, Kevin Morris wrote:
The existing notify.py script was grabbing entries regardless of user suspension. This has been modified to only send notifications to unsuspended users.
This change was written as a solution to https://bugs.archlinux.org/task/65554.
Signed-off-by: Kevin Morris <kevr.gtalk@gmail.com> ---
The issue was in OwnershipEventNotification: `Users.Suspended = 0` was included in the bottom-most SQL fetch, which is not a fetch of the recipients. That part of the stanza was removed, and `Users.Suspended = 0` is only ever used when fetching recipients for notifications.
aurweb/scripts/notify.py | 28 ++++++++++++++++++---------- 1 file changed, 18 insertions(+), 10 deletions(-)
Thanks for identifying the problem and sending an updated patch! Replaced the previous one in pu with this version.