moderator -1 admin requests pending

Bug #558133 reported by jtittsler
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
GNU Mailman
Fix Released
Undecided
Unassigned

Bug Description

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

Revision history for this message
jtittsler (jtittsler-users) wrote :

The file ListAdmin.diff was added: make sure the request db has a version key

Revision history for this message
tkikuchi (tkikuchi-users) wrote :

Logged In: YES
user_id=67709

fixed in CVS: revision: 2.41.2.7

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.