credativ team mailing list archive

["Stefan Rijnhart \(Therp\)" <stefan@xxxxxxxx>]

Stefan Rijnhart (Therp) has proposed merging lp:~therp-nl/openupgrade-addons/7.0-lp1206105-mail_post_script_query_parameters into lp:openupgrade-addons.

Requested reviews:
  OpenUpgrade Committers (openupgrade-committers)
Related bugs:
  Bug #1206105 in OpenUpgrade Addons: "[7.0] mail,  SQL query parameters should be a tuple, list or dict"
  https://bugs.launchpad.net/openupgrade-addons/+bug/1206105

For more details, see:
https://code.launchpad.net/~therp-nl/openupgrade-addons/7.0-lp1206105-mail_post_script_query_parameters/+merge/178272
-- 
https://code.launchpad.net/~therp-nl/openupgrade-addons/7.0-lp1206105-mail_post_script_query_parameters/+merge/178272
Your team OpenUpgrade Committers is requested to review the proposed merge of lp:~therp-nl/openupgrade-addons/7.0-lp1206105-mail_post_script_query_parameters into lp:openupgrade-addons.

=== modified file 'mail/migrations/7.0.1.0/post-migration.py'
--- mail/migrations/7.0.1.0/post-migration.py	2013-07-24 12:44:43 +0000
+++ mail/migrations/7.0.1.0/post-migration.py	2013-08-02 11:32:40 +0000
@@ -49,7 +49,7 @@
                 })
         for row in cr.fetchall():
             body = func(row[1])
-            cr.execute("UPDATE mail_message SET body = %s WHERE id = %s", body, row[0])
+            cr.execute("UPDATE mail_message SET body = %s WHERE id = %s", (body, row[0]))
 
 def create_mail_mail(cr, pool):
     """