minor fixes in new forms

[edumed.git] / contact / views.py

diff --git a/contact/views.py b/contact/views.py
index 245433e..82e0347 100644 (file)
--- a/contact/views.py
+++ b/contact/views.py
@@ -1,38 +1,50 @@
+# -*- coding: utf-8 -*-
+from urllib import unquote
+

 from django.contrib.auth.decorators import permission_required
 from django.contrib.auth.decorators import permission_required

-from django.http import Http404, HttpResponse
+from django.http import Http404

 from django.shortcuts import get_object_or_404, redirect, render
 from django.shortcuts import get_object_or_404, redirect, render

-from django.utils.translation import ugettext_lazy as _

 from fnpdjango.utils.views import serve_file
 from fnpdjango.utils.views import serve_file

+from honeypot.decorators import check_honeypot
+

 from .forms import contact_forms
 from .forms import contact_forms

-from .models import Attachment
+from .models import Attachment, Contact

 
 
 
 

-def form(request, form_tag):
+@check_honeypot
+def form(request, form_tag, force_enabled=False):

     try:
         form_class = contact_forms[form_tag]
     except KeyError:
         raise Http404
     try:
         form_class = contact_forms[form_tag]
     except KeyError:
         raise Http404

+    if (getattr(form_class, 'disabled', False) and
+            not (force_enabled and request.user.is_superuser)):
+        template = getattr(form_class, 'disabled_template', None)
+        if template:
+            return render(request, template, {'title': form_class.form_title})
+        raise Http404

     if request.method == 'POST':
         form = form_class(request.POST, request.FILES)
     if request.method == 'POST':
         form = form_class(request.POST, request.FILES)

-        formsets = []
-        valid = form.is_valid()
-        for formset in getattr(form, 'form_formsets', ()):
-            fset = formset(request.POST, request.FILES)
-            if not fset.is_valid():
-                valid = False
-            formsets.append(fset)
-        if valid:
-            form.save(request, formsets)
-            return redirect('contact_thanks', form_tag)

     else:
         form = form_class(initial=request.GET)
     else:
         form = form_class(initial=request.GET)

-        formsets = []
-        for formset in getattr(form, 'form_formsets', ()):
-            formsets.append(formset())
-    return render(request,
-                ['contact/%s/form.html' % form_tag, 'contact/form.html'],
-                {'form': form, 'formsets': formsets}
-            )
+    formset_classes = getattr(form, 'form_formsets', {})
+    if request.method == 'POST':
+        formsets = {
+            prefix: formset_class(request.POST, request.FILES, prefix=prefix)
+            for prefix, formset_class in formset_classes.iteritems()}
+        if form.is_valid() and all(formset.is_valid() for formset in formsets.itervalues()):
+            contact = form.save(request, formsets.values())
+            if form.result_page:
+                return redirect('contact_results', contact.id, contact.digest())
+            else:
+                return redirect('contact_thanks', form_tag)
+    else:
+        formsets = {prefix: formset_class(prefix=prefix) for prefix, formset_class in formset_classes.iteritems()}
+
+    return render(
+        request, ['contact/%s/form.html' % form_tag, 'contact/form.html'],
+        {'form': form, 'formsets': formsets}
+    )

 
 
 def thanks(request, form_tag):
 
 
 def thanks(request, form_tag):


@@ -41,13 +53,31 @@ def thanks(request, form_tag):
     except KeyError:
         raise Http404
 
     except KeyError:
         raise Http404
 

-    return render(request,
-                ['contact/%s/thanks.html' % form_tag, 'contact/thanks.html'],
-                dict(base_template = getattr(form_class, 'base_template', None))
-            )
+    return render(
+        request, ['contact/%s/thanks.html' % form_tag, 'contact/thanks.html'],
+        {'base_template': getattr(form_class, 'base_template', None)})
+
+
+def results(request, contact_id, digest):
+    contact = get_object_or_404(Contact, id=contact_id)
+    if digest != contact.digest():
+        raise Http404
+    try:
+        form_class = contact_forms[contact.form_tag]
+    except KeyError:
+        raise Http404
+
+    return render(
+        request, 'contact/%s/results.html' % contact.form_tag,
+        {
+            'results': form_class.results(contact),
+            'base_template': getattr(form_class, 'base_template', None),
+        }
+    )

 
 
 @permission_required('contact.change_attachment')
 def attachment(request, contact_id, tag):
     attachment = get_object_or_404(Attachment, contact_id=contact_id, tag=tag)
 
 
 @permission_required('contact.change_attachment')
 def attachment(request, contact_id, tag):
     attachment = get_object_or_404(Attachment, contact_id=contact_id, tag=tag)

-    return serve_file(attachment.file.url)
+    attachment_url = unquote(attachment.file.url)
+    return serve_file(attachment_url)