Merge lp:~openerp-dev/openobject-addons/trunk-bug-1212109-jar into lp:openobject-addons

Proposed by Jaydeep Barot(OpenERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openobject-addons/trunk-bug-1212109-jar
Merge into: lp:openobject-addons
Diff against target: 14 lines (+3/-1)
1 file modified
mail/static/src/js/mail.js (+3/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/trunk-bug-1212109-jar
Reviewer Review Type Date Requested Status
Atul Patel(OpenERP) (community) Approve
OpenERP Core Team Pending
Review via email: mp+184531@code.launchpad.net

Description of the change

Hello,

[trunk]
     --> I have fix this problem.able to reply message from inbox.

Thanks,
Jaydeep Barot.

To post a comment you must log in.
Revision history for this message
Atul Patel(OpenERP) (atp-openerp) wrote :

Hello,

we donot get child record for first enter inbox message.
that's why need to check parent_message.
seems ok

Thanks

review: Approve
8889. By Mehul Mehta(OpenERP)

[Merge] with main main addons

Unmerged revisions

8889. By Mehul Mehta(OpenERP)

[Merge] with main main addons

8888. By Jaydeep Barot(OpenERP)

[FIX] add condition on parent_message in mail.js

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'mail/static/src/js/mail.js'
2--- mail/static/src/js/mail.js 2013-10-09 18:03:37 +0000
3+++ mail/static/src/js/mail.js 2013-10-29 07:31:00 +0000
4@@ -834,7 +834,9 @@
5 // go to the parented message
6 var message = this.parent_thread.parent_message;
7 var parent_message = message.parent_id ? message.parent_thread.parent_message : message;
8- var messages = [parent_message].concat(parent_message.get_childs());
9+ if(parent_message){
10+ var messages = [parent_message].concat(parent_message.get_childs());
11+ }
12 } else if (this.options.emails_from_on_composer) {
13 // get all wall messages if is not a mail.Wall
14 _.each(this.options.root_thread.messages, function (msg) {messages.push(msg); messages.concat(msg.get_childs());});

Subscribers

People subscribed via source and target branches

to all changes: