[14.0][REM] l10n_br_account_payment_order, l10n_br_account_payment_brcobranca: Removendo Campos, Objetos e Visões obsoletos

[mbcosta]

Remove CNAB objects fields views obsolete.

Removendo Campos, Objetos e Visões obsoletos, depende dos PRs #3243 #3360 os commits desses PRs foram incluídos aqui para poder validar o PR, depois do merge será feito o rebase e esses commits deverão sair, mesmo assim já é possível revisar o PR olhando os dois últimos commits.

Com o merge #3337 os objetos l10n_br_cnab.mov.instruction.code, l10n_br_cnab.return.move.code e l10n_br_cnab.boleto.wallet.code, suas Visões e os Campos referentes em outros objetos se tornaram obsoletos, foram mantidos apenas para permitir a migração, esse PR está removendo.

O script de migração deve atualizar os XMLs IDs para o novo objeto l10n_br_cnab.code e remover os objetos l10n_br_cnab.mov.instruction.code, l10n_br_cnab.return.move.code e l10n_br_cnab.boleto.wallet.code.

IMPORTANTE: Não atualize um Banco de Dados em Produção diretamente crie um novo Banco para Homologação em caso de algum erro por favor veja de colocar aqui o LOG com a mensagem.

Na atualização dos Bancos de Dados recomendo rodar uma versão por vez para acompanhar se tudo ocorre sem erros

• 14.0.7.0.0 Separa a Configuração do CNAB do Modo de Pagamento [14.0][REF] l10n_br_account_payment_order, l10n_br_account_payment_brcobranca: Separando a Configuração do CNAB do Modo de Pagamento #3243

• 14.0.8.0.0 Código de Desconto no caso UNICRED 400 e possibilidade de informar códigos além do 0 e 1 [14.0][FIX+IMP] l10n_br_account_payment_order, l10n_br_account_payment_brcobranca: Código de Desconto no caso UNICRED 400 e possibilidade de informar códigos além do 0 e 1 #3360

• 14.0.9.0.0 esse PR

Aproveitando essa "limpeza" estou considerando remover algumas Constantes que pelo o que vi com o grep não estão sendo utilizados:

• CODIGO_OCORRENCIAS https://github.com/OCA/l10n-brazil/blob/14.0/l10n_br_account_payment_order/constants.py#L283

• COD_REGISTROS_REJEITADOS_CNAB400 https://github.com/OCA/l10n-brazil/blob/14.0/l10n_br_account_payment_order/constants.py#L532

• CODIGO_OCORRENCIAS_CNAB200 https://github.com/OCA/l10n-brazil/blob/14.0/l10n_br_account_payment_order/constants.py#L591

Aqui seria importante um retorno do @mileo e @antoniospneto para saber se isso está sendo usado de alguma forma em algum outro modulo.

Outro ponto não foi possível remover o campo cnab_processor do l10n_br_account_payment_order/models/account_payment_mode.py por estar sendo usado pelo modulo l10n_br_cnab_structure é possível mante-lo para aguardar essa refatoração porém com o PR do CNAB Config o campo já vai deixar de aparecer na Visão do Modo de Pagamento, então é preciso avaliar essas alteração no modulo.

cc @rvalyi @renatonlima @marcelsavegnago

[OCA-git-bot]

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

[OCA-git-bot]

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

[rvalyi]

/ocabot merge nobump

[OCA-git-bot]

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-3365-by-rvalyi-bump-nobump, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at d7e0633. Thanks a lot for contributing to OCA. ❤️