Merge pull request #63 from pagseguro/desenvolvimento

fix: errors in checkout page when pagseguro credentials are invalid

Author: s2it-moscou

CHANGELOG.md

@@ -1,5 +1,9 @@
 Changelog
 ---------
+3.2.1
+- Corrigido bug ao exibir o formulário de checkout quando as credencias do PagSeguro eram inválidas.
+  - Adicionada mensagem de credenciais inválidas ao salvar as configurações no admin e ocultando os meios de pagamento do PagSeguro na tela de pagamento caso as credenciais salvas sejam inválidas.
+
 3.2.0
 - Alterado fluxo do checkout transparente para o fluxo padrão do Magento (onepage)
 - Atualizada tela de cofiguração do módulo (admin), adicionando categorias e possibilitando habilitar métodos transparentes (boleto, débito online e cartão de crédito) individualmente

README.md

@@ -30,7 +30,7 @@ Com o módulo instalado e configurado, você pode pode oferecer o PagSeguro como
 
 - Certifique-se de que não há instalação de outros módulos para o PagSeguro em seu sistema;
 - Caso utilize a compilação do Magento, desative-a e limpe-a *(Sistema -> Ferramentas -> Compilação)*;
-- Baixe a última versão do módulo **[nesse link](https://github.com/pagseguro/magento/raw/master/UOL_PagSeguro-3.2.0.tgz)** ou então baixe o repositório como arquivo zip através do botão do GitHub;
+- Baixe a última versão do módulo **[nesse link](https://github.com/pagseguro/magento/raw/master/UOL_PagSeguro-3.2.1.tgz)** ou então baixe o repositório como arquivo zip através do botão do GitHub;
 - Na área administrativa do seu Magento, acesse o menu *Sistema/System -> Magento Connect -> Magento Connect Manager*. Caso tenha uma versão anterior do módulo instalada faça a remoção agora;
 - No Magento Connect Manger, dentro da seção Direct package file upload, clique em **Escolher arquivo/Choose file**, selecione o arquivo UOL_PagSeguro-x.x.x.tgz (baixado anteriormente), clique no botão de upload e acompanhe a instalação do módulo no console da página;
 - Caso utilize a compilação, volte para a área administrativa do Magento, ative-a e execute-a novamente;

UOL_PagSeguro-3.2.0.tgz

@@ -137,17 +137,13 @@ private function removeEnvironmentNotice($table, $id)
      */
     final public function checkCredentials()
     {
-        $date = new DateTime ("now");
-        $date->setTimezone(new DateTimeZone ("America/Sao_Paulo"));
-        $date->sub(new DateInterval ('P1D'));
-        $date->setTime(00, 00, 00);
-        $date = $date->format("Y-m-d\TH:i:s");
+        $yesterday = new DateTime('yesterday'); 
         $useCache = Mage::app()->useCache();
         if ($useCache['config']) {
             Mage::app()->getCacheInstance()->flush();
         }
         try {
-            $this->webserviceHelper()->getTransactionsByDate(1, 1, $date);
+            $this->webserviceHelper()->getTransactionsByDate(1, 1, $yesterday->format('Y-m-d\TH:i'));
             Mage::getConfig()->saveConfig('uol_pagseguro/store/credentials', 1);
         } catch (Exception $e) {
             Mage::getConfig()->saveConfig('uol_pagseguro/store/credentials', 0);

UOL_PagSeguro-3.2.1.tgz

@@ -114,7 +114,7 @@ public function getTransactionsByDate($page, $maxPageResults, $initialDate)
                 array('initial_date' => $initialDate, 'page' => $page, 'max_per_page' => $maxPageResults)
             );
         } catch (Exception $e) {
-            if (trim($e->getMessage()) == '[HTTP 401] - UNAUTHORIZED') {
+            if (trim($e->getMessage()) == '[HTTP 401] - UNAUTHORIZED' || $e->getCode() == 401) {
                 throw new Exception($e->getMessage());
             }
         }

app/code/community/UOL/PagSeguro/Helper/Data.php

@@ -52,4 +52,14 @@ public function getOrderPlaceRedirectUrl()
         return Mage::getUrl('pagseguro/payment/request');
     }
 
+    /**
+     * If pagseguro credentials are invalid, disable payment method
+     * @param Mage_Sales_Model_Quote $quote
+     * @return bool
+     */
+    public function isAvailable($quote = null) 
+    {
+        $enablePaymentMethod = (Mage::getStoreConfig("uol_pagseguro/store/credentials") == 1) ? true : false;
+        return parent::isAvailable($quote) && $enablePaymentMethod;
+    }
 }

app/code/community/UOL/PagSeguro/Helper/Webservice.php

@@ -87,4 +87,15 @@ public function getOrderPlaceRedirectUrl()
     {
         return Mage::getUrl('pagseguro/payment/request');
     }
+    
+    /**
+     * If pagseguro credentials are invalid, disable payment method
+     * @param Mage_Sales_Model_Quote $quote
+     * @return bool
+     */
+    public function isAvailable($quote = null) 
+    {
+        $enablePaymentMethod = (Mage::getStoreConfig("uol_pagseguro/store/credentials") == 1) ? true : false;
+        return parent::isAvailable($quote) && $enablePaymentMethod;
+    }
 }

app/code/community/UOL/PagSeguro/Model/Boleto.php

@@ -65,7 +65,15 @@ public function adminSystemConfigPaymentSave()
         }
 
         if (Mage::getStoreConfig("payment/pagseguro/email") && Mage::getStoreConfig("payment/pagseguro/token")) {
-            Mage::helper('pagseguro')->checkCredentials();
+            try {
+                Mage::helper('pagseguro')->checkCredentials();
+            } catch (Exception $exc) {
+                Mage::getSingleton('core/session')->addError(
+                    'PagSeguro: Credenciais (EMAIL ou TOKEN) inválidas para o AMBIENTE selecionado.'
+                        . 'Não será possível utilizar nenhum tipo de checkout enquanto não '
+                        . 'forem salvas credenciais válidas.'
+                );
+            }
         } else {
             throw new Exception("Certifique-se de que o e-mail e token foram preenchidos.");
         }

app/code/community/UOL/PagSeguro/Model/CreditCard.php

@@ -66,4 +66,14 @@ public function getOrderPlaceRedirectUrl()
         return Mage::getUrl('pagseguro/payment/request');
     }
 
+    /**
+     * If pagseguro credentials are invalid, disable payment method
+     * @param Mage_Sales_Model_Quote $quote
+     * @return bool
+     */
+    public function isAvailable($quote = null) 
+    {
+        $enablePaymentMethod = (Mage::getStoreConfig("uol_pagseguro/store/credentials") == 1) ? true : false;
+        return parent::isAvailable($quote) && $enablePaymentMethod;
+    }
 }