From f22f39f67f02ff263a997835a765446fc24219c1 Mon Sep 17 00:00:00 2001
From: Drew Larson <drew@drwlrsn.com>
Date: Thu, 26 May 2016 18:30:33 -0600
Subject: [PATCH] Saner member sign-up!

---
 bikeshop_project/registration/forms.py        |  2 +-
 .../templates/edit_member_form.html           |  6 ++--
 .../registration/templates/member_form.html   | 14 ++++++----
 bikeshop_project/registration/views.py        | 28 +++++++++++++------
 4 files changed, 31 insertions(+), 19 deletions(-)

diff --git a/bikeshop_project/registration/forms.py b/bikeshop_project/registration/forms.py
index ccc1b43..a1da26d 100644
--- a/bikeshop_project/registration/forms.py
+++ b/bikeshop_project/registration/forms.py
@@ -4,7 +4,7 @@ from registration.models import Member
 
 
 class MemberForm(ModelForm):
-    waiver_substitute = BooleanField(label='I have read and agree to the above terms & conditions.', widget=CheckboxInput(attrs={'class': 'mdl-checkbox__input'}))
+    waiver_substitute = BooleanField(required=False, label='I have read and agree to the above terms & conditions.', widget=CheckboxInput(attrs={'class': 'mdl-checkbox__input'}))
 
     class Meta:
         model = Member
diff --git a/bikeshop_project/registration/templates/edit_member_form.html b/bikeshop_project/registration/templates/edit_member_form.html
index e1ca6b2..288e25f 100644
--- a/bikeshop_project/registration/templates/edit_member_form.html
+++ b/bikeshop_project/registration/templates/edit_member_form.html
@@ -2,16 +2,16 @@
 {% load staticfiles %}
 
 {% block styles %}
-    <link rel="stylesheet" href="{% static 'vendor/md-date-time-picker/dist/css/md-date-time-picker.min.css' %}">
+    <link rel="stylesheet" href="{% static 'vendor/md-date-time-picker/dist/css/mdDateTimePicker.min.css' %}">
 {% endblock %}
 
 {% block scripts %}
     <script src="{% static 'vendor/moment/min/moment.min.js' %}"></script>
     <script src="{% static 'vendor/object.observe/dist/object-observe-lite.min.js' %}"></script>
     <script src="{% static 'vendor/draggabilly/dist/draggabilly.pkgd.min.js' %}"></script>
-    <script src="{% static 'vendor/md-date-time-picker/dist/js/md-date-time-picker.js' %}"></script>
+    <script src="{% static 'vendor/md-date-time-picker/dist/js/mdDateTimePicker.min.js' %}"></script>
     <script>
-    var dateOfBirth = new mdDateTimePicker({
+    var dateOfBirth = new mdDateTimePicker.default({
         type: 'date',
         past: moment().subtract(100, 'years')
     });
diff --git a/bikeshop_project/registration/templates/member_form.html b/bikeshop_project/registration/templates/member_form.html
index 0858db6..9f8328e 100644
--- a/bikeshop_project/registration/templates/member_form.html
+++ b/bikeshop_project/registration/templates/member_form.html
@@ -2,16 +2,16 @@
 {% load staticfiles %}
 
 {% block styles %}
-    <link rel="stylesheet" href="{% static 'vendor/md-date-time-picker/dist/css/md-date-time-picker.min.css' %}">
+    <link rel="stylesheet" href="{% static 'vendor/md-date-time-picker/dist/css/mdDateTimePicker.min.css' %}">
 {% endblock %}
 
 {% block scripts %}
     <script src="{% static 'vendor/moment/min/moment.min.js' %}"></script>
     <script src="{% static 'vendor/object.observe/dist/object-observe-lite.min.js' %}"></script>
     <script src="{% static 'vendor/draggabilly/dist/draggabilly.pkgd.min.js' %}"></script>
-    <script src="{% static 'vendor/md-date-time-picker/dist/js/md-date-time-picker.js' %}"></script>
+    <script src="{% static 'vendor/md-date-time-picker/dist/js/mdDateTimePicker.min.js' %}"></script>
     <script>
-    var dateOfBirth = new mdDateTimePicker({
+    var dateOfBirth = new mdDateTimePicker.default({
         type: 'date',
         past: moment().subtract(100, 'years')
     });
@@ -20,11 +20,11 @@
     });
     Object.observe(dateOfBirth, function(changes) {
         var input = document.getElementById('{{ form.date_of_birth.id_for_label }}');
-        input.value = dateOfBirth.time().format('YYYY-MM-DD');
+        input.value = dateOfBirth.time.format('YYYY-MM-DD');
         input.parentNode.classList.add('is-dirty');
 
         var threshold = moment.duration(18, 'years');
-        var dob = dateOfBirth.time().clone();
+        var dob = dateOfBirth.time.clone();
 
         if (dob.add(threshold).isAfter(moment())) {
             document.getElementById('{{ form.guardian_name.id_for_label }}').disabled = false;
@@ -75,7 +75,9 @@
 
         {% if form.non_field_errors %}
             <div>
-                <span class="error">{{ form.errors }}</span>
+            {% for errors in form.non_field_errors %}
+                <span class="mdl-textfield__error">{{ error }}</span>
+            {% endfor %}
             </div>
         {% endif %}
         <div class="mdl-textfield mdl-js-textfield mdl-textfield--floating-label {% if form.email.errors %}is-invalid{% endif %}">
diff --git a/bikeshop_project/registration/views.py b/bikeshop_project/registration/views.py
index 3fad6e2..03e0582 100644
--- a/bikeshop_project/registration/views.py
+++ b/bikeshop_project/registration/views.py
@@ -1,12 +1,13 @@
-from django.shortcuts import get_object_or_404
+from django.contrib import messages
+from django.http import HttpResponseRedirect
 from django.template.response import TemplateResponse
-from django.utils import timezone
 from django.views.generic import View
+from django.core.urlresolvers import reverse
 
 from .forms import MemberForm
 from .models import Member
 import logging
-logger = logging.getLogger(__file__)
+logger = logging.getLogger('bikeshop')
 
 
 class MemberFormView(View):
@@ -17,18 +18,27 @@ class MemberFormView(View):
             form = MemberForm(instance=member)
         except Member.DoesNotExist:
             form = MemberForm()
+            member = None
 
         context = dict(form=form)
-        if form.instance:
+        if member:
             return TemplateResponse(request, 'edit_member_form.html', context=context)
         return TemplateResponse(request, 'member_form.html', context=context)
 
-    def post(self, request):
-        form = MemberForm(request.POST)
-
+    def post(self, request, member_id=None):
+        try:
+            logger.debug(member_id)
+            member = Member.objects.get(id=member_id)
+            form = MemberForm(request.POST, instance=member)
+        except Member.DoesNotExist:
+            form = MemberForm(request.POST)
+        logger.debug(form)
         if form.is_valid():
-            form.save()
-            return TemplateResponse(request, 'member_created.html')
+            member_instance = form.save()
+            logger.debug(member_instance)
+            return HttpResponseRedirect(reverse('member_edit', kwargs=dict(member_id=member_instance.id)))
+
+        logger.debug(form)
 
         context = {'form': form}
         return TemplateResponse(request, 'member_form.html', context=context)