search

vindi / vindi-woocommerce

Plugin Vindi para pagamento recorrente no Woocommerce
https://vindi.com.br
GNU General Public License v3.0
6 stars 12 forks source link

fix: Duplicação de produtos dentro do WooCommerce #112

Closed devaguia closed 1 year ago

devaguia commented 2 years ago

Github Issue #98

Link: https://github.com/vindi/vindi-woocommerce/issues/98

O erro acontece devido a duplicidade de meta_values que indicam para o mesmo produto no sistema da VINDI.

Quando duplicamos um produto, o WooCommerce faz a duplicação de todo as informações do post, e isso inclui seus meta values, onde é salvo o id do produto no serviço da Vindi.

Por isso, ao duplicar um produto e tentar efetuar a compra com um dos dois produtos, uma inconsistência acontece e o plugin não consegue buscar o ID do produto na Vindi, e com isso não é possível finalizar a compra.

Para resolver isso, criamos uma função para verificar se o produto(WooCommerce) já tem um produto(VINDI) vinculado e se já existe outro produto(WooCommerce) com o id duplicado.

Caso já exista um id vinculado a outro produto, eu removo os metas que viram do produto antigo para que, durante a duplicação seja criado um novo produto na VINDI

Com isso, alteramos 2 arquivos dentro do plugin:

Descrição com imagens: https://boom-wakeboard-7c9.notion.site/Github-Issue-98-181e90683f484fa9be757698af433cac

codeclimate[bot] commented 2 years ago

Code Climate has analyzed commit 709dbb99 and detected 72 issues on this pull request.

Here's the issue category breakdown:

Category Count
Duplication 2
Style 70

View more on Code Climate.

laerte-guimaraes commented 2 years ago

A solução ficou bem legal e funcional 🎉

Estou usando aqui! A única modificação que fiz foi criar uma classe para compartilhar essa query 😉

thalita-nascimento-vindi commented 2 years ago

@aguiarrdev Boa Tarde Mateus, tudo bem?

Poderia nos enviar um commit vazio para destravar o nosso codeclimate?

Aguardamos.

thalita-nascimento-vindi commented 2 years ago

Boa Tarde @aguiarrdev.

Poderia nos enviar um commit vazio pra destravar nosso codeclimate? git commit --allow-empty -m "Empty-Commit"

Aguardamos.

thalita-nascimento-vindi commented 1 year ago

@aguiarrdev boa tarde, vc tem acesso aos detalhes do commit? image

tem alguns alertar de refatoração, consegue validar isso e retornar pra gente subir?

codeclimate[bot] commented 1 year ago

Code Climate has analyzed commit f66e2ee6 and detected 2 issues on this pull request.

Here's the issue category breakdown:

Category Count
Duplication 2

View more on Code Climate.