Kaydet (Commit) 52720c54 authored tarafından R. David Murray's avatar R. David Murray

Add missing iteritems() call to the for loop in mailbox.MH.get_message().

Fixes issue2625.
üst e2942d07
...@@ -911,7 +911,7 @@ class MH(Mailbox): ...@@ -911,7 +911,7 @@ class MH(Mailbox):
_unlock_file(f) _unlock_file(f)
finally: finally:
f.close() f.close()
for name, key_list in self.get_sequences(): for name, key_list in self.get_sequences().iteritems():
if key in key_list: if key in key_list:
msg.add_sequence(name) msg.add_sequence(name)
return msg return msg
......
...@@ -936,6 +936,12 @@ class TestMH(TestMailbox): ...@@ -936,6 +936,12 @@ class TestMH(TestMailbox):
self._box.remove(key1) self._box.remove(key1)
self.assert_(self._box.get_sequences() == {'flagged':[key0]}) self.assert_(self._box.get_sequences() == {'flagged':[key0]})
def test_issue2625(self):
msg0 = mailbox.MHMessage(self._template % 0)
msg0.add_sequence('foo')
key0 = self._box.add(msg0)
refmsg0 = self._box.get_message(key0)
def test_pack(self): def test_pack(self):
# Pack the contents of the mailbox # Pack the contents of the mailbox
msg0 = mailbox.MHMessage(self._template % 0) msg0 = mailbox.MHMessage(self._template % 0)
......
...@@ -202,6 +202,9 @@ Core and Builtins ...@@ -202,6 +202,9 @@ Core and Builtins
Library Library
------- -------
- Issue 2625: added missing iteritems() call to the for loop in
mailbox.MH.get_message().
- Issue 5585: Add the ability to call an initializer to mulitiprocessing.manager - Issue 5585: Add the ability to call an initializer to mulitiprocessing.manager
so that users can install custonm handlers/etc. so that users can install custonm handlers/etc.
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment