[MIG] account_payment_order: Migration to 17.0

account_payment_order/__manifest__.py

@@ -9,7 +9,7 @@
 
 {
     "name": "Account Payment Order",
-    "version": "16.0.1.6.1",
+    "version": "17.0.1.0.0",
     "license": "AGPL-3",
     "author": "ACSONE SA/NV, "
     "Therp BV, "

account_payment_order/hooks.py

@@ -1,7 +1,7 @@
 from odoo.tools import sql
 
 
-def pre_init_hook(cr):
+def pre_init_hook(env):
     """Prepare new partner_bank_id computed field.
 
     Add column to avoid MemoryError on an existing Odoo instance
@@ -12,5 +12,5 @@ def pre_init_hook(cr):
     (see AccountMoveLine._compute_partner_bank_id() in models/account_move_line.py
     and AccountMove._compute_payment_order_ok() in models/account_move.py)
     """
-    if not sql.column_exists(cr, "account_move_line", "partner_bank_id"):
-        sql.create_column(cr, "account_move_line", "partner_bank_id", "int4")
+    if not sql.column_exists(env.cr, "account_move_line", "partner_bank_id"):
+        sql.create_column(env.cr, "account_move_line", "partner_bank_id", "int4")

account_payment_order/models/account_move.py

@@ -24,7 +24,6 @@ class AccountMove(models.Model):
     reference_type = fields.Selection(
         selection=[("none", "Free Reference"), ("structured", "Structured Reference")],
         readonly=True,
-        states={"draft": [("readonly", False)]},
         default="none",
     )
     payment_line_count = fields.Integer(compute="_compute_payment_line_count")

account_payment_order/models/account_payment_order.py

@@ -24,7 +24,6 @@ class AccountPaymentOrder(models.Model):
         ondelete="restrict",
         tracking=True,
         readonly=True,
-        states={"draft": [("readonly", False)]},
         check_company=True,
     )
     payment_type = fields.Selection(
@@ -57,7 +56,6 @@ class AccountPaymentOrder(models.Model):
         string="Bank Journal",
         ondelete="restrict",
         readonly=True,
-        states={"draft": [("readonly", False)]},
         tracking=True,
         check_company=True,
     )
@@ -93,12 +91,10 @@ class AccountPaymentOrder(models.Model):
         default="due",
         tracking=True,
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
     date_scheduled = fields.Date(
         string="Payment Execution Date",
         readonly=True,
-        states={"draft": [("readonly", False)]},
         tracking=True,
         help="Select a requested date of execution if you selected 'Due Date' "
         "as the Payment Execution Date Type.",
@@ -118,7 +114,6 @@ class AccountPaymentOrder(models.Model):
         inverse_name="order_id",
         string="Transactions",
         readonly=True,
-        states={"draft": [("readonly", False)]},
     )
     payment_ids = fields.One2many(
         comodel_name="account.payment",

account_payment_order/readme/CONFIGURE.rst

@@ -0,0 +1,2 @@
+This module adds several options on Payment Modes, cf Invoicing/Accounting >
+Configuration > Management > Payment Modes.

account_payment_order/readme/CONTRIBUTORS.rst

@@ -0,0 +1,32 @@
+* Stéphane Bidoul <[email protected]>
+* Alexis de Lattre <[email protected]>
+* Adrien Peiffer <[email protected]>
+* Stefan Rijnhart
+* Laurent Mignon <[email protected]>
+* Alexandre Fayolle
+* Danimar Ribeiro
+* Erwin van der Ploeg
+* Raphaël Valyi
+* Sandy Carter
+* Angel Moya <[email protected]>
+* Jose María Alzaga <[email protected]>
+* Meyomesse Gilles <[email protected]>
+* Denis Roussel <[email protected]>
+
+* `DynApps <https://www.dynapps.be>`_:
+
+  * Raf Ven <[email protected]>
+* Andrea Stirpe <[email protected]>
+* `Jarsa <https://www.jarsa.com.mx>`_:
+
+  * Alan Ramos <[email protected]>
+* `Tecnativa <https://www.tecnativa.com>`_:
+
+  * Pedro M. Baeza
+  * Carlos Dauden
+  * Carlos Roca
+
+* `Open Source Integrators <https://www.opensourceintegrators.com>`_:
+
+  * Ammar Officewala <[email protected]>
+* Marçal Isern <[email protected]>

account_payment_order/readme/DESCRIPTION.rst

@@ -0,0 +1 @@
+This module adds support for payment orders and debit orders.

account_payment_order/readme/INSTALL.rst

@@ -0,0 +1,7 @@
+This module depends on:
+
+* account_payment_partner
+* base_iban
+* document
+
+This modules is part of the OCA/bank-payment suite.

account_payment_order/readme/USAGE.rst

@@ -0,0 +1,7 @@
+You can create a Payment order via the menu Invoicing/Accounting > Vendors > Payment Orders and then select the move lines to pay.
+
+You can create a Debit order via the menu Invoicing/Accounting > Customers > Debit Orders and then select the move lines to debit.
+
+This module also adds an action *Add to Payment Order* on supplier invoices and *Add to Debit Order* on customer invoices.
+
+You can print a Payment order via the menu Invoicing/Accounting > Vendors > Payment Orders and then select the payment oder to print.

account_payment_order/tests/test_payment_order_outbound.py

@@ -120,8 +120,6 @@ def _create_supplier_refund(self, move, manual=False):
             .with_context(active_model="account.move", active_ids=move.ids)
             .create(
                 {
-                    "date_mode": "entry",
-                    "refund_method": "refund",
                     "journal_id": move.journal_id.id,
                 }
             )
@@ -422,7 +420,6 @@ def test_supplier_refund_reference(self):
         refund_line = self.refund.line_ids.filtered(
             lambda line: line.account_type == "liability_payable"
         )
-        (invoice_line | refund_line).reconcile()
 
         self.env["account.invoice.payment.line.multi"].with_context(
             active_model="account.move", active_ids=self.invoice.ids

account_payment_order/views/account_invoice_view.xml

@@ -21,29 +21,21 @@
                     type="object"
                     string="Add to Debit Order"
                     groups="account_payment_order.group_account_payment"
-                    attrs="{'invisible': ['|', '|',
-                                              ('payment_order_ok', '=', False),
-                                              ('state', '!=', 'posted'),
-                                              ('move_type', 'not in', ('out_invoice', 'out_refund'))
-                                             ]}"
+                    invisible="not payment_order_ok or state != 'posted' or move_type not in ('out_invoice', 'out_refund')"
                 />
                 <button
                     name="create_account_payment_line"
                     type="object"
                     string="Add to Payment Order"
                     groups="account_payment_order.group_account_payment"
