Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversão de valor para float está errada #40

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zhiru
Copy link

@zhiru zhiru commented May 8, 2021

Atualmente na linha 360 do arquivo faz a conversão para valores acima do milhar de forma errada.

Com essa atualização é primeiramente removido o . do milhar no formato brasileiro, depois converte a vírgula em ponto, além disso foi adicionado um preg_replace para manter apenas numeros, virgulas ou pontos

Motivação e contexto

Correção de um bug quando o frete fica acima de R$ 1.000,00

Como isso foi testado?

Testei em três e-commerces.

Essa imagem é referente ao retorno da chamada do correio
image

Esta imagem demonstra o erro que possui na função transformCorreiosService
image

Esta imagem o resultado da correção:
image

Atualmente na linha 360 do arquivo faz a conversão para valores acima do milhar de forma errada.

Com essa atualização é primeiramente removido o . do milhar no formato brasileiro, depois converte a vírgula em ponto, além disso foi adicionado um preg_replace para manter apenas numeros, virgulas ou pontos
@flyingluscas
Copy link
Owner

@zhiru Boooa! Só vou te pedir para escrever alguns testes validando que o problema foi corrigido ;)

@zhiru
Copy link
Author

zhiru commented May 25, 2021

vixe sou um zero a esquerda com testes kk

@JosuehElelyon
Copy link

Estou tentando usar a lib, porem os valores que me é retornado, estão bem diferentes dos quais eu consulto direto no site dos correios.
Mesmo usando os mesmo parametros( largura, altura, comprimento, peso), saberia me informar o porque da diferença ?
Grato.

@zhiru
Copy link
Author

zhiru commented Apr 28, 2022

Estou tentando usar a lib, porem os valores que me é retornado, estão bem diferentes dos quais eu consulto direto no site dos correios. Mesmo usando os mesmo parametros( largura, altura, comprimento, peso), saberia me informar o porque da diferença ? Grato.

Não poderia, se os ceps são os mesmos usados, se todos os parametros passados são iguais, então os valores tem que dar o mesmo resultado. Agora se for comparar com uso de contratos, ferramentas como melhor envio, frenet, dai sim vai dar diferença.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants