diff contact/forms.py @ 819:38db6ec61af3

Contact form can prepopulate subject field now. Also updated code for PEP-8 and added tests.
author Brian Neal <bgneal@gmail.com>
date Sat, 20 Sep 2014 14:05:52 -0500
parents ee87ea74d46b
children 79a71b9d0a2a
line wrap: on
line diff
--- a/contact/forms.py	Sat Sep 13 16:19:46 2014 -0500
+++ b/contact/forms.py	Sat Sep 20 14:05:52 2014 -0500
@@ -8,40 +8,38 @@
 
 
 class ContactForm(forms.Form):
-   """Form used to contact the website admins"""
-   name = forms.CharField(label = "Your Name", max_length = 61,
-         widget = forms.TextInput(attrs = {'size' : 50 }))
-   email = forms.EmailField(label = "Your Email",
-         widget = forms.TextInput(attrs = {'size' : 50 }))
-   subject = forms.CharField(max_length = 64,
-         widget = forms.TextInput(attrs = {'size' : 50 }))
-   honeypot = forms.CharField(max_length = 64, required = False,
-         label = 'If you enter anything in this field your message will be treated as spam')
-   message = forms.CharField(label = "Your Message", 
-         widget = forms.Textarea(attrs = {'rows' : 16, 'cols' : 50}), 
-         max_length = 3000)
+    """Form used to contact the website admins"""
+    name=forms.CharField(label="Your Name", max_length=61,
+            widget=forms.TextInput(attrs={'size': 50 }))
+    email=forms.EmailField(label="Your Email",
+            widget=forms.TextInput(attrs={'size': 50 }))
+    subject=forms.CharField(max_length=64,
+            widget=forms.TextInput(attrs={'size': 50 }))
+    honeypot=forms.CharField(max_length=64, required=False,
+            label='If you enter anything in this field your message will be treated as spam')
+    message=forms.CharField(label="Your Message",
+            widget=forms.Textarea(attrs={'rows': 16, 'cols': 50}),
+            max_length=3000)
 
-   recipient_list = [mail_tuple[1] for mail_tuple in settings.MANAGERS]
+    recipient_list = [mail_tuple[1] for mail_tuple in settings.MANAGERS]
 
-   def clean_honeypot(self):
-      value = self.cleaned_data['honeypot']
-      if value:
-         raise forms.ValidationError(self.fields['honeypot'].label)
-      return value
+    def clean_honeypot(self):
+        value = self.cleaned_data['honeypot']
+        if value:
+            raise forms.ValidationError(self.fields['honeypot'].label)
+        return value
 
-   def save(self):
-      # Send the feedback message email
+    def save(self):
+        # Send the feedback message email
 
-      site = Site.objects.get_current()
+        site = Site.objects.get_current()
 
-      msg = render_to_string('contact/contact_email.txt',
-            {
-               'site_name' : site.name,
-               'user_name' : self.cleaned_data['name'],
-               'user_email' : self.cleaned_data['email'],
-               'message' : self.cleaned_data['message'],
-            })
+        msg = render_to_string('contact/contact_email.txt', {
+                    'site_name': site.name,
+                    'user_name': self.cleaned_data['name'],
+                    'user_email': self.cleaned_data['email'],
+                    'message': self.cleaned_data['message'],
+                })
 
-      subject = site.name + ' Feedback: ' + self.cleaned_data['subject']
-      send_mail(subject, msg, self.cleaned_data['email'], self.recipient_list)
-
+        subject = site.name + ' Feedback: ' + self.cleaned_data['subject']
+        send_mail(subject, msg, self.cleaned_data['email'], self.recipient_list)