Added random eml file name, removed getattr related to self.mailbox.from_email in Message.reply, fix docs - second round of pull requests

django_mailbox/models.py

@@ -332,7 +332,7 @@ class Mailbox(models.Model):
     def _process_message(self, message):
         msg = Message()
         if STORE_ORIGINAL_MESSAGE:
-            msg.eml.save('message.eml', ContentFile(message), save=False)
+            msg.eml.save('%s.eml' % uuid.uuid4(), ContentFile(message), save=False)
         msg.mailbox = self
         if 'subject' in message:
             msg.subject = convert_header_to_unicode(message['subject'])[0:255]
@@ -460,6 +460,7 @@ class Message(models.Model):
     eml = models.FileField(
         _(u'Message as a file'),
         null=True,
+        upload_to="messages",
         help_text=_(u'Original full content of message')
     )
     objects = models.Manager()
@@ -510,7 +511,7 @@ class Message(models.Model):
         pre-set it.
 
         """
-        if not getattr(message, 'from_email', None):
+        if not self.from_email:
             if self.mailbox.from_email:
                 message.from_email = self.mailbox.from_email
             else:
@@ -612,7 +613,7 @@ class Message(models.Model):
     def get_email_object(self):
         """ Returns an `email.message.Message` instance for this message."""
         if self.eml:
-            return email.message_from_file(Message.objects.last().eml)
+            return email.message_from_file(self.eml)
         body = self.get_body()
         if six.PY3:
             flat = email.message_from_bytes(body)

django_mailbox/tests/test_process_email.py

@@ -210,7 +210,7 @@ class TestProcessEmail(EmailMessageTestCase):
         )
         msg = self.mailbox.record_outgoing_message(email_object.message())
 
-        self.assertEqual(msg.outgoing, True)
+        self.assertTrue(msg.outgoing)
 
         actual_from = 'username@example.com'
         reply_email_object = EmailMessage('Test subject',  # subject