diff options
| -rw-r--r-- | core/views/user_registration.py | 9 |
1 files changed, 5 insertions, 4 deletions
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})
|