Skip to content

Commit

Permalink
[FIX] mail_tracking js could break other modules using chat_manager
Browse files Browse the repository at this point in the history
  • Loading branch information
ecino authored and yajo committed May 7, 2018
1 parent 4173794 commit 150d0d0
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 10 deletions.
2 changes: 1 addition & 1 deletion mail_tracking/__manifest__.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
{
"name": "Email tracking",
"summary": "Email tracking system for all mails sent",
"version": "10.0.1.1.0",
"version": "10.0.1.1.1",
"category": "Social Network",
"website": "http://www.tecnativa.com",
"author": "Tecnativa, "
Expand Down
14 changes: 5 additions & 9 deletions mail_tracking/static/src/js/mail_tracking.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,22 +8,18 @@ var core = require('web.core');
var session = require('web.session');
var Model = require('web.Model');
var ActionManager = require('web.ActionManager');
var chat_manager = require('mail.chat_manager');
var ChatThread = require('mail.ChatThread');
var Chatter = require('mail.Chatter');

var _t = core._t;
var MessageModel = new Model('mail.message', session.context);

// chat_manager is a simple dictionary, not an OdooClass
chat_manager._make_message_super = chat_manager.make_message;
chat_manager.make_message = function(data) {
var msg = this._make_message_super(data);
msg.partner_trackings = data.partner_trackings || [];
return msg;
};

ChatThread.include({
_preprocess_message: function (message) {
var msg = this._super.apply(this, arguments);
msg.partner_trackings = msg.partner_trackings || [];
return msg;
},
on_tracking_partner_click: function (event) {
var partner_id = this.$el.find(event.currentTarget).data('partner');
var state = {
Expand Down

0 comments on commit 150d0d0

Please sign in to comment.