From e08d658b2542d8997f7cf8271f62671c06ea5ba8 Mon Sep 17 00:00:00 2001
From: Alexis de Lattre <alexis.delattre@akretion.com>
Date: Tue, 24 Aug 2021 21:54:06 +0200
Subject: [PATCH] [MIG] base_dynamic_list from v10 to v14

---
 base_dynamic_list/__manifest__.py        |  7 ++---
 base_dynamic_list/models/dynamic_list.py | 19 ++++++------
 base_dynamic_list/views/dynamic_list.xml | 38 ++++++------------------
 3 files changed, 21 insertions(+), 43 deletions(-)

diff --git a/base_dynamic_list/__manifest__.py b/base_dynamic_list/__manifest__.py
index 62f60af..6d8e01e 100644
--- a/base_dynamic_list/__manifest__.py
+++ b/base_dynamic_list/__manifest__.py
@@ -1,11 +1,10 @@
-# -*- coding: utf-8 -*-
-# Copyright 2020 Akretion France (http://www.akretion.com)
+# Copyright 2020-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).
 
 {
     'name': 'Base Dynamic List',
-    'version': '10.0.1.0.0',
+    'version': '14.0.1.0.0',
     'category': 'Tools',
     'license': 'AGPL-3',
     'summary': 'Dynamic lists',
@@ -28,7 +27,7 @@ To use it, you need to do 2 or 3 things :
 
 1) Add an entry in the domain field and the object you selected:
 
-domain = fields.Selection(selection_add=[('risk.type', "Risk Type")])
+domain = fields.Selection(selection_add=[('risk.type', "Risk Type")], ondelete={"risk.type": "set null"})
 
 2) Add the many2one field on your object:
 
diff --git a/base_dynamic_list/models/dynamic_list.py b/base_dynamic_list/models/dynamic_list.py
index a65e687..92e9eb1 100644
--- a/base_dynamic_list/models/dynamic_list.py
+++ b/base_dynamic_list/models/dynamic_list.py
@@ -1,5 +1,4 @@
-# -*- coding: utf-8 -*-
-# Copyright 2020 Akretion France (http://www.akretion.com)
+# Copyright 2020-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).
 
@@ -12,7 +11,7 @@ class DynamicList(models.Model):
     _order = 'sequence, id'
 
     name = fields.Char(required=True)
-    sequence = fields.Integer()
+    sequence = fields.Integer(default=10)
     active = fields.Boolean(default=True)
     domain = fields.Selection([], string='Domain', required=True, index=True)
 
@@ -29,7 +28,7 @@ class DynamicListTranslate(models.Model):
     _order = 'sequence, id'
 
     name = fields.Char(translate=True, required=True)
-    sequence = fields.Integer()
+    sequence = fields.Integer(default=10)
     active = fields.Boolean(default=True)
     domain = fields.Selection([], string='Domain', required=True, index=True)
 
@@ -47,7 +46,7 @@ class DynamicListCode(models.Model):
 
     code = fields.Char(required=True)
     name = fields.Char(translate=True, required=True)
-    sequence = fields.Integer()
+    sequence = fields.Integer(default=10)
     active = fields.Boolean(default=True)
     domain = fields.Selection([], string='Domain', required=True, index=True)
 
@@ -61,7 +60,7 @@ class DynamicListCode(models.Model):
     def name_get(self):
         res = []
         for rec in self:
-            res.append((rec.id, u'[%s] %s' % (rec.code, rec.name)))
+            res.append((rec.id, '[%s] %s' % (rec.code, rec.name)))
         return res
 
     @api.model
@@ -74,7 +73,7 @@ class DynamicListCode(models.Model):
                 [('code', '=', name)] + args, limit=limit)
             if recs:
                 return recs.name_get()
