[MERGE] Revisions 8997 ... 8999 of 7.0 branch.

bzr revid: tde@openerp.com-20130426132721-02lkoc41fms9ve1k
2013-04-26 15:27:21 +02:00
parent 71b245598e a75cc23573
commit 0dada27fcd
8 changed files with 116 additions and 23 deletions
@@ -28,7 +28,6 @@ from urlparse import urljoin
 from openerp import tools
 from openerp import SUPERUSER_ID
 from openerp.osv import fields, osv
-from openerp.osv.orm import except_orm
 from openerp.tools.translate import _

 _logger = logging.getLogger(__name__)
@@ -195,6 +194,31 @@ class mail_mail(osv.Model):
            self.unlink(cr, SUPERUSER_ID, [mail.id], context=context)
        return True

+    #------------------------------------------------------
+    # mail_mail formatting, tools and send mechanism
+    #------------------------------------------------------
+
+    def _get_partner_access_link(self, cr, uid, mail, partner=None, context=None):
+        """ Generate URLs for links in mails:
+            - partner is an user and has read access to the document: direct link to document with model, res_id
+        """
+        if partner and partner.user_ids:
+            base_url = self.pool.get('ir.config_parameter').get_param(cr, uid, 'web.base.url')
+            # the parameters to encode for the query and fragment part of url
+            query = {'db': cr.dbname}
+            fragment = {
+                'login': partner.user_ids[0].login,
+                'action': 'mail.action_mail_redirect',
+            }
+            if mail.notification:
+                fragment.update({
+                        'message_id': mail.mail_message_id.id,
+                    })
+            url = urljoin(base_url, "?%s#%s" % (urlencode(query), urlencode(fragment)))
+            return _("""<small>Access this document <a style='color:inherit' href="%s">directly in OpenERP</a></small>""") % url
+        else:
+            return None
+
    def send_get_mail_subject(self, cr, uid, mail, force=False, partner=None, context=None):
        """ If subject is void and record_name defined: '<Author> posted on <Resource>'

@@ -244,6 +268,11 @@ class mail_mail(osv.Model):
        # add footer
        body_footer = self.send_get_mail_body_footer(cr, uid, mail, partner=partner, context=context)
        body = tools.append_content_to_html(body, body_footer, plaintext=False, container_tag='div')
+
+        # generate footer
+        link = self._get_partner_access_link(cr, uid, mail, partner, context=context)
+        if link:
+            body = tools.append_content_to_html(body, link, plaintext=False, container_tag='div')
        return body

    def send_get_email_dict(self, cr, uid, mail, partner=None, context=None):
@@ -59,6 +59,9 @@
                    <field name="type"/>
                    <field name="author_id"/>
                    <field name="partner_ids"/>
+                    <field name="model"/>
+                    <field name="res_id"/>
+                    <field name="parent_id"/>
                    <filter string="To Read"
                            name="message_unread" help="Show messages to read"
                            domain="[('to_read', '=', True)]"/>
@@ -449,6 +449,55 @@ class mail_thread(osv.AbstractModel):
        ir_attachment_obj.unlink(cr, uid, attach_ids, context=context)
        return True

+    def message_redirect_get_inbox_action_xml_id(self, cr, uid, context=None):
+        """ When redirecting towards the Inbox, choose which action xml_id has
+            to be fetched. This method is meant to be inherited, at least in portal
+            because portal users have a different Inbox action than classic users. """
+        return ('mail', 'action_mail_inbox_feeds')
+
+    def message_redirect_action(self, cr, uid, context=None):
+        """ For a given message, return an action that either
+            - opens the form view of the related document if model, res_id, and
+              read access to the document
+            - opens the Inbox with a default search on the conversation if model,
+              res_id
+            - opens the Inbox with context propagated
+
+        """
+        if context is None:
+            context = {}
+
+        # default action is the Inbox action
+        self.pool.get('res.users').browse(cr, SUPERUSER_ID, uid, context=context)
+        act_model, act_id = self.pool.get('ir.model.data').get_object_reference(cr, uid, *self.message_redirect_get_inbox_action_xml_id(cr, uid, context=context))
+        action = self.pool.get(act_model).read(cr, uid, act_id, [])
+
+        # if msg_id specified: try to redirect to the document or fallback on the Inbox
+        msg_id = context.get('params', {}).get('message_id')
+        if not msg_id:
+            return action
+        msg = self.pool.get('mail.message').browse(cr, uid, msg_id, context=context)
+        if msg.model and msg.res_id and self.pool.get(msg.model).check_access_rights(cr, uid, 'read', raise_exception=False):
+            try:
+                self.pool.get(msg.model).check_access_rule(cr, uid, [msg.res_id], 'read', context=context)
+                action = {
+                    'type': 'ir.actions.act_window',
+                    'res_model': msg.model,
+                    'view_type': 'form',
+                    'view_mode': 'form',
+                    'views': [(msg.res_id, 'form')],
+                    'target': 'current',
+                    'res_id': msg.res_id,
+                }
+            except osv.except_osv:
+                action.update({
+                    'context': {
+                        'search_default_model': msg.model,
+                        'search_default_res_id': msg.res_id,
+                    }
+                })
+        return action
+
    #------------------------------------------------------
    # Email specific
    #------------------------------------------------------
@@ -108,6 +108,15 @@
            </field>
        </record>

+        <record id="action_mail_redirect" model="ir.actions.server">
+            <field name="name">Mail Redirection (Document / Inbox)</field>
+            <field name="condition">True</field>
+            <field name="model_id" ref="model_mail_thread"/>
+            <field name="code">action = pool.get('mail.thread').message_redirect_action(cr, uid, context)</field>
+            <field name="state">code</field>
+            <field name="type">ir.actions.server</field>
+        </record>
+
        <!-- MENU -->

        <!-- Top menu item -->
@@ -1841,10 +1841,19 @@ openerp.mail = function (session) {
            this._super(parent, action);

            this.action = _.clone(action);
+            // debugger
            this.domain = this.action.params.domain || this.action.domain || [];
            this.context = _.extend(this.action.params.context || {}, this.action.context || {});

+            // filter some parameters that we will propagate as search_default
            this.defaults = {};
+            for (var key in this.action.context.params) {
+                if (_.indexOf(['model', 'res_id'], key) == -1) {
+                    continue;
+                }
+                this.context['search_default_' + key] = this.action.context.params[key];
+                console.log(this.context);
+            }
            for (var key in this.context) {
                if (key.match(/^search_default_/)) {
                    this.defaults[key.replace(/^search_default_/, '')] = this.context[key];
@@ -20,6 +20,7 @@
 ##############################################################################

 from openerp.addons.mail.tests.test_mail_base import TestMailBase
+from openerp.tools import mute_logger

 MAIL_TEMPLATE = """Return-Path: <whatever-2a840@postmaster.twitter.com>
 To: {to}
