[MIG] delivery_ups_oca: Migration to 17.0

Author: Tisho99

delivery_ups_oca/tests/test_delivery_ups.py

@@ -13,7 +13,7 @@
 _provider_class = _module_ns + ".models.ups_request.UpsRequest"
 
 
-class TestDeliveryUpsBase(common.SavepointCase):
+class TestDeliveryUpsBase(common.TransactionCase):
     @classmethod
     def setUpClass(cls):
         super().setUpClass()

delivery_ups_oca/views/delivery_carrier_view.xml

@@ -13,45 +13,45 @@ License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
                 <page
                     string="UPS"
                     name="configuration_ups_oca"
-                    attrs="{'invisible': [('delivery_type', '!=', 'ups')]}"
+                    invisible="delivery_type != 'ups'"
                 >
                     <group>
                         <group string="Service">
                             <field
                                 name="ups_service_code"
-                                attrs="{'required': [('delivery_type', '=', 'ups')]}"
+                                required="delivery_type == 'ups'"
                             />
                         </group>
                         <group string="Misc">
                             <field
                                 name="ups_shipper_number"
-                                attrs="{'required': [('delivery_type', '=', 'ups')]}"
+                                required="delivery_type == 'ups'"
                             />
                             <field
                                 name="ups_default_packaging_id"
-                                attrs="{'required': [('delivery_type', '=', 'ups')]}"
+                                required="delivery_type == 'ups'"
                                 context="{'default_package_carrier_type': 'ups'}"
                             />
                             <field name="ups_use_packages_from_picking" />
                             <field
                                 name="ups_package_dimension_code"
-                                attrs="{'required': [('delivery_type', '=', 'ups')]}"
+                                required="delivery_type == 'ups'"
                             />
                             <field
                                 name="ups_package_weight_code"
-                                attrs="{'required': [('delivery_type', '=', 'ups')]}"
+                                required="delivery_type == 'ups'"
                             />
                             <field name="ups_tracking_state_update_sync" />
                         </group>
                         <group string="Credentials">
                             <field
                                 name="ups_client_id"
-                                attrs="{'required': [('delivery_type', '=', 'ups')]}"
+                                required="delivery_type == 'ups'"
                             />
                             <field
                                 name="ups_client_secret"
                                 password="True"
-                                attrs="{'required': [('delivery_type', '=', 'ups')]}"
+                                required="delivery_type == 'ups'"
                             />
                             <button
                                 name="ups_update_token"
@@ -63,7 +63,7 @@ License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
                         <group string="Label">
                             <field
                                 name="ups_file_format"
-                                attrs="{'required': [('delivery_type', '=', 'ups')]}"
+                                required="delivery_type == 'ups'"
                             />
                         </group>
                     </group>
@@ -72,7 +72,9 @@ License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
                             <field name="ups_cash_on_delivery" />
                             <field
                                 name="ups_cod_funds_code"
-                                attrs="{'required': [('delivery_type', '=', 'ups'), ('ups_cash_on_delivery', '=', True)], 'invisible': [('ups_cash_on_delivery', '=', False)]}"
+                                required=""
+                                invisible="not ups_cash_on_delivery"
+                                required="delivery_type == 'ups' and ups_cash_on_delivery"
                             />
                         </group>
                     </group>

delivery_ups_oca/views/stock_picking_view.xml

@@ -13,13 +13,7 @@ License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
                     name="ups_get_label"
                     string="UPS Label"
                     type="object"
-                    attrs="{'invisible':[
-                        '|',
-                        '|',
-                        ('carrier_tracking_ref', '=', False),
-                        ('delivery_type', '!=', 'ups'),
-                        ('state', '!=', 'done')
-                    ]}"
+                    invisible="not carrier_tracking_ref or delivery_type != 'ups' or state != 'done'"
                 />
             </xpath>
         </field>