Fix outdated log usage in EmailPlayer._log_part + broken email rendering from previou...

[pyrisk.git] / pyrisk / player / email.py

diff --git a/pyrisk/player/email.py b/pyrisk/player/email.py
index 4feb957d9c44d58e1e92734c107740a5f3cac933..b255aa749d0e040c93def3294f1e6fd25aaf5fcc 100644 (file)
--- a/pyrisk/player/email.py
+++ b/pyrisk/player/email.py
@@ -32,7 +32,6 @@ import sys
 import tempfile
 
 from ..base import Player, PlayerError
-from ..graphics import WorldRenderer
 
 
 # Configure alternative sendmail command in case smtplib is too
@@ -271,17 +270,18 @@ class EmailPlayer (Player):
                 else:
                     body.append('  %s\t%s\t%d' % (terr, terr.player, terr.armies))
         return encodedMIMEText('\n'.join(body), filename='world')
-    def _rendered_world_part(self, log, world):
+    def _rendered_world_part(self, world, log):
         start_event = log[0]
         players = start_event.players
         body = self.world_renderer.render(world, players)
         filename,subtype = \
             self.world_renderer.filename_and_mime_image_type(world)
         part = MIMEImage(body, subtype)
-        part.add_header('Content-Disposition', 'attachment', filename)
+        part.add_header('Content-Disposition', 'attachment', filename=filename)
         return part
     def _log_part(self, log):
-        return encodedMIMEText('\n'.join(log), filename='log')
+        return encodedMIMEText('\n'.join([str(e) for e in log]),
+                               filename='log')
     def report(self, world, log):
         """Send reports about death and game endings.