From 6fd2f14e3b368359aa630d730fdb37829e3fa9ef Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Thibault=20Delavall=C3=A9e?= <tde@openerp.com>
Date: Thu, 20 Dec 2012 13:17:52 +0100
Subject: [PATCH] [CLEAN] Small cleanup of code.

bzr revid: tde@openerp.com-20121220121752-31a5ulkwg1k7v961
---
 addons/mail/mail_message.py         |  8 +-------
 addons/mail/mail_message_subtype.py | 11 +++++++----
 2 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/addons/mail/mail_message.py b/addons/mail/mail_message.py
index 7ff6e749ad8..d545f3782b4 100644
--- a/addons/mail/mail_message.py
+++ b/addons/mail/mail_message.py
@@ -350,15 +350,9 @@ class mail_message(osv.Model):
         vote_nb = len(message.vote_user_ids)
         has_voted = uid in [user.id for user in message.vote_user_ids]
 
-        # TDE tmp hack
-        if not message.body:
-            body = ''
-        else:
-            body = html_email_clean(message.body or '')
-
         return {'id': message.id,
                 'type': message.type,
-                'body': body,
+                'body': html_email_clean(message.body or ''),
                 'model': message.model,
                 'res_id': message.res_id,
                 'record_name': message.record_name,
diff --git a/addons/mail/mail_message_subtype.py b/addons/mail/mail_message_subtype.py
index f4383ff9957..bef6e427c3b 100644
--- a/addons/mail/mail_message_subtype.py
+++ b/addons/mail/mail_message_subtype.py
@@ -38,13 +38,16 @@ class mail_message_subtype(osv.osv):
                     'precisely tune the notifications the user want to receive on its wall.'),
         'description': fields.text('Description', translate=True,
             help='Description that will be added in the message posted for this '\
-                    'subtype. If void, no message will be added.'),
+                    'subtype. If void, the name will be added instead.'),
         'parent_id': fields.many2one('mail.message.subtype', string='Parent',
-            ondelete='set null'),
+            ondelete='set null',
+            help='Parent subtype, used for automatic subscription.'),
         'relation_field': fields.char('Relation field',
-            help='Field used to link the related model to the subtype model'),
+            help='Field used to link the related model to the subtype model when '\
+                    'using automatic subscription on a related document. The field '\
+                    'is used to compute getattr(related_document.relation_field).'),
         'res_model': fields.char('Model',
-            help="Model the subtype applies to. If False, this subtype exists for all models."),
+            help="Model the subtype applies to. If False, this subtype applies to all models."),
         'default': fields.boolean('Default',
             help="Activated by default when subscribing."),
     }