Skip to content

Commit 8e4e721

Browse files
committed
Python2.7 import compatibility fix
1 parent 03564f7 commit 8e4e721

File tree

2 files changed

+21
-4
lines changed

2 files changed

+21
-4
lines changed

sendgrid/helpers/mail/mail.py

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
"""v3/mail/send response body builder"""
22
from .personalization import Personalization
33
from .header import Header
4+
from .email import Email
5+
from .content import Content
46

57

68
class Mail(object):
@@ -154,9 +156,14 @@ def from_EmailMessage(cls, message):
154156
subject=message.get('Subject'),
155157
to_email=Email(message.get('To')),
156158
)
159+
try:
160+
body = message.get_content()
161+
except AttributeError:
162+
# Python2
163+
body = message.get_payload()
157164
mail.add_content(Content(
158165
message.get_content_type(),
159-
message.get_content()
166+
body.strip()
160167
))
161168
for k, v in message.items():
162169
mail.add_header(Header(k, v))

test/test_mail.py

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,12 @@
22
import json
33
import unittest
44

5-
from email.message import EmailMessage
5+
try:
6+
from email.message import EmailMessage
7+
except ImportError:
8+
# Python2
9+
from email import message
10+
EmailMessage = message.Message
611

712
from sendgrid.helpers.mail import (
813
ASM,
@@ -562,7 +567,12 @@ def test_directly_setting_substitutions(self):
562567

563568
def test_from_emailmessage(self):
564569
message = EmailMessage()
565-
message.set_content('message that is not urgent')
570+
body = 'message that is not urgent'
571+
try:
572+
message.set_content(body)
573+
except AttributeError:
574+
# Python2
575+
message.set_payload(body)
566576
message.set_default_type('text/plain')
567577
message['Subject'] = 'URGENT TITLE'
568578
message['From'] = 'test@example.com'
@@ -576,4 +586,4 @@ def test_from_emailmessage(self):
576586
self.assertEqual(len(mail.contents), 1)
577587
content = mail.contents[0]
578588
self.assertEqual(content.type, 'text/plain')
579-
self.assertEqual(content.value, 'message that is not urgent\n')
589+
self.assertEqual(content.value, 'message that is not urgent')

0 commit comments

Comments
 (0)