fixup! fixup! [OU-ADD] account

Author: hbrunn

openupgrade_scripts/scripts/account/18.0.1.3/pre-migration.py

@@ -134,23 +134,6 @@ def adapt_account_move_sending_data(env):
             '{author_user_id}', sending_data::jsonb->'sp_user_id')
         WHERE sending_data IS NOT NULL AND sending_data::jsonb ? 'sp_user_id'""",
     )
-    # mail_template_id -> mail_template:
-    env.cr.execute(
-        """
-        SELECT id, sending_data::jsonb->'mail_template_id' as mail_template_id
-        FROM account_move
-        WHERE sending_data IS NOT NULL AND sending_data::jsonb ? 'mail_template_id'""",
-    )
-    for move_id, mail_template_id in env.cr.fetchall():
-        mail_template = env.ref(mail_template_id)
-        openupgrade.logged_query(
-            env.cr,
-            f"""
-            UPDATE account_move
-            SET sending_data = jsonb_set(sending_data::jsonb - 'mail_template_id',
-                '{{mail_template}}', {mail_template})
-            WHERE id = {move_id}""",
-        )
     # send_mail: True -> 'sending_methods': {"email"}:
     openupgrade.logged_query(
         env.cr,

openupgrade_scripts/scripts/account/18.0.1.3/upgrade_analysis_work.txt

@@ -78,7 +78,7 @@ account      / account.lock_exception   / lock_date (date)              : NEW
 account      / account.lock_exception   / lock_date_field (selection)   : NEW required, selection_keys: ['fiscalyear_lock_date', 'purchase_lock_date', 'sale_lock_date', 'tax_lock_date']
 account      / account.lock_exception   / reason (char)                 : NEW
 account      / account.lock_exception   / user_id (many2one)            : NEW relation: res.users, hasdefault: default
-# NOTHING TO DO? (new model)
+# NOTHING TO DO: new feature
 
 account      / account.move             / amount_untaxed_in_currency_signed (float): NEW isfunction: function, stored
 # DONE: pre-migration: pre-created and filled
@@ -101,7 +101,7 @@ account      / account.move             / journal_group_id (many2one)   : NEW re
 # NOTHING TO DO: store = False
 
 account      / account.move             / made_sequence_gap (boolean)   : NEW isfunction: function, stored
-# NOTHING TO DO?
+# NOTHING TO DO: will be computed by ORM
 
 account      / account.move             / origin_payment_id (many2one)  : NEW relation: account.payment
 account      / account.move             / payment_id (many2one)         : DEL relation: account.payment

openupgrade_scripts/scripts/account/tests/data.py

@@ -0,0 +1,13 @@
+env = locals().get("env")
+# call sending wizard on some moves asynchronously
+action = (
+    env["account.move"]
+    .search([("move_type", "=", "out_invoice")])
+    .action_send_and_print()
+)
+env[action["res_model"]].with_context(**action["context"]).create(
+    {
+        "checkbox_download": False,
+    }
+).action_send_and_print()
+env.cr.commit()