diff options
| author | bwarsaw | 2007-04-03 02:03:18 +0000 |
|---|---|---|
| committer | bwarsaw | 2007-04-03 02:03:18 +0000 |
| commit | 43f516b5d72a3d20f13957cee7f9c8a6699120ad (patch) | |
| tree | 98078bf1e0796a187df7584440c35ca3b50fdcf7 /Mailman/initialize.py | |
| parent | 6362c6e0a021eb5939d14b8ae4a2ed7b80371580 (diff) | |
| download | mailman-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
