[16.0][l10n_br_account_payment_order][l10n_br_account_payment_brcobranca] unify cnab code objects

[rvalyi]

port de #3337

[OCA-git-bot]

Hi @mbcosta,
some modules you are maintaining are being modified, check this out!

[rvalyi]

faltou adaptar o script de migração, vou botar em rascunho até la.

[antoniospneto]

@rvalyi aproveitando, eu to refatorando o script de migração do cnab na 14.0, se puder esperar um pouco, acho que vele a pena ver a minha proposta

[mbcosta]

@rvalyi @antoniospneto atualizei o script de migração e fiz outras pequenas alterações necessárias, segue o que foi feito:

• Copiei diretamente o arquivo l10n_br_account_payment_order/migrations/14.0.6.0.0/post-migration.py da v14 colocando o Antonio como autor, mas foi preciso corrigir o script porque os Códigos de Retorno que geram Baixa/Liquidação e o Código de Carteira não estavam sendo copiados, pelo menos na v16 ( na v14 é preciso confirmar)

Um ponto aqui, até onde vi é desnecessário tanto na v14 quanto na v16 criar os campos via script como foi feito no l10n_br_account_payment_order/migrations/14.0.6.0.0/pre-migration.py o Openupgradelib tem uma função para adicionar campos def add_fields mas no docstring está dizendo que a partir da v12 isso seria desnecessário e não tive erros ao não usar

E para não rodar o script em Banco de Dados já migrados é verificado se exite algum Modo de Pagamento com o campo sending_code_id preenchido, o campo é fundamental para o uso do CNAB, e para testar é possível simular mudando a versão do modulo para 16.0.3.0.0 e copiando o mesmo script da versão 16.0.2.0.0.

• Com esse script de migração foi possível remover campos e visões que se tornaram obsoletos, fazendo boa parte do que foi feito no PR [14.0][REM] l10n_br_account_payment_order, l10n_br_account_payment_brcobranca: Removendo Campos, Objetos e Visões obsoletos #3365 porém não foi possível remover os objetos que foram unificados isso gera erro porque o programa parece buscar os antigos XML que tem esses objetos, depois de rodar o script e depois da 16.0.2.0.0 isso deve deixar de causar erros mas para evitar o problema que foi visto na v14, migrar múltiplas versões de uma vez l10n_br_account_payment_order/migrations/14.0.7.0.0/post-migration.py #3408 eu recomendo ou deixar para remover os objetos daqui 6 meses ou apenas na v17, deixei um TODO com o LOG de erro completo para deixar claro o problema

# TODO: Ao remover os objetos que ficaram obsoletos com a unificação dos Códigos
#  no l10n_br_cnab.code acontece o erro abaixo, isso deixa de acontecer em
#  versões posteriores a 16.0.2.0.0, porém para evitar problemas como o que ocorreu
#  na v14 será melhor aguardar mesmo que por 6 meses ou mesmo apenas na migração
#  para a v17 para remover esses objetos e assim evitar qualquer possibilidade
#  de problema com migrações
# ERROR db odoo.modules.registry: Failed to load registry
# Traceback (most recent call last):
#  File "/usr/local/lib/python3.10/site-packages/odoo/tools/convert.py",
#  line 698, in _tag_root
#    f(rec)
#  File "/usr/local/lib/python3.10/site-packages/odoo/tools/convert.py", line 515,
#  in _tag_record
#    record = env['ir.model.data']._load_xmlid(xid)
#  File "/usr/local/lib/python3.10/site-packages/odoo/addons/base/models/ir_model.py",
#  line 2162, in _load_xmlid
#    record = self.env.ref(xml_id, raise_if_not_found=False)
#  File "/usr/local/lib/python3.10/site-packages/odoo/api.py", line 600, in ref
#    record = self[res_model].browse(res_id)
#  File "/usr/local/lib/python3.10/site-packages/odoo/api.py", line 550, in __getitem__
#    return self.registry[model_name](self, (), ())
#  File "/usr/local/lib/python3.10/site-packages/odoo/modules/registry.py",
#  line 190, in __getitem__
#    return self.models[model_name]
# KeyError: 'l10n_br_cnab.mov.instruction.code'

• Foi preciso atualizar o PR devido as atualizações do pre-commit, foi feito em commits em separado

• Parece que na migração [16.0][MIG] l10n_br_account_payment_order #3222 faltou adaptar as Visões para as alterações que foram feitas na v16, algumas referencias sobre a mudança

https://github.com/odoo/odoo/issues/109447

https://github.com/odoo/odoo/pull/102119

https://github.com/odoo/odoo/pull/102190/files

Os PRs estão dentro desse "bash" para evitar uma ligação direta com esses PRs e acabarem "poluindo" os PRs, segue images

O problema que acontecia aqui

Modo de Pagamento

Antes do PR

Com esse PR

Linhas de Movimentações Financeiras

Antes do PR

Com esse PR

Se acreditarem que vai levar um tempo para Aprovar esse PR posso ver de extrair esse commit que corrige essas Visões, aqui é necessário para permitir a Revisão.

Com o que foi feito no PR acredito e peço para alterar o PR de Rascunho/Draft para Pronto para Revisão/Ready to Review