Merge pull request #94 from EduardoSorokin/fix-payment-methods-list

Fix payment methods list
pagseguro · Jun 15, 2018 · 00994a1 · 00994a1
2 parents 51c2bf2 + 2434331
commit 00994a1
Show file tree

Hide file tree

Showing 11 changed files with 109 additions and 108 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 Changelog
 ---------
+3.16.4
+- Fix: ajuste na regra css do botão "btn-pagseguro"
+- Fix: ajuste na exibição dos meios de pagamento do checkout transparente
+
 3.16.2
 - Fix: exibição das opções de banco no método de pagamento Débito Online
 

diff --git a/README.md b/README.md
@@ -33,7 +33,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.16.2.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.16.4.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;

diff --git a/UOL_PagSeguro-3.16.2.tgz b/UOL_PagSeguro-3.16.2.tgz
diff --git a/UOL_PagSeguro-3.16.4.tgz b/UOL_PagSeguro-3.16.4.tgz
diff --git a/UOL_PagSeguro.xml b/UOL_PagSeguro.xml
@@ -1,5 +1,5 @@
 <_>
-   <form_key>p14kBk6BRl5ky9IZ</form_key>
+   <form_key>t8LFePNMsJQzQEJE</form_key>
    <name>UOL_PagSeguro</name>
    <channel>community</channel>
    <version_ids>
@@ -9,9 +9,10 @@
    <description>Aceite mais de 25 meios de pagamentos com apenas 1 contrato. Seus clientes podem parcelar no cartão em até 18x e se preferir, você também pode oferecer parcelamento sem acréscimo.</description>
    <license>Apache Software License</license>
    <license_uri>http://opensource.org/licenses/apachepl.php</license_uri>
-   <version>3.16.2</version>
+   <version>3.16.4</version>
    <stability>stable</stability>
-   <notes>- Fix of the list view of banks in online debit</notes>
+   <notes>- Fixes: button &quot;btn-pagseguro&quot; css rule
+- Fixes: payment methods list - direct payment</notes>
    <authors>
       <name>
          <name>pagseguro</name>
@@ -44,8 +45,8 @@
             <max/>
          </max>
          <files>
-            <files>                                                                                                                            </files>
-            <files>                                                                                                                            </files>
+            <files>                                                                                                                                    </files>
+            <files>                                                                                                                                    </files>
          </files>
       </package>
       <extension>

diff --git a/app/code/community/UOL/PagSeguro/etc/config.xml b/app/code/community/UOL/PagSeguro/etc/config.xml
@@ -19,7 +19,7 @@ limitations under the License.
 <config>
     <modules>
         <UOL_PagSeguro>
-            <version>3.16.2</version>
+            <version>3.16.4</version>
         </UOL_PagSeguro>
     </modules>
     <global>

diff --git a/app/design/frontend/base/default/template/uol/pagseguro/form/boleto.phtml b/app/design/frontend/base/default/template/uol/pagseguro/form/boleto.phtml
@@ -1,29 +1,26 @@
-<style>
-  #dt_method_<?php echo $this->getMethodCode() ?> {
-    display: none;
-  }
-</style>
-
 <script>
   //<![CDATA[
-  var pagseguro_boleto_options = jQuery('#dt_method_<?php echo $this->getMethodCode() ?>')
-  paymentMethods()
-  var BoletoOptions = false
+  var pagseguro_boleto_dt = jQuery('#dt_method_<?php echo $this->getMethodCode() ?>');
+
+  if (!alreadyGetPaymentMethods) {
+    pagseguro_boleto_dt.hide();
+  }
 
