Commit Graph

120 Commits

Author SHA1 Message Date
Christophe Simonis 504a04c823 [MERGE] forward port of branch 7.0 up to 117b636 2015-09-25 14:56:58 +02:00
Christophe Simonis 117b636d3f [FIX] mail: catch database errors when sending mails 2015-09-25 14:43:00 +02:00
Denis Ledoux ecd8f63d75 [MERGE] forward port of branch 7.0 up to 60a361c 2014-12-15 17:34:21 +01:00
Christophe Combelles e38974c5b6 [FIX] mail: access link generaTION
Catch only the error related to the access of the linked record.
Previous was hidding error to access the ir.config_parameter and eventual other
errors from the orm.
Uses SUPERUSER_ID to access instance URL (e.g. portal has no access to it).
Fixes #4234 #4234
2014-12-15 14:08:14 +01:00
Denis Ledoux f7a76cbb17 [MERGE] forward port of branch 7.0 up to be7c894 2014-11-04 17:54:48 +01:00
Manuel Vázquez Acosta 2fb2d9f956 [FIX] mail: partners with missing/invalid emails must not halt notifications to others 2014-11-04 17:32:32 +01:00
Denis Ledoux 2c9a116c74 [MERGE] forward port of branch 7.0 up to 245bb4e 2014-08-12 14:16:35 +02:00
Denis Ledoux 245bb4ebdf [FIX] mail: to/from/reply-to addresses formated according to RFC2822
Use formataddr method from email.utils lib, which do the job correctly
2014-08-12 13:40:45 +02:00
Denis Ledoux 4f1dbde8d5 [FIX] mail: attachments present for first recipient, but not for the others
attachments should be an array and not a generator [x for x in y] instead of (x for x in y)

bzr revid: dle@openerp.com-20140505123852-kjc6z0npiot9c54l
2014-05-05 14:38:52 +02:00
Olivier Dony 61d2122987 [FIX] mail.mail: avoid prefetching attachment data in memory while processing mail queue, could fail due to memory limit
Specifically when there are many emails in the queue, prefetching
all their attachments at once when processing the first one
could fail, so not even one mail would be sent.

Added explicit logging when an email triggers a MemoryError,
as well as logging when each email is successfully sent.

bzr revid: odo@openerp.com-20140502144431-r3brgagl4gel4wmt
2014-05-02 16:44:31 +02:00
Olivier Dony c3e812f34a [MERGE] Forward-port of latest 7.0 bugfixes, up to rev. 10025 rev-id odo@openerp.com-20140430102552-qkwv20b20nve64th
bzr revid: odo@openerp.com-20140430110048-6971xplq8k073tto
2014-04-30 13:00:48 +02:00
Olivier Dony c43fdfe32b [FIX] mail: avoid catching (transient) MemoryErrors when sending outgoing mail
This would permanently mark the mail as failed while
there is a good chance it will be sent properly next
time it is retried (provided the memory situation
improves or the server/worker has restarted).

bzr revid: odo@openerp.com-20140430102552-qkwv20b20nve64th
2014-04-30 12:25:52 +02:00
Thibault Delavallée 11e0e64769 [FIX] mail: notification url: only web, not login
bzr revid: tde@openerp.com-20140226170002-ndr090idip3cyx3n
2014-02-26 18:00:02 +01:00
Thibault Delavallée bb795e929d [FIX] mail: fixed sign-in URL in notification emails, use the new web/login controller introduced in saas-3.
bzr revid: tde@openerp.com-20140225095054-kswinebsgsj5fex1
2014-02-25 10:50:54 +01:00
Thibault Delavallée b92ad39005 [FIX] email_template: support of email_recipients in send_mail.
email_recipients is a char fields holding a comma separated list of partner_ids
that are the email recipients. This field is correctly handled in the composer
but was not taken into account in the send_mail method email_template.
However due to model limitations, this field can only be taken into account
when sending the email directly using force_send. When the email is
queued, no field is available to hold the value of email_recipients.
This is already fixed in trunk/v8 as a field has been added to handle
this value.

[FIX] mail: when sending email to some recipients_ids, filter to have only
the existing partner_ids. Trusting email template to generate valid
partner_ids is a bit risky.

[TESTS] email_template: added tests for send_mail

bzr revid: tde@openerp.com-20131220144652-h18yam60vpedbh7x
2013-12-20 15:46:52 +01:00
jke-openerp 306dd6e397 [IMP] Add span with class for footer received in mail
[FIX] convert partner_to  from template_mail (string with comma) to recipent_ids (list of partner_id) in model mail_mail

bzr revid: jke@openerp.com-20131219133144-zbt100e8030wd927
2013-12-19 14:31:44 +01:00
jke-openerp cd4f71cf36 [REF] rb r2g
bzr revid: jke@openerp.com-20131124130241-s1hjdl08y5dcsnxs
2013-11-24 14:02:41 +01:00
jke-openerp 5f9ee3b2d2 [IMP] Bck not running
bzr revid: jke@openerp.com-20131123113627-csxj0qx8s030dhi7
2013-11-23 12:36:27 +01:00
Thibault Delavallée 62e88a16e4 [FIX] mail: mail_mail: fixed url embedded in notification emails.
When having emails not linked to a notification (mail.notification = False)
the link is generated without any clue about the source record.
However we can try to use model and res_id defined on the mail_mail record
to correctly format a link.

