savoirfairelinux-openerp team mailing list archive

["Laurent Mignon \(Acsone\)" <laurent.mignon@xxxxxxxxx>]

Laurent Mignon (Acsone) has proposed merging lp:~acsone-openerp/e-commerce-addons/7.0-bug-1280131-lmi into lp:e-commerce-addons.

Requested reviews:
  extra-addons-commiter (extra-addons-commiter)
Related bugs:
  Bug #1280131 in e-commerce-addons: "sale_payment_method: Running migration [0.1>]  failed"
  https://bugs.launchpad.net/e-commerce-addons/+bug/1280131

For more details, see:
https://code.launchpad.net/~acsone-openerp/e-commerce-addons/7.0-bug-1280131-lmi/+merge/206422

[FIX] https://bugs.launchpad.net/e-commerce-addons/+bug/1280131
-- 
https://code.launchpad.net/~acsone-openerp/e-commerce-addons/7.0-bug-1280131-lmi/+merge/206422
Your team extra-addons-commiter is requested to review the proposed merge of lp:~acsone-openerp/e-commerce-addons/7.0-bug-1280131-lmi into lp:e-commerce-addons.

=== modified file 'sale_payment_method/migrations/0.1/post-migration.py'
--- sale_payment_method/migrations/0.1/post-migration.py	2013-09-26 08:41:34 +0000
+++ sale_payment_method/migrations/0.1/post-migration.py	2014-02-14 11:02:02 +0000
@@ -27,9 +27,10 @@
             "many2many with the name payment_ids")
 
 def migrate(cr, version):
-    cr.execute("INSERT INTO account_voucher_sale_order_rel"
-               "(sale_order_id, account_voucher_id) "
-               "(SELECT id, payment_id FROM "
-               " sale_order "
-               "WHERE payment_id IS NOT NULL )")
+    if version:
+        cr.execute("INSERT INTO account_voucher_sale_order_rel"
+                   "(sale_order_id, account_voucher_id) "
+                   "(SELECT id, payment_id FROM "
+                   " sale_order "
+                   "WHERE payment_id IS NOT NULL )")