From 63dcfd44dedeeee5216a944437d2f65ad76a62d6 Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Tue, 15 Oct 2024 17:27:59 +0200
Subject: [PATCH 01/21] Initial wizard commit

Co-authored-by: Pere Montagud Ferragud <pere.montagud@somenergia.coop>
---
 som_auvidi/__terp__.py                        |  3 ++
 som_auvidi/security/ir.model.access.csv       |  4 ++
 som_auvidi/tests/__init__.py                  |  0
 som_auvidi/wizard/__init__.py                 |  1 +
 .../wizard_change_to_indexada_auvidi_multi.py | 54 +++++++++++++++++++
 ...wizard_change_to_indexada_auvidi_multi.xml | 45 ++++++++++++++++
 6 files changed, 107 insertions(+)
 create mode 100644 som_auvidi/security/ir.model.access.csv
 create mode 100644 som_auvidi/tests/__init__.py
 create mode 100644 som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
 create mode 100644 som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.xml

diff --git a/som_auvidi/__terp__.py b/som_auvidi/__terp__.py
index 2b6cc8ce5..5fdd4d015 100644
--- a/som_auvidi/__terp__.py
+++ b/som_auvidi/__terp__.py
@@ -10,6 +10,7 @@
         "base",
         "giscedata_serveis_generacio",
         "giscedata_polissa_category",
+        "som_indexada",
     ],
     "init_xml": [],
     "demo_xml": [],
@@ -17,6 +18,8 @@
         "giscedata_polissa_view.xml",
         "giscedata_serveis_generacio_data.xml",
         "giscedata_serveis_generacio_view.xml",
+        "wizard/wizard_change_to_indexada_auvidi_multi.xml",
+        "security/ir.model.access.csv",
     ],
     "active": False,
     "installable": True,
diff --git a/som_auvidi/security/ir.model.access.csv b/som_auvidi/security/ir.model.access.csv
new file mode 100644
index 000000000..24d3d1ccc
--- /dev/null
+++ b/som_auvidi/security/ir.model.access.csv
@@ -0,0 +1,4 @@
+"id","name","model_id:id","group_id:id","perm_read","perm_write","perm_create","perm_unlink"
+"access_wizard_change_to_indexada_auvidi_multi_r","wizard.change.to.indexada.auvidi.multi","model_wizard_change_to_indexada_auvidi_multi","giscedata_serveis_generacio.group_serveis_generacio_r",1,0,0,0
+"access_wizard_change_to_indexada_auvidi_multi_rcw","wizard.change.to.indexada.auvidi.multi","model_wizard_change_to_indexada_auvidi_multi","giscedata_serveis_generacio.group_serveis_generacio_w",1,1,1,0
+"access_wizard_change_to_indexada_auvidi_multi_rcwd","wizard.change.to.indexada.auvidi.multi","model_wizard_change_to_indexada_auvidi_multi","giscedata_serveis_generacio.group_serveis_generacio_u",1,1,1,1
diff --git a/som_auvidi/tests/__init__.py b/som_auvidi/tests/__init__.py
new file mode 100644
index 000000000..e69de29bb
diff --git a/som_auvidi/wizard/__init__.py b/som_auvidi/wizard/__init__.py
index 12f5403ba..800098eb7 100644
--- a/som_auvidi/wizard/__init__.py
+++ b/som_auvidi/wizard/__init__.py
@@ -1,3 +1,4 @@
 # -*- encoding: utf-8 -*-
 from __future__ import absolute_import
 from . import wizard_load_servei_gen_records_from_file
+from . import wizard_change_to_indexada_auvidi_multi
diff --git a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
new file mode 100644
index 000000000..65617a579
--- /dev/null
+++ b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
@@ -0,0 +1,54 @@
+# -*- coding: utf-8 -*-
+from osv import osv, fields
+
+
+class WizardChangeToIndexadaAuvidiMulti(osv.osv_memory):
+
+    _name = "wizard.change.to.indexada.auvidi.multi"
+
+    def change_to_indexada_auvidi_multi(self, cursor, uid, ids, context=None):
+        if context is None:
+            context = {}
+
+        wiz_og = self.browse(cursor, uid, ids[0], context=context)
+
+        pol_ids = context.get("active_ids")
+        wz_chng_to_indx_obj = self.pool.get("wizard.change.to.indexada")
+        pol_obj = self.pool.get("giscedata.polissa")
+        failed_polisses = []
+        for pol_id in pol_ids:
+            ctx = {
+                "active_id": pol_id,
+                "business_pricelist": wiz_og.pricelist.id,
+                "coeficient_k": wiz_og.coeficient_k,
+            }
+            params = {"change_type": "from_period_to_index"}
+            wiz_id = wz_chng_to_indx_obj.create(cursor, uid, params, context=ctx)
+            wiz = wz_chng_to_indx_obj.browse(cursor, uid, [wiz_id])[0]
+            try:
+                wz_chng_to_indx_obj.change_to_indexada(cursor, uid, [wiz.id], context=ctx)
+            except Exception:
+                pol_name = pol_obj.read(cursor, uid, pol_id, ["name"])
+                failed_polisses.append(pol_name["name"])
+
+        info = ""
+        if failed_polisses:
+            info += "\nLes pòlisses següents han fallat: {}".format(str(failed_polisses))
+        if not failed_polisses:
+            info = "Procés acabat correctament!"
+
+        wiz_og.write({"state": "end", "info": info})
+
+    _columns = {
+        "state": fields.selection([("init", "Init"), ("end", "End")], "State"),
+        "info": fields.text("Description"),
+        "pricelist": fields.many2one("product.pricelist", "Tarifa"),
+        "coeficient_k": fields.float("Coeficient K", digits=(9, 3)),
+    }
+
+    _defaults = {
+        "state": lambda *a: "init",
+    }
+
+
+WizardChangeToIndexadaAuvidiMulti()
diff --git a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.xml b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.xml
new file mode 100644
index 000000000..35bf1fde6
--- /dev/null
+++ b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.xml
@@ -0,0 +1,45 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<openerp>
+    <data>
+        <record id="view_change_to_indexada_auvidi_multi" model="ir.ui.view">
+            <field name="name">wizard.change.to.indexada.auvidi.multi.form</field>
+            <field name="model">wizard.change.to.indexada.auvidi.multi</field>
+            <field name="type">form</field>
+            <field name="arch" type="xml">
+                <form string="Canviar mode facturació de les polisses   ">
+                    <field name="state" invisible="1"/>
+                    <group attrs="{'invisible': [('state','!=','init')]}">
+                        <group colspan="4" col="4">
+                            <field name="pricelist"/>
+                            <field name="coeficient_k"/>
+                            <button name="change_to_indexada_auvidi_multi"
+                                    icon="gtk-ok" string="Canviar a indexada amb auvidi" type="object"/>
+                        </group>
+                    </group>
+                    <group attrs="{'invisible': [('state','!=','end')]}">
+                        <field name="info" colspan="4" readonly="1" nolabel="1"/>
+                        <button icon="gtk-close" special="cancel" string="Finalitzar" type="object" colspan="4"/>
+                    </group>
+                </form>
+            </field>
+        </record>
+        <record id="action_change_to_indexada_auvidi_multi" model="ir.actions.act_window">
+            <field name="name">Wizard canviar polisses massivament a indexada amb auvidi</field>
+            <field name="type">ir.actions.act_window</field>
+            <field name="res_model">wizard.change.to.indexada.auvidi.multi</field>
+            <field name="view_type">form</field>
+            <field name="view_mode">form</field>
+            <field name="domain">[]</field>
+            <field name="target">new</field>
+            <field name="view_id" ref="view_change_to_indexada_auvidi_multi"/>
+        </record>
+        <record id="wizard_change_to_indexada_auvidi_multi" model="ir.values">
+            <field name="object" eval="1"/>
+            <field name="name">Canviar les polisses a indexada amb auvidi</field>
+            <field name="key2">client_action_multi</field>
+            <field name="key">action</field>
+            <field name="model">giscedata.polissa</field>
+            <field name="value" eval="'ir.actions.act_window,'+str(ref('action_change_to_indexada_auvidi_multi'))" />
+        </record>
+    </data>
+</openerp>

From 7016db5124b4a5fba2b48acb2bf30948c8b8f72b Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Wed, 16 Oct 2024 14:26:12 +0200
Subject: [PATCH 02/21] WIP create modcon when periodes to indexada

Co-authored-by: Pere Montagud Ferragud <pere.montagud@somenergia.coop>
---
 .../wizard_change_to_indexada_auvidi_multi.py | 82 +++++++++++++++----
 1 file changed, 66 insertions(+), 16 deletions(-)

diff --git a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
index 65617a579..80beccee7 100644
--- a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
@@ -13,32 +13,82 @@ def change_to_indexada_auvidi_multi(self, cursor, uid, ids, context=None):
         wiz_og = self.browse(cursor, uid, ids[0], context=context)
 
         pol_ids = context.get("active_ids")
-        wz_chng_to_indx_obj = self.pool.get("wizard.change.to.indexada")
-        pol_obj = self.pool.get("giscedata.polissa")
         failed_polisses = []
+        error_polisses = []
         for pol_id in pol_ids:
-            ctx = {
-                "active_id": pol_id,
-                "business_pricelist": wiz_og.pricelist.id,
-                "coeficient_k": wiz_og.coeficient_k,
-            }
-            params = {"change_type": "from_period_to_index"}
-            wiz_id = wz_chng_to_indx_obj.create(cursor, uid, params, context=ctx)
-            wiz = wz_chng_to_indx_obj.browse(cursor, uid, [wiz_id])[0]
-            try:
-                wz_chng_to_indx_obj.change_to_indexada(cursor, uid, [wiz.id], context=ctx)
-            except Exception:
-                pol_name = pol_obj.read(cursor, uid, pol_id, ["name"])
-                failed_polisses.append(pol_name["name"])
+
+            mode = self.get_current_mode(cursor, uid, pol_id)
+            md = self.get_last_pending_modcon(cursor, uid, pol_id)
+
+            if mode == 'atr':  # in periods
+                if not md:
+                    # Periods withe MODCON pending to indexed --> modcon pending Indexed + auvidi
+                    # Create the MODCON to indexed
+                    if not self.change_to_indexada_auvidi(
+                        cursor, uid, pol_id, wiz_og.pricelist.id, wiz_og.coeficient_k
+                    ):
+                        self.add_to_list(cursor, uid, pol_id, failed_polisses)
+                        break
+
+                md = self.get_last_pending_modcon(cursor, uid, pol_id)
+                # Periods with MODCON pending to indexed and auvidi --> don't do anything
+                # Periods with MODCON pending to indexed --> modify the MODCON to add the auvdi
+                md.te_auvidi = True
+
+            else:  # in indexed
+                if not md:
+                    # Indexed with out MODCON to periods --> create MODCON to auvidi
+                    pass
+                else:
+                    # Indexed with MODCON to periods --> error, cannot be done!
+                    self.add_to_list(cursor, uid, pol_id, error_polisses)
 
         info = ""
         if failed_polisses:
             info += "\nLes pòlisses següents han fallat: {}".format(str(failed_polisses))
-        if not failed_polisses:
+        if error_polisses:
+            info += "\nLes pòlisses següents no es pot fer el canvi: {}".format(str(error_polisses))
+        if not failed_polisses and not error_polisses:
             info = "Procés acabat correctament!"
 
         wiz_og.write({"state": "end", "info": info})
 
