Capture the user's name at signup time

Ref #18, #15

odp/identity/forms.py

@@ -4,6 +4,10 @@
 
 
 class SignupForm(FlaskForm):
+    name = StringField(
+        label='Full name',
+        validators=[input_required()],
+    )
     email = StringField(
         label='Email address',
         filters=[lambda s: s.lower() if s else s],

odp/identity/templates/signup.html

@@ -8,6 +8,7 @@
 {% block content %}
     <form action="{{ url_for('signup.signup', token=token) }}" method="POST">
         {{ form.csrf_token }}
+        {{ render_field(form.name) }}
         {{ render_field(form.email) }}
         {{ render_field(form.password) }}
         {{ render_field(form.confirm_password) }}

odp/identity/views/signup.py

@@ -38,8 +38,9 @@ def signup():
                 if form.validate():
                     email = form.email.data
                     password = form.password.data
+                    name = form.name.data
                     try:
-                        create_user_account(email, password)
+                        create_user_account(email, password, name)
 
                         # the signup (and login) is completed via email verification
                         send_verification_email(email, challenge)

odp/lib/users.py

@@ -160,7 +160,7 @@ def validate_email_verification(email):
     return user.id
 
 
-def create_user_account(email, password=None):
+def create_user_account(email, password=None, name=None):
     """
     Create a new user account with the specified credentials.
     Password may be omitted if the user is externally authenticated.
@@ -172,6 +172,7 @@ def create_user_account(email, password=None):
 
     :param email: the input email address
     :param password: (optional) the input plain-text password
+    :param name: (optional) the user's personal name
     :return: the new user id
 
     :raises ODPEmailInUse: if the email address is already associated with a user account
@@ -191,6 +192,7 @@ def create_user_account(email, password=None):
             active=True,
             verified=False,
             superuser=False,
+            name=name,
         )
         user.save()