https://issues.apache.org/SpamAssassin/show_bug.cgi?id=7021
--- Comment #17 from Ivo Truxa <[email protected]> --- I had to do two small bug-fixes to allow proper relearning. When message tracking is enabled, and a previously trained message is relearned, TxRep first forgets the original (recorded) score, and then relearns the new one. TxRep incorrectly deduced the old score twice, and additionally, when SQL storage was used, the count could not be decremented. The later requires a minor change in the SQLBasedAddrList.pm module, but it won't have any impact on AWL - it stays fully compatible. No change needed for the DBBasedAddrList.pm. There are the diffs for TxRep.pm and for SQLBasedAddrList.pm here: TxRep: 1370,1371c1370,1371 < if ($key eq 'MSG_ID') { < $delta = $self->total(); --- > if ($key eq 'MSG_ID' && $self->count() > 0) { > $delta = $self->total() / $self->count(); SQLBasedAddrList: 331c331 < "SET count = count + 1, totscore = totscore + ? ". --- > "SET count = $entry->{count}, totscore = totscore + ? ". Please let me know if there is a different way to submit such modifications - for example if you prefer that I commit the changes through the SpamAssassin repository at GitHub. -- You are receiving this mail because: You are the assignee for the bug.
