[PATCH v2] Exclude suspended Users from being notified
lfleischer at archlinux.org
Sat Jul 25 21:47:36 UTC 2020
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
> Signed-off-by: Kevin Morris <kevr.gtalk at 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.
More information about the aur-dev