Added support of model and res_id in message_redirect_action. This method
now can redirect based on a message_id, or based on model / res_id.

bzr revid: tde@openerp.com-20131018144924-j4f22hen3lsua7cb
2013-10-18 16:49:24 +02:00
Thibault Delavallée ed62d1dac7 [REF] mass_mailing: first refactor
Mail statistics are now stored onto a separated object (mail.mail.statistics), allowing to
handle emails separately from statistics (among other removing mail.mail entries while keeping
statistics).

Everything linnked to opened/replied/bounce is not managed by mass_mailing, removed added code
in mail module.

bzr revid: tde@openerp.com-20130913115408-322cyjipdg680as6
2013-09-13 13:54:08 +02:00
Thibault Delavallée ce8c987089 [IMP] mail, email_template, mass_mailing: improvev mass mailing management in mail.compose.message + form view cleanings + some fixes in mail_mail about set_opened/replied/bounced, now checking that ids effectively exist
bzr revid: tde@openerp.com-20130910151901-nhuwvtrkbwekyh7x
2013-09-10 17:19:01 +02:00
Thibault Delavallée 903ca548c7 [IMP] mass_mailing: added kanban views for mass_mailing, implemented segment for campaigns. Moved sparkline and gauge widgets to web.
Segment: template, date, mails.
Campaign: grouped statistics.

bzr revid: tde@openerp.com-20130903145936-nwy250w1suxmtbl3
2013-09-03 16:59:36 +02:00
Thibault Delavallée b35a44f66d [IMP] mail_mail, mail_message: various improvements to try to improve message creation time
Replaced some read by browse
Moved get_reply_to from mail_mail to mail_message
Hint: specifying email_from, reply_to help to enhance computation time

[REF] mail: cleaned some tests, renamed a file, moved mail_group tests into a dedicated file

bzr revid: tde@openerp.com-20130827133058-ko0g0ib0f0jihmdk
2013-08-27 15:30:58 +02:00
Thibault Delavallée 982915b7d0 [MERGE] Sync with trunk
bzr revid: tde@openerp.com-20130807145147-ltn86n2a87q8y88g
2013-08-07 16:51:47 +02:00
Thibault Delavallée 2eece10cd4 [FIX] mail_mail: computation of reply_to after create to avoid
interfering with automatic message_id creation of mail_message
[FIX] mail_thread: fixed issues with private messages through mailgateway
(wrong route checking, variable erasing)
[TEST] mail: added test for the second bug

bzr revid: tde@openerp.com-20130807142418-3h5qxdt3ekosj9x6
2013-08-07 16:24:18 +02:00
Thibault Delavallée fd100054cc [FIX] mail_mail: fixed reply_to computation (before create was leading to a wrong message_id to the embedded mail_message); mail_thread; fixed private discussion not going through the route checking; improved replying through mailgateway now incrementing the replied field of mail_mail.
bzr revid: tde@openerp.com-20130807130118-yggvmontssofxt0q
2013-08-07 15:01:18 +02:00
Thibault Delavallée 38a534dee0 [IMP] mail: first implementation of tracking and bounce management.
Added 2 fields on mail_mail to count read/bounce.
Added bounce alias bounce-mail_id-model-res_id.
Added message_receive_bounce method that try to incremetn message_bounce field.

