[14.0][FORWARD-PORT][l10n_br_account_payment_brcobranca] - Santander CNAB 400

[kaynnan]

Port da pr #2012 para a v14.0

cc @marcelsavegnago @antoniospneto

Issue #1578

[OCA-git-bot]

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

[rvalyi]

LGTM

[rvalyi]

/ocabot merge minor

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-2848-by-rvalyi-bump-minor, awaiting test results.

[OCA-git-bot]

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

[mbcosta]

@antoniospneto @kaynnan acredito que ficou faltando incluir os Códigos de Instrução do Movimento
e o Código de Retorno do Movimento https://github.com/OCA/l10n-brazil/tree/14.0/l10n_br_account_payment_order/data/cnab_codes ou vai ser incluído em outro PR ? Porque sem isso não é possível nem Usar esse caso do CNAB 400 Santader e portanto nem Testar, isso está no README do modulo:

https://github.com/OCA/l10n-brazil/blob/14.0/l10n_br_account_payment_order/readme/CONFIGURE.rst

Verifique se o Banco e o tipo CNAB usado 240 ou 400 possuem os Códigos de Instrução do Movimento e os Códigos de Retorno do Movimento em:

• Faturamento > Configurações > Administração > CNAB Código de Movimento de Instrução
• Faturamento > Configurações > Administração > CNAB Código de Retorno do Movimento

Caso seja preciso cadastrar por favor considere fazer um PR nesse modulo acrescentando em l10n_br_account_payment_order/data/cnab_codes/banco_X_cnab_Y_Z.xml assim em proximas implementações já não será preciso cadastra-los.

https://github.com/OCA/l10n-brazil/blob/14.0/l10n_br_account_payment_order/readme/ROADMAP.rst

• Mapear e incluir os codigos dos bancos de cada CNAB 240 / 400, aqui devido a quantidade de possibilidades se trata de um "roadmap" constante onde contamos com PRs de outros contribuidores que irão implementar um caso que ainda não esteja cadastrado, apesar do codigo permitir que o cadastro seja feito na tela nesses casos.

Acredito que essa orientação foi passada na implementação do Banco AILOS, e também é importante ter os Testes para evitar que alterações ou atualizações tanto aqui na Localização quanto no BRCobranca não afetem um caso já Homologado, uma regressão, que pode acontecer devido a quantidade de diferenças entre os Bancos e CNABs e por a bilbioteca BRCobranca ser algo externo que pode ter alterações sem o teste da Localização.

[kaynnan]

@mbcosta Grato pela revisão, fiz a refatoração seguindo o que foi apontado, está disponível na PR #2870