+    def get_last_pending_modcon(self, cursor, uid, pol_id):
+        pol_obj = self.pool.get("giscedata.polissa")
+        pol = pol_obj.browse(cursor, uid, pol_id)
+        if (pol.modcontractuals_ids[0].state == "pendent"
+            and pol.mode_facturacio != pol.modcontractuals_ids[0].mode_facturacio
+                and pol.modcontractuals_ids[0].mode_facturacio):
+            return pol.modcontractuals_ids[0]
+        else:
+            return None
+
+    def get_current_mode(self, cursor, uid, pol_id):
+        pol_obj = self.pool.get("giscedata.polissa")
+        data = pol_obj.read(cursor, uid, pol_id, ['mode_facturacio'])
+        return data['mode_facturacio']
+
+    def add_to_list(self, cursor, uid, pol_id, list_to_add):
+        pol_obj = self.pool.get("giscedata.polissa")
+        pol_name = pol_obj.read(cursor, uid, pol_id, ["name"])
+        list_to_add.append(pol_name["name"])
+
+    def change_to_indexada_auvidi(self, cursor, uid, pol_id, pricelist_id, coeficient_k):
+        wz_chng_to_indx_obj = self.pool.get("wizard.change.to.indexada")
+        ctx = {
+            "active_id": pol_id,
+            "business_pricelist": pricelist_id,
+            "coeficient_k": coeficient_k,
+        }
+        params = {"change_type": "from_period_to_index"}
+        wiz_id = wz_chng_to_indx_obj.create(cursor, uid, params, context=ctx)
+        wiz = wz_chng_to_indx_obj.browse(cursor, uid, [wiz_id])[0]
+        try:
+            wz_chng_to_indx_obj.change_to_indexada(cursor, uid, [wiz.id], context=ctx)
+            return True
+        except Exception:
+            return False
+
     _columns = {
         "state": fields.selection([("init", "Init"), ("end", "End")], "State"),
         "info": fields.text("Description"),

From 60738d5c5c243338d679a0c6e29eaf820418205b Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Wed, 16 Oct 2024 14:34:57 +0200
Subject: [PATCH 03/21] changed break for continue ...

Co-authored-by: Pere Montagud Ferragud <pere.montagud@somenergia.coop>
---
 .../wizard_change_to_indexada_auvidi_multi.py  | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
index 80beccee7..7ed534d4f 100644
--- a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
@@ -27,8 +27,8 @@ def change_to_indexada_auvidi_multi(self, cursor, uid, ids, context=None):
                     if not self.change_to_indexada_auvidi(
                         cursor, uid, pol_id, wiz_og.pricelist.id, wiz_og.coeficient_k
                     ):
-                        self.add_to_list(cursor, uid, pol_id, failed_polisses)
-                        break
+                        failed_polisses.append(pol_id)
+                        continue
 
                 md = self.get_last_pending_modcon(cursor, uid, pol_id)
                 # Periods with MODCON pending to indexed and auvidi --> don't do anything
@@ -41,13 +41,15 @@ def change_to_indexada_auvidi_multi(self, cursor, uid, ids, context=None):
                     pass
                 else:
                     # Indexed with MODCON to periods --> error, cannot be done!
-                    self.add_to_list(cursor, uid, pol_id, error_polisses)
+                    error_polisses.append(pol_id)
 
         info = ""
         if failed_polisses:
-            info += "\nLes pòlisses següents han fallat: {}".format(str(failed_polisses))
+            failed_str = self.get_list_polissa_names(cursor, uid, failed_polisses)
+            info += "\nLes pòlisses següents han fallat: {}".format(failed_str)
         if error_polisses:
-            info += "\nLes pòlisses següents no es pot fer el canvi: {}".format(str(error_polisses))
+            error_str = self.get_list_polissa_names(cursor, uid, error_polisses)
+            info += "\nLes pòlisses següents no es pot fer el canvi: {}".format(error_str)
         if not failed_polisses and not error_polisses:
             info = "Procés acabat correctament!"
 
@@ -68,10 +70,10 @@ def get_current_mode(self, cursor, uid, pol_id):
         data = pol_obj.read(cursor, uid, pol_id, ['mode_facturacio'])
         return data['mode_facturacio']
 
-    def add_to_list(self, cursor, uid, pol_id, list_to_add):
+    def get_list_polissa_names(self, cursor, uid, pol_ids):
         pol_obj = self.pool.get("giscedata.polissa")
-        pol_name = pol_obj.read(cursor, uid, pol_id, ["name"])
-        list_to_add.append(pol_name["name"])
+        pol_data = pol_obj.read(cursor, uid, pol_ids, ["name"])
+        return ', '.join([pol['name'] for pol in pol_data])
 
     def change_to_indexada_auvidi(self, cursor, uid, pol_id, pricelist_id, coeficient_k):
         wz_chng_to_indx_obj = self.pool.get("wizard.change.to.indexada")

From 172f48f24595174c7a47f614ddfa5edbbd9beb39 Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Wed, 16 Oct 2024 16:14:57 +0200
Subject: [PATCH 04/21] simplification, refactor and error handling

---
 .../wizard_change_to_indexada_auvidi_multi.py | 111 ++++++++++++------
 ...wizard_change_to_indexada_auvidi_multi.xml |   6 +-
 2 files changed, 81 insertions(+), 36 deletions(-)

diff --git a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
index 7ed534d4f..c5de8e063 100644
--- a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
@@ -1,5 +1,6 @@
 # -*- coding: utf-8 -*-
 from osv import osv, fields
+from datetime import date, timedelta
 
 
 class WizardChangeToIndexadaAuvidiMulti(osv.osv_memory):
@@ -13,8 +14,14 @@ def change_to_indexada_auvidi_multi(self, cursor, uid, ids, context=None):
         wiz_og = self.browse(cursor, uid, ids[0], context=context)
 
         pol_ids = context.get("active_ids")
-        failed_polisses = []
-        error_polisses = []
+        faileds = []
+        from_wainting_periodes = []
+        from_periodes = []
+        from_waiting_indexed = []
+        from_already_there = []
+        from_indexed = []
+        msg = []
+
         for pol_id in pol_ids:
 
             mode = self.get_current_mode(cursor, uid, pol_id)
@@ -24,36 +31,68 @@ def change_to_indexada_auvidi_multi(self, cursor, uid, ids, context=None):
                 if not md:
                     # Periods withe MODCON pending to indexed --> modcon pending Indexed + auvidi
                     # Create the MODCON to indexed
-                    if not self.change_to_indexada_auvidi(
-                        cursor, uid, pol_id, wiz_og.pricelist.id, wiz_og.coeficient_k
-                    ):
-                        failed_polisses.append(pol_id)
-                        continue
-
-                md = self.get_last_pending_modcon(cursor, uid, pol_id)
-                # Periods with MODCON pending to indexed and auvidi --> don't do anything
-                # Periods with MODCON pending to indexed --> modify the MODCON to add the auvdi
-                md.te_auvidi = True
-
+                    if not self.change_to_indexada(cursor, uid, pol_id):
+                        faileds.append(pol_id)
+                    else:
+                        md = self.get_last_pending_modcon(cursor, uid, pol_id)
+                        md.te_auvidi = True
+                        from_periodes.append(pol_id)
+                else:
+                    if md.te_auvidi:
+                        # Periods with MODCON pending to indexed and auvidi --> don't do anything
+                        from_already_there.append(pol_id)
+                    else:
+                        # Periods with MODCON pending to indexed --> modify the MODCON to add auvdi
+                        md.te_auvidi = True
+                        from_waiting_indexed.append(pol_id)
             else:  # in indexed
                 if not md:
                     # Indexed with out MODCON to periods --> create MODCON to auvidi
-                    pass
+                    if not self.create_auvidi_pending_modcon(cursor, uid, pol_id):
+                        faileds.append(pol_id)
+                    else:
+                        from_indexed.append(pol_id)
                 else:
                     # Indexed with MODCON to periods --> error, cannot be done!
-                    error_polisses.append(pol_id)
+                    from_wainting_periodes.append(pol_id)
+
+        if from_periodes:
+            pols = self.get_list_polissa_names(cursor, uid, from_periodes)
+            msg.append("Pòlisses que estan a periodes:")
+            msg.append(" - Creada modcon a indexada + auvidi: {}".format(pols))
+
+        if from_waiting_indexed:
+            pols = self.get_list_polissa_names(cursor, uid, from_waiting_indexed)
+            msg.append("Pòlisses que estan a periodes i tenen modcon a indexada:")
+            msg.append(" - Modcon modificada, afegit auvidi: {}".format(pols))
+
+        if from_already_there:
+            pols = self.get_list_polissa_names(cursor, uid, from_already_there)
+            msg.append("Pòlisses que estan a periodes i tenen modcon a indexada amb auvidi:")
+            msg.append(" - No fem res: {}".format(pols))
+
+        if from_indexed:
+            pols = self.get_list_polissa_names(cursor, uid, from_indexed)
+            msg.append("Pòlisses que estan a indexada:")
+            msg.append(" - Creada modcon a auvidi: {}".format(pols))
+
+        if from_wainting_periodes:
+            pols = self.get_list_polissa_names(cursor, uid, from_wainting_periodes)
+            msg.append("ERROR Pòlisses que estan a indexada amb modcon pendent a periodes:")
+            msg.append(" - {}".format(pols))
+
+        if faileds:
+            pols = self.get_list_polissa_names(cursor, uid, faileds)
+            msg.append("ERROR Pòlisses que han fallat al crear modcon:")
+            msg.append(" - {}".format(pols))
 
-        info = ""
-        if failed_polisses:
-            failed_str = self.get_list_polissa_names(cursor, uid, failed_polisses)
-            info += "\nLes pòlisses següents han fallat: {}".format(failed_str)
-        if error_polisses:
-            error_str = self.get_list_polissa_names(cursor, uid, error_polisses)
-            info += "\nLes pòlisses següents no es pot fer el canvi: {}".format(error_str)
-        if not failed_polisses and not error_polisses:
-            info = "Procés acabat correctament!"
+        if not faileds and not from_wainting_periodes:
+            msg.append("Procés acabat correctament!")
 
-        wiz_og.write({"state": "end", "info": info})
+        wiz_og.write({"state": "end", "info": "\n".join(msg)})
+
+    def quit_auvidi_multi(self, cursor, uid, ids, context=None):
+        pass
 
     def get_last_pending_modcon(self, cursor, uid, pol_id):
         pol_obj = self.pool.get("giscedata.polissa")
@@ -75,13 +114,9 @@ def get_list_polissa_names(self, cursor, uid, pol_ids):
         pol_data = pol_obj.read(cursor, uid, pol_ids, ["name"])
         return ', '.join([pol['name'] for pol in pol_data])
 
-    def change_to_indexada_auvidi(self, cursor, uid, pol_id, pricelist_id, coeficient_k):
+    def change_to_indexada(self, cursor, uid, pol_id):
         wz_chng_to_indx_obj = self.pool.get("wizard.change.to.indexada")
-        ctx = {
-            "active_id": pol_id,
-            "business_pricelist": pricelist_id,
-            "coeficient_k": coeficient_k,
-        }
+        ctx = {"active_id": pol_id}
         params = {"change_type": "from_period_to_index"}
         wiz_id = wz_chng_to_indx_obj.create(cursor, uid, params, context=ctx)
         wiz = wz_chng_to_indx_obj.browse(cursor, uid, [wiz_id])[0]
@@ -91,11 +126,21 @@ def change_to_indexada_auvidi(self, cursor, uid, pol_id, pricelist_id, coeficien
         except Exception:
             return False
 
+    def create_auvidi_pending_modcon(self, cursor, uid, pol_id):
+        pol_obj = self.pool.get("giscedata.polissa")
+        tomorrow = (date.today() + timedelta(days=1)).strftime("%Y-%m-%d")
+        auvidi_flag = {
+            'te_auvidi': True,
+        }
+        try:
+            pol_obj.create_contracte(cursor, uid, pol_id, tomorrow, auvidi_flag)
+            return True
+        except Exception:
+            return False
+
     _columns = {
         "state": fields.selection([("init", "Init"), ("end", "End")], "State"),
         "info": fields.text("Description"),
-        "pricelist": fields.many2one("product.pricelist", "Tarifa"),
-        "coeficient_k": fields.float("Coeficient K", digits=(9, 3)),
     }
 
     _defaults = {
diff --git a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.xml b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.xml
index 35bf1fde6..a2f1c6348 100644
--- a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.xml
+++ b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.xml
@@ -10,10 +10,10 @@
                     <field name="state" invisible="1"/>
                     <group attrs="{'invisible': [('state','!=','init')]}">
                         <group colspan="4" col="4">
-                            <field name="pricelist"/>
-                            <field name="coeficient_k"/>
                             <button name="change_to_indexada_auvidi_multi"
-                                    icon="gtk-ok" string="Canviar a indexada amb auvidi" type="object"/>
+                                    icon="gtk-undo" string="Canviar a indexada amb auvidi" type="object"/>
+                            <button name="quit_auvidi_multi"
+                                    icon="gtk-redo" string="Treure d'auvidi" type="object"/>
                         </group>
                     </group>
                     <group attrs="{'invisible': [('state','!=','end')]}">

From f208525caa89904019b172d7136cf6236ad888a0 Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Wed, 16 Oct 2024 16:54:39 +0200
Subject: [PATCH 05/21] adding the alredy in auvidi option

---
 .../wizard_change_to_indexada_auvidi_multi.py | 23 +++++++++++++++----
 1 file changed, 19 insertions(+), 4 deletions(-)

diff --git a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
index c5de8e063..98cf446ac 100644
--- a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
@@ -20,6 +20,7 @@ def change_to_indexada_auvidi_multi(self, cursor, uid, ids, context=None):
         from_waiting_indexed = []
         from_already_there = []
         from_indexed = []
+        from_auvidi = []
         msg = []
 
         for pol_id in pol_ids:
@@ -47,11 +48,15 @@ def change_to_indexada_auvidi_multi(self, cursor, uid, ids, context=None):
                         from_waiting_indexed.append(pol_id)
             else:  # in indexed
                 if not md:
-                    # Indexed with out MODCON to periods --> create MODCON to auvidi
-                    if not self.create_auvidi_pending_modcon(cursor, uid, pol_id):
-                        faileds.append(pol_id)
+                    if self.get_current_auvidi(cursor, uid, pol_id):
+                        # Indexed and auvidiwith out MODCON to periods --> don't do anything
+                        from_auvidi.append(pol_id)
                     else:
-                        from_indexed.append(pol_id)
+                        # Indexed with out MODCON to periods --> create MODCON to auvidi
+                        if not self.create_auvidi_pending_modcon(cursor, uid, pol_id):
+                            faileds.append(pol_id)
+                        else:
+                            from_indexed.append(pol_id)
                 else:
                     # Indexed with MODCON to periods --> error, cannot be done!
                     from_wainting_periodes.append(pol_id)
@@ -76,6 +81,11 @@ def change_to_indexada_auvidi_multi(self, cursor, uid, ids, context=None):
             msg.append("Pòlisses que estan a indexada:")
             msg.append(" - Creada modcon a auvidi: {}".format(pols))
 
+        if from_auvidi:
+            pols = self.get_list_polissa_names(cursor, uid, from_auvidi)
+            msg.append("Pòlisses que estan a indexada amb auvidi:")
+            msg.append(" - No fem res: {}".format(pols))
+
         if from_wainting_periodes:
             pols = self.get_list_polissa_names(cursor, uid, from_wainting_periodes)
             msg.append("ERROR Pòlisses que estan a indexada amb modcon pendent a periodes:")
@@ -109,6 +119,11 @@ def get_current_mode(self, cursor, uid, pol_id):
         data = pol_obj.read(cursor, uid, pol_id, ['mode_facturacio'])
         return data['mode_facturacio']
 
+    def get_current_auvidi(self, cursor, uid, pol_id):
+        pol_obj = self.pool.get("giscedata.polissa")
+        data = pol_obj.read(cursor, uid, pol_id, ['te_auvidi'])
+        return data['te_auvidi']
+
     def get_list_polissa_names(self, cursor, uid, pol_ids):
         pol_obj = self.pool.get("giscedata.polissa")
         pol_data = pol_obj.read(cursor, uid, pol_ids, ["name"])

From a8edd7288571914dc7fb73b3e7ddcdcf1499fb8e Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Thu, 17 Oct 2024 16:06:33 +0200
Subject: [PATCH 06/21] modify the condition and enforce the te_auvidi write

Co-authored-by: Pere Montagud Ferragud <pere.montagud@somenergia.coop>
---
 .../wizard_change_to_indexada_auvidi_multi.py | 20 +++++++++++--------
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
index 98cf446ac..39e998f0f 100644
--- a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
@@ -36,7 +36,7 @@ def change_to_indexada_auvidi_multi(self, cursor, uid, ids, context=None):
                         faileds.append(pol_id)
                     else:
                         md = self.get_last_pending_modcon(cursor, uid, pol_id)
-                        md.te_auvidi = True
+                        self.set_auvidi(cursor, uid, md.id)
                         from_periodes.append(pol_id)
                 else:
                     if md.te_auvidi:
@@ -44,7 +44,7 @@ def change_to_indexada_auvidi_multi(self, cursor, uid, ids, context=None):
                         from_already_there.append(pol_id)
                     else:
                         # Periods with MODCON pending to indexed --> modify the MODCON to add auvdi
-                        md.te_auvidi = True
+                        self.set_auvidi(cursor, uid, md.id)
                         from_waiting_indexed.append(pol_id)
             else:  # in indexed
                 if not md:
@@ -107,12 +107,12 @@ def quit_auvidi_multi(self, cursor, uid, ids, context=None):
     def get_last_pending_modcon(self, cursor, uid, pol_id):
         pol_obj = self.pool.get("giscedata.polissa")
         pol = pol_obj.browse(cursor, uid, pol_id)
-        if (pol.modcontractuals_ids[0].state == "pendent"
-            and pol.mode_facturacio != pol.modcontractuals_ids[0].mode_facturacio
-                and pol.modcontractuals_ids[0].mode_facturacio):
-            return pol.modcontractuals_ids[0]
-        else:
-            return None
+        if pol.modcontractuals_ids[0].state == "pendent":
+            if (pol.mode_facturacio != pol.modcontractuals_ids[0].mode_facturacio
+                    or pol.mode_facturacio == 'index'):
+                if pol.modcontractuals_ids[0].mode_facturacio:
+                    return pol.modcontractuals_ids[0]
+        return None
 
     def get_current_mode(self, cursor, uid, pol_id):
         pol_obj = self.pool.get("giscedata.polissa")
@@ -124,6 +124,10 @@ def get_current_auvidi(self, cursor, uid, pol_id):
         data = pol_obj.read(cursor, uid, pol_id, ['te_auvidi'])
         return data['te_auvidi']
 
+    def set_auvidi(self, cursor, uid, modcon_id, value=True):
+        mod_obj = self.pool.get("giscedata.polissa.modcontractual")
+        mod_obj.write(cursor, uid, modcon_id, {'te_auvidi': value})
+
     def get_list_polissa_names(self, cursor, uid, pol_ids):
         pol_obj = self.pool.get("giscedata.polissa")
         pol_data = pol_obj.read(cursor, uid, pol_ids, ["name"])

From 4785a46ae18d6606c3fe7ef12c51f6d6bc0e7108 Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Thu, 17 Oct 2024 17:07:06 +0200
Subject: [PATCH 07/21] adding the quit auvidi action logic

iCo-authored-by: Pere Montagud Ferragud <pere.montagud@somenergia.coop>
---
 .../wizard_change_to_indexada_auvidi_multi.py | 132 +++++++++++++++++-
 1 file changed, 127 insertions(+), 5 deletions(-)

diff --git a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
index 39e998f0f..3fdddf113 100644
--- a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
@@ -53,7 +53,7 @@ def change_to_indexada_auvidi_multi(self, cursor, uid, ids, context=None):
                         from_auvidi.append(pol_id)
                     else:
                         # Indexed with out MODCON to periods --> create MODCON to auvidi
-                        if not self.create_auvidi_pending_modcon(cursor, uid, pol_id):
+                        if not self.create_auvidi_pending_modcon(cursor, uid, pol_id, True):
                             faileds.append(pol_id)
                         else:
                             from_indexed.append(pol_id)
@@ -101,8 +101,130 @@ def change_to_indexada_auvidi_multi(self, cursor, uid, ids, context=None):
 
         wiz_og.write({"state": "end", "info": "\n".join(msg)})
 
-    def quit_auvidi_multi(self, cursor, uid, ids, context=None):
-        pass
+    def quit_auvidi_multi(self, cursor, uid, ids, context=None):  # noqa: C901
+        if context is None:
+            context = {}
+
+        wiz_og = self.browse(cursor, uid, ids[0], context=context)
+
+        pol_ids = context.get("active_ids")
+        faileds = []
+        from_already_there = []
+        msg = []
+
+        for pol_id in pol_ids:
+
+            mode = self.get_current_mode(cursor, uid, pol_id)
+            auvidi = self.get_current_auvidi(cursor, uid, pol_id)
+            md = self.get_last_pending_modcon(cursor, uid, pol_id)
+
+            faileds = []
+            from_indexed_auvidi = []
+            from_indexed_waiting_auvidi = []
+            from_indexed_waiting_no_auvidi = []
+            from_indexed_no_auvidi = []
+            from_periods_auvidi = []
+            from_periods_waiting_indexada_auvidi = []
+            from_periods_waiting_no_auvidi = []
+            from_already_there = []
+
+            if mode == 'index':  # in indexed
+                if auvidi:  # has auvidi
+                    if not md:  # does not have any pending modcon
+                        # Indexed + auvidi --> create MODCON to quit auvidi
+                        if not self.create_auvidi_pending_modcon(cursor, uid, pol_id, False):
+                            faileds.append(pol_id)
+                        else:
+                            from_indexed_auvidi.append(pol_id)
+                    else:  # has a pending MODCON
+                        # Indexed + auvidi + MODCON --> modify the MODCON to quit auvidi
+                        if md.te_auvidi:
+                            self.set_auvidi(cursor, uid, md.id, False)
+                            from_indexed_waiting_auvidi.append(pol_id)
+                        else:
+                            from_indexed_waiting_no_auvidi.append(pol_id)
+                else:  # does not has auvidi
+                    if md and md.te_auvidi:
+                        # Indexed + MODCON auvidi --> modify the MODCON to quit auvidi
+                        self.set_auvidi(cursor, uid, md.id, False)
+                        from_indexed_waiting_auvidi.append(pol_id)
+                    else:
+                        from_indexed_no_auvidi.append(pol_id)
+
+            else:  # in periods
+                if auvidi:  # periods and auvidi --> Error!!
+                    if not md:  # does not have any pending modcon
+                        if not self.create_auvidi_pending_modcon(cursor, uid, pol_id, False):
+                            faileds.append(pol_id)
+                        else:
+                            from_periods_auvidi.append(pol_id)
+                    else:
+                        # periods + auvidi + MODCON --> modify the MODCON to quit auvidi
+                        if md.te_auvidi:
+                            self.set_auvidi(cursor, uid, md.id, False)
+                            from_periods_waiting_indexada_auvidi.append(pol_id)
+                        else:
+                            from_periods_waiting_no_auvidi.append(pol_id)
+                else:  # periods and not auvidi
+                    if not md:  # does not have any pending modcon
+                        # Periods no auvidi and no modcon --> don't do anything
+                        from_already_there.append(pol_id)
+                    else:
+                        if md.te_auvidi:
+                            self.set_auvidi(cursor, uid, md.id, False)
+                            from_periods_waiting_indexada_auvidi.append(pol_id)
+                        else:
+                            from_periods_waiting_no_auvidi.append(pol_id)
+
+        if from_indexed_auvidi:
+            pols = self.get_list_polissa_names(cursor, uid, from_indexed_auvidi)
+            msg.append("Pòlisses que estan a indexada amb auvidi:")
+            msg.append(" - Creada modcon per treure auvidi: {}".format(pols))
+
+        if from_indexed_waiting_auvidi:
+            pols = self.get_list_polissa_names(cursor, uid, from_indexed_waiting_auvidi)
+            msg.append("Pòlisses que estan a indexada amb modcon pendent per activar auvidi:")
+            msg.append(" - Modcon modificada, tret auvidi: {}".format(pols))
+
+        if from_indexed_waiting_no_auvidi:
+            pols = self.get_list_polissa_names(cursor, uid, from_indexed_waiting_no_auvidi)
+            msg.append("Pòlisses que ja estan a indexada amb modcon pendent per treure auvidi:")
+            msg.append(" - no fem res: {}".format(pols))
+
+        if from_indexed_no_auvidi:
+            pols = self.get_list_polissa_names(cursor, uid, from_indexed_no_auvidi)
+            msg.append("Pòlisses que estan a indexada sense auvidi i sense modcon pendent:")
+            msg.append(" - no fem res: {}".format(pols))
+
+        if from_periods_auvidi:
+            pols = self.get_list_polissa_names(cursor, uid, from_periods_auvidi)
+            msg.append("Pòlisses que estan a periodes amb auvidi!!!:")
+            msg.append(" - Creada modcon per treure auvidi: {}".format(pols))
+
+        if from_periods_waiting_indexada_auvidi:
+            pols = self.get_list_polissa_names(cursor, uid, from_periods_waiting_indexada_auvidi)
+            msg.append("Pòlisses que estan a periodes amb modcon per passar a indexada + auvidi:")
+            msg.append(" - Modcon modificada, tret auvidi: {}".format(pols))
+
+        if from_periods_waiting_no_auvidi:
+            pols = self.get_list_polissa_names(cursor, uid, from_periods_waiting_no_auvidi)
+            msg.append("Pòlisses que estan a periodes amb modcon per passar a indexada SENSE auvidi:")  # noqa: E501
+            msg.append(" - no fem res: {}".format(pols))
+
+        if from_already_there:
+            pols = self.get_list_polissa_names(cursor, uid, from_already_there)
+            msg.append("Pòlisses que estan a periodes i no tenen modcon a indexada ni auvidi:")
+            msg.append(" - No fem res: {}".format(pols))
+
+        if faileds:
+            pols = self.get_list_polissa_names(cursor, uid, faileds)
+            msg.append("ERROR Pòlisses que han fallat al crear modcon:")
+            msg.append(" - {}".format(pols))
+
+        if not faileds:
+            msg.append("Procés acabat correctament!")
+
+        wiz_og.write({"state": "end", "info": "\n".join(msg)})
 
     def get_last_pending_modcon(self, cursor, uid, pol_id):
         pol_obj = self.pool.get("giscedata.polissa")
@@ -145,11 +267,11 @@ def change_to_indexada(self, cursor, uid, pol_id):
         except Exception:
             return False
 
-    def create_auvidi_pending_modcon(self, cursor, uid, pol_id):
+    def create_auvidi_pending_modcon(self, cursor, uid, pol_id, auvidi_value):
         pol_obj = self.pool.get("giscedata.polissa")
         tomorrow = (date.today() + timedelta(days=1)).strftime("%Y-%m-%d")
         auvidi_flag = {
-            'te_auvidi': True,
+            'te_auvidi': auvidi_value,
         }
         try:
             pol_obj.create_contracte(cursor, uid, pol_id, tomorrow, auvidi_flag)

From 59055d0f9eb6d81a2c794b7ac3820caea65f5120 Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Fri, 18 Oct 2024 13:03:36 +0200
Subject: [PATCH 08/21] Test code

Co-authored-by: Pere Montagud Ferragud <pere.montagud@somenergia.coop>
---
 som_auvidi/tests/__init__.py                  |   1 +
 ..._wizard_change_to_indexada_auvidi_multi.py | 109 ++++++++++++++++++
 .../wizard_change_to_indexada_auvidi_multi.py |   2 +-
 3 files changed, 111 insertions(+), 1 deletion(-)
 create mode 100644 som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py

diff --git a/som_auvidi/tests/__init__.py b/som_auvidi/tests/__init__.py
index e69de29bb..d2d24ee7b 100644
--- a/som_auvidi/tests/__init__.py
+++ b/som_auvidi/tests/__init__.py
@@ -0,0 +1 @@
+from test_wizard_change_to_indexada_auvidi_multi import *
diff --git a/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
new file mode 100644
index 000000000..994a9e4eb
--- /dev/null
+++ b/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
@@ -0,0 +1,109 @@
+# -*- coding: utf-8 -*-
+from destral import testing
+from destral.transaction import Transaction
+import mock
+
+
+class WizardChangeToIndexadaAuvidiMultiBaseTests(testing.OOTestCase):
+    def setUp(self):
+        self.txn = Transaction().start(self.database)
+
+        self.cursor = self.txn.cursor
+        self.uid = self.txn.user
+        self.pool = self.openerp.pool
+        self.wiz_obj = self.pool.get("wizard.change.to.indexada.auvidi.multi")
+        self.polissa_obj = self.pool.get("giscedata.polissa")
+        self.imd_obj = self.pool.get("ir.model.data")
+
+    def tearDown(self):
+        self.txn.stop()
+
+    def get_model(self, model_name):
+        return self.openerp.pool.get(model_name)
+
+    def open_polissa(self, xml_id, mode_facturacio='atr'):
+        polissa_id = self.imd_obj.get_object_reference(
+            self.cursor, self.uid, "giscedata_polissa", xml_id
+        )[1]
+        self.polissa_obj.write(self.cursor, self.uid, [polissa_id], {
+                               'mode_facturacio': mode_facturacio})
+
+        self.polissa_obj.send_signal(self.cursor, self.uid, [polissa_id], ["validar", "contracte"])
+
+        return polissa_id
+
+
+class WizardChangeToIndexadaAuvidiMultiTests(WizardChangeToIndexadaAuvidiMultiBaseTests):
+
+    def test__create_auvidi_pending_modcon__auvidi_True_index(self):
+        polissa_id = self.open_polissa("polissa_tarifa_018", 'index')
+        self.wiz_obj.create_auvidi_pending_modcon(self.cursor, self.uid, polissa_id, True)
+        md = self.wiz_obj.get_last_pending_modcon(self.cursor, self.uid, polissa_id)
+        self.assertEqual(md.te_auvidi, True)
+
+    def test__create_auvidi_pending_modcon__auvidi_True_periods(self):
+        polissa_id = self.open_polissa("polissa_tarifa_018")
+        self.wiz_obj.create_auvidi_pending_modcon(self.cursor, self.uid, polissa_id, True)
+
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(p.modcontractuals_ids[0].te_auvidi, True)
+
+    @mock.patch("poweremail.poweremail_send_wizard.poweremail_send_wizard.send_mail")
+    def test__inperiods_without_modcontoindex_to_auvidi(self, mocked_send_mail):
+        # Periods without MODCON pending to indexed --> modcon pending Indexed + auvidi
+        polissa_id = self.open_polissa("polissa_tarifa_018")
+
+        context = {"active_id": polissa_id, "active_ids": [polissa_id]}
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.change_to_indexada_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+
+        wiz = self.wiz_obj.browse(self.cursor, self.uid, wiz_id)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(p.modcontractuals_ids[0].te_auvidi, True)
+        self.assertEqual(p.modcontractuals_ids[0].mode_facturacio, u'index')
+        self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        self.assertIn(u"Pòlisses que estan a periodes:", wiz.info)
+        self.assertIn(u" - Creada modcon a indexada + auvidi: {}".format(p.name), wiz.info)
+
+    @mock.patch("poweremail.poweremail_send_wizard.poweremail_send_wizard.send_mail")
+    def test__inperiods_with_modcontoindex_to_auvidi(self, mocked_send_mail):
+        # Periods with MODCON pending to indexed and auvidi --> don't do anything
+        polissa_id = self.open_polissa("polissa_tarifa_018")
+
+        context = {"active_id": polissa_id, "active_ids": [polissa_id]}
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.change_to_indexada_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.change_to_indexada_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+
+        wiz = self.wiz_obj.browse(self.cursor, self.uid, wiz_id)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(p.modcontractuals_ids[0].te_auvidi, True)
+        self.assertEqual(p.modcontractuals_ids[0].mode_facturacio, u'index')
+        self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        self.assertIn(
+            u"Pòlisses que estan a periodes i tenen modcon a indexada amb auvidi:", wiz.info)
+        self.assertIn(u" - No fem res: {}".format(p.name), wiz.info)
+
+    @mock.patch("poweremail.poweremail_send_wizard.poweremail_send_wizard.send_mail")
+    def test__inperiods_with_modcontoindex_no_auvidi(self, mocked_send_mail):
+        # Periods with MODCON pending to indexed and auvidi --> don't do anything
+        polissa_id = self.open_polissa("polissa_tarifa_018")
+
+        context = {"active_id": polissa_id, "active_ids": [polissa_id]}
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.change_to_indexada_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.wiz_obj.set_auvidi(self.cursor, self.uid, p.modcontractuals_ids[0].id, False)
+
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.change_to_indexada_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+
+        wiz = self.wiz_obj.browse(self.cursor, self.uid, wiz_id)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(p.modcontractuals_ids[0].te_auvidi, True)
+        self.assertEqual(p.modcontractuals_ids[0].mode_facturacio, u'index')
+        self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        self.assertIn(u"Pòlisses que estan a periodes i tenen modcon a indexada:", wiz.info)
+        self.assertIn(u" - Modcon modificada, afegit auvidi: {}".format(p.name), wiz.info)
diff --git a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
index 3fdddf113..51a0ca644 100644
--- a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
@@ -30,7 +30,7 @@ def change_to_indexada_auvidi_multi(self, cursor, uid, ids, context=None):
 
             if mode == 'atr':  # in periods
                 if not md:
-                    # Periods withe MODCON pending to indexed --> modcon pending Indexed + auvidi
+                    # Periods without MODCON pending to indexed --> modcon pending Indexed + auvidi
                     # Create the MODCON to indexed
                     if not self.change_to_indexada(cursor, uid, pol_id):
                         faileds.append(pol_id)

From 7afc8e77c9d1b9f4d625b7b277121019e2319a2b Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Fri, 18 Oct 2024 13:37:35 +0200
Subject: [PATCH 09/21] More test code

Co-authored-by: Pere Montagud Ferragud <pere.montagud@somenergia.coop>
---
 ..._wizard_change_to_indexada_auvidi_multi.py | 72 ++++++++++++++++++-
 1 file changed, 69 insertions(+), 3 deletions(-)

diff --git a/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
index 994a9e4eb..7b4360a07 100644
--- a/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
@@ -21,12 +21,14 @@ def tearDown(self):
     def get_model(self, model_name):
         return self.openerp.pool.get(model_name)
 
-    def open_polissa(self, xml_id, mode_facturacio='atr'):
+    def open_polissa(self, xml_id, mode_facturacio='atr', te_auvidi=False):
         polissa_id = self.imd_obj.get_object_reference(
             self.cursor, self.uid, "giscedata_polissa", xml_id
         )[1]
         self.polissa_obj.write(self.cursor, self.uid, [polissa_id], {
-                               'mode_facturacio': mode_facturacio})
+            'mode_facturacio': mode_facturacio,
+            'te_auvidi': te_auvidi,
+        })
 
         self.polissa_obj.send_signal(self.cursor, self.uid, [polissa_id], ["validar", "contracte"])
 
@@ -88,7 +90,7 @@ def test__inperiods_with_modcontoindex_to_auvidi(self, mocked_send_mail):
 
     @mock.patch("poweremail.poweremail_send_wizard.poweremail_send_wizard.send_mail")
     def test__inperiods_with_modcontoindex_no_auvidi(self, mocked_send_mail):
-        # Periods with MODCON pending to indexed and auvidi --> don't do anything
+        # Periods with MODCON pending to indexed --> modify the MODCON to add auvdi
         polissa_id = self.open_polissa("polissa_tarifa_018")
 
         context = {"active_id": polissa_id, "active_ids": [polissa_id]}
@@ -107,3 +109,67 @@ def test__inperiods_with_modcontoindex_no_auvidi(self, mocked_send_mail):
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
         self.assertIn(u"Pòlisses que estan a periodes i tenen modcon a indexada:", wiz.info)
         self.assertIn(u" - Modcon modificada, afegit auvidi: {}".format(p.name), wiz.info)
+
+    @mock.patch("poweremail.poweremail_send_wizard.poweremail_send_wizard.send_mail")
+    def test__inindex_with_auvidi_without_modcon(self, mocked_send_mail):
+        # Indexed and auvidiwith out MODCON to periods --> don't do anything
+        polissa_id = self.open_polissa("polissa_tarifa_018", 'index', True)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        prev_modcons = len(p.modcontractuals_ids)
+
+        context = {"active_id": polissa_id, "active_ids": [polissa_id]}
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.change_to_indexada_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+
+        wiz = self.wiz_obj.browse(self.cursor, self.uid, wiz_id)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(p.te_auvidi, True)
+        self.assertEqual(p.mode_facturacio, u'index')
+        self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
+        self.assertIn(u"Pòlisses que estan a indexada amb auvidi:", wiz.info)
+        self.assertIn(u" - No fem res: {}".format(p.name), wiz.info)
+
+    @mock.patch("poweremail.poweremail_send_wizard.poweremail_send_wizard.send_mail")
+    def test__inindex_without_auvidi_without_modcon(self, mocked_send_mail):
+        # Indexed with out MODCON to periods --> create MODCON to auvidi
+        polissa_id = self.open_polissa("polissa_tarifa_018", 'index', False)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        prev_modcons = len(p.modcontractuals_ids)
+
+        context = {"active_id": polissa_id, "active_ids": [polissa_id]}
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.change_to_indexada_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+
+        wiz = self.wiz_obj.browse(self.cursor, self.uid, wiz_id)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(p.modcontractuals_ids[0].te_auvidi, True)
+        self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        self.assertEqual(len(p.modcontractuals_ids), prev_modcons + 1)
+        self.assertIn(u"Pòlisses que estan a indexada:", wiz.info)
+        self.assertIn(u" - Creada modcon a auvidi: {}".format(p.name), wiz.info)
+
+    @mock.patch("poweremail.poweremail_send_wizard.poweremail_send_wizard.send_mail")
+    def test__inindex_with_modcon_to_period(self, mocked_send_mail):
+        # Indexed with MODCON to periods --> error, cannot be done!
+        polissa_id = self.open_polissa("polissa_tarifa_018", 'index', False)
+        context = {"active_id": polissa_id, "active_ids": [polissa_id]}
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.change_to_indexada_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.wiz_obj.set_auvidi(self.cursor, self.uid, p.modcontractuals_ids[0].id, False)
+        mod_obj = self.pool.get("giscedata.polissa.modcontractual")
+        mod_obj.write(
+            self.cursor, self.uid, p.modcontractuals_ids[0].id, {'mode_facturacio': 'atr'})
+        prev_modcons = len(p.modcontractuals_ids)
+
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.change_to_indexada_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+
+        wiz = self.wiz_obj.browse(self.cursor, self.uid, wiz_id)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(p.modcontractuals_ids[0].mode_facturacio, 'atr')
+        self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
+        self.assertIn(
+            u"ERROR Pòlisses que estan a indexada amb modcon pendent a periodes:", wiz.info)
+        self.assertIn(u" - {}".format(p.name), wiz.info)

From db73276674f7053bc880169d0522fdb5208a5a8e Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Fri, 18 Oct 2024 15:46:18 +0200
Subject: [PATCH 10/21] stronger test code

Co-authored-by: Pere Montagud Ferragud <pere.montagud@somenergia.coop>
---
 .../test_wizard_change_to_indexada_auvidi_multi.py   | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
index 7b4360a07..61048064c 100644
--- a/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
@@ -45,15 +45,21 @@ def test__create_auvidi_pending_modcon__auvidi_True_index(self):
 
     def test__create_auvidi_pending_modcon__auvidi_True_periods(self):
         polissa_id = self.open_polissa("polissa_tarifa_018")
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        prev_modcons = len(p.modcontractuals_ids)
+
         self.wiz_obj.create_auvidi_pending_modcon(self.cursor, self.uid, polissa_id, True)
 
         p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, True)
