Skip to content

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

Open
zhiru wants to merge 1 commit into
flyingluscas:masterfrom
aireset:master
Open

Conversão de valor para float está errada#40
zhiru wants to merge 1 commit into
flyingluscas:masterfrom
aireset:master

Conversation

@zhiru

@zhiru zhiru commented May 8, 2021

Copy link
Copy Markdown

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
Copy Markdown
Owner

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

@zhiru

zhiru commented May 25, 2021

Copy link
Copy Markdown
Author

vixe sou um zero a esquerda com testes kk

@JosuehElelyon

Copy link
Copy Markdown

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

zhiru commented Apr 28, 2022

Copy link
Copy Markdown
Author

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.

3 participants