Juan Rial (OpenERP) has proposed merging lp:~openerp-dev/openerp-int/mck_addons-jri into lp:~openerp-dev/openerp-int/mck_addons.
Requested reviews: tfr (Openerp) (tfr) For more details, see: https://code.launchpad.net/~openerp-dev/openerp-int/mck_addons-jri/+merge/144908 * Added some access rights I seem to have forgotten to commit * Fixed bug where partner validation mails could not be sent anymore * Now only sends the non-tracked version of the mail if cc-nontracked is actually populated -- https://code.launchpad.net/~openerp-dev/openerp-int/mck_addons-jri/+merge/144908 Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openerp-int/mck_addons.
=== modified file 'mck_common/security/ir.model.access.csv' --- mck_common/security/ir.model.access.csv 2012-12-07 14:03:23 +0000 +++ mck_common/security/ir.model.access.csv 2013-01-25 12:12:29 +0000 @@ -8,4 +8,4 @@ access_mail_message_uber_admin,mail.message,mck_common.model_mail_message,mck_base_menu.uber_user,1,1,1,1 preview_email_id,Email_preview_rule,mck_common.model_email_preview,mck_base_menu.practice_user,1,1,1,1 track_log_base,track log base,mck_common.model_mck_base_tracklog,base.group_no_one,1,1,1,1 - +track_log_contact,track log Contact,base_contact.model_res_partner_contact,base.group_no_one,1,0,0,0 === modified file 'mck_common/wizard/send_mail.py' --- mck_common/wizard/send_mail.py 2012-12-06 11:12:19 +0000 +++ mck_common/wizard/send_mail.py 2013-01-25 12:12:29 +0000 @@ -172,9 +172,13 @@ headers = {} bodies = {} bodies_nontracked = {} - for invitee_id in mail.invitee_ids: - bodies_nontracked[invitee_id.id] = mail.body_html if mail.subtype == 'html' else mail.body_text - bodies[invitee_id.id] = mail.body_html if mail.subtype == 'html' else mail.body_text + if context.get('active_model') in ['mck.event', 'mck.publication']: + to_mail_ids = [x.id for x in mail.invitee_ids] + else: + to_mail_ids = context.get('active_ids') + for to_mail_id in to_mail_ids: + bodies_nontracked[to_mail_id] = mail.body_html if mail.subtype == 'html' else mail.body_text + bodies[to_mail_id] = mail.body_html if mail.subtype == 'html' else mail.body_text # Overwrite body if we have a template with a tracker URL if template is not None and 'tracker_url' in template and template['tracker_url']: if re.search('\?', template['tracker_url']): @@ -239,10 +243,11 @@ model=mail.model, email_cc=to_email(email_cc), email_cc_nontracked=to_email(email_cc_nontracked), email_bcc=to_email(email_bcc), reply_to=reply_to, attachments=attachment, references=references, res_id=active_id, include_recipients=['to', 'cc', 'bcc'], subtype=mail.subtype, headers=headers, cell_id=cell_id ,context=context) - mail_message.schedule_with_attach(cr, uid, email_from, to_email(email_to), subject, rendered_body_nontracked, - model=mail.model, email_cc=to_email(email_cc), email_cc_nontracked=to_email(email_cc_nontracked), email_bcc=to_email(email_bcc), reply_to=reply_to, - attachments=attachment, references=references, res_id=active_id, include_recipients=['cc-nontracked'], - subtype=mail.subtype, headers=headers, cell_id=cell_id ,context=context) + if email_cc_nontracked: + mail_message.schedule_with_attach(cr, uid, email_from, to_email(email_to), subject, rendered_body_nontracked, + model=mail.model, email_cc=to_email(email_cc), email_cc_nontracked=to_email(email_cc_nontracked), email_bcc=to_email(email_bcc), reply_to=reply_to, + attachments=attachment, references=references, res_id=active_id, include_recipients=['cc-nontracked'], + subtype=mail.subtype, headers=headers, cell_id=cell_id ,context=context) if template is not None and 'tracker_url' in template and template['tracker_url']: self.pool.get('mck.base.tracklog').generate_event_log(cr, uid, curr_date, 'mail_sent', '127.0.0.1', '', template_track_str, invitee_track_strings[active_id]) else: @@ -259,7 +264,8 @@ # in normal mode, we send the email immediately, as the user expects us to (delay should be sufficiently small) mail_message.send(cr, uid, [msg_id1], context=context) - mail_message.send(cr, uid, [msg_id2], context=context) + if email_cc_nontracked: + mail_message.send(cr, uid, [msg_id2], context=context) if template is not None and 'tracker_url' in template and template['tracker_url']: # TODO: TFR, please verify this! self.pool.get('mck.base.tracklog').generate_event_log(cr, uid, curr_date, 'mail_sent', '127.0.0.1', '', template_track_str, invitee_track_strings[active_ids[0]]) @@ -271,7 +277,7 @@ 'type': 'ir.actions.act_window', 'target': 'new', 'context' : {'old_active_model' : active_model } - } + } def preview_email(self, cr, uid, ids, context=None): context = context or {} === modified file 'mck_event/security/ir.model.access.csv' --- mck_event/security/ir.model.access.csv 2013-01-09 14:18:11 +0000 +++ mck_event/security/ir.model.access.csv 2013-01-25 12:12:29 +0000 @@ -11,6 +11,13 @@ event_mail_analysis_report_practice_user,mail_analysis,mck_event.model_mck_event_mail_analysis,mck_base_menu.practice_user,1,1,1,1 access_event_tracklog_practice_user,mail_analysis,mck_event.model_mck_event_tracklog,mck_base_menu.practice_user,1,0,0,0 access_event_tracklog_practice_admin,mail_analysis,mck_event.model_mck_event_tracklog,mck_base_menu.practice_admin,1,1,1,1 +<<<<<<< TREE track_log_event,track log Event,mck_event.model_mck_event_tracklog,base.group_no_one,1,1,1,1track_log_event,track log Event,mck_event.model_mck_event_tracklog,base.group_no_one,1,1,1,1 track_log_event_invite,tracklogger mck_invite,mck_event.model_mck_invitee,base.group_no_one,1,0,0,0 -track_log_event_stage,Tracklogger stage event,mck_event.model_event_stage,base.group_no_one,1,0,0,0 \ No newline at end of file +track_log_event_stage,Tracklogger stage event,mck_event.model_event_stage,base.group_no_one,1,0,0,0======= +track_log_event_tracklog,track log Event Tracklog,mck_event.model_mck_event_tracklog,base.group_no_one,1,1,1,1 +track_log_event,track log Event,mck_event.model_mck_event,base.group_no_one,1,0,0,0 +track_log_invitee,track log Invitee,mck_event.model_mck_invitee,base.group_no_one,1,1,0,0 +track_log_invitee_history,track log Invitee History,mck_event.model_history_invitee,base.group_no_one,1,1,1,0 +track_log_stage,track log Stage,mck_event.model_event_stage,base.group_no_one,1,0,0,0 +>>>>>>> MERGE-SOURCE === modified file 'mck_publication/security/ir.model.access.csv' --- mck_publication/security/ir.model.access.csv 2013-01-09 14:18:11 +0000 +++ mck_publication/security/ir.model.access.csv 2013-01-25 12:12:29 +0000 @@ -13,6 +13,14 @@ publication_mail_analysis_report_practice_user,mail_analysis,mck_publication.model_mck_publication_mail_analysis,mck_base_menu.practice_user,1,1,1,1 access_publication_tracklog_practice_user,mail_analysis,mck_publication.model_mck_publication_tracklog,mck_base_menu.practice_user,1,0,0,0 access_publication_tracklog_practice_admin,mail_analysis,mck_publication.model_mck_publication_tracklog,mck_base_menu.practice_admin,1,1,1,1 +<<<<<<< TREE track_log_pub,track log Pub,mck_publication.model_mck_publication_tracklog,base.group_no_one,1,1,1,1 track_log_recipient,Tracklogger recipient,mck_publication.model_mck_recipient,base.group_no_one,1,0,0,0 track_log_stage,Tracklogger stage recipient,mck_publication.model_publication_stage,base.group_no_one,1,0,0,0 +======= +track_log_publication_tracklog,track log Publication Tracklog,mck_publication.model_mck_publication_tracklog,base.group_no_one,1,1,1,1 +track_log_publication,track log Publication Tracklog,mck_publication.model_mck_publication,base.group_no_one,1,0,0,0 +track_log_recipient,track log Recipient,mck_publication.model_mck_recipient,base.group_no_one,1,1,0,0 +track_log_recipient_history,track log Recipient History,mck_publication.model_history_recipient,base.group_no_one,1,1,1,0 +track_log_stage,track log Stage,mck_publication.model_publication_stage,base.group_no_one,1,0,0,0 +>>>>>>> MERGE-SOURCE
_______________________________________________ Mailing list: https://launchpad.net/~openerp-dev-gtk Post to : openerp-dev-gtk@lists.launchpad.net Unsubscribe : https://launchpad.net/~openerp-dev-gtk More help : https://help.launchpad.net/ListHelp