Ticket #61: password_field.2.patch

django/contrib/admin/templates/widget/password.html

@@ +1 @@
+{% load i18n %}
+<p> 
+   {{ bound_field.form_fields.0 }} <span class="help">{% trans "Enter new password (or leave blank to keep existing password)." %}</span><br />
+   {{ bound_field.form_fields.1 }} <span class="help">{% trans "Confirm new password." %}</span>
+</p>
+<p>
+   {{ bound_field.form_fields.2 }}
+</p>
+ No newline at end of file

django/contrib/auth/models.py

@@ -19 +19 @@
         return hsh == sha.new(salt+raw_password).hexdigest()
     raise ValueError, "Got unknown password algorithm type in password."
 
+def encrypt_password(raw_password):
+    import sha, random
+    algo = 'sha1'
+    salt = sha.new(str(random.random())).hexdigest()[:5]
+    hsh = sha.new(salt+raw_password).hexdigest()
+    return '%s$%s$%s' % (algo, salt, hsh)
+
 class SiteProfileNotAvailable(Exception):
     pass
 
@@ -91 +98 @@
     first_name = models.CharField(_('first name'), maxlength=30, blank=True)
     last_name = models.CharField(_('last name'), maxlength=30, blank=True)
     email = models.EmailField(_('e-mail address'), blank=True)
-    password = models.CharField(_('password'), maxlength=128, help_text=_("Use '[algo]$[salt]$[hexdigest]'"))
+    password = models.PasswordField(_('password'), maxlength=128, show_raw=True, encrypt_func=encrypt_password, help_text=_("Above is the encrypted password. Use '[algo]$[salt]$[hexdigest]'."))
     is_staff = models.BooleanField(_('staff status'), help_text=_("Designates whether the user can log into this admin site."))
     is_active = models.BooleanField(_('active'), default=True, help_text=_("Designates whether this user can log into the Django admin. Unselect this instead of deleting accounts."))
     is_superuser = models.BooleanField(_('superuser status'), help_text=_("Designates that this user has all permissions without explicitly assigning them."))
@@ -138 +145 @@
         return full_name.strip()
 
     def set_password(self, raw_password):
-        import sha, random
-        algo = 'sha1'
-        salt = sha.new(str(random.random())).hexdigest()[:5]
-        hsh = sha.new(salt+raw_password).hexdigest()
-        self.password = '%s$%s$%s' % (algo, salt, hsh)
+        self.password = encrypt_pasword(raw_password)
 
     def check_password(self, raw_password):
         """

django/db/models/fields/__init__.py

@@ -385 +385 @@
                 raise validators.ValidationError, gettext_lazy("This field cannot be null.")
         return str(value)
 
+class PasswordField(CharField):
+    def __init__(self, *args, **kwargs):
+        self.show_raw = kwargs.pop('show_raw', False)
+        self.encrypt_func = kwargs.pop('encrypt_func', None)
+        if not self.show_raw:
+            # Since raw will be a hidden field in this case, we can't use maxlength.
+            # The other solution is adding maxlength=None to HiddenField.__init__ in django.forms.
+            kwargs.pop('maxlength')
+        super(PasswordField, self).__init__(*args, **kwargs)
+
+    def get_manipulator_field_objs(self):
+        return [forms.PasswordField,
+                forms.PasswordField,
+                self.show_raw and forms.TextField or forms.HiddenField]
+
+    def get_manipulator_field_names(self, name_prefix):
+        return [name_prefix + self.name,
+                name_prefix + self.name + '_confirm',
+                name_prefix + self.name + '_raw']
+
+    def get_manipulator_fields(self, opts, manipulator, change, name_prefix='', rel=False, follow=True):
+        field_list = Field.get_manipulator_fields(self, opts, manipulator, change, name_prefix, rel, follow)
+        if not self.blank:
+            # TODO: Making this work in a related context is over my head.
+            v = validators.AlwaysMatchesOtherField(field_list[1].field_name, gettext_lazy('New password and confirm password fields must match.'))
+            field_list[0].validator_list.append(v)
+            field_list[0].is_required = field_list[1].is_required = False
+            #field_list[0].maxlength = field_list[1].maxlength = self.maxlength
+            field_list[0].length = field_list[1].length = 10
+            field_list[2].length = 60
+        return field_list
+
+    def get_manipulator_new_data(self, new_data, rel=False):
+        change, confirm, raw = self.get_manipulator_field_names('')
+        if rel:
+            # TODO: I don't know if this rel stuff is right.
+            change = new_data.get(change, [None])[0]
+            confirm = new_data.get(confirm, [None])[0]
+            raw = new_data.get(raw, [None])[0]
+        else:
+            change = new_data.get(change, None)
+            confirm = new_data.get(confirm, None)
+            raw = new_data.get(raw, None)
+        # The change == confirm test should be done by the validator, but
+        # it can't hurt to check again.
+        if change and change == confirm:
+            return self.encrypt(change)
+        if raw is not None:
+            return raw
+        return self.get_default()
+
+    def flatten_data(self,follow, obj = None):
+        val = self._get_val_from_obj(obj)
+        change, confirm, raw = self.get_manipulator_field_names('')
+        return {raw: val,
+                change: '',
+                confirm: ''}
+    
+    def encrypt(self, raw_password):
+        """
+        Attempt to encrypt password using the encrypt_func function.
+        The function should take only one argument: the raw password.
+        """
+        if callable(self.encrypt_func):
+            try:
+                return self.encrypt_func(raw_password)
+            except TypeError:  # Most likely the function received incorrect parameters.
+                pass
+        return raw_password
+
 # TODO: Maybe move this into contrib, because it's specialized.
 class CommaSeparatedIntegerField(CharField):
     def get_manipulator_field_objs(self):

docs/model-api.txt

@@ -322 +322 @@
 
 The admin represents this as a ``<select>`` box with "Unknown", "Yes" and "No" choices.
 
+``PasswordField``
+~~~~~~~~~~~~~~~~~  
+
+A ``PasswordField`` is similar to ``TextField`` but the characters that are
+entered are masked, typically by asterisks (*), when entered into a form. Note
+that though the data is masked on entry, it is sent as clear text to the
+server and stored as plain text in the database. Additional measures (such as
+using HTTPS) are needed to ensure the security of data sent from a form.
+
+The user is required to enter the same password in a password confirmation box
+to ensure the password was correctly entered.
+
+Has a two optional arguments:
+
+    ======================  ===================================================
+    Argument                Description
+    ======================  ===================================================
+    ``show_raw``            Either ``True`` or ``False``. Shows the existing
+                            password value as stored in the database in a text
+                            input box. Defaults to ``False``, which passes the
+                            existing password as a hidden input value.
+
+    ``encrypt_func``        An optional function to encrypt a new passwords.
+                            The function should take one argument (the raw
+                            password) and return the encrypted password.
+                            This happens on the server when a new password is
+                            entered.
+    ======================  ===================================================
+
+If ``show_raw`` is set to ``True``, the user can enter a password in to the raw
+text input box without it being processed by ``encrypt_func``.
+
 ``PhoneNumberField``
 ~~~~~~~~~~~~~~~~~~~~