Re: [PATCH v2] Exclude suspended Users from being notified

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 ---

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.