diff options
| author | Patrick Maia <patrickjourdanmaia@gmail.com> | 2014-10-13 21:15:01 +0000 |
|---|---|---|
| committer | Patrick Maia <patrickjourdanmaia@gmail.com> | 2014-10-13 21:33:14 +0000 |
| commit | 69dccf3fd6979d797d49c9a158725cb654c4ec2d (patch) | |
| tree | 3f629bcfb94137560620a3ef56cfd45675587068 /service/pixelated | |
| parent | 21eee8ccf485b8aca8d5677ba0f0de97fc0afc22 (diff) | |
#89 - separates manage draft and send mail responsibilities
Diffstat (limited to 'service/pixelated')
| -rw-r--r-- | service/pixelated/adapter/mail_service.py | 3 | ||||
| -rw-r--r-- | service/pixelated/user_agent.py | 18 |
2 files changed, 12 insertions, 9 deletions
diff --git a/service/pixelated/adapter/mail_service.py b/service/pixelated/adapter/mail_service.py index a6241821..d26c7c8f 100644 --- a/service/pixelated/adapter/mail_service.py +++ b/service/pixelated/adapter/mail_service.py @@ -45,7 +45,8 @@ class MailService: def send(self, last_draft_ident, mail): self.mail_sender.sendmail(mail) - self.mailboxes.drafts().remove(last_draft_ident) + if last_draft_ident: + self.mailboxes.drafts().remove(last_draft_ident) return self.mailboxes.sent().add(mail) def thread(self, thread_id): diff --git a/service/pixelated/user_agent.py b/service/pixelated/user_agent.py index 04be0b90..da8b69a0 100644 --- a/service/pixelated/user_agent.py +++ b/service/pixelated/user_agent.py @@ -72,12 +72,10 @@ def send_mail(): _mail = InputMail.from_dict(request.json) draft_id = request.json.get('ident') if draft_id: - _mail = mail_service.send(draft_id, _mail) - search_engine.index_mail(_mail) search_engine.remove_from_index(draft_id) - else: - _mail = draft_service.create_draft(_mail) - search_engine.index_mail(mail_service.mail(_mail.ident)) + _mail = mail_service.send(draft_id, _mail) + search_engine.index_mail(_mail) + return respond_json(_mail.as_dict()) except Exception as error: return respond_json({'message': '\n'.join(list(error.args))}, status_code=500) @@ -86,10 +84,14 @@ def send_mail(): @app.route('/mails', methods=['PUT']) def update_draft(): _mail = InputMail.from_dict(request.json) - new_revision = draft_service.update_draft(request.json['ident'], _mail) - ident = new_revision.ident + draft_id = request.json.get('ident') + if draft_id: + ident = draft_service.update_draft(draft_id, _mail).ident + search_engine.remove_from_index(draft_id) + else: + ident = draft_service.create_draft(_mail).ident + search_engine.index_mail(mail_service.mail(ident)) - search_engine.remove_from_index(request.json['ident']) return respond_json({'ident': ident}) |