-
Notifications
You must be signed in to change notification settings - Fork 12
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
Melhoria na criação de pedidos com produtos variáveis #180
Merged
leticia-santos-vindi
merged 113 commits into
vindi:master
from
lucastgama:apiki_produto_variavel
Jul 15, 2024
Merged
Melhoria na criação de pedidos com produtos variáveis #180
leticia-santos-vindi
merged 113 commits into
vindi:master
from
lucastgama:apiki_produto_variavel
Jul 15, 2024
Conversation
This file contains 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
…com as novas versões do PHP 8.3 e do WC 8.5.2
…remainder na condiçao quando o resto menor ou igual ao preço total, antes tinha um else e esse else foi removido para que o code climate pare com a issue
…era remainder - fullprice agora é apenas fullprice
… duplicação do desconto e na aplicação incorreta do desconto percentual em produtos simples.
…alizar estado da assinatura
…atico na função config_discount_cycles
…iar nas regras do code climate
…iar nas regras do code climate
…a pegar o valor do id da vindi de seus produtos variaveis
…criação de uma nova função chamada build_product_from_order_item_bill
Code Climate has analyzed commit 70c4d68 and detected 0 issues on this pull request. View more on Code Climate. |
leticia-santos-vindi
approved these changes
Jul 15, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
O que mudou
Na função
build_product_from_order_item
do arquivoPaymentProcessor.php
, foram introduzidas duas novas condições. Uma delas valida se o item é um produto variável; nesse caso, o produto variável receberá o ID do produto "pai", pois o sistema da Vindi registra apenas um produto variável como pai. Além disso, foi implementada uma condição para reduzir o uso de rotas, garantindo que o ID do produto da Vindi seja buscado no banco de dados em vez de pela rotafindProductByCode
. Caso o ID não seja encontrado no banco de dados, a rota é utilizada, otimizando assim o tempo de requisições quando não necessárias.Motivação
O cliente notificou que ao realizar compras com produtos variáveis, o sistema não estava identificando corretamente o ID do produto na Vindi, resultando em uma mensagem de erro que impedia a conclusão do pedido.
Solução proposta
Para resolver esse problema, foi necessário modificar o ID do produto enviado pela rota, verificando que o mesmo ID já estava registrado no banco de dados do WordPress. Dessa forma, não era mais necessário utilizar a rota para validação. No entanto, por questões de segurança, foi implementada uma validação para verificar se esse ID está registrado no banco de dados. Caso não esteja, a rota da Vindi é utilizada para essa finalidade.
Como testar