-        return super(DynamicListCode, self).name_search(
+        return super().name_search(
             name=name, args=args, operator=operator, limit=limit)
 
 
@@ -85,7 +84,7 @@ class DynamicListCodeTranslate(models.Model):
 
     code = fields.Char(required=True)
     name = fields.Char(translate=True, required=True)
-    sequence = fields.Integer()
+    sequence = fields.Integer(default=10)
     active = fields.Boolean(default=True)
     domain = fields.Selection([], string='Domain', required=True, index=True)
 
@@ -99,7 +98,7 @@ class DynamicListCodeTranslate(models.Model):
     def name_get(self):
         res = []
         for rec in self:
-            res.append((rec.id, u'[%s] %s' % (rec.code, rec.name)))
+            res.append((rec.id, '[%s] %s' % (rec.code, rec.name)))
         return res
 
     @api.model
@@ -112,5 +111,5 @@ class DynamicListCodeTranslate(models.Model):
                 [('code', '=', name)] + args, limit=limit)
             if recs:
                 return recs.name_get()
-        return super(DynamicListCodeTranslate, self).name_search(
+        return super().name_search(
             name=name, args=args, operator=operator, limit=limit)
diff --git a/base_dynamic_list/views/dynamic_list.xml b/base_dynamic_list/views/dynamic_list.xml
index 28b2ca1..e3542af 100644
--- a/base_dynamic_list/views/dynamic_list.xml
+++ b/base_dynamic_list/views/dynamic_list.xml
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <!--
-  Copyright 2020 Akretion France (http://www.akretion.com/)
+  Copyright 2020-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).
 -->
@@ -15,16 +15,11 @@
     <field name="arch" type="xml">
         <form>
             <sheet>
-                <div class="oe_button_box" name="button_box">
-                    <button name="toggle_active" type="object"
-                        class="oe_stat_button" icon="fa-archive">
-                        <field name="active" widget="boolean_button"
-                            options='{"terminology": "archive"}'/>
-                    </button>
-                </div>
+                <widget name="web_ribbon" title="Archived" bg_color="bg-danger" attrs="{'invisible': [('active', '=', True)]}"/>
                 <group name="main">
                     <field name="name"/>
                     <field name="domain" invisible="not context.get('dynamic_list_main_view')"/>
+                    <field name="active" invisible="1"/>
                 </group>
             </sheet>
         </form>
@@ -70,16 +65,11 @@
     <field name="arch" type="xml">
         <form>
             <sheet>
-                <div class="oe_button_box" name="button_box">
-                    <button name="toggle_active" type="object"
-                        class="oe_stat_button" icon="fa-archive">
-                        <field name="active" widget="boolean_button"
-                            options='{"terminology": "archive"}'/>
-                    </button>
-                </div>
+                <widget name="web_ribbon" title="Archived" bg_color="bg-danger" attrs="{'invisible': [('active', '=', True)]}"/>
                 <group name="main">
                     <field name="name"/>
                     <field name="domain" invisible="not context.get('dynamic_list_translate_main_view')"/>
+                    <field name="active" invisible="1"/>
                 </group>
             </sheet>
         </form>
@@ -125,17 +115,12 @@
     <field name="arch" type="xml">
         <form>
             <sheet>
-                <div class="oe_button_box" name="button_box">
-                    <button name="toggle_active" type="object"
-                        class="oe_stat_button" icon="fa-archive">
-                        <field name="active" widget="boolean_button"
-                            options='{"terminology": "archive"}'/>
-                    </button>
-                </div>
+                <widget name="web_ribbon" title="Archived" bg_color="bg-danger" attrs="{'invisible': [('active', '=', True)]}"/>
                 <group name="main">
                     <field name="code"/>
                     <field name="name"/>
                     <field name="domain" invisible="not context.get('dynamic_list_code_main_view')"/>
+                    <field name="active" invisible="1"/>
                 </group>
             </sheet>
         </form>
@@ -183,17 +168,12 @@
     <field name="arch" type="xml">
         <form>
             <sheet>
-                <div class="oe_button_box" name="button_box">
-                    <button name="toggle_active" type="object"
-                        class="oe_stat_button" icon="fa-archive">
-                        <field name="active" widget="boolean_button"
-                            options='{"terminology": "archive"}'/>
-                    </button>
-                </div>
+                <widget name="web_ribbon" title="Archived" bg_color="bg-danger" attrs="{'invisible': [('active', '=', True)]}"/>
                 <group name="main">
                     <field name="code"/>
                     <field name="name"/>
                     <field name="domain" invisible="not context.get('dynamic_list_code_translate_main_view')"/>
+                    <field name="active" invisible="1"/>
                 </group>
             </sheet>
         </form>