+        self.assertEqual(len(p.modcontractuals_ids), prev_modcons + 1)
 
     @mock.patch("poweremail.poweremail_send_wizard.poweremail_send_wizard.send_mail")
     def test__inperiods_without_modcontoindex_to_auvidi(self, mocked_send_mail):
         # Periods without MODCON pending to indexed --> modcon pending Indexed + auvidi
         polissa_id = self.open_polissa("polissa_tarifa_018")
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        prev_modcons = len(p.modcontractuals_ids)
 
         context = {"active_id": polissa_id, "active_ids": [polissa_id]}
         wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
@@ -64,6 +70,7 @@ def test__inperiods_without_modcontoindex_to_auvidi(self, mocked_send_mail):
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, True)
         self.assertEqual(p.modcontractuals_ids[0].mode_facturacio, u'index')
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        self.assertEqual(len(p.modcontractuals_ids), prev_modcons + 1)
         self.assertIn(u"Pòlisses que estan a periodes:", wiz.info)
         self.assertIn(u" - Creada modcon a indexada + auvidi: {}".format(p.name), wiz.info)
 
@@ -75,6 +82,8 @@ def test__inperiods_with_modcontoindex_to_auvidi(self, mocked_send_mail):
         context = {"active_id": polissa_id, "active_ids": [polissa_id]}
         wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
         self.wiz_obj.change_to_indexada_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        prev_modcons = len(p.modcontractuals_ids)
 
         wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
         self.wiz_obj.change_to_indexada_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
