add update forms

[edumed.git] / contact / forms.py

diff --git a/contact/forms.py b/contact/forms.py
index 019b38e..73be1db 100644 (file)
--- a/contact/forms.py
+++ b/contact/forms.py
@@ -13,6 +13,7 @@ from . import mailing
 
 
 contact_forms = {}
+update_forms = {}
 admin_list_width = 0
 
 
@@ -21,10 +22,14 @@ class ContactFormMeta(forms.Form.__class__):
         global admin_list_width
         model = super(ContactFormMeta, cls).__new__(cls, name, bases, attrs)
         if model.form_tag:
-            assert model.form_tag not in contact_forms, 'Duplicate form_tag.'
-            if model.admin_list:
-                admin_list_width = max(admin_list_width, len(model.admin_list))
-            contact_forms[model.form_tag] = model
+            if model.form_type == 'create':
+                assert model.form_tag not in contact_forms, 'Duplicate form_tag.'
+                if model.admin_list:
+                    admin_list_width = max(admin_list_width, len(model.admin_list))
+                contact_forms[model.form_tag] = model
+            elif model.form_type == 'update':
+                assert model.form_tag not in update_forms, 'Duplicate form_tag.'
+                update_forms[model.form_tag] = model
         return model
 
 
@@ -33,6 +38,7 @@ class ContactForm(forms.Form):
     __metaclass__ = ContactFormMeta
 
     form_tag = None
+    form_type = 'create'
     old_form_tags = []
     form_title = _('Contact form')
     submit_label = _('Submit')
@@ -45,6 +51,26 @@ class ContactForm(forms.Form):
     required_css_class = 'required'
     contact = NotImplemented
 
+    def __init__(self, *args, **kwargs):
+        self.instance = kwargs.pop('instance', None)
+        super(ContactForm, self).__init__(*args, **kwargs)
+        if not self.is_bound and self.instance:
+            # files are omitted (not necessary for now)
+            self.fields['contact'].initial = self.instance.contact
+            body = self.instance.body
+            for field, value in body.iteritems():
+                if field in self.fields:
+                    self.fields[field].initial = value
+
+    def get_dictionary(self, contact):
+        site = Site.objects.get_current()
+        return {
+            'form_tag': self.form_tag,
+            'site_name': getattr(self, 'site_name', site.name),
+            'site_domain': getattr(self, 'site_domain', site.domain),
+            'contact': contact,
+        }
+
     def save(self, request, formsets=None):
         from .models import Attachment, Contact
         body = {}
@@ -61,11 +87,21 @@ class ContactForm(forms.Form):
                 if sub_body:
                     body.setdefault(f.form_tag, []).append(sub_body)
 
-        contact = Contact.objects.create(
-            body=body,
-            ip=request.META['REMOTE_ADDR'],
-            contact=self.cleaned_data['contact'],
-            form_tag=self.form_tag)
+        if self.instance:
+            contact = self.instance
+            contact.body = body
+            email_changed = contact.contact != self.cleaned_data['contact']
+            contact.contact = self.cleaned_data['contact']
+            assert contact.form_tag == self.form_tag
+            contact.save()
+        else:
+            contact = Contact.objects.create(
+                body=body,
+                ip=request.META['REMOTE_ADDR'],
+                contact=self.cleaned_data['contact'],
+                form_tag=self.form_tag)
+            email_changed = True
+        # not intended to be used with update forms
         for name, value in self.cleaned_data.items():
             if isinstance(value, UploadedFile):
                 attachment = Attachment(contact=contact, tag=name)
@@ -73,12 +109,7 @@ class ContactForm(forms.Form):
                 attachment.save()
 
         site = Site.objects.get_current()
-        dictionary = {
-            'form_tag': self.form_tag,
-            'site_name': getattr(self, 'site_name', site.name),
-            'site_domain': getattr(self, 'site_domain', site.domain),
-            'contact': contact,
-        }
+        dictionary = self.get_dictionary(contact)
         context = RequestContext(request)
         mail_managers_subject = render_to_string([
                 'contact/%s/mail_managers_subject.txt' % self.form_tag,
@@ -95,25 +126,25 @@ class ContactForm(forms.Form):
         except ValidationError:
             pass
         else:
-            mail_subject = render_to_string([
-                    'contact/%s/mail_subject.txt' % self.form_tag,
-                    'contact/mail_subject.txt', 
-                ], dictionary, context).strip()
-            if self.result_page:
-                mail_body = render_to_string(
-                    'contact/%s/results_email.txt' % contact.form_tag,
-                    {
-                        'contact': contact,
-                        'results': self.results(contact),
-                    }, context)
-            else:
-                mail_body = render_to_string([
-                        'contact/%s/mail_body.txt' % self.form_tag,
-                        'contact/mail_body.txt',
-                    ], dictionary, context)
-            send_mail(mail_subject, mail_body, 'no-reply@%s' % site.domain, [contact.contact], fail_silently=True)
-            if self.mailing or (self.mailing_field and self.cleaned_data[self.mailing_field]):
-                email = self.cleaned_data['contact']
-                mailing.subscribe(email)
+            if not self.instance:
+                mail_subject = render_to_string([
+                        'contact/%s/mail_subject.txt' % self.form_tag,
+                        'contact/mail_subject.txt',
+                    ], dictionary, context).strip()
+                if self.result_page:
+                    mail_body = render_to_string(
+                        'contact/%s/results_email.txt' % contact.form_tag,
+                        {
+                            'contact': contact,
+                            'results': self.results(contact),
+                        }, context)
+                else:
+                    mail_body = render_to_string([
+                            'contact/%s/mail_body.txt' % self.form_tag,
+                            'contact/mail_body.txt',
+                        ], dictionary, context)
+                send_mail(mail_subject, mail_body, 'no-reply@%s' % site.domain, [contact.contact], fail_silently=True)
+            if email_changed and (self.mailing or (self.mailing_field and self.cleaned_data[self.mailing_field])):
+                mailing.subscribe(contact.contact)
 
         return contact