@@ -123,6 +124,7 @@ class TestMailgateway(TestMailBase):
        self.assertEqual(partner_info['partner_id'], p_b_id,
                        'mail_thread: message_find_partner_from_emails wrong partner found')

+    @mute_logger('openerp.addons.mail.mail_thread', 'openerp.osv.orm')
    def test_10_message_process(self):
        """ Testing incoming emails processing. """
        cr, uid, user_raoul = self.cr, self.uid, self.user_raoul
@@ -365,6 +367,7 @@ class TestMailgateway(TestMailBase):
        self.assertEqual(msg.body, '<pre>\nPlease call me as soon as possible this afternoon!\n\n--\nSylvie\n</pre>',
                            'message_process: plaintext incoming email incorrectly parsed')

+    @mute_logger('openerp.addons.mail.mail_thread', 'openerp.osv.orm')
    def test_20_thread_parent_resolution(self):
        """ Testing parent/child relationships are correctly established when processing incoming mails """
        cr, uid = self.cr, self.uid
@@ -20,6 +20,7 @@
 ##############################################################################

 import portal
+import mail_thread
 import mail_mail
 import wizard
 import acquirer
@@ -21,7 +21,6 @@

 from openerp import SUPERUSER_ID
 from openerp.osv import osv
-from openerp.osv.orm import except_orm
 from openerp.tools.translate import _


@@ -29,27 +28,18 @@ class mail_mail(osv.Model):
    """ Update of mail_mail class, to add the signin URL to notifications. """
    _inherit = 'mail.mail'

-    def send_get_mail_body_footer(self, cr, uid, mail, partner=None, context=None):
-        """ add a signin link inside the body of a mail.mail
-            :param mail: mail.mail browse_record
-            :param partner: browse_record of the specific recipient partner
-            :return: the resulting body_html
+    def _get_partner_access_link(self, cr, uid, mail, partner=None, context=None):
+        """ Generate URLs for links in mails:
+            - partner is not an user: signup_url
+            - partner is an user: fallback on classic URL
        """
-        partner_obj = self.pool.get('res.partner')
-        if partner:
+        if partner and not partner.user_ids:
            contex_signup = dict(context or {}, signup_valid=True)
-            partner = partner_obj.browse(cr, SUPERUSER_ID, partner.id, context=contex_signup)
-            body_footer = _("""<small>Access your messages and documents through <a style='color:inherit' href="%s">our Customer Portal</a></small>""") % partner.signup_url
-            # partner is an user: add a link to the document if read access
-            if partner.user_ids and mail.model and mail.res_id \
-                    and self.check_access_rights(cr, partner.user_ids[0].id, 'read', raise_exception=False):
-                related_user = partner.user_ids[0]
-                try:
-                    self.pool[mail.model].check_access_rule(cr, related_user.id, [mail.res_id], 'read', context=context)
-                    url = partner_obj._get_signup_url_for_action(cr, related_user.id, [partner.id], action='', res_id=mail.res_id, model=mail.model, context=context)[partner.id]
-                    text = _("""<small>Access this document <a style='color:inherit' href="%s">directly in OpenERP</a></small>""") % url
-                except except_orm, e:
-                    pass
-            return body_footer
+            partner = self.pool.get('res.partner').browse(cr, SUPERUSER_ID, partner.id, context=contex_signup)
+            return _("""<small>Access your messages and documents through <a style='color:inherit' href="%s">our Customer Portal</a></small>""") % partner.signup_url
        else:
-            return super(mail_mail, self).send_get_mail_body_footer(cr, uid, mail, partner=partner, context=context)
+            return super(mail_mail, self)._get_partner_access_link(cr, uid, mail, partner=partner, context=context)
+
+    def send_get_mail_body(self, cr, uid, mail, partner=None, context=None):
+        """ TODO: remove me in 8.0 """
+        return super(mail_mail, self).send_get_mail_body(cr, uid, mail, partner=partner, context=context)