@@ -84,6 +93,7 @@ def test__inperiods_with_modcontoindex_to_auvidi(self, mocked_send_mail):
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, True)
         self.assertEqual(p.modcontractuals_ids[0].mode_facturacio, u'index')
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
         self.assertIn(
             u"Pòlisses que estan a periodes i tenen modcon a indexada amb auvidi:", wiz.info)
         self.assertIn(u" - No fem res: {}".format(p.name), wiz.info)
@@ -98,6 +108,7 @@ def test__inperiods_with_modcontoindex_no_auvidi(self, mocked_send_mail):
         self.wiz_obj.change_to_indexada_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
         p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
         self.wiz_obj.set_auvidi(self.cursor, self.uid, p.modcontractuals_ids[0].id, False)
+        prev_modcons = len(p.modcontractuals_ids)
 
         wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
         self.wiz_obj.change_to_indexada_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
@@ -107,6 +118,7 @@ def test__inperiods_with_modcontoindex_no_auvidi(self, mocked_send_mail):
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, True)
         self.assertEqual(p.modcontractuals_ids[0].mode_facturacio, u'index')
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
         self.assertIn(u"Pòlisses que estan a periodes i tenen modcon a indexada:", wiz.info)
         self.assertIn(u" - Modcon modificada, afegit auvidi: {}".format(p.name), wiz.info)
 

