Do not notify if status is unchanged in pkg_flag()

Check if a package actually changed its status before sending an email
to prevent from spamming. Addresses FS#31745.

Signed-off-by: Lukas Fleischer <archlinux@cryptocrack.de>
This commit is contained in:
Lukas Fleischer 2012-09-30 23:37:29 +02:00
parent e2f3bee01e
commit e9ed60566e

View file

@ -699,9 +699,13 @@ function pkg_flag ($atype, $ids, $action=true, $dbh=NULL) {
$q.= "AND MaintainerUID = " . uid_from_sid($_COOKIE["AURSID"], $dbh); $q.= "AND MaintainerUID = " . uid_from_sid($_COOKIE["AURSID"], $dbh);
} }
$dbh->exec($q);
if ($action) { if ($action) {
$q.= " AND OutOfDateTS IS NULL";
}
$affected_pkgs = $dbh->exec($q);
if ($action && $affected_pkgs > 0) {
# Notify of flagging by email # Notify of flagging by email
$f_name = username_from_sid($_COOKIE['AURSID'], $dbh); $f_name = username_from_sid($_COOKIE['AURSID'], $dbh);
$f_email = email_from_sid($_COOKIE['AURSID'], $dbh); $f_email = email_from_sid($_COOKIE['AURSID'], $dbh);