[Tickets #2549] NEW: Reporting multiple messages as spam fails

[bugs@bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=2549
-----------------------------------------------------------------------
 Ticket             | 2549
 Created By         | horde.org at spamvrij.kicks-ass.org
 Summary            | Reporting multiple messages as spam fails
 Queue              | IMP
 Version            | 4.0.4-RC1
 State              | Unconfirmed
 Priority           | 1. Low
 Type               | Bug
 Owners             | 
+New Attachment     | IMP-H3-404-RC1-Spam.php.diff
-----------------------------------------------------------------------


horde.org at spamvrij.kicks-ass.org (2005-09-02 18:13) wrote:

Symptom description: When using a 'spam reporting email address' is set and
multiple messages are selected in the message-list and reported to the
system administrator, then only the first message is reported, and instead
of just once, it's done for the number of selected messages.

Problem description: The variable $raw_msg in imp/lib/Spam.php is only
retrieved once, because the code checks for !isset($raw_msg).

Solution: either reset the variable after the loop, or give a null-value at
loop-start. I found the bug in IMP-H3-4.03 and made a private fix. The bug
is still present in IMP-H3-4.0.4RC1. Attached is a diff suggestion for
4.0.4RC1 (using diff -uNr).

Grtz, Paul Molensky