Opened 17 years ago

Closed 17 years ago

#7068 closed (fixed)

contrib.auth.forms are not i18n enabled

Reported by: Maciej Wiśniowski Owned by: nobody
Component: Contrib apps Version: newforms-admin
Severity: Keywords: password auth reset
Cc: pigletto@… Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

AuthenticationForm, PasswordResetForm, PasswordChangeForm, AdminPasswordChangeForm have no 'label' set for it's fields, eg.:

password1 = forms.CharField(max_length=60, widget=forms.PasswordInput)

so field labels are not internationalized.
Patch is attached.

Would be nice if this can be applied with ticket 6905 ​http://code.djangoproject.com/ticket/6905.

Attachments (1)

auth_translations_v7436.diff​ (2.2 KB ) - added by Maciej Wiśniowski 17 years ago.
patch against svn version 7436

Download all attachments as: .zip

Change History (5)

by Maciej Wiśniowski, 17 years ago

Attachment: auth_translations_v7436.diff​ added

patch against svn version 7436

comment:1 by Alex Gaynor, 17 years ago

Triage Stage: Unreviewed → Fixed on a branch

This has been fixed in the newforms-admin branch

comment:2 by Maciej Wiśniowski, 17 years ago

Sorry but where is that fix? I can't see it fixed at newforms-admin branch, revision 7442:

​http://code.djangoproject.com/browser/django/branches/newforms-admin/django/contrib/auth/forms.py?rev=7442

comment:3 by Alex Gaynor, 17 years ago

Triage Stage: Fixed on a branch → Accepted

Whoops, boneheaded move on my part, it looks like those are only set on one of the forms(user creation).

comment:4 by Brian Rosner, 17 years ago

Resolution: → fixed
Status: new → closed

(In [7449]) newforms-admin: Fixed #6774, #7068 -- Use ugettext_lazy instead of ugettext in django/contrib/auth/forms.py. Marked a few strings for translation in the forms.

Note: See TracTickets for help on using tickets.
Back to Top