banking-addons-drivers team mailing list archive

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

Stefan Rijnhart (Therp) has proposed merging lp:~therp-nl/banking-addons/70-communication-communication2-clarify+migration into lp:~banking-addons-drivers/banking-addons/70-communication-communication2-clarify.

Requested reviews:
  Banking Addons Drivers (banking-addons-drivers)

For more details, see:
https://code.launchpad.net/~therp-nl/banking-addons/70-communication-communication2-clarify+migration/+merge/212589
-- 
https://code.launchpad.net/~therp-nl/banking-addons/70-communication-communication2-clarify+migration/+merge/212589
Your team Banking Addons Drivers is requested to review the proposed merge of lp:~therp-nl/banking-addons/70-communication-communication2-clarify+migration into lp:~banking-addons-drivers/banking-addons/70-communication-communication2-clarify.

=== modified file 'account_banking/__openerp__.py'
--- account_banking/__openerp__.py	2014-03-17 07:38:04 +0000
+++ account_banking/__openerp__.py	2014-03-25 11:55:20 +0000
@@ -26,7 +26,7 @@
 
 {
     'name': 'Account Banking',
-    'version': '0.2',
+    'version': '0.3',
     'license': 'AGPL-3',
     'author': 'Banking addons community',
     'website': 'https://launchpad.net/banking-addons',

=== added directory 'account_banking/migrations/7.0.0.3'
=== added file 'account_banking/migrations/7.0.0.3/pre-migration.py'
--- account_banking/migrations/7.0.0.3/pre-migration.py	1970-01-01 00:00:00 +0000
+++ account_banking/migrations/7.0.0.3/pre-migration.py	2014-03-25 11:55:20 +0000
@@ -0,0 +1,44 @@
+# -*- coding: utf-8 -*-
+##############################################################################
+#
+#    Copyright (C) 2014 Akretion (http://www.akretion.com/)
+#    @author: Alexis de Lattre <alexis.delattre@xxxxxxxxxxxx>
+#
+#    This program is free software: you can redistribute it and/or modify
+#    it under the terms of the GNU Affero General Public License as
+#    published by the Free Software Foundation, either version 3 of the
+#    License, or (at your option) any later version.
+#
+#    This program is distributed in the hope that it will be useful,
+#    but WITHOUT ANY WARRANTY; without even the implied warranty of
+#    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+#    GNU Affero General Public License for more details.
+#
+#    You should have received a copy of the GNU Affero General Public License
+#    along with this program.  If not, see <http://www.gnu.org/licenses/>.
+#
+##############################################################################
+
+def table_exists(cr, table):
+    """ Check whether a certain table or view exists """
+    cr.execute(
+        'SELECT count(relname) FROM pg_class WHERE relname = %s',
+        (table,))
+    return cr.fetchone()[0] == 1
+
+def migrate(cr, version):
+    """
+    Migration script for semantic changes in account_banking_payment_export.
+    Putting the same script in this module for users migrating from 6.1,
+    before the export module was refactored out.
+    """
+    if not version or not table_exists(cr, 'payment_line'):
+        return
+    cr.execute(
+        "UPDATE payment_line SET communication = communication2, "
+        "communication2 = null "
+        "FROM payment_order "
+        "WHERE payment_line.order_id = payment_order.id "
+        "AND payment_order.state in ('draft', 'open') "
+        "AND payment_line.state = 'normal' "
+        "AND communication2 is not null")

=== modified file 'account_banking_payment_export/migrations/7.0.0.1.165/pre-migration.py'
--- account_banking_payment_export/migrations/7.0.0.1.165/pre-migration.py	2014-03-25 06:46:32 +0000
+++ account_banking_payment_export/migrations/7.0.0.1.165/pre-migration.py	2014-03-25 11:55:20 +0000
@@ -21,6 +21,8 @@
 
 
 def migrate(cr, version):
+    if not version:
+        return
     cr.execute(
         "UPDATE payment_line SET communication = communication2, "
         "communication2 = null "
@@ -28,5 +30,4 @@
         "WHERE payment_line.order_id = payment_order.id "
         "AND payment_order.state in ('draft', 'open') "
         "AND payment_line.state = 'normal' "
-        "AND communication is null"
         "AND communication2 is not null")