Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

Fixed #14014 -- Ensure that the "save and add another" button for use…

…rs actually does what it says. Thanks to Ramiro for the report.

git-svn-id: http://code.djangoproject.com/svn/django/trunk@13503 bcc190cf-cafb-0310-a4f2-bffc1f526a37
  • Loading branch information...
commit cefee67b7d531090b21a2b4457bb36fa5557d1f3 1 parent 2ab3b52
@freakboy3742 freakboy3742 authored
View
1  django/contrib/admin/templates/admin/auth/user/add_form.html
@@ -4,7 +4,6 @@
{% block form_top %}
{% if not is_popup %}
<p>{% trans "First, enter a 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 a username and password." %}</p>
{% endif %}
View
15 tests/regressiontests/admin_views/tests.py
@@ -2126,6 +2126,7 @@ def tearDown(self):
self.client.logout()
def test_user_creation(self):
+ user_count = User.objects.count()
response = self.client.post('/test_admin/admin/auth/user/add/', {
'username': 'newuser',
'password1': 'newpassword',
@@ -2134,6 +2135,7 @@ def test_user_creation(self):
})
new_user = User.objects.order_by('-id')[0]
self.assertRedirects(response, '/test_admin/admin/auth/user/%s/' % new_user.pk)
+ self.assertEquals(User.objects.count(), user_count + 1)
self.assertNotEquals(new_user.password, UNUSABLE_PASSWORD)
def test_password_mismatch(self):
@@ -2155,3 +2157,16 @@ def test_user_fk_popup(self):
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"')
+
+ def test_user_add_another(self):
+ user_count = User.objects.count()
+ response = self.client.post('/test_admin/admin/auth/user/add/', {
+ 'username': 'newuser',
+ 'password1': 'newpassword',
+ 'password2': 'newpassword',
+ '_addanother': '1',
+ })
+ new_user = User.objects.order_by('-id')[0]
+ self.assertRedirects(response, '/test_admin/admin/auth/user/add/')
+ self.assertEquals(User.objects.count(), user_count + 1)
+ self.assertNotEquals(new_user.password, UNUSABLE_PASSWORD)

0 comments on commit cefee67

Please sign in to comment.
Something went wrong with that request. Please try again.