[WIP][FIX] _convert_field_bootstrap_4to5_sql: consider jsonb columns

[chienandalu]

ref: OCA/OpenUpgrade#4263

Options for the fix:

• This WIP: consider tranlated fields in _convert_field_bootstrap_4to5_sql
• Use the orm method in the issued OpenUpgrade script

cc @Tecnativa TT46660

[dansanti]

hello, here I do:
1.- make import:

openupgradelib/openupgradelib/openupgrade_160.py

Line 11 in 35f51c4

from psycopg2 import sql

from psycopg2 import sql, extras

2.- delete from

openupgradelib/openupgradelib/openupgrade_160.py

Line 396 in 35f51c4

[code for code, *_ in cr.fetchall()]

to

openupgradelib/openupgradelib/openupgrade_160.py

Line 399 in 35f51c4

# query = "SELECT id, {field}->>%s FROM {table}"

3.- then adding to your code:

openupgradelib/openupgradelib/openupgrade_160.py

Line 409 in 35f51c4

for id_, old_content in cr.fetchall():

`

for id_, old_content in cr.fetchall():
    new_content = {}
    if column_type == "jsonb":
        for lang, lang_old_content in old_content.items():
            lang_new_content = convert_string_bootstrap_4to5(lang_old_content)
            new_content[lang] = lang_new_content
        new_content = extras.Json(new_content)
        old_content = extras.Json(old_content)
    else:
        new_content = convert_string_bootstrap_4to5(old_content)
    if old_content != new_content:
        cr.execute(
            sql.SQL("UPDATE {table} SET {field} = %s WHERE id = %s").format(
                **format_query_args
            ),
            (
                new_content,
                id_,
            ),
        )

`

[hbrunn]

@chienandalu I ran into the same problem and propose to just have psycopg2 sort it out: #357

[pedrobaeza]

Closed in favor of #357