bzr revid: tde@openerp.com-20130806151143-7dw6xlj8n7mh0nqe
2013-08-06 17:11:43 +02:00
Thibault Delavallée a3a0d3dbbf [FIX] mail: res was not created if there are no email in email_list when sending emails
bzr revid: tde@openerp.com-20130726101008-whwcl1ssh2fsxhbw
2013-07-26 12:10:08 +02:00
Thibault Delavallée f1f2b2ba06 [IMP] mail: mail.message now has mail_server_id from template_id.
bzr revid: tde@openerp.com-20130722131725-i30xd4b03nui3hyw
2013-07-22 15:17:25 +02:00
Martin Trigaux d2d72d8416 [MERGE] forward port of saas-1 up to revision 8749 (mat@openerp.com-20130702135856-kmg7d8anw17nkbd8)
bzr revid: mat@openerp.com-20130702144646-y1wqs5mz4rzoh50g
2013-07-02 16:46:46 +02:00
Denis Ledoux 6b45fe6ae9 [FIX]mail: _postprocess_sent_message send all mail, not only type = email
bzr revid: dle@openerp.com-20130614132418-hfhf61vfmq28zmv0
2013-06-14 15:24:18 +02:00
Christophe Simonis 201a383c9c [FIX] mail: reply_to is set to email_from as a last resort
bzr revid: chs@openerp.com-20130614103005-9x5r0b22b25jll1g
2013-06-14 12:30:05 +02:00
Christophe Simonis 1ae74e8907 [FIX] mail: default value for email_reply_to variable (merge messup)
bzr revid: chs@openerp.com-20130614094439-iqji8aimvaf4x5t6
2013-06-14 11:44:39 +02:00
Christophe Simonis aa133b4d29 [MERGE] forward port of branch saas-1 up to revid 8745 chs@openerp.com-20130613181503-82234mo34wxb0oap
bzr revid: chs@openerp.com-20130614091924-z5ta02kwhmwcrgox
2013-06-14 11:19:24 +02:00
Thibault Delavallée a1a6698558 [CLEAN] mail: cleaned reply_to code, finally.
bzr revid: tde@openerp.com-20130607091611-7cpzlxzngkrs86a0
2013-06-07 11:16:11 +02:00
Thibault Delavallée 216cb2122e [IMP] mail: mass_mailing: avoid having 'Followers of' when having a specified reply_to coming from the mail_message
bzr revid: tde@openerp.com-20130606152814-iqftppk06s7g8weg
2013-06-06 17:28:14 +02:00
Thibault Delavallée 30a7720452 [IMP] mail_message: added reply_to to the form view of mail.message
bzr revid: tde@openerp.com-20130606152538-4u3s1k4vvkotxz6c
2013-06-06 17:25:38 +02:00
Thibault Delavallée 1ef65f1577 [FIX] mail_mail: reply_to: use tools.email_split to avoid issues when handling email addresses.
bzr revid: tde@openerp.com-20130606151440-atdhk834ny248xqq
2013-06-06 17:14:40 +02:00
Thibault Delavallée a1e942c0de [FIX] mail.compose.message: fixed reply_to from template in mass mailing.
bzr revid: tde@openerp.com-20130606142814-jyw76nhaeivotu6k
2013-06-06 16:28:14 +02:00
Thibault Delavallée 9c57f6e1cd [FIX] mail: various fixes
Mass mailing: emails due to mass mailing are not queued instead of sent immediately.
As most of cases go through mail_followers._notify(), a new argument is added that
allows to control the force_send (like what is done in email_template).
Chatter: better display of recipients when having a message_post due to mass mailing
(not always displaying logged a note).
Notified people: when having no subtype, just avoid notifying the followers but
take into account specified recipients (partner_ids).

bzr revid: tde@openerp.com-20130606102437-43s4xp96b5wa3r1q
2013-06-06 12:24:37 +02:00
Martin Trigaux ccb38117ee [MERGE] Forward porting of saas-1 until rev 8739
bzr revid: mat@openerp.com-20130605090453-e4mg37kms5j1r18n
2013-06-05 11:04:53 +02:00
Thibault Delavallée 27839fead3 [CLEAN] auth_signup, mail_mail: better management of exceptions when sending emails, delegated to mail_mail.
bzr revid: tde@openerp.com-20130604151508-7bd8gxhkr86fuybp
2013-06-04 17:15:08 +02:00
Thibault Delavallée 8b42f9bb1f [FIX] mail_mail: removed column reply_to, as this column already exists
in mail_message. As mail_mail inherits from mail_message, there is no need
to duplicate it. Removing the column should be safe, because reading will
fetch the value from mail_message even if the mail_mail column has been
deleted.

bzr revid: tde@openerp.com-20130529144441-giku7wbwrc3kg5qk
2013-05-29 16:44:41 +02:00
Thibault Delavallée c93c8935ad [IMP] mail: now may raise exception instead of just logging it. They are used in auth_signup to avoid having the process considering the invite/reset password email was sent when it was not.
bzr revid: tde@openerp.com-20130524114857-8wy5uob23i2d6b6p
2013-05-24 13:48:57 +02:00
Thibault Delavallée cd278d83ab [MERGE] Sync with trunk
bzr revid: tde@openerp.com-20130522103305-yi5ql1chdx9qxxlx
bzr revid: tde@openerp.com-20130522114236-qi2s3i2dl2lemdvh
2013-05-22 13:42:36 +02:00
Thibault Delavallée faefb9fad1 [MERGE] Sync with trunk
bzr revid: tde@openerp.com-20130522103426-wjn0ttqdbf72g713
2013-05-22 12:34:26 +02:00
Thibault Delavallée a05b0bebbe [FORWARD] Forward port of 7.0 branch until revision 9143.
bzr revid: tde@openerp.com-20130521122359-b5vii7gv8arluz06
2013-05-21 14:23:59 +02:00
Thibault Delavallée bc4964de13 [IMP] mail: now defines a catchall email alias, used to format the
reply_to of notification emails when having a document without alias, or for
private discussions using external mail readers.

bzr revid: tde@openerp.com-20130516150900-khb1d1ygyuos1wrl
2013-05-16 17:09:00 +02:00
Thibault Delavallée 8a0adc0f33 [FIX] mail: notification email: notification sent for private discussions now have a correct subject, aka the subject of the original email.
bzr revid: tde@openerp.com-20130515133332-g3wtttsnasfz0y8b
2013-05-15 15:33:32 +02:00
Thibault Delavallée c08b877370 [FIX] mail: fixed redirection action that was not redirecting to the discussion when failing to open the document.
bzr revid: tde@openerp.com-20130429093329-w0qxiiipx1mkz6qt
2013-04-29 11:33:29 +02:00