Luke Plant avatar Luke Plant committed e16c6a3

Fixed bug with emails being treated as case sensitive, which causes confusion.

Comments (0)

Files changed (2)

cciw/bookings/tests.py

         self.assertEqual(BookingAccount.objects.all().count(), 1)
         self.assertEqual(len(mail.outbox), 1)
 
+    def test_complete_form_existing_email_different_case(self):
+        BookingAccount.objects.create(email="booker@bookers.com")
+        self.assertEqual(BookingAccount.objects.all().count(), 1)
+        resp = self.client.post(self.url,
+                                {'email': 'BOOKER@bookers.com'})
+        self.assertEqual(BookingAccount.objects.all().count(), 1)
+        self.assertEqual(len(mail.outbox), 1)
+
     def test_skip_if_logged_in(self):
         # This assumes verification process works
         # Check redirect to step 3 - account details

cciw/bookings/views.py

     def form_valid(self, form):
         email = form.cleaned_data['email']
         try:
-            account = BookingAccount.objects.get(email=email)
-        except BookingAccount.DoesNotExist:
+            account = BookingAccount.objects.filter(email__iexact=email)[0]
+        except IndexError:
             # Ensure we use NULLs, not empty strings, or we will not be able to
             # create more than one, as they will have same 'name and post_code'
             account = BookingAccount.objects.create(email=email,
Tip: Filter by directory path e.g. /media app.js to search for public/media/app.js.
Tip: Use camelCasing e.g. ProjME to search for ProjectModifiedEvent.java.
Tip: Filter by extension type e.g. /repo .js to search for all .js files in the /repo directory.
Tip: Separate your search with spaces e.g. /ssh pom.xml to search for src/ssh/pom.xml.
Tip: Use ↑ and ↓ arrow keys to navigate and return to view the file.
Tip: You can also navigate files with Ctrl+j (next) and Ctrl+k (previous) and view the file with Ctrl+o.
Tip: You can also navigate files with Alt+j (next) and Alt+k (previous) and view the file with Alt+o.