oops, sent to the wrong list, meant for pmg-de...@pve.proxmox.com
On 3/3/20 9:33 AM, Dominik Csapak wrote:
by using 'LIMIT 10' for the spamlevels, we only got the first 10 spamlevels back from the database (whatever they were) this is only ok if there are only <= 10 different spamlevels in the database but not if there are more the call site of this uses the combined spam count of this sub for calculating the 'rest' (meaning the mails with spam level 0) but this is obviously wrong if not all spamlevels are counted so simply return all available levels Signed-off-by: Dominik Csapak <d.csa...@proxmox.com> --- src/PMG/Statistic.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/PMG/Statistic.pm b/src/PMG/Statistic.pm index b00fed4..6d27930 100755 --- a/src/PMG/Statistic.pm +++ b/src/PMG/Statistic.pm @@ -431,7 +431,7 @@ sub total_spam_stat {my $sth = $rdb->{dbh}->prepare("SELECT spamlevel, COUNT(spamlevel) AS count FROM CStatistic " ."WHERE virusinfo IS NULL and time >= ? AND time < ? AND ptime > 0 AND spamlevel > 0 " . - "GROUP BY spamlevel ORDER BY spamlevel LIMIT 10"); + "GROUP BY spamlevel ORDER BY spamlevel"); $sth->execute($from, $to);my $res = $sth->fetchall_arrayref({});
_______________________________________________ pve-devel mailing list pve-devel@pve.proxmox.com https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
