[MIG] sale_confirm_wizard from v10 to v14

sale_confirm_wizard/__init__.py

@@ -1,4 +1,2 @@
-# -*- coding: utf-8 -*-
+from . import models
-
-from . import sale
 from . import wizard

sale_confirm_wizard/__manifest__.py

@@ -1,10 +1,10 @@
-# -*- coding: utf-8 -*-
+# Copyright 2017-2021 Akretion France (http://www.akretion.com/)
-# © 2017 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+# @author: Alexis de Lattre <alexis.delattre@akretion.com>
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
 {
     'name': 'Sale Confirm Wizard',
-    'version': '10.0.1.0.0',
+    'version': '14.0.1.0.0',
     'category': 'Sales',
     'license': 'AGPL-3',
     'summary': 'Open a wizard when you confirm a sale order to update important info',
@@ -31,7 +31,8 @@ This module has been written by Alexis de Lattre from Akretion
     'depends': ['sale'],
     'data': [
         'wizard/sale_confirm_view.xml',
-        'sale_view.xml',
+        'views/sale_order.xml',
+        'security/ir.model.access.csv',
         ],
     'installable': True,
 }

sale_confirm_wizard/models/__init__.py

@@ -0,0 +1 @@
+from . import sale_order

sale_confirm_wizard/models/sale_order.py

@@ -1,5 +1,4 @@
-# -*- coding: utf-8 -*-
+# Copyright 2020-2021 Akretion France (http://www.akretion.com/)
-# Copyright 2020 Akretion France (http://www.akretion.com/)
 # @author: Alexis de Lattre <alexis.delattre@akretion.com>
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
@@ -13,6 +12,6 @@ class SaleOrder(models.Model):
         """This method is designed to be inherited.
         For example, inherit it if you don't want to start the wizard in
         some scenarios"""
-        action = self.env.ref(
+        action = self.sudo().env.ref(
             'sale_confirm_wizard.sale_confirm_action').read()[0]
         return action

sale_confirm_wizard/security/ir.model.access.csv

@@ -0,0 +1,2 @@
+id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
+access_sale_confirm_wizard,Full access on sale.confirm wizard,model_sale_confirm,sales_team.group_sale_salesman,1,1,1,1

sale_confirm_wizard/views/sale_order.xml

@@ -1,6 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
-  © 2017 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+  Copyright 2017-2021 Akretion France (http://www.akretion.com/)
+  @author: Alexis de Lattre <alexis.delattre@akretion.com>
   License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 -->
 
@@ -11,10 +12,10 @@
     <field name="model">sale.order</field>
     <field name="inherit_id" ref="sale.view_order_form"/>
     <field name="arch" type="xml">
-        <button name="action_confirm" type="object" states="draft" position="attributes">
+        <button id="action_confirm" position="attributes">
             <attribute name="name">sale_confirm_wizard_button</attribute>
         </button>
-        <button name="action_confirm" type="object" states="sent" position="attributes">
+        <button name="action_confirm" attrs="{'invisible': [('state', 'not in', ['draft'])]}" position="attributes">
             <attribute name="name">sale_confirm_wizard_button</attribute>
         </button>
     </field>

sale_confirm_wizard/wizard/__init__.py

@@ -1,3 +1 @@
-# -*- coding: utf-8 -*-
-
 from . import sale_confirm

sale_confirm_wizard/wizard/sale_confirm.py

@@ -1,10 +1,10 @@
-# -*- coding: utf-8 -*-
+# Copyright 2017-2021 Akretion France (http://www.akretion.com/)
-# © 2017 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+# @author: Alexis de Lattre <alexis.delattre@akretion.com>
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
 from odoo import models, fields, api, _
 from odoo.exceptions import UserError
-from odoo.addons.base.res.res_partner import WARNING_MESSAGE
+from odoo.addons.base.models.res_partner import WARNING_MESSAGE
 
 
 class SaleConfirm(models.TransientModel):
@@ -19,12 +19,12 @@ class SaleConfirm(models.TransientModel):
     partner_invoice_id = fields.Many2one(
         'res.partner', 'Invoice Address', required=True)
     show_partner_invoice_id = fields.Many2one(
-        related='partner_invoice_id', readonly=True,
+        related='partner_invoice_id',
         string='Detailed Invoice Address')
     partner_shipping_id = fields.Many2one(
         'res.partner', 'Delivery Address', required=True)
     show_partner_shipping_id = fields.Many2one(
-        related='partner_shipping_id', readonly=True,
+        related='partner_shipping_id',
         string='Detailed Delivery Address')
     sale_warn = fields.Selection(
         WARNING_MESSAGE, 'Sale Warning Type', readonly=True)
@@ -46,7 +46,7 @@ class SaleConfirm(models.TransientModel):
 
     @api.model
     def default_get(self, fields):
-        res = super(SaleConfirm, self).default_get(fields)
+        res = super().default_get(fields)
         assert self._context.get('active_model') == 'sale.order',\
             'active_model should be sale.order'
         order = self.env['sale.order'].browse(self._context.get('active_id'))
@@ -54,7 +54,6 @@ class SaleConfirm(models.TransientModel):
         res.update(default)
         return res
 
-    @api.multi
     def _prepare_update_so(self):
         self.ensure_one()
         return {
@@ -64,7 +63,6 @@ class SaleConfirm(models.TransientModel):
             'partner_shipping_id': self.partner_shipping_id.id,
             }
 
-    @api.multi
     def confirm(self):
         self.ensure_one()
         partner = self.sale_id.partner_id.commercial_partner_id

sale_confirm_wizard/wizard/sale_confirm_view.xml

@@ -1,6 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
-  © 2017 Akretion (Alexis de Lattre <alexis.delattre@akretion.com>)
+  Copyright 2017-2021 Akretion France (http://www.akretion.com/)
+  @author: Alexis de Lattre <alexis.delattre@akretion.com>
   License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 -->