Ticket #12202: 12202_with_test.diff

new file django/contrib/admin/templates/registration/password_reset_subject.html

@@ +1 @@
+{% load i18n %}{% autoescape off %}
+{% blocktrans %}Password reset on {{ site_name }}{% endblocktrans %}
+{% endautoescape %}
+ No newline at end of file

django/contrib/auth/fixtures/authtestdata.json

@@ -31 +31 @@
             "groups": [], 
             "user_permissions": [], 
             "password": "sha1$6efc0$f93efe9fd7542f25a7be94871ea45aa95de57161", 
-            "email": "testclient@example.com", 
+            "email": "testclient2@example.com",
             "date_joined": "2006-12-17 07:03:31"
         }
     },

django/contrib/auth/forms.py

@@ -117 +117 @@
             raise forms.ValidationError(_("That e-mail address doesn't have an associated user account. Are you sure you've registered?"))
         return email
 
-    def save(self, domain_override=None, email_template_name='registration/password_reset_email.html',
-             use_https=False, token_generator=default_token_generator, from_email=None, request=None):
+    def save(self, domain_override=None,
+             subject_template_name='registration/password_reset_subject.html',
+             email_template_name='registration/password_reset_email.html',
+             use_https=False, token_generator=default_token_generator,
+             from_email=None, request=None):
         """
         Generates a one-use only link for resetting password and sends to the user
         """
@@ -130 +133 @@
                 domain = current_site.domain
             else:
                 site_name = domain = domain_override
-            t = loader.get_template(email_template_name)
             c = {
                 'email': user.email,
                 'domain': domain,
@@ -140 +142 @@
                 'token': token_generator.make_token(user),
                 'protocol': use_https and 'https' or 'http',
             }
-            send_mail(_("Password reset on %s") % site_name,
-                t.render(Context(c)), from_email, [user.email])
+            subject = loader.render_to_string(subject_template_name, c)
+            subject = ''.join(subject.splitlines())
+            email = loader.render_to_string(email_template_name, c)
+            send_mail(subject, email, from_email, [user.email])
 
 class SetPasswordForm(forms.Form):
     """

django/contrib/auth/tests/forms.py

@@ -242 +242 @@
         self.assertTrue(form.is_valid())
         self.assertEqual(form.cleaned_data['email'], u'jsmith3@example.com')
 
+    def test_custom_email_subject(self):
+        import os
+        from django.conf import settings
+        from django.core import mail
+
+        old_TEMPLATE_DIRS = settings.TEMPLATE_DIRS
+        settings.TEMPLATE_DIRS = (os.path.join(os.path.dirname(__file__), 'templates'),)
+        data = {'email': 'testclient@example.com'}
+        form = PasswordResetForm(data)
+        try:
+            self.assertTrue(form.is_valid())
+            form.save()
+
+            self.assertEqual(len(mail.outbox), 1)
+            self.assertEqual(mail.outbox[0].subject, u'Custom password reset on example.com')
+        finally:
+            settings.TEMPLATE_DIRS = old_TEMPLATE_DIRS
 
     def test_bug_5605(self):
         # bug #5605, preserve the case of the user name (before the @ in the

new file django/contrib/auth/tests/templates/registration/password_reset_subject.html

@@ +1 @@
+{% load i18n %}{% autoescape off %}
+{% blocktrans %}Custom password reset on {{ site_name }}{% endblocktrans %}
+{% endautoescape %}