summaryrefslogtreecommitdiff
path: root/Mailman/bin
diff options
context:
space:
mode:
authorbwarsaw2007-04-03 02:03:18 +0000
committerbwarsaw2007-04-03 02:03:18 +0000
commit43f516b5d72a3d20f13957cee7f9c8a6699120ad (patch)
tree98078bf1e0796a187df7584440c35ca3b50fdcf7 /Mailman/bin
parent6362c6e0a021eb5939d14b8ae4a2ed7b80371580 (diff)
downloadmailman-43f516b5d72a3d20f13957cee7f9c8a6699120ad.tar.gz
mailman-43f516b5d72a3d20f13957cee7f9c8a6699120ad.tar.zst
mailman-43f516b5d72a3d20f13957cee7f9c8a6699120ad.zip
Moved the session.expire() to the MailList.Load() method, via
DBContext.api_load(). This is where it really ought to be based on the internal semantics of .Load()/.Lock(). MailListMapperExtension.populate_instance(): Checking the state of the isnew flag is not sufficient to know whether the MailList object we're getting is brand spankin' new or not. It turns out that when we session.expire() the MailList object, the next time SA loads this from the db, the populate_instance() will get called with isnew=True, even though the object really isn't new. Instead, check to make sure InitTempVars() isn't incorrectly called twice. Note that I might move this test, but I wanted to check in something that works, and then see if this is what we expect from the SA guys (this flag appears underdocumented). LockFile.py: Add some additional debugging.
Diffstat (limited to 'Mailman/bin')
0 files changed, 0 insertions, 0 deletions