From 0de2e0ee64a7c7f93d10ff480cbbe5493638a2cc Mon Sep 17 00:00:00 2001 From: Sylvain LE GAL Date: Fri, 26 Apr 2024 23:23:50 +0200 Subject: [PATCH] [OU-FIX] point_of_sale: improve pre-migration execution, replacing n SQL requests n=len(pos_orders) by 1 SQL request --- .../scripts/point_of_sale/16.0.1.0.1/pre-migration.py | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/openupgrade_scripts/scripts/point_of_sale/16.0.1.0.1/pre-migration.py b/openupgrade_scripts/scripts/point_of_sale/16.0.1.0.1/pre-migration.py index 1f9a9ed3cf18..7cc0f47e269a 100644 --- a/openupgrade_scripts/scripts/point_of_sale/16.0.1.0.1/pre-migration.py +++ b/openupgrade_scripts/scripts/point_of_sale/16.0.1.0.1/pre-migration.py @@ -1,6 +1,5 @@ # SPDX-FileCopyrightText: 2024 Tecnativa - Pedro M. Baeza # SPDX-License-Identifier: AGPL-3.0-or-later -import uuid from openupgradelib import openupgrade @@ -10,12 +9,9 @@ def _assign_pos_order_token(env): env, [("access_token", "pos.order", "pos_order", "char", False, "point_of_sale")], ) - env.cr.execute("SELECT id FROM pos_order") - for row in env.cr.fetchall(): - env.cr.execute( - "UPDATE pos_order SET access_token = %s WHERE id = %s", - (str(uuid.uuid4()), row[0]), - ) + openupgrade.logged_query( + env.cr, "UPDATE pos_order SET access_token = gen_random_uuid();" + ) @openupgrade.migrate()