From 2da9a7a5b193ee506305c3eed500238119344300 Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Fri, 18 Oct 2024 16:12:53 +0200
Subject: [PATCH 11/21] more tests

---
 ..._wizard_change_to_indexada_auvidi_multi.py | 36 +++++++++++++++++++
 1 file changed, 36 insertions(+)

diff --git a/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
index 61048064c..f32637279 100644
--- a/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
@@ -54,6 +54,42 @@ def test__create_auvidi_pending_modcon__auvidi_True_periods(self):
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, True)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons + 1)
 
+    def test__get_last_pending_modcon__no_pending_modcon(self):
+        polissa_id = self.open_polissa("polissa_tarifa_018", 'index')
+
+        md = self.wiz_obj.get_last_pending_modcon(self.cursor, self.uid, polissa_id)
+
+        self.assertEqual(md, None)
+
+    def test__get_last_pending_modcon__pending_modcon_no_mode_facturacio_change(self):
+        polissa_id = self.open_polissa("polissa_tarifa_018", 'atr')
+        self.wiz_obj.create_auvidi_pending_modcon(self.cursor, self.uid, polissa_id, True)
+
+        md = self.wiz_obj.get_last_pending_modcon(self.cursor, self.uid, polissa_id)
+
+        self.assertEqual(md, None)
+
+    @mock.patch("poweremail.poweremail_send_wizard.poweremail_send_wizard.send_mail")
+    def test__get_last_pending_modcon__pending_modcon_to_indexed(self, mocked_send_mail):
+        polissa_id = self.open_polissa("polissa_tarifa_018", 'atr')
+        context = {"active_id": polissa_id, "active_ids": [polissa_id]}
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.change_to_indexada_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+
+        md = self.wiz_obj.get_last_pending_modcon(self.cursor, self.uid, polissa_id)
+
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(md.id, p.modcontractuals_ids[0].id)
+
+    def test__get_last_pending_modcon__pending_modcon_indexed(self):
+        polissa_id = self.open_polissa("polissa_tarifa_018", 'index')
+        self.wiz_obj.create_auvidi_pending_modcon(self.cursor, self.uid, polissa_id, True)
+
+        md = self.wiz_obj.get_last_pending_modcon(self.cursor, self.uid, polissa_id)
+
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(md.id, p.modcontractuals_ids[0].id)
+
     @mock.patch("poweremail.poweremail_send_wizard.poweremail_send_wizard.send_mail")
     def test__inperiods_without_modcontoindex_to_auvidi(self, mocked_send_mail):
         # Periods without MODCON pending to indexed --> modcon pending Indexed + auvidi

