Patches item #970383, was opened at 2004-06-10 13:57 Message generated for change (Comment added) made by tkikuchi You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300103&aid=970383&group_id=103 Category: list administration Group: Mailman 2.1

Status: Closed Resolution: None Priority: 5 Submitted By: Jim Tittsler (jtittsler) Assigned to: Nobody/Anonymous (nobody) Summary: moderator -1 admin requests pending

Initial Comment: ListAdmin.NumRequestsPending() presumes that the request database always has at least one entry... a dummy "request" containing the version number of the request database. bin/newlist does not create a request database for each list, so the moderator gets a daily mail saying there are -1 requests pending. I propose that when __opendb() fails because the file didn't exist, the version number be added to the __db dict. Two additional points: - perhaps __open_db should ALWAYS add the version to the db, in case a request.pck somehow was created without a version key (especially since the version isn't currently checked) - the closedb setting of the version could *probably* be safely removed ----------------------------------------------------------------------

Comment By: Tokio Kikuchi (tkikuchi) Date: 2004-09-16 01:06

Message: Logged In: YES user_id=67709 fixed in CVS: revision: 2.41.2.7 ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300103&aid=970383&group_id=103