[14.0][IMP] l10n_br_account_payment_brcobranca: Código de instrução de protesto não pode ter apenas 1 char

[DiegoParadeda]

Caso a instrução de protesto (no cadastro do diário do banco) tenha apenas um dígito, as linhas de detalhe do arquivo ficam com um caracter a menos.

Com a correção:

[OCA-git-bot]

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

[antoniospneto]

Será que isso se aplica a todos os bancos? Não poderia haver algum banco em que o código de protesto seja apenas um caractere? Eu também enfrentei esse problema e propus uma correção diretamente na biblioteca brcobranca, mas somente para o banco específico que estava apresentando o problema:

https://github.com/kivanio/brcobranca/pull/262

[mbcosta]

Valeu @DiegoParadeda , para resolver esse e outros problemas similares o que estou buscando fazer no PR https://github.com/OCA/l10n-brazil/pull/3243 é mudar os campos "Códigos de Desconto/Multa/Protesto/etc" de CHAR para objetos com isso esse problema deve ser resolvido porque cada Banco CNAB vai poder cadastrar os próprios Códigos então o valor vai poder ser informado no cadastro.

Devido a quantidade de código no PR o campo Código do Protesto ainda é um CHAR mas assim que aprovado o PR esse e os outros Códigos deverão ser simples de implementar por isso seria importante uma revisão do PR agradeço quem puder ajudar.

Se acharem melhor aprovar esse PR por enquanto pare ter mais tempo na Revisão do "Unificando os Códigos CNAB e separando a Configuração do CNAB do Modo de Pagamento" não vejo problemas.

[DiegoParadeda]

Valeu @DiegoParadeda , para resolver esse e outros problemas similares o que estou buscando fazer no PR #3243 é mudar os campos "Códigos de Desconto/Multa/Protesto/etc" de CHAR para objetos com isso esse problema deve ser resolvido porque cada Banco CNAB vai poder cadastrar os próprios Códigos então o valor vai poder ser informado no cadastro.

Devido a quantidade de código no PR o campo Código do Protesto ainda é um CHAR mas assim que aprovado o PR esse e os outros Códigos deverão ser simples de implementar por isso seria importante uma revisão do PR agradeço quem puder ajudar.

Se acharem melhor aprovar esse PR por enquanto pare ter mais tempo na Revisão do "Unificando os Códigos CNAB e separando a Configuração do CNAB do Modo de Pagamento" não vejo problemas.

@mbcosta seu PR muda bastante esse funcionamento, vou fechar esse PR e volto a trabalhar nisso se necessário. Assim que possível reviso o #3243 também, obrigado

[mbcosta]

@DiegoParadeda importante também o comentário do @antoniospneto sobre a correção ser feita diretamente na Lib BRCobranca, sempre que for possível ou dependendo do problema isso sempre deve ser considerado, um exemplo disso é que idealmente nos gostaríamos que o Arquivo de Retorno fosse padronizado assim evitando a necessidade de tratamento de diferenças no l10n_br_account_payment_brcobranca como o campo Data https://github.com/OCA/l10n-brazil/blob/14.0/l10n_br_account_payment_brcobranca/parser/cnab_file_parser.py#L106 isso é parte da ideia de Modularizarão, cada modulo ou biblioteca fica responsável e restrito a algo específico, no caso o modulo l10n_br_account_payment_brcobranca deveria ser especifico sobre a Comunicação com a LIB, mas por necessidade também precisamos resolver/tratar fazer o DE PARA dessas diferenças entre os Bancos e outro ponto é que alterações na Lib precisam ser validadas pelo mantenedor do projeto então pode ocorrer de incluir um código por aqui com um TODO "aguardando merge do PR 123 na Lib" .