Corrige envio de frete para faturas avulsas

[st3llaris]

O que mudou

Foi corrigido o comportamento que impediam que compras avulsas com frete pudessem ser realizadas.

Motivação

Na issue #83, foi reportado um Internal Server Error ao finalizar compras avulsas.

Solução proposta

Estamos corrigindo esse comportamento, pois anteriormente a function VindiHelpers::get_matching_subscription($this->order, $order_item); estava retornando null para compras avulsas e, por conta disso, estava ocorrendo uma exceção na chamada da function get_shipping_method().

Como testar

• Crie um produto simples no WooCommerce
• Configure um frete para esse produto
• Realize a compra e veja se foi feita normalmente e o produto frete foi criado na Vindi como esperado

[luiz-dalcico-vindi]

A motivação desse trecho de código veio desse PR (#46)

Para garantir que tudo vai funcionar, o cenário de criar múltiplas assinaturas com múltiplos valores de fretes distintos deve ser coberto.

@victorsinged você pode fazer um check nessa função:

if ($this->is_subscription_type($product))

se for true, deixa como está hoje. Caso contrário, aciona a tua mudança sugerida no PR.

[leo-araujo]

@laerte-guimaraes @victorsinged @luizdalcicovindi
Realizei os testes após o último commit, acredito que agora esteja funcionando conforme esperado.

Segue abaixo teste realizado:

Compra de múltiplas assinaturas no mesmo carrinho com fretes diferentes:

Tela de finalização da compra:

Assinatura semestral:

Assinatura mensal:

Conseguem ver algum ponto adicional que possa ter impacto e precisa ser validado?

[codeclimate[bot]]

Code Climate has analyzed commit 5dbb0bc7 and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category	Count
Complexity	1

View more on Code Climate.