-                    attrs="{'invisible': ['|', '|',
-                                              ('payment_order_ok', '=', False),
-                                              ('state', '!=', 'posted'),
-                                              ('move_type', 'not in', ('in_invoice', 'in_refund'))
-                                             ]}"
+                    invisible="not payment_order_ok or state != 'posted' or move_type not in ('in_invoice', 'in_refund')"
                 />
             </button>
             <div name="button_box" position="inside">
                 <button
                     name="action_payment_lines"
                     type="object"
-                    attrs="{'invisible': [('payment_line_count', '=', 0)]}"
+                    invisible="not payment_line_count"
                     icon="fa-bars"
                     help="Payment Order Lines"
                     groups="account_payment_order.group_account_payment"
@@ -61,9 +53,9 @@
             <field name="payment_reference" position="before">
                 <field
                     name="reference_type"
-                    attrs="{'readonly':[('state','!=','draft')],
-                            'invisible': [('move_type', 'not in', ('out_invoice', 'out_refund'))],
-                            'required': [('move_type', 'in', ('out_invoice', 'out_refund'))]}"
+                    readonly="state != 'draft'"
+                    invisible="move_type not in ('out_invoice', 'out_refund')"
+                    required="move_type in ('out_invoice', 'out_refund')"
                 />
             </field>
         </field>

account_payment_order/views/account_payment_line.xml

@@ -28,7 +28,7 @@
                             name="partner_bank_id"
                             context="{'default_partner_id': partner_id}"
                             domain="[('partner_id', '=', partner_id)]"
-                            attrs="{'required': [('bank_account_required', '=', True)]}"
+                            required="bank_account_required"
                         />
                         <field name="bank_account_required" invisible="1" />
                         <field name="communication_type" />

account_payment_order/views/account_payment_mode.xml

@@ -12,19 +12,19 @@
                 <group
                     name="payment_order_options"
                     string="Options for Payment Orders"
-                    attrs="{'invisible': [('payment_order_ok', '=', False)]}"
+                    invisible="not payment_order_ok"
                 >
                     <field
                         name="no_debit_before_maturity"
-                        attrs="{'invisible': ['|', ('payment_type', '!=', 'inbound'), ('payment_order_ok', '!=', True)]}"
+                        invisible="payment_type != 'inbound' or not payment_order_ok"
                     />
                     <field name="default_date_prefered" />
                     <field name="group_lines" />
                 </group>
                 <group
                     name="payment_order_create_defaults"
                     string="Select Move Lines to Pay - Default Values"
