Fix para frete com desconto/grátis em áreas com restrição de entrega

[alex-andrade]

Correção de bug ao utilizar promoção de frete grátis em áreas com restrição de entrega.
Durante testes na versão 4.5.0, se o cliente utilizar um cupom/regra para permitir frete grátis, e se, este cliente tiver em área de restrição, não é dado desconto. O desconto aparece apenas em áreas sem restrições.
Para fins de testes, este foi um dos ceps que utilizei para testar 04843-640 (com restrição de entrega)

[pedro-teixeira]

Esse código não está fazendo sentido, se podemos ter cotação com erro só precisa retirar esse if. @jarjar123, alguma sugestão aqui?

[jarjar123]

Nenhuma sugestão. Remover o IF é melhor mesmo.
Em 20/07/2015 19:04, "Pedro Teixeira" notifications@github.com escreveu:

In
app/code/community/PedroTeixeira/Correios/Model/Carrier/CorreiosMethod.php
#100 (comment)
:

@@ -114,6 +114,8 @@ public function collectRates(Mage_Shipping_Model_Rate_Request $request)

     $this->_filterMethodByItemRestriction();
     if ($this->_getQuotes()->getError()) {

•        //Fix to allow discounts on delivery areas with restrictions
  

Esse código não está fazendo sentido, se podemos ter cotação com erro só
precisa retirar esse if. @jarjar123 https://github.com/jarjar123,
alguma sugestão aqui?

—
Reply to this email directly or view it on GitHub
https://github.com/pedro-teixeira/correios/pull/100/files#r35049885.

[alex-andrade]

No caso, se for remover apenas o if, em localhost tive problemas, não apresentando mais cotação no carrinho.

[pedro-teixeira]

@alex-andrade, não faz sentido, com ou sem if é exatamente o mesmo código.

[jarjar123]

O erro procede.
Mas comigo o patch não corrigiu nada.
Dá uma conferida no cache e no compilador. Desativa tudo.

[alex-andrade]

@pedro-teixeira esquece que eu falei, realmente não faz sentido hehe. @jarjar123, aqui deu certo, entretanto vou criar uma instalação limpa para novos testes.

[jarjar123]

Sorry! Funcionou sim @alex-andrade
Duas coisas importantes pra vc concluir sua PR.

1. Manter Exibir Método ativo.
   
2. Remover o condicional IF, mas não esquecer de manter a linha $this->_getQuotes();, senão não tem cotação.

[alex-andrade]

Perfeito @jarjar123 , o codigo simplificado irá ficar assim:

$this->_filterMethodByItemRestriction();

//Show Quotes
$this->_getQuotes();

// Use descont codes
$this->_updateFreeMethodQuote($request);

return $this->_result;

Só uma questão, sou novo no Git, faço uma nova PR ou edita esta mesmo?

[jarjar123]

Eu editaria a mesma PR.
E como a alteração envolve apenas 2 linhas, ainda faria isso no editor online do github.

[pedro-teixeira]

@jarjar123 esse está aprovado?

[jarjar123]

Bom dia.
Sim, a PR está Ok