From 3e184540c77083a7c6f5d5c78d7d465542a5bcb6 Mon Sep 17 00:00:00 2001
From: David <david.vidal@tecnativa.com>
Date: Fri, 22 Dec 2017 19:23:40 +0100
Subject: [PATCH] [10.0][FIX] mail_tracking_mailgun: Avoid key dict errors

---
 mail_tracking_mailgun/__manifest__.py               | 2 +-
 mail_tracking_mailgun/models/mail_tracking_email.py | 6 ++++--
 mail_tracking_mailgun/models/res_partner.py         | 6 +++---
 3 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/mail_tracking_mailgun/__manifest__.py b/mail_tracking_mailgun/__manifest__.py
index 9fe931e0..f271654f 100644
--- a/mail_tracking_mailgun/__manifest__.py
+++ b/mail_tracking_mailgun/__manifest__.py
@@ -7,7 +7,7 @@
 {
     "name": "Mail tracking for Mailgun",
     "summary": "Mail tracking and Mailgun webhooks integration",
-    "version": "10.0.1.1.0",
+    "version": "10.0.1.1.1",
     "category": "Social Network",
     "website": "https://odoo-community.org/",
     "author": "Tecnativa, "
diff --git a/mail_tracking_mailgun/models/mail_tracking_email.py b/mail_tracking_mailgun/models/mail_tracking_email.py
index 2364762b..b017d4e9 100644
--- a/mail_tracking_mailgun/models/mail_tracking_email.py
+++ b/mail_tracking_mailgun/models/mail_tracking_email.py
@@ -9,7 +9,7 @@ import json
 import requests
 from datetime import datetime
 from odoo import _, api, fields, models
-from odoo.exceptions import ValidationError
+from odoo.exceptions import UserError, ValidationError
 
 import logging
 _logger = logging.getLogger(__name__)
@@ -221,6 +221,8 @@ class MailTrackingEmail(models.Model):
         """
         api_key, api_url, domain, validation_key = self._mailgun_values()
         for tracking in self:
+            if not tracking.mail_message_id:
+                raise UserError(_('There is no tracked message!'))
             message_id = tracking.mail_message_id.message_id.replace(
                 "<", "").replace(">", "")
             res = requests.get(
@@ -242,7 +244,7 @@ class MailTrackingEmail(models.Model):
                 if not self.env['mail.tracking.event'].search(
                         [('mailgun_id', '=', item["id"])]):
                     mapped_event_type = self._mailgun_event_type_mapping.get(
-                        item["event"]) or False
+                        item["event"], item["event"])
                     metadata = self._mailgun_metadata(
                         mapped_event_type, item, {})
                     tracking.event_create(mapped_event_type, metadata)
diff --git a/mail_tracking_mailgun/models/res_partner.py b/mail_tracking_mailgun/models/res_partner.py
index 9657647c..651b5cd7 100644
--- a/mail_tracking_mailgun/models/res_partner.py
+++ b/mail_tracking_mailgun/models/res_partner.py
@@ -8,8 +8,8 @@
 import requests
 import json
 
-from openerp import _, api, models
-from openerp.exceptions import UserError
+from odoo import _, api, models
+from odoo.exceptions import UserError
 
 
 class ResPartner(models.Model):
@@ -30,7 +30,7 @@ class ResPartner(models.Model):
             body = _('Email has been bounced: %s\n'
                      'Reason: %s\n'
                      'Event: %s') % (partner.email, reason,
-                                     event['Message-Id'] or '')
+                                     event.get('Message-Id', _('unknown')))
             partner.message_post(body=body)
 
     @api.multi