We will work on Apr 26th (Saturday) and will be off from Apr 30th (Wednesday) until May 2nd (Friday) for public holiday in our country

Commit cfd948a2 authored by Alessio Zampatti's avatar Alessio Zampatti

Add try/catch on originKeys call, add error message for card methods on frontend and backend

parent cee5877b
...@@ -1395,7 +1395,12 @@ class Data extends AbstractHelper ...@@ -1395,7 +1395,12 @@ class Data extends AbstractHelper
$client = $this->initializeAdyenClient($storeId); $client = $this->initializeAdyenClient($storeId);
$service = $this->createAdyenCheckoutUtilityService($client); $service = $this->createAdyenCheckoutUtilityService($client);
$response = $service->originKeys($params); try {
$response = $service->originKeys($params);
}
catch(\Exception $e){
$this->adyenLogger->error($e->getMessage());
}
$originKey = ""; $originKey = "";
......
...@@ -33,6 +33,7 @@ $ccExpYear = $block->getInfoData('cc_exp_year'); ...@@ -33,6 +33,7 @@ $ccExpYear = $block->getInfoData('cc_exp_year');
<fieldset class="admin__fieldset payment-method" id="payment_form_<?php /* @noEscape */ <fieldset class="admin__fieldset payment-method" id="payment_form_<?php /* @noEscape */
echo $code; ?>" style="display:none"> echo $code; ?>" style="display:none">
<span id="noApiKey" class="message message-error error" style="visibility: hidden"><?php echo $block->escapeHtml(__('Please configure an API Key in your Adyen Required Settings'));?></span>
<div class="field admin__field"> <div class="field admin__field">
<label for="<?php /* @noEscape */ <label for="<?php /* @noEscape */
echo $code; ?>_cc_owner"> echo $code; ?>_cc_owner">
...@@ -96,6 +97,10 @@ echo $code; ?>" style="display:none"> ...@@ -96,6 +97,10 @@ echo $code; ?>" style="display:none">
*/ */
var secureFieldsInitialize = function () { var secureFieldsInitialize = function () {
if (!"<?php echo $block->getCheckoutOriginKeys(); ?>") {
document.getElementById('noApiKey').style="visibility: visible; display:inline-block";
return;
}
var cardNode = document.getElementById("cardContainer-<?php /* @noEscape */ echo $code; ?>"); var cardNode = document.getElementById("cardContainer-<?php /* @noEscape */ echo $code; ?>");
var checkout = new AdyenCheckout({ var checkout = new AdyenCheckout({
......
...@@ -71,6 +71,9 @@ define( ...@@ -71,6 +71,9 @@ define(
renderSecureFields: function () { renderSecureFields: function () {
var self = this; var self = this;
self.placeOrderAllowed(false); self.placeOrderAllowed(false);
if (!self.getOriginKey()) {
return;
}
installments.setInstallments(0); installments.setInstallments(0);
......
...@@ -206,6 +206,9 @@ define( ...@@ -206,6 +206,9 @@ define(
renderSecureCVC: function () { renderSecureCVC: function () {
var self = this; var self = this;
if (!self.getOriginKey()) {
return;
}
var oneClickCardNode = document.getElementById('cvcContainer-' + self.value); var oneClickCardNode = document.getElementById('cvcContainer-' + self.value);
var checkout = new AdyenCheckout({ var checkout = new AdyenCheckout({
......
...@@ -39,6 +39,10 @@ ...@@ -39,6 +39,10 @@
<div class="payment-method-content"> <div class="payment-method-content">
<!-- ko ifnot: (getOriginKey()) -->
<span class="message message-error error"><!-- ko text: $t('Please configure an API Key in your Adyen Required Settings')--><!-- /ko --></span>
<!--/ko-->
<div class="payment-method-billing-address"> <div class="payment-method-billing-address">
<!-- ko foreach: $parent.getRegion(getBillingAddressFormName()) --> <!-- ko foreach: $parent.getRegion(getBillingAddressFormName()) -->
<!-- ko template: getTemplate() --><!-- /ko --> <!-- ko template: getTemplate() --><!-- /ko -->
......
...@@ -45,6 +45,10 @@ ...@@ -45,6 +45,10 @@
</div> </div>
<div class="payment-method-content"> <div class="payment-method-content">
<!-- ko ifnot: (getOriginKey()) -->
<span class="message message-error error"><!-- ko text: $t('Please configure an API Key in your Adyen Required Settings')--><!-- /ko --></span>
<!--/ko-->
<!-- ko foreach: $parent.getRegion(getMessageName()) --> <!-- ko foreach: $parent.getRegion(getMessageName()) -->
<!-- ko template: getTemplate() --><!-- /ko --> <!-- ko template: getTemplate() --><!-- /ko -->
<!--/ko--> <!--/ko-->
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment