From 009276e9b3ba00b40cd73c8bf034becd348bb9c2 Mon Sep 17 00:00:00 2001 From: Andy McKay Date: Mon, 22 Aug 2011 12:28:57 -0700 Subject: [PATCH] fix thank you email for new email field (bug 676424) --- apps/amo/tests/test_views.py | 16 ++++++++++++++++ apps/amo/views.py | 3 ++- 2 files changed, 18 insertions(+), 1 deletion(-) diff --git a/apps/amo/tests/test_views.py b/apps/amo/tests/test_views.py index 56d7f5fb0d2..aa3c381ca60 100644 --- a/apps/amo/tests/test_views.py +++ b/apps/amo/tests/test_views.py @@ -4,6 +4,7 @@ from django import http, test from django.conf import settings from django.core.cache import cache +from django.core import mail import commonware.log from lxml import etree @@ -354,6 +355,21 @@ def test_subscription_event(self, urlopen): eq_(response.status_code, 200) eq_(SubscriptionEvent.objects.count(), 1) + @patch('amo.views.urllib2.urlopen') + def test_mail(self, urlopen): + urlopen.return_value = self.urlopener('VERIFIED') + add = Addon.objects.create(enable_thankyou=True, + support_email='a@a.com', + type=amo.ADDON_EXTENSION) + Contribution.objects.create(addon_id=add.pk, + uuid='123') + response = self.client.post(self.url, {u'action_type': u'PAY', + u'sender_email': u'a@a.com', + u'status': u'COMPLETED', + u'tracking_id': u'123'}) + eq_(response.status_code, 200) + eq_(len(mail.outbox), 1) + def test_get_not_allowed(self): response = self.client.get(self.url) assert isinstance(response, http.HttpResponseNotAllowed) diff --git a/apps/amo/views.py b/apps/amo/views.py index d8247275fc8..add60d5331c 100644 --- a/apps/amo/views.py +++ b/apps/amo/views.py @@ -273,7 +273,8 @@ def _log_error_with_data(msg, request): # embedded payments. for old, new in [('payment_status', 'status'), ('item_number', 'tracking_id'), - ('txn_id', 'tracking_id')]: + ('txn_id', 'tracking_id'), + ('payer_email', 'sender_email')]: if old not in post and new in post: post[old] = post[new]