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.

Sign up for GitHub

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

Jump to bottom

Frete de mil reais exibido por 1 real #32

Closed
jarjar123 opened this issue Nov 17, 2014 · 0 comments · Fixed by #40 or #47
Closed

Frete de mil reais exibido por 1 real #32

jarjar123 opened this issue Nov 17, 2014 · 0 comments · Fixed by #40 or #47
Assignees
@pedro-teixeira
Milestone
v4.4.0

Comments

@jarjar123
Copy link
Contributor

Reparei que alguns pedidos grandes e pesados apareciam com frete de 1 real ou 2 reais.
Ao verificar o conteúdo do webservice notei que os valores superavam os mil reais.
Aparentemente o erro está na leitura do valor do XML. O separador decimal está sendo convertido corretamente, mas o separador de milhar não está sendo removido. E o método floatval, executado em seguida, acaba convertendo valores de mil reais para 1 real.
@pedro-teixeira pedro-teixeira added this to the v4.5.0 milestone Nov 17, 2014
@pedro-teixeira pedro-teixeira self-assigned this Nov 17, 2014
jarjar123 pushed a commit to jarjar123/correios that referenced this issue Dec 12, 2014
@rafaelpatro
pedro-teixeira#32 Corrige a leitura do valor de frete
53c67ed
@jarjar123 jarjar123 mentioned this issue Dec 12, 2014
Closed
@pedro-teixeira pedro-teixeira modified the milestones: v4.4.0, v4.5.0 Dec 14, 2014
This was referenced Dec 15, 2014
Merged
Closed
pedro-teixeira added a commit that referenced this issue Dec 15, 2014
@pedro-teixeira
Merge pull request #40 from jarjar123/correcao_frete_mil_reais
afe46b2 
#32 Corrige a leitura do valor de frete
jarjar123 pushed a commit to jarjar123/correios that referenced this issue Dec 15, 2014
@rafaelpatro
Revert "pedro-teixeira#32 Corrige a leitura do valor de frete"
830bd22 
This reverts commit 53c67ed.
@jarjar123 jarjar123 mentioned this issue Dec 22, 2014
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pedro-teixeira pedro-teixeira

Labels
None yet
Projects
None yet
Milestone
v4.4.0
2 participants
@pedro-teixeira @jarjar123