Merge lp:~therp-nl/ocb-addons/6.1_lp1044359 into lp:ocb-addons/6.1

Proposed by Holger Brunn (Therp)
Status: Merged
Merged at revision: 6735
Proposed branch: lp:~therp-nl/ocb-addons/6.1_lp1044359
Merge into: lp:ocb-addons/6.1
Diff against target: 19 lines (+8/-1)
1 file modified
mail/mail_message.py (+8/-1)
To merge this branch: bzr merge lp:~therp-nl/ocb-addons/6.1_lp1044359
Reviewer Review Type Date Requested Status
Stefan Rijnhart (Opener) Approve
Review via email: mp+162938@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'mail/mail_message.py'
2--- mail/mail_message.py 2012-09-24 14:02:33 +0000
3+++ mail/mail_message.py 2013-05-08 08:06:26 +0000
4@@ -534,7 +534,14 @@
5 mail_server_id=message.mail_server_id.id,
6 context=context)
7 if res:
8- message.write({'state':'sent', 'message_id': res})
9+ message.write(
10+ {
11+ 'state':'sent',
12+ 'message_id': res,
13+ 'user_id': message.user_id.id or uid,
14+ 'date': message.date
15+ or datetime.datetime.now(),
16+ })
17 message_sent = True
18 else:
19 message.write({'state':'exception'})

Subscribers

People subscribed via source and target branches