[PATCH v2] Exclude suspended Users from being notified

Lukas Fleischer 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
> https://bugs.archlinux.org/task/65554.
> 
> 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 mailing list