From cad4bbf073663db828fa7ac43b27ae00556e2797 Mon Sep 17 00:00:00 2001
From: Jefferson Stachelski <jstachel@thoughtworks.com>
Date: Fri, 30 Jan 2015 14:04:16 -0200
Subject: 249 Implemented a feature that send an email when something was wrong
 to send email

---
 service/pixelated/resources/mails_resource.py | 59 +++++++++++++--------------
 1 file changed, 29 insertions(+), 30 deletions(-)

(limited to 'service/pixelated/resources/mails_resource.py')

diff --git a/service/pixelated/resources/mails_resource.py b/service/pixelated/resources/mails_resource.py
index 6cf3df8e..4280d7ec 100644
--- a/service/pixelated/resources/mails_resource.py
+++ b/service/pixelated/resources/mails_resource.py
@@ -3,15 +3,14 @@ from pixelated.adapter.model.mail import InputMail
 from pixelated.resources import respond_json, respond_json_deferred
 from twisted.web import server
 from twisted.web.resource import Resource
-
-
-def _format_exception(e):
-    exception_info = map(str, list(e.args))
-    return '\n'.join(exception_info)
+from leap.common.events import (
+    register,
+    unregister,
+    events_pb2 as proto
+)
 
 
 class MailsUnreadResource(Resource):
-
     isLeaf = True
 
     def __init__(self, mail_service, search_engine):
@@ -29,7 +28,6 @@ class MailsUnreadResource(Resource):
 
 
 class MailsReadResource(Resource):
-
     isLeaf = True
 
     def __init__(self, mail_service, search_engine):
@@ -47,7 +45,6 @@ class MailsReadResource(Resource):
 
 
 class MailsDeleteResource(Resource):
-
     isLeaf = True
 
     def __init__(self, mail_service, search_engine):
@@ -73,6 +70,15 @@ class MailsDeleteResource(Resource):
 
 class MailsResource(Resource):
 
+    def _register_smtp_error_handler(self):
+
+        def on_error(event):
+            delivery_error_mail = InputMail.delivery_error_template(delivery_address=event.content)
+            delivery_error_mail = self._mail_service.mailboxes.inbox().add(delivery_error_mail)
+            self._search_engine.index_mail(delivery_error_mail)
+
+        register(signal=proto.SMTP_SEND_MESSAGE_ERROR, callback=on_error)
+
     def __init__(self, search_engine, mail_service, draft_service):
         Resource.__init__(self)
         self.putChild('delete', MailsDeleteResource(mail_service, search_engine))
@@ -82,9 +88,11 @@ class MailsResource(Resource):
         self._draft_service = draft_service
         self._mail_service = mail_service
         self._search_engine = search_engine
+        self._register_smtp_error_handler()
 
     def render_GET(self, request):
-        mail_ids, total = self._search_engine.search(request.args.get('q')[0], request.args.get('w')[0], request.args.get('p')[0])
+        mail_ids, total = self._search_engine.search(request.args.get('q')[0], request.args.get('w')[0],
+                                                     request.args.get('p')[0])
         mails = self._mail_service.mails(mail_ids)
 
         response = {
@@ -97,27 +105,18 @@ class MailsResource(Resource):
         return respond_json(response, request)
 
     def render_POST(self, request):
-        try:
-            content_dict = json.loads(request.content.read())
-            _mail = InputMail.from_dict(content_dict)
-            draft_id = content_dict.get('ident')
-            if draft_id:
-                self._search_engine.remove_from_index(draft_id)
-            sendDeferred = self._mail_service.send(draft_id, _mail)
-
-            def onSuccess(mail):
-                self._search_engine.index_mail(mail)
-                respond_json_deferred(mail.as_dict(), request)
-
-            def onError(error):
-                respond_json_deferred({'message': _format_exception(error)}, request, status_code=422)
-
-            sendDeferred.addCallback(onSuccess)
-            sendDeferred.addErrback(onError)
-
-            return server.NOT_DONE_YET
-        except Exception as error:
-            return respond_json({'message': _format_exception(error)}, request, status_code=422)
+        content_dict = json.loads(request.content.read())
+        _mail = InputMail.from_dict(content_dict)
+        draft_id = content_dict.get('ident')
+
+        self._mail_service.send(_mail)
+        sent_mail = self._mail_service.move_to_send(draft_id, _mail)
+        self._search_engine.index_mail(sent_mail)
+
+        if draft_id:
+            self._search_engine.remove_from_index(draft_id)
+
+        return respond_json(sent_mail.as_dict(), request)
 
     def render_PUT(self, request):
         content_dict = json.loads(request.content.read())
-- 
cgit v1.2.3