[IMP] util.dispatch_by_dbuuid

[Pirols]

Companion of https://github.com/odoo/upgrade-platform/pull/705

[robodoo]

[KangOl]

Actually, prioritizing the origin.database.uuid was a mistake.
I would like to change it, but it will break a lot of custom scripts, even if most of them are now dead code as the databases have been upgraded.

I don't like the idea of introducing a new neuter flag only set by the upgrade platform.

Let's keep your ancestor=True, but change the implementation of dispatch_by_dbuuid to first try the real dbuuid. This should be retro-compatible

def dispatch_by_dbuuid(cr, version, callbacks):
    query = """
       SELECT value
         FROM ir_config_parameter
        WHERE key IN ('database.uuid', 'origin.database.uuid')
     ORDER BY key ASC
    """
    cr.execute(query)
    for uuid, in cr.fetchall():
        if uuid in callbacks:
            func = callbacks[uuid]
            _logger.info("calling dbuuid-specific function `%s`", func.__name__)
            func(cr, version)
            break

[Pirols]

Actually, prioritizing the origin.database.uuid was a mistake.

I haven't looked into the history of its introduction but to me it looks like this was made necessary to have a consistent dispatch among test and production requests, due to this.

I would like to change it, but it will break a lot of custom scripts, even if most of them are now dead code as the databases have been upgraded.

I don't like the idea of introducing a new neuter flag only set by the upgrade platform.

The issue is caused by the overload of the origin.database.uuid icp key, which is why I opted for the introduction and usage of a dedicated (and overwritable¹) key.

Footnotes

1. see the companion PR for the actual details. ↩