From 19c19132688e0ca812666e4869e2ac54731fc4c9 Mon Sep 17 00:00:00 2001
From: Issac Kelly <issac.kelly@gmail.com>
Date: Tue, 15 Jul 2014 15:48:42 -0700
Subject: [PATCH] Allow Templated Subjects.

---
 docs/settings.rst    | 4 ++++
 helpdesk/lib.py      | 2 +-
 helpdesk/settings.py | 3 +++
 3 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/docs/settings.rst b/docs/settings.rst
index 38bb35fb..b6a5f03b 100644
--- a/docs/settings.rst
+++ b/docs/settings.rst
@@ -86,6 +86,10 @@ These changes are visible throughout django-helpdesk
   
   **Default:** ``HELPDESK_AUTO_SUBSCRIBE_ON_TICKET_RESPONSE = False``
 
+- **HELPDESK_EMAIL_SUBJECT_TEMPLATE ** Subject template for templated emails. ``%(subject)s`` represents the subject wording from the email template (e.g. "(Closed)").
+
+  **Default:** ``HELPDESK_EMAIL_SUBJECT_TEMPLATE = "{{ ticket.ticket }} {{ ticket.title|safe }} %(subject)s"``
+
 
 Options shown on public pages
 -----------------------------
diff --git a/helpdesk/lib.py b/helpdesk/lib.py
index 39bc3074..0b66da41 100644
--- a/helpdesk/lib.py
+++ b/helpdesk/lib.py
@@ -104,7 +104,7 @@ def send_templated_mail(template_name, email_context, recipients, sender=None, b
         ).render(context)
 
     subject_part = loader.get_template_from_string(
-        "{{ ticket.ticket }} {{ ticket.title|safe }} %s" % t.subject
+        settings.HELPDESK_EMAIL_SUBJECT_TEMPLATE % t.subject
         ).render(context)
 
     if isinstance(recipients,(str,unicode)):
diff --git a/helpdesk/settings.py b/helpdesk/settings.py
index 20140721..52fb4900 100644
--- a/helpdesk/settings.py
+++ b/helpdesk/settings.py
@@ -80,6 +80,9 @@ HELPDESK_STAFF_ONLY_TICKET_OWNERS = getattr(settings, 'HELPDESK_STAFF_ONLY_TICKE
 HELPDESK_STAFF_ONLY_TICKET_CC = getattr(settings, 'HELPDESK_STAFF_ONLY_TICKET_CC', False)
 
 
+# allow the subject to have a configurable template.
+HELPDESK_EMAIL_SUBJECT_TEMPLATE = getattr(settings, 'HELPDESK_EMAIL_SUBJECT_TEMPLATE', "{{ ticket.ticket }} {{ ticket.title|safe }} %(subject)s")
+
 
 ''' options for staff.create_ticket view '''
 # hide the 'assigned to' / 'Case owner' field from the 'create_ticket' view?