From 14d408353c9a55d384c6b4ff4979243d2e15222a Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Fri, 18 Oct 2024 17:39:44 +0200
Subject: [PATCH 12/21] test code for quit_auvidi_multi

---
 ..._wizard_change_to_indexada_auvidi_multi.py | 227 ++++++++++++++++++
 1 file changed, 227 insertions(+)

diff --git a/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
index f32637279..7b35fb7c3 100644
--- a/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
@@ -221,3 +221,230 @@ def test__inindex_with_modcon_to_period(self, mocked_send_mail):
         self.assertIn(
             u"ERROR Pòlisses que estan a indexada amb modcon pendent a periodes:", wiz.info)
         self.assertIn(u" - {}".format(p.name), wiz.info)
+
+    def test__quit_auvidi_multi__indexed_auvidi_no_modcon(self):
+        # Indexed + auvidi --> create MODCON to quit auvidi
+        polissa_id = self.open_polissa("polissa_tarifa_018", 'index', True)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        prev_modcons = len(p.modcontractuals_ids)
+
+        context = {"active_id": polissa_id, "active_ids": [polissa_id]}
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.quit_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+
+        wiz = self.wiz_obj.browse(self.cursor, self.uid, wiz_id)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
+        self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        self.assertEqual(len(p.modcontractuals_ids), prev_modcons + 1)
+        self.assertIn(
+            u"Pòlisses que estan a indexada amb auvidi:", wiz.info)
+        self.assertIn(u" - Creada modcon per treure auvidi: {}".format(p.name), wiz.info)
+
+    def test__quit_auvidi_multi__indexed_auvidi_modcon_no_auvidi(self):
+        # Indexed + auvidi + MODCON no au --> modify the MODCON to quit auvidi
+        polissa_id = self.open_polissa("polissa_tarifa_018", 'index', True)
+        self.wiz_obj.create_auvidi_pending_modcon(self.cursor, self.uid, polissa_id, False)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        prev_modcons = len(p.modcontractuals_ids)
+
+        context = {"active_id": polissa_id, "active_ids": [polissa_id]}
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.quit_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+
+        wiz = self.wiz_obj.browse(self.cursor, self.uid, wiz_id)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
+        self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
+        self.assertIn(
+            u"Pòlisses que ja estan a indexada amb modcon pendent per treure auvidi:", wiz.info)
+        self.assertIn(u" - no fem res: {}".format(p.name), wiz.info)
+
+    def test__quit_auvidi_multi__indexed_auvidi_modcon_yes_auividi(self):
+        # Indexed + auvidi + MODCON no au --> modify the MODCON to quit auvidi
+        polissa_id = self.open_polissa("polissa_tarifa_018", 'index', True)
+        self.wiz_obj.create_auvidi_pending_modcon(self.cursor, self.uid, polissa_id, True)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        prev_modcons = len(p.modcontractuals_ids)
+
+        context = {"active_id": polissa_id, "active_ids": [polissa_id]}
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.quit_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+
+        wiz = self.wiz_obj.browse(self.cursor, self.uid, wiz_id)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
+        self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
+        self.assertIn(
+            u"Pòlisses que estan a indexada amb modcon pendent per activar auvidi:", wiz.info)
+        self.assertIn(u" - Modcon modificada, tret auvidi: {}".format(p.name), wiz.info)
+
+    def test__quit_auvidi_multi__indexed_no_auvidi_modcon_yes_auvidi(self):
+        # Indexed + MODCON auvidi --> modify the MODCON to quit auvidi
+        polissa_id = self.open_polissa("polissa_tarifa_018", 'index', False)
+        self.wiz_obj.create_auvidi_pending_modcon(self.cursor, self.uid, polissa_id, True)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        prev_modcons = len(p.modcontractuals_ids)
+
+        context = {"active_id": polissa_id, "active_ids": [polissa_id]}
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.quit_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+
+        wiz = self.wiz_obj.browse(self.cursor, self.uid, wiz_id)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
+        self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
+        self.assertIn(
+            u"Pòlisses que estan a indexada amb modcon pendent per activar auvidi:", wiz.info)
+        self.assertIn(u" - Modcon modificada, tret auvidi: {}".format(p.name), wiz.info)
+
+    def test__quit_auvidi_multi__indexed_no_auvidi_modcon_no_auvidi(self):
+        # Indexed + MODCON no auvidi --> do nothing
+        polissa_id = self.open_polissa("polissa_tarifa_018", 'index', False)
+        self.wiz_obj.create_auvidi_pending_modcon(self.cursor, self.uid, polissa_id, False)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        prev_modcons = len(p.modcontractuals_ids)
+
+        context = {"active_id": polissa_id, "active_ids": [polissa_id]}
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.quit_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+
+        wiz = self.wiz_obj.browse(self.cursor, self.uid, wiz_id)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
+        self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
+        self.assertIn(
+            u"Pòlisses que estan a indexada sense auvidi i sense modcon pendent:", wiz.info)
+        self.assertIn(u" - no fem res: {}".format(p.name), wiz.info)
+
+    def test__quit_auvidi_multi__periods_auvidi_no_modcon(self):
+        # periods and auvidi --> Error!!
+        polissa_id = self.open_polissa("polissa_tarifa_018", 'atr', True)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        prev_modcons = len(p.modcontractuals_ids)
+
+        context = {"active_id": polissa_id, "active_ids": [polissa_id]}
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.quit_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+
+        wiz = self.wiz_obj.browse(self.cursor, self.uid, wiz_id)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
+        self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        self.assertEqual(len(p.modcontractuals_ids), prev_modcons + 1)
+        self.assertIn(
+            u"Pòlisses que estan a periodes amb auvidi!!!:", wiz.info)
+        self.assertIn(u" - Creada modcon per treure auvidi: {}".format(p.name), wiz.info)
+
+    @mock.patch("poweremail.poweremail_send_wizard.poweremail_send_wizard.send_mail")
+    def test__quit_auvidi_multi__periods_auvidi_modcon_no_auvidi(self, mocked_send_mail):
+        # periods + auvidi + MODCON no auvidi --> nothing
+        polissa_id = self.open_polissa("polissa_tarifa_018", 'atr', True)
+        context = {"active_id": polissa_id, "active_ids": [polissa_id]}
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.change_to_indexada_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.wiz_obj.set_auvidi(self.cursor, self.uid, p.modcontractuals_ids[0].id, False)
+        prev_modcons = len(p.modcontractuals_ids)
+
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.quit_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+
+        wiz = self.wiz_obj.browse(self.cursor, self.uid, wiz_id)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
+        self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
+        self.assertIn(
+            u"Pòlisses que estan a periodes amb modcon per passar a indexada SENSE auvidi:",
+            wiz.info
+        )
+        self.assertIn(u" - no fem res: {}".format(p.name), wiz.info)
+
+    @mock.patch("poweremail.poweremail_send_wizard.poweremail_send_wizard.send_mail")
+    def test__quit_auvidi_multi__periods_auvidi_modcon_yes_auvidi(self, mocked_send_mail):
+        # periods + auvidi + MODCON --> modify the MODCON to quit auvidi
+        polissa_id = self.open_polissa("polissa_tarifa_018", 'atr', True)
+        context = {"active_id": polissa_id, "active_ids": [polissa_id]}
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.change_to_indexada_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        prev_modcons = len(p.modcontractuals_ids)
+
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.quit_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+
+        wiz = self.wiz_obj.browse(self.cursor, self.uid, wiz_id)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
+        self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
+        self.assertIn(
+            u"Pòlisses que estan a periodes amb modcon per passar a indexada + auvidi:", wiz.info)
+        self.assertIn(u" - Modcon modificada, tret auvidi: {}".format(p.name), wiz.info)
+
+    def test__quit_auvidi_multi__periods_no_auvidi_no_modcon(self):
+        # Periods no auvidi and no modcon --> don't do anything
+        polissa_id = self.open_polissa("polissa_tarifa_018", 'atr', False)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        prev_modcons = len(p.modcontractuals_ids)
+
+        context = {"active_id": polissa_id, "active_ids": [polissa_id]}
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.quit_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+
+        wiz = self.wiz_obj.browse(self.cursor, self.uid, wiz_id)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
+        self.assertIn(
+            u"Pòlisses que estan a periodes i no tenen modcon a indexada ni auvidi:", wiz.info)
+        self.assertIn(u" - No fem res: {}".format(p.name), wiz.info)
+
+    @mock.patch("poweremail.poweremail_send_wizard.poweremail_send_wizard.send_mail")
+    def test__quit_auvidi_multi__periods_no_auvidi_modcon_yes_auvidi(self, mocked_send_mail):
+        # Periods no auvidi and modcon to set auvidi --> modify the MODCON to quit auvidi
+        polissa_id = self.open_polissa("polissa_tarifa_018", 'atr', False)
+        context = {"active_id": polissa_id, "active_ids": [polissa_id]}
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.change_to_indexada_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        prev_modcons = len(p.modcontractuals_ids)
+
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.quit_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+
+        wiz = self.wiz_obj.browse(self.cursor, self.uid, wiz_id)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
+        self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
+        self.assertIn(
+            u"Pòlisses que estan a periodes amb modcon per passar a indexada + auvidi:", wiz.info)
+        self.assertIn(u" - Modcon modificada, tret auvidi: {}".format(p.name), wiz.info)
+
+    @mock.patch("poweremail.poweremail_send_wizard.poweremail_send_wizard.send_mail")
+    def test__quit_auvidi_multi__periods_no_auvidi_modcon_no_auvidi(self, mocked_send_mail):
+        # Periods no auvidi and modcon neutral --> don't do anything
+        polissa_id = self.open_polissa("polissa_tarifa_018", 'atr', False)
+        self.wiz_obj.change_to_indexada(self.cursor, self.uid, polissa_id)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        prev_modcons = len(p.modcontractuals_ids)
+
+        context = {"active_id": polissa_id, "active_ids": [polissa_id]}
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.quit_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+
+        wiz = self.wiz_obj.browse(self.cursor, self.uid, wiz_id)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
+        self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
+        self.assertIn(
+            u"Pòlisses que estan a periodes amb modcon per passar a indexada SENSE auvidi:",
+            wiz.info
+        )
+        self.assertIn(u" - no fem res: {}".format(p.name), wiz.info)

From 7be78104beeb6774b4e1283aa7f2da63cf37c08b Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Tue, 22 Oct 2024 17:33:44 +0200
Subject: [PATCH 13/21] Better but slower way to modify existing modcon

---
 som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
index 51a0ca644..d87886863 100644
--- a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
@@ -248,7 +248,10 @@ def get_current_auvidi(self, cursor, uid, pol_id):
 
     def set_auvidi(self, cursor, uid, modcon_id, value=True):
         mod_obj = self.pool.get("giscedata.polissa.modcontractual")
