Validação do cEANTrib - Melhoria

[Gabriel Bonzanini]

Olá.

Reparei que o teste que dispara a validação do campo cEANTrib através da função ValidarGTIN está assim:

Ao meu ver, deveria estar assim:

Em anexo o arquivo com a alteração, caso entendam que esteja correta.

Grato pela atenção.

pcnNFeW.pas

[RickMao]

Amigo o exemplo valida tanto o GTIN Tributado como o não Tributado uma vez que esqueça de um ou outro pra não válidar errado, pois, se existir um dos dois campos preenchido deve ter nos dois.

 

cEAN = '' vazio e cEANTrib = '789...' ou cEAN = '789...' vazio e cEANTrib = '' dispara um evento de erro, visto que se preencher um deve preencher o outro.

 

Então o seu exemplo causa um erro na validação, melhor dizendo não valida a regra de ter os dois preenchido mesmo que sejam distintos.

Editado 4 Agosto, 2018 por RickMao
complementando

[Gabriel Bonzanini]

3 horas atrás, RickMao disse:

Amigo o exemplo valida tanto o GTIN Tributado como o não Tributado uma vez que esqueça de um ou outro pra não válidar errado, pois, se existir um dos dois campos preenchido deve ter nos dois.

 

cEAN = '' vazio e cEANTrib = '789...' ou cEAN = '789...' vazio e cEANTrib = '' dispara um evento de erro, visto que se preencher um deve preencher o outro.

 

Então o seu exemplo causa um erro na validação, melhor dizendo não valida a regra de ter os dois preenchido mesmo que sejam distintos.

@RickMao grato pela atenção. Entendi a lógica. Mas, sendo assim, quando cEANTrib estiver em branco, vai passar desnecessariamente pela validação, pois obviamente um texto em branco não é um código de barras válido, concorda?

Vou efetuar mais alguns testes, tanto no ambiente de produção quanto no de homologação, e depois posto mais alguns detalhes aqui. Notei que as validações quanto a esse campo são diferentes (provavelmente pelo fato do ambiente de homologação já conter as alterações que entrarão em vigor no fim do ano).

Obrigado novamente pela atenção.

[BigWings]

13 horas atrás, Gabriel Bonzanini disse:

Em anexo o arquivo com a alteração, caso entendam que esteja correta.

Grato pela atenção.

pcnNFeW.pas

Me pareceu correta.

Enviado para o repositório.

Obrigado pela contribuição.