[FIX] l10n_br_account_payment_brcobranca: discount value

[DiegoParadeda]

Nesse PR estou tentando corrigir o seguinte erro na geração do arquivo de pagamento Itaú CNAB400:

Discussão

@mbcosta você poderia me ajudar com um review desse trecho, por favor? Se entendi corretamente o valor cod_desconto deveria ser False para o banco Itaú, mas daí resulta no erro da lib.

Também debuguei colocando cod_desconto="x" pra ver se o valor "x" aparecia no arquivo de remessa e não apareceu. Ou seja, parece que a lib retorna erro se cod_desconto=False porém quando fornecido um valor ela não utiliza para compor o arquivo.

[OCA-git-bot]

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

[antoniospneto]

boa @DiegoParadeda, tbm peguei esse erro e ia abrir uma PR, mas eu ia sugerir diferente, assim:
ali na linha onde adiciona o valor ao dicionario, adicionaria uma verificação:

if cod_desconto:
    linhas_pagamentos["cod_desconto"] = discount_code

pois se não tem desconto não precisamos informar o campo "cod_desconto" no dicionario.
pois ai pode deixar que a lib brcobranca vai se responsabilizar em preencher os valores com ZEROS ou BRANCOS, conforme o layout de cada banco.

dessa forma mantemos o comportamento de antes, nos caso que não tinham descontos.

cc @mbcosta

[antoniospneto]

verificar meu comentario

[DiegoParadeda]

boa @DiegoParadeda, tbm peguei esse erro e ia abrir uma PR, mas eu ia sugerir diferente, assim: ali na linha onde adiciona o valor ao dicionario, adicionaria uma verificação:

if cod_desconto:
    linhas_pagamentos["cod_desconto"] = discount_code

pois se não tem desconto não precisamos informar o campo "cod_desconto" no dicionario. pois ai pode deixar que a lib brcobranca vai se responsabilizar em preencher os valores com ZEROS ou BRANCOS, conforme o layout de cada branco.

dessa forma mantemos o comportamento de antes, nos caso que não tinham descontos.

cc @mbcosta

@antoniospneto ficou melhor mesmo, vou atualizar aqui

[antoniospneto]

hot fix

/ocabot merge patch

[antoniospneto]

/ocabot merge patch

[rvalyi]

Hadouken
/ocabot merge patch

[OCA-git-bot]

On my way to merge this fine PR!
Prepared branch 14.0-ocabot-merge-pr-3384-by-rvalyi-bump-patch, awaiting test results.

[OCA-git-bot]

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