Jigar Amin  (OpenERP) has proposed merging 
lp:~openerp-dev/openobject-addons/trunk-bug-832056-jam into 
lp:openobject-addons.

Requested reviews:
  Bhumika (OpenERP) (sbh-openerp)
Related bugs:
  Bug #832056 in OpenERP Addons: "[6.0] mail_gateway: crm_claim.message_new 
fails when bad 'priority' value is given from mail_gateway"
  https://bugs.launchpad.net/openobject-addons/+bug/832056

For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-832056-jam/+merge/72995

Changes : Bug Fix
 - Bug #832056:[6.0] mail_gateway: crm_claim.message_new fails when bad 
'priority' value is given from mail_gateway

Kindly Review This
Thank You
-- 
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-832056-jam/+merge/72995
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-addons/trunk-bug-832056-jam.
=== modified file 'mail_gateway/mail_gateway.py'
--- mail_gateway/mail_gateway.py	2011-08-03 10:06:02 +0000
+++ mail_gateway/mail_gateway.py	2011-08-26 06:06:28 +0000
@@ -483,7 +483,11 @@
             msg['in-reply-to'] = msg_txt.get('In-Reply-To')
 
         if 'X-Priority' in fields:
-            msg['priority'] = msg_txt.get('X-Priority', '3 (Normal)').split(' ')[0]
+            priority = msg_txt.get('X-Priority', '3 (Normal)').split(' ')[0]
+            # Avoid future validation errors for this non-standard header
+            if priority not in [str(x) for x in range(1,6)]:
+                priority = "3"
+            msg['priority'] = priority
 
         if not msg_txt.is_multipart() or 'text/plain' in msg.get('Content-Type', ''):
             encoding = msg_txt.get_content_charset()

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to