-        mod_obj.write(cursor, uid, modcon_id, {'te_auvidi': value})
+        if not value:
+            mod_obj.write(cursor, uid, modcon_id, {'te_auvidi': value})
+        pol_id = mod_obj.read(cursor, uid, modcon_id, ['polissa_id'])['polissa_id'][0]
+        self.create_auvidi_pending_modcon(cursor, uid, pol_id, value)
 
     def get_list_polissa_names(self, cursor, uid, pol_ids):
         pol_obj = self.pool.get("giscedata.polissa")

From 35a23a1b2663422a8bf96fd806f35c447768f4b4 Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Tue, 22 Oct 2024 17:35:10 +0200
Subject: [PATCH 14/21] test code for observacions field

---
 ..._wizard_change_to_indexada_auvidi_multi.py | 26 +++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
index 7b35fb7c3..f8ab1f304 100644
--- a/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
@@ -106,6 +106,9 @@ def test__inperiods_without_modcontoindex_to_auvidi(self, mocked_send_mail):
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, True)
         self.assertEqual(p.modcontractuals_ids[0].mode_facturacio, u'index')
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        observacions = p.modcontractuals_ids[0].observacions
+        self.assertIn(u'Mode facturació: ATR → Indexada', observacions)
+        self.assertIn(u'Té AUVIDI: False → True', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons + 1)
         self.assertIn(u"Pòlisses que estan a periodes:", wiz.info)
         self.assertIn(u" - Creada modcon a indexada + auvidi: {}".format(p.name), wiz.info)
@@ -129,6 +132,9 @@ def test__inperiods_with_modcontoindex_to_auvidi(self, mocked_send_mail):
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, True)
         self.assertEqual(p.modcontractuals_ids[0].mode_facturacio, u'index')
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        observacions = p.modcontractuals_ids[0].observacions
+        self.assertIn(u'Mode facturació: ATR → Indexada', observacions)
+        self.assertIn(u'Té AUVIDI: False → True', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
         self.assertIn(
             u"Pòlisses que estan a periodes i tenen modcon a indexada amb auvidi:", wiz.info)
@@ -154,6 +160,9 @@ def test__inperiods_with_modcontoindex_no_auvidi(self, mocked_send_mail):
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, True)
         self.assertEqual(p.modcontractuals_ids[0].mode_facturacio, u'index')
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        observacions = p.modcontractuals_ids[0].observacions
+        self.assertIn(u'Mode facturació: ATR → Indexada', observacions)
+        self.assertIn(u'Té AUVIDI: False → True', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
         self.assertIn(u"Pòlisses que estan a periodes i tenen modcon a indexada:", wiz.info)
         self.assertIn(u" - Modcon modificada, afegit auvidi: {}".format(p.name), wiz.info)
@@ -192,6 +201,9 @@ def test__inindex_without_auvidi_without_modcon(self, mocked_send_mail):
         p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, True)
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        observacions = p.modcontractuals_ids[0].observacions
+        self.assertNotIn(u'Mode facturació: ATR → Indexada', observacions)
+        self.assertIn(u'Té AUVIDI: False → True', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons + 1)
         self.assertIn(u"Pòlisses que estan a indexada:", wiz.info)
         self.assertIn(u" - Creada modcon a auvidi: {}".format(p.name), wiz.info)
@@ -236,6 +248,8 @@ def test__quit_auvidi_multi__indexed_auvidi_no_modcon(self):
         p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        observacions = p.modcontractuals_ids[0].observacions
+        self.assertIn(u'Té AUVIDI: True → False', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons + 1)
         self.assertIn(
             u"Pòlisses que estan a indexada amb auvidi:", wiz.info)
@@ -256,6 +270,8 @@ def test__quit_auvidi_multi__indexed_auvidi_modcon_no_auvidi(self):
         p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        observacions = p.modcontractuals_ids[0].observacions
