diff --git a/core/views/user_registration.py b/core/views/user_registration.py index 88b2fa8..2bfacec 100644 --- a/core/views/user_registration.py +++ b/core/views/user_registration.py @@ -19,11 +19,12 @@ from troggle.core.views.auth import expologout """ This is the new individual user login registration, instead of everyone signing -in as "expo". This will be useful for the kanban expo organisation tool. +in as "expo". This will be useful for the kanban expo organisation tool.. maybe? """ todo = """ - Make all this work with New people who have never been on expo before +- Stop anyone re-registering an email for an id which already has an email - login automatically, and redirect to control panel ? """ @@ -135,7 +136,7 @@ def register(request, url_username=None): similar to the "expo" user (with cavey:beery password) but specific to an individual. - We only allow this to be done ONCE for each user-id. + We should only allow this to be done ONCE for each user-id. But this constraint seems to be broken. """ warning = "" @@ -143,7 +144,7 @@ def register(request, url_username=None): logged_in = (identified_login := is_identified_user(request.user)) if logged_in: - # logged in as a known real person with a USer logon + # logged in as a known real person with a User logon return re_register_email(request) # discarding url_username if not request.user.is_anonymous: @@ -153,7 +154,7 @@ def register(request, url_username=None): if url_username: # if provided in URL if Person.objects.filter(slug=url_username).count() != 1: # not an old expoer, so redirect to the other form - print(Person.objects.filter(slug=url_username).count()) + print(url_username, Person.objects.filter(slug=url_username).count()) return HttpResponseRedirect("/accounts/newregister/") initial_values.update({"username": url_username})