From 9f2ce9d7c5b74a504c2b375a5928a6518b746236 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?V=C3=AD=C3=B0ir=20Valberg=20Gu=C3=B0mundsson?=
 <valberg@orn.li>
Date: Sat, 20 Feb 2016 05:18:12 +0100
Subject: [PATCH] Adding profile update functionality

---
 .gitignore                                    |  1 +
 bornhack/templates/base.html                  |  2 +
 profiles/forms.py                             | 11 +++++
 .../templates/profiles/profile_detail.html    |  5 +++
 profiles/templates/profiles/profile_form.html | 20 ++++++++++
 profiles/urls.py                              |  5 ++-
 profiles/views.py                             | 40 ++++++++++++++-----
 7 files changed, 71 insertions(+), 13 deletions(-)
 create mode 100644 profiles/forms.py
 create mode 100644 profiles/templates/profiles/profile_form.html
diff --git a/.gitignore b/.gitignore
index ee6cf3bb..f6170ce7 100644
--- a/.gitignore
+++ b/.gitignore
@@ -3,3 +3,4 @@ __pycache__/
 db.sqlite3
 *.sw*
 .env
+*.pyc
diff --git a/bornhack/templates/base.html b/bornhack/templates/base.html
index 9c988265..46349dbb 100644
--- a/bornhack/templates/base.html
+++ b/bornhack/templates/base.html
@@ -1,4 +1,5 @@
 {% load static from staticfiles %}
+{% load bootstrap3 %}
 <!DOCTYPE html>
 <html lang="en">
   <head>
@@ -49,6 +50,7 @@
     </nav>
 
     <div id="main" class="container container-fluid">
+      {% bootstrap_messages %}
       {% block content %}{% endblock %}
     </div>
     <script src="{% static 'js/bootstrap.min.js' %}">
diff --git a/profiles/forms.py b/profiles/forms.py
new file mode 100644
index 00000000..81af6302
--- /dev/null
+++ b/profiles/forms.py
@@ -0,0 +1,11 @@
+from django import forms
+
+from . import models
+
+
+class ProfileForm(forms.ModelForm):
+    email = forms.EmailField()
+
+    class Meta:
+        model = models.Profile
+        fields = []
diff --git a/profiles/templates/profiles/profile_detail.html b/profiles/templates/profiles/profile_detail.html
index 62eb9e5d..de27f23e 100644
--- a/profiles/templates/profiles/profile_detail.html
+++ b/profiles/templates/profiles/profile_detail.html
@@ -5,5 +5,10 @@
     <p class="lead">
       Yeah! You now have a BornHack user!
     </p>
+    <p>
+        <a href="{% url 'profiles:update' %}">
+            Edit your profile
+        </a>
+    </p>
   </div>
 {% endblock %}
diff --git a/profiles/templates/profiles/profile_form.html b/profiles/templates/profiles/profile_form.html
new file mode 100644
index 00000000..cd2760d1
--- /dev/null
+++ b/profiles/templates/profiles/profile_form.html
@@ -0,0 +1,20 @@
+{% extends "base.html" %}
+
+{% load account %}
+{% load bootstrap3 %}
+
+{% block content %}
+
+  <div class="row">
+    <div class="col-md-6 col-md-offset-3">
+      <h1>Edit profile</h1>
+
+      <form class="login" method="POST">
+        {% csrf_token %}
+        {% bootstrap_form form %}
+        <button class="form-control btn btn-black" type="submit">Save</button>
+      </form>
+    </div>
+  </div>
+
+{% endblock %}
diff --git a/profiles/urls.py b/profiles/urls.py
index 4e551d8c..54c8123c 100644
--- a/profiles/urls.py
+++ b/profiles/urls.py
@@ -1,7 +1,8 @@
 from django.conf.urls import url
 
-from .views import ProfileDetail
+from .views import ProfileDetail, ProfileUpdate
 
 urlpatterns = [
     url(r'^$', ProfileDetail.as_view(), name='detail'),
-]
\ No newline at end of file
+    url(r'^edit$', ProfileUpdate.as_view(), name='update'),
+]
diff --git a/profiles/views.py b/profiles/views.py
index f599279f..c6213b66 100644
--- a/profiles/views.py
+++ b/profiles/views.py
@@ -1,16 +1,34 @@
-from django.contrib.auth.decorators import login_required
-from django.utils.decorators import method_decorator
-from django.views.generic import DetailView
-
-from .models import Profile
+from django.contrib.auth.mixins import LoginRequiredMixin
+from django.views.generic import DetailView, UpdateView
+from django.core.urlresolvers import reverse_lazy
+from django.contrib import messages
 
 
-class ProfileDetail(DetailView):
-    model = Profile
+from . import models, forms
 
-    @method_decorator(login_required)
-    def get(self, request, *args, **kwargs):
-        return super().get(request, *args, **kwargs)
+
+class ProfileDetail(LoginRequiredMixin, DetailView):
+    model = models.Profile
 
     def get_object(self, queryset=None):
-        return Profile.objects.get(user=self.request.user)
+        return models.Profile.objects.get(user=self.request.user)
+
+
+class ProfileUpdate(LoginRequiredMixin, UpdateView):
+    model = models.Profile
+    form_class = forms.ProfileForm
+    success_url = reverse_lazy('profiles:detail')
+
+    def get_object(self, queryset=None):
+        return models.Profile.objects.get(user=self.request.user)
+
+    def get_form_kwargs(self):
+        kwargs = super(ProfileUpdate, self).get_form_kwargs()
+        kwargs['initial'] = {'email': self.object.user.email}
+        return kwargs
+
+    def form_valid(self, form, **kwargs):
+        self.object.user.email = form.cleaned_data['email']
+        self.object.user.save()
+        messages.info(self.request, 'Your profile has been updated.')
+        return super(ProfileUpdate, self).form_valid(form, **kwargs)