-  pagseguroBoletoOptions = function (res) {
-    if (!BoletoOptions) {
-      BoletoOptions = true
-      if (!res['error']) {
-        if (res['paymentMethods']) {
-          if (res['paymentMethods'] && res['paymentMethods']['BOLETO'] && res['paymentMethods']['BOLETO']['options']['BOLETO']['status'] === 'AVAILABLE') {
-            pagseguro_boleto_options.show()
-          } else {
-            pagseguro_creditcard_dt.find('input').prop('disabled', true)
-          }
+  setBoletoSessionId("<?php echo $this->getBoletoSession(); ?>");
+
+  paymentMethods();
+
+  pagseguroBoletoOptions = function(res) {
+    if (!res['error']) {
+      if (res['paymentMethods']) {
+        if (res['paymentMethods'] && res['paymentMethods']['BOLETO'] && res['paymentMethods']['BOLETO']['options']['BOLETO']['status'] === 'AVAILABLE') {
+          pagseguro_boleto_dt.show();
+        } else {
+          pagseguro_creditcard_dt.find('input').prop('disabled', true);
         }
-      } else {
-        console.log(res['error'])
       }
+    } else {
+      console.log(res['error']);
     }
   }
   //]]>
@@ -41,9 +38,6 @@
             }
         }
     }
-
-    //set pagseguro session
-    setBoletoSessionId("<?= $this->getBoletoSession();?>");
 //]]>
 </script>
 

diff --git a/app/design/frontend/base/default/template/uol/pagseguro/form/creditcard.phtml b/app/design/frontend/base/default/template/uol/pagseguro/form/creditcard.phtml
@@ -1,51 +1,51 @@
-<style>
-  #dt_method_<?php echo $this->getMethodCode() ?> {
-    display: none;
-  }
-</style>
-
 <script>
   //<![CDATA[
-  var pagseguro_creditcard_dt = jQuery('#dt_method_<?php echo $this->getMethodCode() ?>')
-  var pagseguro_credit_card_options = jQuery('#pagseguro-credit-card-options')
-  paymentMethods()
-  var CreditcardOptions = false
+  var pagseguro_credit_card_dt = jQuery('#dt_method_<?php echo $this->getMethodCode() ?>');
+
+  if (!alreadyGetPaymentMethods) {
+    pagseguro_credit_card_dt.hide();
+  }
 
-  pagseguroCreditcardOptions = function (res) {
-    if (!CreditcardOptions) {
+  setCreditCardSessionId("<?= $this->getCreditCardSession(); ?>");
+
+  paymentMethods();
+
+  pagseguroCreditcardOptions = function(res) {
+      var pagseguro_credit_card_options = jQuery('#pagseguro-credit-card-options');
+
       pagseguro_credit_card_options.on('click', 'a', function () {
         pagseguro_credit_card_options.find('li').css('display', 'inline-flex')
         pagseguro_credit_card_options.find('li.more_cc').css('display', 'none')
       })
+
       if (!res['error']) {
         if (res['paymentMethods'] && res['paymentMethods']['CREDIT_CARD'] && res['paymentMethods']['CREDIT_CARD']['options']) {
-          var k = 0
-          pagseguro_credit_card_options.empty()
+          var k = 0;
+          pagseguro_credit_card_options.empty();
           jQuery.each(res['paymentMethods']['CREDIT_CARD']['options'], function (i, item) {
             if (item['status'] === 'AVAILABLE') {
-              ++k
+              ++k;
               if (k > 0) {
-                pagseguro_creditcard_dt.show()
+                pagseguro_credit_card_dt.show();
               }
               if (k <= 6) {
-                pagseguro_credit_card_options.append('<li style="display: inline-flex; padding: 0 3px 3px 0; margin: 0;"><img src="https://stc.pagseguro.uol.com.br' + item['images']['SMALL']['path'] + '"></li>')
+                pagseguro_credit_card_options.append('<li style="display: inline-flex; padding: 0 3px 3px 0; margin: 0;"><img src="https://stc.pagseguro.uol.com.br' + item['images']['SMALL']['path'] + '"></li>');
                 if (k === 6) {
-                  pagseguro_credit_card_options.append('<li style="display: inline-flex; padding: 0 3px 3px 0; margin: 0;" class="more_cc"><a >mais</a></li>')
+                  pagseguro_credit_card_options.append('<li style="display: inline-flex; padding: 0 3px 3px 0; margin: 0;" class="more_cc"><a >mais</a></li>');
                 }
               }
               if (k > 6) {
-                pagseguro_credit_card_options.append('<li style="display: none; padding: 0 3px 3px 0; margin: 0;"><img src="https://stc.pagseguro.uol.com.br' + item['images']['SMALL']['path'] + '"></li>')
+                pagseguro_credit_card_options.append('<li style="display: none; padding: 0 3px 3px 0; margin: 0;"><img src="https://stc.pagseguro.uol.com.br' + item['images']['SMALL']['path'] + '"></li>');
               }
             }
           })
           if (k === 0) {
-            pagseguro_onlinedebit_dt.find('input').prop('disabled', true)
+            pagseguro_onlinedebit_dt.find('input').prop('disabled', true);
           }
         }
       } else {
-        console.log(res['error'])
+        console.log(res['error']);
       }
-    }
   }
   //]]>
 </script>
@@ -62,9 +62,6 @@
             }
         }
     }
-
-    //set pagseguro credit card session
-    setCreditCardSessionId("<?= $this->getCreditCardSession(); ?>");
 //]]>
 </script>
 

diff --git a/app/design/frontend/base/default/template/uol/pagseguro/form/onlinedebit.phtml b/app/design/frontend/base/default/template/uol/pagseguro/form/onlinedebit.phtml
@@ -1,61 +1,55 @@
-<style>
- #dt_method_<?php echo $this->getMethodCode() ?> {
-   display: none;
- }
-</style>
-
 <script>
   //<![CDATA[
-  paymentMethods();
+  var pagseguro_onlinedebit_dt = jQuery('#dt_method_<?php echo $this->getMethodCode() ?>');
+  var pagseguro_onlinedebit_options = jQuery('#pagseguro-online-debit-options');
+
+  if (!alreadyGetPaymentMethods) {
+    pagseguro_onlinedebit_dt.hide();
+  } else {
+    pagseguro_onlinedebit_options.html(onlineDebitBanksHtml);
+  }
+
+  setOnlineDebitSessionId("<?php echo $this->getOnlineDebitSession(); ?>");
 
-  pagseguroOnlinedebitOptions = function (res) {
-    var pagseguro_onlinedebit_dt = jQuery('#dt_method_<?php echo $this->getMethodCode() ?>');
-    var pagseguro_online_debit_options = jQuery('#pagseguro-online-debit-options');
-    var OnlinedebitOptions = false;
+  paymentMethods();
 
-    if (!OnlinedebitOptions) {
-      if (!res['error']) {
-        if (res['paymentMethods'] && res['paymentMethods']['ONLINE_DEBIT'] && res['paymentMethods']['ONLINE_DEBIT']['options']) {
-          var k = 0;
-          pagseguro_online_debit_options.empty();
-          jQuery.each(res['paymentMethods']['ONLINE_DEBIT']['options'], function (i, item) {
-            if (item['status'] === 'AVAILABLE') {
-              ++k;
-              if (k > 0) {
-                pagseguro_onlinedebit_dt.show();
-              }
-              pagseguro_online_debit_options.append('<div><label>' +
-                '<input class="input-radio required-entry" type="radio" id="debitbankName" name="debitbankName" value="' + i + '" required onclick="validateDebitBankName(this)">'
-                + item['displayName'] + '</label></div>');
-            }
-          });
-          if (k === 0) {
-            pagseguro_onlinedebit_dt.find('input').prop('disabled', true);
+  pagseguroOnlinedebitOptions = function(res) {
+    if (!res['error']) {
+      if (res['paymentMethods'] && res['paymentMethods']['ONLINE_DEBIT'] && res['paymentMethods']['ONLINE_DEBIT']['options']) {
+        onlineDebitBanksHtml = ''; // variável declarada no arquivo "direct-payments.js" (para funcionar com o plugin IWD\OPC)
+        var onlineDebitOptions = res['paymentMethods']['ONLINE_DEBIT']['options'];
+        jQuery.each(onlineDebitOptions, function (i, item) {
+          if (item['status'] === 'AVAILABLE') {
+            onlineDebitBanksHtml += '<div><label><input class="input-radio required-entry" type="radio" id="debitbankName" name="debitbankName" value="' + i + '" required onclick="validateDebitBankName(this)">'
+              + item['displayName'] + '</label></div>';
           }
+        });
+        if (onlineDebitBanksHtml !== '') {
+          pagseguro_onlinedebit_dt.show();
+          pagseguro_onlinedebit_options.html(onlineDebitBanksHtml);
+        } else {
+          pagseguro_onlinedebit_dt.find('input').prop('disabled', true);
         }
-      } else {
-        console.log(res['error']);
       }
+    } else {
+      console.log(res['error']);
     }
   }
   //]]>
 </script>
 
 <script type="text/javascript">
 //<![CDATA[
-    // get value from one step checkout guest (if it is enabled) document to be used in pagseguro direct payment
-    if(document.getElementById('billing:taxvat') !== null && document.getElementById('billing:taxvat').onchange === null) {
-        document.getElementById('billing:taxvat').onchange = function (){
-            var documents =  document.querySelectorAll('.pagseguro-document');
-            var i;
-            for (i=0; i < documents.length; i++) {
-                documents[i].value = document.getElementById('billing:taxvat').value;
-            }
-        }
+  // get value from one step checkout guest (if it is enabled) document to be used in pagseguro direct payment
+  if(document.getElementById('billing:taxvat') !== null && document.getElementById('billing:taxvat').onchange === null) {
+    document.getElementById('billing:taxvat').onchange = function (){
+      var documents =  document.querySelectorAll('.pagseguro-document');
+      var i;
+      for (i=0; i < documents.length; i++) {
+        documents[i].value = document.getElementById('billing:taxvat').value;
+      }
     }
-
-    //set pagseguro session
-    setOnlineDebitSessionId("<?= $this->getOnlineDebitSession();?>");
+  }
 //]]>
 </script>
 

diff --git a/skin/frontend/base/default/uol/pagseguro/css/styles.css b/skin/frontend/base/default/uol/pagseguro/css/styles.css
@@ -2079,8 +2079,7 @@ a.btn-pagseguro:focus {
 }
 
 /* btn-pagseguro */
-button:not(.primary),
-.btn-pagseguro {
+.btn-pagseguro:not(.primary) {
     color: #4F7743;
     font-size: 1.2em;
     white-space: nowrap;
@@ -2174,4 +2173,4 @@ input.form-control, .full-width {
 .pagseguro-payment-retry-button-set {
     padding-top: 1rem;
     text-align: center;
-}
+}
diff --git a/skin/frontend/base/default/uol/pagseguro/js/direct-payment.js b/skin/frontend/base/default/uol/pagseguro/js/direct-payment.js
@@ -329,19 +329,31 @@ function creditCardCodeMask(code) {
 /**
  * @type {boolean}
  */
-var alreadyGetPaymentMethods = false
+var alreadyGetPaymentMethods = false;
+
+/**
+ * @type {string}
+ */
+var onlineDebitBanksHtml = '';
 
 /**
  *
  *
  */
-function paymentMethods () {
+function paymentMethods() {
   if (!alreadyGetPaymentMethods) {
+    alreadyGetPaymentMethods = true;
     PagSeguroDirectPayment.getPaymentMethods({
-      success: function (res) {
-        pagseguroBoletoOptions(res)
-        pagseguroCreditcardOptions(res)
-        pagseguroOnlinedebitOptions(res)
+      success: function(res) {
+        if (typeof pagseguroCreditcardOptions === "function") {
+          pagseguroCreditcardOptions(res);  
+        }
+        if (typeof pagseguroBoletoOptions === "function") {
+          pagseguroBoletoOptions(res);  
+        }
+        if (typeof pagseguroOnlinedebitOptions === "function") {
+          pagseguroOnlinedebitOptions(res);  
+        }
       }
     })
   }