Ticket #5605: lowercase-email.diff

django/contrib/auth/models.py

@@ -104 +104 @@
     def create_user(self, username, email, password=None):
         "Creates and saves a User with the given username, e-mail and password."
         now = datetime.datetime.now()
-        user = self.model(None, username, '', '', email.strip().lower(), 'placeholder', False, True, False, now, now)
+        # Make only the domain lowercase.
+        parts = email.strip().split('@', 1)
+        if len(parts) > 1:
+            parts[1] = parts[1].lower()
+        email = '@'.join(parts)
+        user = self.model(None, username, '', '', email, 'placeholder', False, True, False, now, now)
         if password:
             user.set_password(password)
         else:

tests/regressiontests/auth_backends/tests.py

@@ -75 +75 @@
 False
 >>> user.has_perms(['auth.test2', 'auth.test3'])
 False
+
+# bug #5605, preserve the case of the user name (before the @ in the email address)
+# when creating a user.
+
+>>> user = User.objects.create_user('test2', 'tesT@EXAMple.com', 'test')
+>>> user.email
+'tesT@example.com'
+>>> user = User.objects.create_user('test3', 'tesT', 'test')
+>>> user.email
+'tesT'
 """}

docs/authentication.txt

@@ -168 +168 @@
 The ``User`` model has a custom manager that has the following helper functions:
 
     * ``create_user(username, email, password=None)`` -- Creates, saves and
-      returns a ``User``. The ``username``, ``email`` and ``password`` are set
-      as given, and the ``User`` gets ``is_active=True``.
+      returns a ``User``. The ``username`` and ``password`` are set as
+      given, the domain portion of ``email`` gets lowercased, and the
+      ``User`` gets ``is_active=True``.
 
       If no password is provided, ``set_unusable_password()`` will be called.