-                    attrs="{'invisible': [('payment_order_ok', '=', False)]}"
+                    invisible="not payment_order_ok"
                 >
                     <field name="default_journal_ids" widget="many2many_tags" />
                     <field name="default_payment_mode" />

account_payment_order/views/account_payment_order.xml

@@ -10,46 +10,46 @@
                         name="%(account_payment_line_create_action)d"
                         type="action"
                         string="Create Payment Lines from Journal Items"
-                        states="draft"
+                        invisible="state != 'draft'"
                         class="oe_highlight"
                     />
                     <button
                         name="draft2open"
                         type="object"
-                        states="draft"
+                        invisible="state != 'draft'"
                         string="Confirm Payments"
                         class="oe_highlight"
                     />
                     <button
                         name="open2generated"
                         type="object"
-                        states="open"
+                        invisible="state != 'open'"
                         string="Generate Payment File"
                         class="oe_highlight"
                     />
                     <button
                         name="generated2uploaded"
                         type="object"
-                        states="generated"
+                        invisible="state != 'generated'"
                         string="File Successfully Uploaded"
                         class="oe_highlight"
                     />
                     <button
                         name="cancel2draft"
                         type="object"
-                        states="cancel"
+                        invisible="state != 'cancel'"
                         string="Back to Draft"
                     />
                     <button
                         name="action_cancel"
                         type="object"
-                        states="draft,open,generated"
+                        invisible="state not in ['draft', 'open', 'generated']"
                         string="Cancel Payments"
                     />
                     <button
                         name="action_uploaded_cancel"
                         type="object"
-                        states="uploaded"
+                        invisible="state != 'uploaded'"
                         string="Cancel Payments"
                     />
                     <field
@@ -83,11 +83,13 @@
                         <group name="head-left">
                             <field
                                 name="payment_mode_id"
+                                readonly="state != 'draft'"
                                 domain="[('payment_order_ok', '=', True), ('payment_type', '=', payment_type)]"
                             />
                             <field name="allowed_journal_ids" invisible="1" />
                             <field
                                 name="journal_id"
+                                readonly="state != 'draft'"
                                 domain="[('id', 'in', allowed_journal_ids)]"
                             />
                             <field name="bank_account_link" invisible="1" />
@@ -99,14 +101,16 @@
                             <field name="payment_type" invisible="0" />
                             <field
                                 name="payment_count"
-                                attrs="{'invisible': [('state', 'in', ('draft', 'cancel'))]}"
+                                invisible="state in ('draft', 'cancel')"
                             />
                         </group>
                         <group name="head-right">
-                            <field name="date_prefered" />
+                            <field name="date_prefered" readonly="state != 'draft'" />
                             <field
                                 name="date_scheduled"
-                                attrs="{'invisible': [('date_prefered', '!=', 'fixed')], 'required': [('date_prefered', '=', 'fixed')]}"
+                                readonly="state != 'draft'"
+                                invisible="date_prefered != 'fixed'"
+                                required="date_prefered == 'fixed'"
                             />
                             <field name="date_generated" />
                             <field name="generated_user_id" />
@@ -118,13 +122,14 @@
                         <page name="payment-lines" string="Transactions">
                             <field
                                 name="payment_line_ids"
+                                readonly="state != 'draft'"
                                 context="{'default_payment_type': payment_type}"
                             />
                         </page>
                         <page
                             name="payment-lines"
                             string="Payment Transactions"
-                            attrs="{'invisible': [('state', 'in', ('draft', 'cancel'))]}"
+                            invisible="state in ('draft', 'cancel')"
                         >
                             <field
                                 name="payment_ids"

account_payment_order/views/account_payment_views.xml

@@ -17,10 +17,7 @@
         <field name="inherit_id" ref="account.view_account_payment_form" />
         <field name="arch" type="xml">
             <field name="ref" position="after">
-                <field
-                    name="payment_reference"
-                    attrs="{'invisible': [('payment_reference', '=', False)]}"
-                />
+                <field name="payment_reference" invisible="not payment_reference" />
             </field>
         </field>
     </record>

account_payment_order/wizard/account_payment_line_create_view.xml

@@ -15,11 +15,13 @@
                     <field name="date_type" />
                     <field
                         name="move_date"
-                        attrs="{'required': [('date_type', '=', 'move')], 'invisible': [('date_type', '!=', 'move')]}"
+                        required="date_type == 'move'"
+                        invisible="date_type != 'move'"
                     />
                     <field
                         name="due_date"
-                        attrs="{'required': [('date_type', '=', 'due')], 'invisible': [('date_type', '!=', 'due')]}"
+                        required="date_type == 'due'"
+                        invisible="date_type != 'due'"
                     />
                     <field
                         name="journal_ids"