[ mailman-Patches-1318883 ] Line following Approved: line in body of post is stripped

[SourceForge.net]

Patches item #1318883, was opened at 2005-10-08 14:12
Message generated for change (Comment added) made by msapiro
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1318883&group_id=103

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: list administration
Group: Mailman 2.1
>Status: Closed
Resolution: None
Priority: 5
Submitted By: Mark Sapiro (msapiro)
>Assigned to: Mark Sapiro (msapiro)
Summary: Line following Approved: line in body of post is stripped

Initial Comment:
If a post contains an [Aa]pproved?: line as the first
non-blank line of the first text/plain part (and
doesn't contain an Approved?: header), the line itself
is deleted, but so is the first of the remaining lines.
If this first remaining line is a blank line which
preceeded the Approved: line, no harm is done, but if
there are no blank lines preceeding the Approved: line,
the line following the Approved: line which may be
non-blank is stripped.

This occurs because after the Approved: line is
removed, the remaining lines are re-joined beginning
with line 1 and not line 0.

The patch to Mailman/Handlers/Approve.py corrects this.

----------------------------------------------------------------------

>Comment By: Mark Sapiro (msapiro)
Date: 2005-11-12 18:46

Message:
Logged In: YES 
user_id=1123998

Patch has been commited in CVS

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=300103&aid=1318883&group_id=103