[Mailman-Developers] [ mailman-Bugs-609692 ] Non-member hold reason is wrong

[noreply@sourceforge.net]

Bugs item #609692, was opened at 2002-09-15 15:07
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=100103&aid=609692&group_id=103

Category: mail delivery
Group: 2.1 beta
Status: Open
Resolution: None
Priority: 5
Submitted By: Les Niles (lniles)
Assigned to: Nobody/Anonymous (nobody)
Summary: Non-member hold reason is wrong

Initial Comment:
Maybe I'm missing something, but it would seem 
that Handlers.Moderate.process() should call 
hold.hold_for_approval() with a fourth arg of 
Hold.NonMemberPost instead of Hold.ModeratedPost 
at line 85.  The latter causes confusion for the typical 
case of a member (person) posting from other than 
her subscribed email address, since she gets back a 
message saying that the post was to a moderated 
list.  And the default reject message for the 
moderator, something about the posting being 
deemed inappropriate, is not appropriate. :)


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

>Comment By: Les Niles (lniles)
Date: 2002-09-18 09:56

Message:
Logged In: YES 
user_id=92931

Oops -- Hold.NonMemberPost should replace 
Hold.ModeratedPost at line 101, not line 85.  (This is 
in Sept 3 2002 CVS.)


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

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