summaryrefslogtreecommitdiff
path: root/Mailman/initialize.py
diff options
context:
space:
mode:
authorbwarsaw2007-04-03 02:03:18 +0000
committerbwarsaw2007-04-03 02:03:18 +0000
commit43f516b5d72a3d20f13957cee7f9c8a6699120ad (patch)
tree98078bf1e0796a187df7584440c35ca3b50fdcf7 /Mailman/initialize.py
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/initialize.py')
0 files changed, 0 insertions, 0 deletions