Merge branch 'master' of github.com:webnotes/erpnext
diff --git a/startup/schedule_handlers.py b/startup/schedule_handlers.py
index 668c11d..54b9892 100644
--- a/startup/schedule_handlers.py
+++ b/startup/schedule_handlers.py
@@ -26,7 +26,7 @@
* recurring invoice
"""
# pull emails
- from support.doctype.support_ticket import get_support_mails
+ from support.doctype.support_ticket.get_support_mails import get_support_mails
run_fn(get_support_mails)
# bulk email
diff --git a/support/doctype/newsletter/newsletter.py b/support/doctype/newsletter/newsletter.py
index c4b622a..48ed21a 100644
--- a/support/doctype/newsletter/newsletter.py
+++ b/support/doctype/newsletter/newsletter.py
@@ -87,7 +87,7 @@
args = self.dt_map[doctype]
- sender = self.doc.send_from or webnotes.utils.get_email_id(self.doc.owner)
+ sender = self.doc.send_from or webnotes.utils.get_formatted_email(self.doc.owner)
recipients = self.doc.test_email_id.split(",")
from webnotes.utils.email_lib.bulk import send
send(recipients = recipients, sender = sender,
@@ -109,7 +109,7 @@
recipients = self.get_recipients(query_key)
else:
recipients = query_key
- sender = self.doc.send_from or webnotes.utils.get_email_id(self.doc.owner)
+ sender = self.doc.send_from or webnotes.utils.get_formatted_email(self.doc.owner)
args = self.dt_map[doctype]
self.send_count[doctype] = self.send_count.setdefault(doctype, 0) + \
len(recipients)
diff --git a/support/doctype/support_ticket/__init__.py b/support/doctype/support_ticket/__init__.py
index da1755f..e69de29 100644
--- a/support/doctype/support_ticket/__init__.py
+++ b/support/doctype/support_ticket/__init__.py
@@ -1,183 +0,0 @@
-# ERPNext - web based ERP (http://erpnext.com)
-# Copyright (C) 2012 Web Notes Technologies Pvt Ltd
-#
-# This program is free software: you can redistribute it and/or modify
-# it under the terms of the GNU General Public License as published by
-# the Free Software Foundation, either version 3 of the License, or
-# (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program. If not, see <http://www.gnu.org/licenses/>.
-
-from __future__ import unicode_literals
-import webnotes
-from webnotes.utils import cstr, cint
-
-from webnotes.utils.email_lib.receive import POP3Mailbox
-
-class SupportMailbox(POP3Mailbox):
- def __init__(self):
- """
- settings_doc must contain
- use_ssl, host, username, password
- """
- from webnotes.model.doc import Document
-
- # extract email settings
- self.email_settings = Document('Email Settings','Email Settings')
- if not self.email_settings.fields.get('sync_support_mails'): return
-
- s = Document('Support Email Settings')
- s.use_ssl = self.email_settings.support_use_ssl
- s.host = self.email_settings.support_host
- s.username = self.email_settings.support_username
- s.password = self.email_settings.support_password
-
- POP3Mailbox.__init__(self, s)
-
- def check_mails(self):
- """
- returns true if there are active sessions
- """
- self.auto_close_tickets()
- return webnotes.conn.sql("select user from tabSessions where time_to_sec(timediff(now(), lastupdate)) < 1800")
-
- def process_message(self, mail):
- """
- Updates message from support email as either new or reply
- """
- from home import update_feed
-
- content, content_type = '[Blank Email]', 'text/plain'
- if mail.text_content:
- content, content_type = mail.text_content, 'text/plain'
- else:
- content, content_type = mail.html_content, 'text/html'
-
- thread_list = mail.get_thread_id()
-
- email_id = mail.mail['From']
- if "<" in mail.mail['From']:
- import re
- re_result = re.findall('(?<=\<)(\S+)(?=\>)', mail.mail['From'])
- if re_result and re_result[0]:
- email_id = re_result[0]
-
- from webnotes.utils import decode_email_header
-
- full_email_id = decode_email_header(mail.mail['From'])
-
- for thread_id in thread_list:
- exists = webnotes.conn.sql("""\
- SELECT name
- FROM `tabSupport Ticket`
- WHERE name=%s AND raised_by REGEXP %s
- """ , (thread_id, '(' + email_id + ')'))
- if exists and exists[0] and exists[0][0]:
- st = webnotes.get_obj('Support Ticket', thread_id)
-
- from core.doctype.communication.communication import make
-
- make(content=content, sender=full_email_id, doctype="Support Ticket",
- name=thread_id, lead = st.doc.lead, contact=st.doc.contact)
-
- st.doc.status = 'Open'
- st.doc.save()
-
- update_feed(st, 'on_update')
- # extract attachments
- self.save_attachments(st.doc, mail.attachments)
- return
-
- from webnotes.model.doctype import get_property
- opts = get_property('Support Ticket', 'options', 'naming_series')
- # new ticket
- from webnotes.model.doc import Document
- d = Document('Support Ticket')
- d.description = content
-
- d.subject = decode_email_header(mail.mail['Subject'])
-
- d.raised_by = full_email_id
- d.content_type = content_type
- d.status = 'Open'
- d.naming_series = opts and opts.split("\n")[0] or 'SUP'
- try:
- d.save(1)
- try:
- # extract attachments
- self.save_attachments(d, mail.attachments)
- except Exception, e:
- self.description += "\n\n[Did not pull attachment]"
- except:
- d.description = 'Unable to extract message'
- d.save(1)
- else:
- # send auto reply
- if cint(self.email_settings.send_autoreply):
- if "mailer-daemon" not in d.raised_by.lower():
- self.send_auto_reply(d)
-
-
- def save_attachments(self, doc, attachment_list=[]):
- """
- Saves attachments from email
-
- attachment_list is a list of dict containing:
- 'filename', 'content', 'content-type'
- """
- from webnotes.utils.file_manager import save_file, add_file_list
- for attachment in attachment_list:
- fid = save_file(attachment['filename'], attachment['content'], 'Support')
- status = add_file_list('Support Ticket', doc.name, fid, fid)
- if not status:
- doc.description = doc.description \
- + "\nCould not attach: " + cstr(attachment['filename'])
- doc.save()
-
- def send_auto_reply(self, d):
- """
- Send auto reply to emails
- """
- from webnotes.utils import cstr
-
- signature = self.email_settings.fields.get('support_signature') or ''
-
- response = self.email_settings.fields.get('support_autoreply') or ("""
-A new Ticket has been raised for your query. If you have any additional information, please
-reply back to this mail.
-
-We will get back to you as soon as possible
-----------------------
-Original Query:
-
-""" + d.description + "\n----------------------\n" + cstr(signature))
-
- from webnotes.utils.email_lib import sendmail
-
- sendmail(\
- recipients = [cstr(d.raised_by)], \
- sender = cstr(self.email_settings.fields.get('support_email')), \
- subject = '['+cstr(d.name)+'] ' + cstr(d.subject), \
- msg = cstr(response))
-
- def auto_close_tickets(self):
- """
- Auto Closes Waiting for Customer Support Ticket after 15 days
- """
- webnotes.conn.sql("update `tabSupport Ticket` set status = 'Closed' where status = 'Waiting for Customer' and date_sub(curdate(),interval 15 Day) > modified")
-
-
-def get_support_mails():
- """
- Gets new emails from support inbox and updates / creates Support Ticket records
- """
- import webnotes
- from webnotes.utils import cint
- if cint(webnotes.conn.get_value('Email Settings', None, 'sync_support_mails')):
- SupportMailbox().get_messages()
\ No newline at end of file
diff --git a/support/doctype/support_ticket/get_support_mails.py b/support/doctype/support_ticket/get_support_mails.py
new file mode 100644
index 0000000..e5e99f5
--- /dev/null
+++ b/support/doctype/support_ticket/get_support_mails.py
@@ -0,0 +1,94 @@
+# ERPNext - web based ERP (http://erpnext.com)
+# Copyright (C) 2012 Web Notes Technologies Pvt Ltd
+#
+# This program is free software: you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program. If not, see <http://www.gnu.org/licenses/>.
+
+from __future__ import unicode_literals
+import webnotes
+from webnotes.utils import cstr, cint
+from webnotes.utils.email_lib import sendmail
+from webnotes.utils.email_lib.receive import POP3Mailbox
+from core.doctype.communication.communication import make
+
+class SupportMailbox(POP3Mailbox):
+ def setup(self):
+ self.email_settings = webnotes.doc("Email Settings", "Email Settings")
+ self.settings = webnotes._dict({
+ "use_ssl": self.email_settings.support_use_ssl,
+ "host": self.email_settings.support_host,
+ "username": self.email_settings.support_username,
+ "password": self.email_settings.support_password
+ })
+
+ def check_mails(self):
+ self.auto_close_tickets()
+ return webnotes.conn.sql("select user from tabSessions where \
+ time_to_sec(timediff(now(), lastupdate)) < 1800")
+
+ def process_message(self, mail):
+ thread_id = mail.get_thread_id()
+ ticket = None
+
+ if thread_id and webnotes.conn.exists("Support Ticket", thread_id):
+ ticket = webnotes.model_wrapper("Support Ticket", thread_id)
+ ticket.doc.status = 'Open'
+ ticket.doc.save()
+
+ else:
+ ticket = webnotes.model_wrapper([{
+ "doctype":"Support Ticket",
+ "description": mail.content,
+ "subject": mail.mail["Subject"],
+ "raised_by": mail.from_email,
+ "content_type": mail.content_type,
+ "status": "Open"
+ }])
+ ticket.insert()
+
+ if cint(self.email_settings.send_autoreply):
+ if "mailer-daemon" not in mail.from_email.lower():
+ self.send_auto_reply(ticket.doc)
+
+ mail.save_attachments_in_doc(ticket.doc)
+
+ make(content=mail.content, sender=mail.from_email,
+ doctype="Support Ticket", name=ticket.doc.name,
+ lead = ticket.doc.lead, contact=ticket.doc.contact)
+
+ def send_auto_reply(self, d):
+ signature = self.email_settings.fields.get('support_signature') or ''
+ response = self.email_settings.fields.get('support_autoreply') or ("""
+A new Ticket has been raised for your query. If you have any additional information, please
+reply back to this mail.
+
+We will get back to you as soon as possible
+----------------------
+Original Query:
+
+""" + d.description + "\n----------------------\n" + cstr(signature))
+
+ sendmail(\
+ recipients = [cstr(d.raised_by)], \
+ sender = cstr(self.email_settings.fields.get('support_email')), \
+ subject = '['+cstr(d.name)+'] ' + cstr(d.subject), \
+ msg = cstr(response))
+
+ def auto_close_tickets(self):
+ webnotes.conn.sql("""update `tabSupport Ticket` set status = 'Closed'
+ where status = 'Waiting for Customer'
+ and date_sub(curdate(),interval 15 Day) > modified""")
+
+def get_support_mails():
+ if cint(webnotes.conn.get_value('Email Settings', None, 'sync_support_mails')):
+ SupportMailbox()
\ No newline at end of file
diff --git a/support/doctype/support_ticket/support_ticket.js b/support/doctype/support_ticket/support_ticket.js
index 159dddd..28b08f8 100644
--- a/support/doctype/support_ticket/support_ticket.js
+++ b/support/doctype/support_ticket/support_ticket.js
@@ -49,11 +49,17 @@
var wrapper = cur_frm.fields_dict['thread_html'].wrapper;
var comm_list = wn.model.get("Communication", {"support_ticket": doc.name})
- comm_list.push({
- "sender": doc.raised_by,
- "creation": doc.creation,
- "modified": doc.creation,
- "content": doc.description});
+
+ var sortfn = function (a, b) { return (b.creation > a.creation) ? 1 : -1; }
+ comm_list = comm_list.sort(sortfn);
+
+ if(!comm_list.length || (comm_list[0].sender != doc.raised_by)) {
+ comm_list.push({
+ "sender": doc.raised_by,
+ "creation": doc.creation,
+ "modified": doc.creation,
+ "content": doc.description});
+ }
cur_frm.communication_view = new wn.views.CommunicationList({
list: comm_list,