set duplicate as a reject reason in email for theme queue

This commit is contained in:
Kevin Ngo 2013-04-02 21:27:24 -07:00
Родитель 5651bd25a5
Коммит 4b2a6a60e3
2 изменённых файлов: 4 добавлений и 4 удалений

Просмотреть файл

@ -25,6 +25,8 @@ def send_mail(cleaned_data, theme_lock):
reason = None
if reject_reason:
reason = rvw.THEME_REJECT_REASONS[reject_reason]
elif action == rvw.ACTION_DUPLICATE:
reason = _('Duplicate Submission')
comment = cleaned_data['comment']
emails = set(theme.addon.authors.values_list('email', flat=True))
@ -45,13 +47,11 @@ def send_mail(cleaned_data, theme_lock):
subject = _('A problem with your Theme submission')
template = 'reviewers/themes/emails/reject.html'
theme.addon.update(status=amo.STATUS_REJECTED)
reason = (rvw.THEME_REJECT_REASONS[reject_reason])
elif action == rvw.ACTION_DUPLICATE:
subject = _('A problem with your Theme submission')
template = 'reviewers/themes/emails/reject.html'
theme.addon.update(status=amo.STATUS_REJECTED)
reason = 'Duplicate'
elif action == rvw.ACTION_FLAG:
subject = _('Theme submission flagged for review')

Просмотреть файл

@ -2136,12 +2136,12 @@ class ThemeReviewTestMixin(object):
recipient_list=[settings.THEMES_EMAIL]),
mock.call('A problem with your Theme submission',
'reviewers/themes/emails/reject.html',
{'reason': None,
{'reason': mock.ANY,
'comment': u'duplicate',
'theme': themes[2],
'base_url': 'http://testserver'},
headers={'Reply-To': settings.THEMES_EMAIL},
recipient_list=set([])),
recipient_list=set([])),
mock.call('A problem with your Theme submission',
'reviewers/themes/emails/reject.html',
{'reason': mock.ANY,