Ticket #14012: 14012.2.diff

django/contrib/admin/templates/admin/auth/user/add_form.html

@@ -2 +2 @@
 {% load i18n %}
 
 {% block form_top %}
-    <p>{% trans "First, enter a username and password. Then, you'll be able to edit more user options." %}</p>
+  {% if not is_popup %}
+    <p>{% trans "First, enter an username and password. Then, you'll be able to edit more user options." %}</p>
     <input type="hidden" name="_continue" value="1" />
+  {% else %}
+    <p>{% trans "Enter an username and password." %}</p>
+  {% endif %}
 {% endblock %}
 
 {% block after_field_sets %}

tests/regressiontests/admin_views/models.py

@@ -10 +10 @@
 from django.db import models
 from django import forms
 from django.forms.models import BaseModelFormSet
+from django.contrib.auth.models import User
 from django.contrib.contenttypes import generic
 from django.contrib.contenttypes.models import ContentType
 
@@ -579 +580 @@
 class PizzaAdmin(admin.ModelAdmin):
     readonly_fields = ('toppings',)
 
+class Album(models.Model):
+    owner = models.ForeignKey(User)
+    title = models.CharField(max_length=30)
+
 admin.site.register(Article, ArticleAdmin)
 admin.site.register(CustomArticle, CustomArticleAdmin)
 admin.site.register(Section, save_as=True, inlines=[ArticleInline])
@@ -625 +630 @@
 admin.site.register(ChapterXtra1)
 admin.site.register(Pizza, PizzaAdmin)
 admin.site.register(Topping)
+admin.site.register(Album)

tests/regressiontests/admin_views/tests.py

@@ -2113 +2113 @@
         response = self.client.get('/test_admin/admin/admin_views/pizza/add/')
         self.assertEqual(response.status_code, 200)
 
-class IncompleteFormTest(TestCase):
+class UserAdminTest(TestCase):
     """
-    Tests validation of a ModelForm that doesn't explicitly have all data
-    corresponding to model fields. Model validation shouldn't fail
-    such a forms.
+    Tests user CRUD functionality.
     """
     fixtures = ['admin-views-users.xml']
 
@@ -2149 +2147 @@
         self.assert_('password' not in adminform.form.errors)
         self.assertEquals(adminform.form.errors['password2'],
                           [u"The two password fields didn't match."])
+
+    def test_user_fk_popup(self):
+        response = self.client.get('/test_admin/admin/admin_views/album/add/')
+        self.failUnlessEqual(response.status_code, 200)
+        self.assertContains(response, '/test_admin/admin/auth/user/add')
+        self.assertContains(response, 'class="add-another" id="add_id_owner" onclick="return showAddAnotherPopup(this);"')
+        response = self.client.get('/test_admin/admin/auth/user/add/?_popup=1')
+        self.assertNotContains(response, 'name="_continue"')