Corrige bill_items inválido quando a quantidade não é um valor inteiro#154
Open
stenioanibal wants to merge 2 commits intovindi:masterfrom
Open
Corrige bill_items inválido quando a quantidade não é um valor inteiro#154stenioanibal wants to merge 2 commits intovindi:masterfrom
stenioanibal wants to merge 2 commits intovindi:masterfrom
Conversation
Para produtos em que uma caixa, por exemplo, representa 1.5 metros quadrados e o múltiplo de venda ocorre em cima dessa quantidade, o módulo não consegue processar o pagamento pelo fato de que a API apenas aceita um número inteiro na quantidade do produto.
leo-araujo
reviewed
Jul 31, 2020
includes/class-vindi-payment.php
Outdated
| $discount_item = array( | ||
| 'type' => 'discount', | ||
| 'vindi_id' => $item['id'], | ||
| 'vindi_id' => ceil($item['id']), |
There was a problem hiding this comment.
@stenioanibal Boa noite Stenio, tudo bem? :)
O vindi_id neste caso é utilizado para identificação do cupom de desconto na Vindi. Caso o objetivo seja arredondar a quantidade enviada nas cobranças avulsas a alteração deve ser realizada na função build_product_items_for_bill, arredondando o valor recebido no parâmetro 'quantity' contido na variável $item.
Author
There was a problem hiding this comment.
Boa tarde @LeoAraujo01 !
Falha minha! De alguma forma, alterei errado aqui :(. No meu local, a alteração está correta.
Author
There was a problem hiding this comment.
Fiz um novo commit desfazendo o erro e aplicando o arredondamento no local correto. Novamente, peço desculpas pela desatenção que tive.
O arredondamento havia sido inserido no local errado aqui no github. Este commit remove esse problema e coloca o mesmo no local correto, arredondando a quantidade dos produtos para a bill.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Para produtos em que uma caixa, por exemplo, representa 1.5 metros quadrados e o múltiplo de venda ocorre em cima dessa quantidade, o módulo não consegue processar o pagamento pelo fato de que a API apenas aceita um número inteiro na quantidade do produto.
O que mudou
Descreva em poucas palavras o que muda para usuários do projeto. Esta descrição será utilizada para publicar o changelog.
Motivação
Descreva a motivação deste PR. Explique qual é o problema que queremos resolver.
Solução proposta
Descreva a solução proposta para resolver o problema mencionado na seção "Motivação", incluindo os detalhes técnicos necessários para entender essa solução.
Como testar
Descreva detalhadamente, passo a passo, como testar este PR. Especifique requisitos, como por exemplo, acesso prévio em alguma ferramenta.