Correção: NumeroDocumento recebendo indevidamente NossoNumero em algumas units

[RobertoSchuster]

Olá pessoal, 

Por gentileza, alguém consegue me explicar o motivo do seguinte código abaixo estar presente nas units do ACBrBoleto? (ACBrBancoAmazonia, ACBrBancoBradesco, ACBrBancoBrasil, ACBrBancoCaixa, ACBrBancoHSBC, ACBrUniprime)

// Quando o numero documento vier em branco
if Trim(NumeroDocumento) = '' then
  NumeroDocumento := NossoNumero;

ou

// prevenir quando o seunumero não vem informado no arquivo, altera para NossoNumero do banco
wSeuNumero := StringReplace(SeuNumero, '0','',[rfReplaceAll]);
if (AnsiSameText(wSeuNumero, EmptyStr)) then
begin
  SeuNumero       := NossoNumero;
  NumeroDocumento := NossoNumero
end;

Isso é alguma prática definida nos manuais?

Toda vez eu tenho que remover estes trechos de código dos fontes porque, no meu ponto de vista, não tem sentido.
São dois campos diferentes: NossoNumero é uma numeração do banco, que pode ser diferente de NumeroDocumento, o qual corresponde a uma numeração interna da empresa, justamente para ser controlada pelo ERP do jeito que cada empresa quiser.

Minha sugestão é que cada um controle NumeroDocumento e SeuNumero em seu próprio ERP do jeito que preferir, quem quiser igualar o NumeroDocumento ao NossoNumero, que faça em seus próprios fontes.
Do jeito que está agora, este tratamento está sendo imposto, mascarando o valor real que vem no arquivo de retorno. 
Sem falar que está sendo implementado em apenas algumas units. Ou seja, nem é uma prática padrão.

Fontes em anexo.

PS: Na unit ACBrbancoBrasil, aproveitei e substituí DataMoraJuros por DataMulta na função GerarRegistroTransacao400 (na função GerarRegistroTransacao240 já estava correto).
PS: Estão indo junto as alterações que enviei no tópico Correção na Comparação de Datas (TDatetime) com Null pois ainda não foram comitadas.

Concordam?

ACBrBancoBrasil.pas

ACBrBancoHSBC.pas

ACBrBancoBancoob.pas

ACBrBancoBrasil.pas

ACBrBancoHSBC.pas

ACBrBancoCaixa.pas

ACBrUniprime.pas

ACBrBancoBradesco.pas

ACBrBancoAmazonia.pas

ACBrBancoItau.pas

ACBrBancoCecred.pas

ACBrBancoCaixaSICOB.pas

[José M. S. Junior]

Bom dia, 

Como sabemos, boa parte da implementação das Units de Bancos são contribuições de diversas pessoas e nesses pontos realmente não seguem um padrão, muitas vezes também por que o layout dos bancos também não seguem um padrão...

Nestes casos acima também acho que está correto o ajuste. Vamos considerar a analise para os devidos ajustes. Obrigado.

[RobertoSchuster]

Em 16/03/2018 at 08:19, José M. S. Junior disse:

Bom dia, 

Como sabemos, boa parte da implementação das Units de Bancos são contribuições de diversas pessoas e nesses pontos realmente não seguem um padrão, muitas vezes também por que o layout dos bancos também não seguem um padrão...

Nestes casos acima também acho que está correto o ajuste. Vamos considerar a analise para os devidos ajustes. Obrigado.

Olá José,

Por gentileza, existe alguma previsão para estas alterações serem avaliadas e "comitadas"?
Pergunto apenas para me agendar aqui.

Obrigado.

[José M. S. Junior]

Bom dia,

Para organização das tarefas, adicionamos as contribuições ref. Boleto em uma lista e analisamos todas de uma vez para atualização no SVN, creio que na próxima semana, consiga atualizar essa e as demais contribuições...

[RobertoSchuster]

Boa tarde, alguma posição sobre as correções?

[José M. S. Junior]

Boa tarde, está em andamento para atualização.

[José M. S. Junior]

Favor atualizar os fontes para testes...

[RobertoSchuster]

Olá José, obrigado pelo retorno.

Pelo que verifiquei as correções estão OK.

Novamente obrigado.

[Ogm TI]

Boa tarde,

Estamos passando por uma situação parecida, na geração do arquivo do Santander com o ACBrBancoSantander.pas, esta gravando o a informação que passamos para a variavel NossoNumero para NumeroDocumento.

Nossos fontes estão atualizados.

[Juliana Tamizou]

Bom dia.

Por favor crie um novo tópico para sua situação, uma vez que este envolve diversos bancos e não é recente.

Att.