+        self.assertIn(u'Té AUVIDI: True → False', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
         self.assertIn(
             u"Pòlisses que ja estan a indexada amb modcon pendent per treure auvidi:", wiz.info)
@@ -276,6 +292,8 @@ def test__quit_auvidi_multi__indexed_auvidi_modcon_yes_auividi(self):
         p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        observacions = p.modcontractuals_ids[0].observacions
+        self.assertIn(u'Té AUVIDI: True → False', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
         self.assertIn(
             u"Pòlisses que estan a indexada amb modcon pendent per activar auvidi:", wiz.info)
@@ -335,6 +353,8 @@ def test__quit_auvidi_multi__periods_auvidi_no_modcon(self):
         p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        observacions = p.modcontractuals_ids[0].observacions
+        self.assertIn(u'Té AUVIDI: True → False', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons + 1)
         self.assertIn(
             u"Pòlisses que estan a periodes amb auvidi!!!:", wiz.info)
@@ -358,6 +378,8 @@ def test__quit_auvidi_multi__periods_auvidi_modcon_no_auvidi(self, mocked_send_m
         p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        observacions = p.modcontractuals_ids[0].observacions
+        self.assertIn(u'Té AUVIDI: True → False', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
         self.assertIn(
             u"Pòlisses que estan a periodes amb modcon per passar a indexada SENSE auvidi:",
@@ -382,6 +404,8 @@ def test__quit_auvidi_multi__periods_auvidi_modcon_yes_auvidi(self, mocked_send_
         p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        observacions = p.modcontractuals_ids[0].observacions
+        self.assertIn(u'Té AUVIDI: True → False', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
         self.assertIn(
             u"Pòlisses que estan a periodes amb modcon per passar a indexada + auvidi:", wiz.info)
@@ -421,6 +445,8 @@ def test__quit_auvidi_multi__periods_no_auvidi_modcon_yes_auvidi(self, mocked_se
         p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
+        observacions = p.modcontractuals_ids[0].observacions
+        self.assertIn(u'Té AUVIDI: True → False', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
         self.assertIn(
             u"Pòlisses que estan a periodes amb modcon per passar a indexada + auvidi:", wiz.info)

From 3f67a7f6bd5324d528bff430e73a19f1a69610f4 Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Wed, 23 Oct 2024 16:00:38 +0200
Subject: [PATCH 15/21] add a protection to avoid crate a bad modcon

---
 .../wizard/wizard_change_to_indexada_auvidi_multi.py       | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
index d87886863..c4a68d2c9 100644
--- a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
@@ -259,6 +259,13 @@ def get_list_polissa_names(self, cursor, uid, pol_ids):
         return ', '.join([pol['name'] for pol in pol_data])
 
     def change_to_indexada(self, cursor, uid, pol_id):
+        pol_obj = self.pool.get("giscedata.polissa")
+        pol = pol_obj.browse(cursor, uid, pol_id)
+        last_modcon_date = pol.modcontractuals_ids[0].data_final
+        today = date.today().strftime("%Y-%m-%d")
+        if last_modcon_date < today:
+            return False
+
         wz_chng_to_indx_obj = self.pool.get("wizard.change.to.indexada")
         ctx = {"active_id": pol_id}
         params = {"change_type": "from_period_to_index"}

From d8a78243642c5ec986b2b14b2f1aec2202b7a5af Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Wed, 23 Oct 2024 16:02:39 +0200
Subject: [PATCH 16/21] test code to test the last commit code

---
 ..._wizard_change_to_indexada_auvidi_multi.py | 30 +++++++++++++++++++
 1 file changed, 30 insertions(+)

diff --git a/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
index f8ab1f304..46810754d 100644
--- a/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
@@ -1,9 +1,18 @@
 # -*- coding: utf-8 -*-
 from destral import testing
 from destral.transaction import Transaction
+from datetime import date, timedelta
 import mock
 
 
+def today_str():
+    return date.today().strftime("%Y-%m-%d")
+
+
+def today_minus_str(d):
+    return (date.today() - timedelta(days=d)).strftime("%Y-%m-%d")
+
+
 class WizardChangeToIndexadaAuvidiMultiBaseTests(testing.OOTestCase):
     def setUp(self):
         self.txn = Transaction().start(self.database)
@@ -14,6 +23,7 @@ def setUp(self):
         self.wiz_obj = self.pool.get("wizard.change.to.indexada.auvidi.multi")
         self.polissa_obj = self.pool.get("giscedata.polissa")
         self.imd_obj = self.pool.get("ir.model.data")
+        self.mod_obj = self.pool.get("giscedata.polissa.modcontractual")
 
     def tearDown(self):
         self.txn.stop()
@@ -234,6 +244,26 @@ def test__inindex_with_modcon_to_period(self, mocked_send_mail):
             u"ERROR Pòlisses que estan a indexada amb modcon pendent a periodes:", wiz.info)
         self.assertIn(u" - {}".format(p.name), wiz.info)
 
+    @mock.patch("poweremail.poweremail_send_wizard.poweremail_send_wizard.send_mail")
+    def test__inperiods_without_modcontoindex_to_auvidi_with_bad_current_modcon(self, mocked_send_mail):  # noqa E501
+        # Periods without MODCON pending to indexed --> modcon pending Indexed + auvidi
+        polissa_id = self.open_polissa("polissa_tarifa_018")
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.mod_obj.write(self.cursor, self.uid, p.modcontractuals_ids[0].id, {
+            'data_final': today_minus_str(2)
+        })
+        prev_modcons = len(p.modcontractuals_ids)
+
+        context = {"active_id": polissa_id, "active_ids": [polissa_id]}
+        wiz_id = self.wiz_obj.create(self.cursor, self.uid, {}, context=context)
+        self.wiz_obj.change_to_indexada_auvidi_multi(self.cursor, self.uid, [wiz_id], context)
+
+        wiz = self.wiz_obj.browse(self.cursor, self.uid, wiz_id)
+        p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
+        self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
+        self.assertIn(u"ERROR Pòlisses que han fallat al crear modcon:", wiz.info)
+        self.assertIn(u" - {}".format(p.name), wiz.info)
+
     def test__quit_auvidi_multi__indexed_auvidi_no_modcon(self):
         # Indexed + auvidi --> create MODCON to quit auvidi
         polissa_id = self.open_polissa("polissa_tarifa_018", 'index', True)

From ce114b8bde2d4f06806f4dccf488c207091d2702 Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Mon, 28 Oct 2024 11:23:10 +0100
Subject: [PATCH 17/21] modify the change to indexada to alow change te_auvidi
 at the same time

Co-authored-by: Pere Montagud Ferragud <pere.montagud@somenergia.coop>
---
 som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py | 4 +---
 som_indexada/wizard/wizard_change_to_indexada.py            | 3 +++
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
index c4a68d2c9..5cfa89db7 100644
--- a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
@@ -32,11 +32,9 @@ def change_to_indexada_auvidi_multi(self, cursor, uid, ids, context=None):
                 if not md:
                     # Periods without MODCON pending to indexed --> modcon pending Indexed + auvidi
                     # Create the MODCON to indexed
-                    if not self.change_to_indexada(cursor, uid, pol_id):
+                    if not self.change_to_indexada(cursor, uid, pol_id, {'te_auvidi': True}):
                         faileds.append(pol_id)
                     else:
-                        md = self.get_last_pending_modcon(cursor, uid, pol_id)
-                        self.set_auvidi(cursor, uid, md.id)
                         from_periodes.append(pol_id)
                 else:
                     if md.te_auvidi:
diff --git a/som_indexada/wizard/wizard_change_to_indexada.py b/som_indexada/wizard/wizard_change_to_indexada.py
index 14a92018d..61ecca7fd 100644
--- a/som_indexada/wizard/wizard_change_to_indexada.py
+++ b/som_indexada/wizard/wizard_change_to_indexada.py
@@ -264,6 +264,9 @@ def change_to_indexada(self, cursor, uid, ids, context=None):
             "coeficient_k": coeficient_k,
             "coeficient_d": False,
         }
+        if context.get("te_auvidi", False):
+            new_modcon_vals['te_auvidi'] = True
+
         try:
             polissa.send_signal("modcontractual")
             polissa_obj.write(cursor, uid, polissa.id, new_modcon_vals, context=context)

From e3b509e0bdbdc042241b5b78c0ba7965a1411ccc Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Mon, 28 Oct 2024 11:38:29 +0100
Subject: [PATCH 18/21] wrong function...

---
 .../wizard/wizard_change_to_indexada_auvidi_multi.py       | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
index 5cfa89db7..a626546ce 100644
--- a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
@@ -32,7 +32,7 @@ def change_to_indexada_auvidi_multi(self, cursor, uid, ids, context=None):
                 if not md:
                     # Periods without MODCON pending to indexed --> modcon pending Indexed + auvidi
                     # Create the MODCON to indexed
-                    if not self.change_to_indexada(cursor, uid, pol_id, {'te_auvidi': True}):
+                    if not self.change_to_indexada(cursor, uid, pol_id):
                         faileds.append(pol_id)
                     else:
                         from_periodes.append(pol_id)
@@ -265,7 +265,10 @@ def change_to_indexada(self, cursor, uid, pol_id):
             return False
 
         wz_chng_to_indx_obj = self.pool.get("wizard.change.to.indexada")
-        ctx = {"active_id": pol_id}
+        ctx = {
+            "active_id": pol_id,
+            "te_auvidi": True,
+        }
         params = {"change_type": "from_period_to_index"}
         wiz_id = wz_chng_to_indx_obj.create(cursor, uid, params, context=ctx)
         wiz = wz_chng_to_indx_obj.browse(cursor, uid, [wiz_id])[0]

From a7b6eab5ec40e3967373b548a77dc5fd706a38be Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Mon, 28 Oct 2024 17:46:11 +0100
Subject: [PATCH 19/21] test code

---
 .../tests/test_wizard_change_to_indexada_auvidi_multi.py      | 4 +---
 som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py   | 4 ++--
 2 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
index 46810754d..e716a2857 100644
--- a/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
@@ -475,8 +475,6 @@ def test__quit_auvidi_multi__periods_no_auvidi_modcon_yes_auvidi(self, mocked_se
         p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
-        observacions = p.modcontractuals_ids[0].observacions
-        self.assertIn(u'Té AUVIDI: True → False', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
         self.assertIn(
             u"Pòlisses que estan a periodes amb modcon per passar a indexada + auvidi:", wiz.info)
@@ -486,7 +484,7 @@ def test__quit_auvidi_multi__periods_no_auvidi_modcon_yes_auvidi(self, mocked_se
     def test__quit_auvidi_multi__periods_no_auvidi_modcon_no_auvidi(self, mocked_send_mail):
         # Periods no auvidi and modcon neutral --> don't do anything
         polissa_id = self.open_polissa("polissa_tarifa_018", 'atr', False)
-        self.wiz_obj.change_to_indexada(self.cursor, self.uid, polissa_id)
+        self.wiz_obj.change_to_indexada(self.cursor, self.uid, polissa_id, False)
         p = self.polissa_obj.browse(self.cursor, self.uid, polissa_id)
         prev_modcons = len(p.modcontractuals_ids)
 
diff --git a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
index a626546ce..d23a9efd7 100644
--- a/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/wizard/wizard_change_to_indexada_auvidi_multi.py
@@ -256,7 +256,7 @@ def get_list_polissa_names(self, cursor, uid, pol_ids):
         pol_data = pol_obj.read(cursor, uid, pol_ids, ["name"])
         return ', '.join([pol['name'] for pol in pol_data])
 
-    def change_to_indexada(self, cursor, uid, pol_id):
+    def change_to_indexada(self, cursor, uid, pol_id, auvidi=True):
         pol_obj = self.pool.get("giscedata.polissa")
         pol = pol_obj.browse(cursor, uid, pol_id)
         last_modcon_date = pol.modcontractuals_ids[0].data_final
@@ -267,7 +267,7 @@ def change_to_indexada(self, cursor, uid, pol_id):
         wz_chng_to_indx_obj = self.pool.get("wizard.change.to.indexada")
         ctx = {
             "active_id": pol_id,
-            "te_auvidi": True,
+            "te_auvidi": auvidi,
         }
         params = {"change_type": "from_period_to_index"}
         wiz_id = wz_chng_to_indx_obj.create(cursor, uid, params, context=ctx)

From b96fbdcbfe8f1cec26e69eb0088772e938b8f77c Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Tue, 3 Dec 2024 12:14:55 +0100
Subject: [PATCH 20/21] =?UTF-8?q?=F0=9F=94=A7=20the=20visible=20name=20of?=
 =?UTF-8?q?=20the=20original=20field=20changed,=20so=20test=20changes?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 ..._wizard_change_to_indexada_auvidi_multi.py | 20 +++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
index e716a2857..c268e56dc 100644
--- a/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
@@ -118,7 +118,7 @@ def test__inperiods_without_modcontoindex_to_auvidi(self, mocked_send_mail):
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
         observacions = p.modcontractuals_ids[0].observacions
         self.assertIn(u'Mode facturació: ATR → Indexada', observacions)
-        self.assertIn(u'Té AUVIDI: False → True', observacions)
+        self.assertIn(u'Té Autoconsum Virtual: False → True', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons + 1)
         self.assertIn(u"Pòlisses que estan a periodes:", wiz.info)
         self.assertIn(u" - Creada modcon a indexada + auvidi: {}".format(p.name), wiz.info)
@@ -144,7 +144,7 @@ def test__inperiods_with_modcontoindex_to_auvidi(self, mocked_send_mail):
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
         observacions = p.modcontractuals_ids[0].observacions
         self.assertIn(u'Mode facturació: ATR → Indexada', observacions)
-        self.assertIn(u'Té AUVIDI: False → True', observacions)
+        self.assertIn(u'Té Autoconsum Virtual: False → True', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
         self.assertIn(
             u"Pòlisses que estan a periodes i tenen modcon a indexada amb auvidi:", wiz.info)
@@ -172,7 +172,7 @@ def test__inperiods_with_modcontoindex_no_auvidi(self, mocked_send_mail):
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
         observacions = p.modcontractuals_ids[0].observacions
         self.assertIn(u'Mode facturació: ATR → Indexada', observacions)
-        self.assertIn(u'Té AUVIDI: False → True', observacions)
+        self.assertIn(u'Té Autoconsum Virtual: False → True', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
         self.assertIn(u"Pòlisses que estan a periodes i tenen modcon a indexada:", wiz.info)
         self.assertIn(u" - Modcon modificada, afegit auvidi: {}".format(p.name), wiz.info)
@@ -213,7 +213,7 @@ def test__inindex_without_auvidi_without_modcon(self, mocked_send_mail):
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
         observacions = p.modcontractuals_ids[0].observacions
         self.assertNotIn(u'Mode facturació: ATR → Indexada', observacions)
-        self.assertIn(u'Té AUVIDI: False → True', observacions)
+        self.assertIn(u'Té Autoconsum Virtual: False → True', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons + 1)
         self.assertIn(u"Pòlisses que estan a indexada:", wiz.info)
         self.assertIn(u" - Creada modcon a auvidi: {}".format(p.name), wiz.info)
@@ -279,7 +279,7 @@ def test__quit_auvidi_multi__indexed_auvidi_no_modcon(self):
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
         observacions = p.modcontractuals_ids[0].observacions
-        self.assertIn(u'Té AUVIDI: True → False', observacions)
+        self.assertIn(u'Té Autoconsum Virtual: True → False', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons + 1)
         self.assertIn(
             u"Pòlisses que estan a indexada amb auvidi:", wiz.info)
@@ -301,7 +301,7 @@ def test__quit_auvidi_multi__indexed_auvidi_modcon_no_auvidi(self):
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
         observacions = p.modcontractuals_ids[0].observacions
-        self.assertIn(u'Té AUVIDI: True → False', observacions)
+        self.assertIn(u'Té Autoconsum Virtual: True → False', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
         self.assertIn(
             u"Pòlisses que ja estan a indexada amb modcon pendent per treure auvidi:", wiz.info)
@@ -323,7 +323,7 @@ def test__quit_auvidi_multi__indexed_auvidi_modcon_yes_auividi(self):
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
         observacions = p.modcontractuals_ids[0].observacions
-        self.assertIn(u'Té AUVIDI: True → False', observacions)
+        self.assertIn(u'Té Autoconsum Virtual: True → False', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
         self.assertIn(
             u"Pòlisses que estan a indexada amb modcon pendent per activar auvidi:", wiz.info)
@@ -384,7 +384,7 @@ def test__quit_auvidi_multi__periods_auvidi_no_modcon(self):
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
         observacions = p.modcontractuals_ids[0].observacions
-        self.assertIn(u'Té AUVIDI: True → False', observacions)
+        self.assertIn(u'Té Autoconsum Virtual: True → False', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons + 1)
         self.assertIn(
             u"Pòlisses que estan a periodes amb auvidi!!!:", wiz.info)
@@ -409,7 +409,7 @@ def test__quit_auvidi_multi__periods_auvidi_modcon_no_auvidi(self, mocked_send_m
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
         observacions = p.modcontractuals_ids[0].observacions
-        self.assertIn(u'Té AUVIDI: True → False', observacions)
+        self.assertIn(u'Té Autoconsum Virtual: True → False', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
         self.assertIn(
             u"Pòlisses que estan a periodes amb modcon per passar a indexada SENSE auvidi:",
@@ -435,7 +435,7 @@ def test__quit_auvidi_multi__periods_auvidi_modcon_yes_auvidi(self, mocked_send_
         self.assertEqual(p.modcontractuals_ids[0].te_auvidi, False)
         self.assertEqual(p.modcontractuals_ids[0].state, u'pendent')
         observacions = p.modcontractuals_ids[0].observacions
-        self.assertIn(u'Té AUVIDI: True → False', observacions)
+        self.assertIn(u'Té Autoconsum Virtual: True → False', observacions)
         self.assertEqual(len(p.modcontractuals_ids), prev_modcons)
         self.assertIn(
             u"Pòlisses que estan a periodes amb modcon per passar a indexada + auvidi:", wiz.info)

From 62d1031e871d1d7265f3b8731c667ad12fb02793 Mon Sep 17 00:00:00 2001
From: Fran Cubero <francisco.cubero@somenergia.coop>
Date: Tue, 3 Dec 2024 12:26:31 +0100
Subject: [PATCH 21/21] =?UTF-8?q?=F0=9F=94=A7=20test=20the=20tester?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py b/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
index c268e56dc..e447794d4 100644
--- a/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
+++ b/som_auvidi/tests/test_wizard_change_to_indexada_auvidi_multi.py
@@ -3,6 +3,7 @@
 from destral.transaction import Transaction
 from datetime import date, timedelta
 import mock
+# test
 
 
 def today_str():