From b4fee849b0182c632424a808b265fbe9da1be47f Mon Sep 17 00:00:00 2001
From: Matias Vasquez <m.vasquezac@gmail.com>
Date: Thu, 7 Nov 2024 12:44:09 -0300
Subject: [PATCH 01/11] feat: add style for secondary button

---
 plugin/css/tbk.css | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

diff --git a/plugin/css/tbk.css b/plugin/css/tbk.css
index e15ed8c..1b7673f 100644
--- a/plugin/css/tbk.css
+++ b/plugin/css/tbk.css
@@ -330,6 +330,21 @@
     box-shadow: 0 2px 4px rgba(0, 0, 0, 0.1);
 }
 
+.button.tbk-button-secondary {
+    background: white;
+    border-color: #D5006C;
+    color: #D5006C;
+    transition: background-color 0.3s ease;
+}
+
+.button.tbk-button-secondary:hover,
+.button.tbk-button-secondary:focus {
+    background: white;
+    border-color: #D5006C;
+    color: #D5006C;
+    box-shadow: 0 2px 4px rgba(0, 0, 0, 0.1);
+}
+
 .log-container {
     width: calc(100vw - 480px);
     max-height: 600px;

From 36dc248454fd2d519f82814e5ff454847f2f2226 Mon Sep 17 00:00:00 2001
From: Matias Vasquez <m.vasquezac@gmail.com>
Date: Thu, 7 Nov 2024 12:46:47 -0300
Subject: [PATCH 02/11] feat: add id for logs elements

---
 plugin/templates/admin/log.php | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/plugin/templates/admin/log.php b/plugin/templates/admin/log.php
index d2fbfb9..8dbcf09 100644
--- a/plugin/templates/admin/log.php
+++ b/plugin/templates/admin/log.php
@@ -7,7 +7,7 @@
 $lineCountTitle = "Cantidad de líneas que posee el último archivo de registro creado";
 ?>
 
-<div class="tbk-box">
+<div class="tbk-box" id="logs-container">
     <h3 class="tbk_title_h3">Información de Registros</h3>
     <div class="tbk-plugin-info-container">
         <div class="info-column">
@@ -35,7 +35,7 @@
                 <input type="hidden" name="page" value="transbank_webpay_plus_rest">
                 <input type="hidden" name="tbk_tab" value="logs">
 
-                <select class="select label" name="log_file" <?= !$folderHasLogs ? "disabled" : '' ?>>
+                <select class="select label" name="log_file" id="log_file" <?= !$folderHasLogs ? "disabled" : '' ?>>
                     <?php
                     $options = '';
 

From 75e4bc61d4716d78e81ca3793c89e6610173fe27 Mon Sep 17 00:00:00 2001
From: Matias Vasquez <m.vasquezac@gmail.com>
Date: Thu, 7 Nov 2024 12:47:54 -0300
Subject: [PATCH 03/11] feat: add download button

---
 plugin/templates/admin/log.php | 1 +
 1 file changed, 1 insertion(+)

diff --git a/plugin/templates/admin/log.php b/plugin/templates/admin/log.php
index 8dbcf09..61a2634 100644
--- a/plugin/templates/admin/log.php
+++ b/plugin/templates/admin/log.php
@@ -58,6 +58,7 @@
                 <input type="submit" class="button button-primary tbk-button-primary"
                     <?= !$folderHasLogs ? 'disabled' : '' ?>
                     value="Ver">
+                <button class="button button-secondary tbk-button-secondary" id="btnDownload">Descargar</button>
             </form>
         </div>
     </div>

From 2c469993323017ef98914b06a596a8fd81cdee95 Mon Sep 17 00:00:00 2001
From: Matias Vasquez <m.vasquezac@gmail.com>
Date: Thu, 7 Nov 2024 12:51:01 -0300
Subject: [PATCH 04/11] feat: add function to check download permission

---
 plugin/webpay-rest.php | 47 ++++++++++++++++++++++++++++++++++++++++--
 1 file changed, 45 insertions(+), 2 deletions(-)

diff --git a/plugin/webpay-rest.php b/plugin/webpay-rest.php
index dc4aa9e..ece643e 100644
--- a/plugin/webpay-rest.php
+++ b/plugin/webpay-rest.php
@@ -41,8 +41,7 @@
 add_action('wp_loaded', 'woocommerceTransbankInit');
 add_action('admin_init', 'on_transbank_rest_webpay_plugins_loaded');
 
-add_action('wp_ajax_check_connection', ConnectionCheck::class.'::check');
-add_action('wp_ajax_check_exist_tables', TableCheck::class.'::check');
+add_action('wp_ajax_checkCanDownloadLogFile', 'checkCanDownloadLogFile');
 add_action('wp_ajax_get_transaction_status', [new TransactionStatusController(), 'getStatus']);
 add_action('woocommerce_before_cart', 'transbank_rest_before_cart');
 
@@ -274,3 +273,47 @@ function () {
         }
     );
 }
+
+function fileExistsInFolder($fileName, $folderPath)
+{
+    $filesInFolder = array_filter(scandir($folderPath), function ($file) use ($folderPath) {
+        return is_file($folderPath . '/' . $file);
+    });
+
+    return in_array($fileName, array_values($filesInFolder));
+}
+
+function checkCanDownloadLogFile()
+{
+    $response = [
+        'canDownload' => false,
+        'downloadUrl' => '',
+        'error' => ''
+    ];
+
+    if (!is_user_logged_in()) {
+        $response['error'] = 'Debes iniciar sesión para poder descargar';
+        wp_send_json_error($response);
+    }
+
+    if (!current_user_can('manage_options')) {
+        $response['error'] = 'No tienes permisos para descargar';
+        wp_send_json_error($response);
+    }
+
+    $baseUploadDir = wp_upload_dir();
+    $tbkLogsFolder = '/transbank_webpay_plus_rest/logs/';
+    $logName = sanitize_text_field($_POST['file']);
+    $folderPath = $baseUploadDir['basedir'] . $tbkLogsFolder;
+    $fileExists = fileExistsInFolder($logName, $folderPath);
+
+    if (!$fileExists) {
+        $response['error'] = 'No existe el archivo solicitado';
+        wp_send_json_error($response);
+    }
+
+    $response['canDownload'] = true;
+    $response['downloadUrl'] = $baseUploadDir['baseurl'] . $tbkLogsFolder . $logName;
+
+    wp_send_json_success($response);
+}

From eccabfcdcb261afec3b5a462aa50ce5ee2050a6a Mon Sep 17 00:00:00 2001
From: Matias Vasquez <m.vasquezac@gmail.com>
Date: Thu, 7 Nov 2024 12:52:36 -0300
Subject: [PATCH 05/11] feat: handle download response

---
 plugin/js/admin.js | 63 ++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 63 insertions(+)

diff --git a/plugin/js/admin.js b/plugin/js/admin.js
index 14cad5e..ff89a6d 100644
--- a/plugin/js/admin.js
+++ b/plugin/js/admin.js
@@ -113,4 +113,67 @@ jQuery(function($) {
             notice_id: noticeId
         });
     });
+
+    function checkPermission(fileToDownload) {
+        return $.ajax({
+            url: ajaxurl,
+            type: 'POST',
+            data: {
+                action: 'checkCanDownloadLogFile',
+                file: fileToDownload
+            }
+        }).then(function (response) {
+            console.log(response);
+            return response.data;
+        }).catch(function () {
+            return false;
+        });
+    }
+
+    function showNotice(title, message, type = 'success') {
+        const notice = $('<div>')
+            .addClass(`is-dismissible notice notice-${type}`)
+            .prepend(`<p><strong>${title}</strong><br>${message}</p>`);
+
+        const dismissButton = $('<button>')
+            .addClass('notice-dismiss');
+
+        notice.append(dismissButton);
+
+
+        notice.find('.notice-dismiss').on('click', function () {
+            notice.fadeOut(300, function () {
+                notice.remove();
+            });
+        });
+
+        $('#logs-container').prepend(notice);
+    }
+
+    $('#btnDownload').on('click', function (e) {
+        e.preventDefault();
+        const logFileSelected = $('#log_file').val();
+
+        if (!logFileSelected) {
+            showNotice('Error en la descarga', 'Debes seleccionar un archivo', 'error');
+            return;
+        }
+
+        checkPermission(logFileSelected).then(function (checkResponse) {
+            console.log(checkResponse);
+            if (!checkResponse) {
+                showNotice('Error en la descarga', 'No es posible descargar el archivo', 'error');
+                return;
+            }
+            if (checkResponse?.canDownload) {
+                window.location.href = checkResponse.downloadUrl;
+                return;
+            }
+            showNotice('Error en la descarga', checkResponse.error, 'error');
+
+        });
+    });
+
+
+
 })

From ec72f3ec10d0b94826fff32f3f97e5a371410693 Mon Sep 17 00:00:00 2001
From: Matias Vasquez <m.vasquezac@gmail.com>
Date: Thu, 7 Nov 2024 13:00:11 -0300
Subject: [PATCH 06/11] refactor: remove console.log

---
 plugin/js/admin.js | 2 --
 1 file changed, 2 deletions(-)

diff --git a/plugin/js/admin.js b/plugin/js/admin.js
index ff89a6d..7ad78af 100644
--- a/plugin/js/admin.js
+++ b/plugin/js/admin.js
@@ -123,7 +123,6 @@ jQuery(function($) {
                 file: fileToDownload
             }
         }).then(function (response) {
-            console.log(response);
             return response.data;
         }).catch(function () {
             return false;
@@ -160,7 +159,6 @@ jQuery(function($) {
         }
 
         checkPermission(logFileSelected).then(function (checkResponse) {
-            console.log(checkResponse);
             if (!checkResponse) {
                 showNotice('Error en la descarga', 'No es posible descargar el archivo', 'error');
                 return;

From 826ae2e849138cd5b383051440687820e4401174 Mon Sep 17 00:00:00 2001
From: Matias Vasquez <m.vasquezac@gmail.com>
Date: Thu, 7 Nov 2024 13:04:13 -0300
Subject: [PATCH 07/11] feat: restore deleted actions

---
 plugin/webpay-rest.php | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/plugin/webpay-rest.php b/plugin/webpay-rest.php
index ece643e..8705d9a 100644
--- a/plugin/webpay-rest.php
+++ b/plugin/webpay-rest.php
@@ -41,6 +41,8 @@
 add_action('wp_loaded', 'woocommerceTransbankInit');
 add_action('admin_init', 'on_transbank_rest_webpay_plugins_loaded');
 
+add_action('wp_ajax_check_connection', ConnectionCheck::class . '::check');
+add_action('wp_ajax_check_exist_tables', TableCheck::class . '::check');
 add_action('wp_ajax_checkCanDownloadLogFile', 'checkCanDownloadLogFile');
 add_action('wp_ajax_get_transaction_status', [new TransactionStatusController(), 'getStatus']);
 add_action('woocommerce_before_cart', 'transbank_rest_before_cart');

From 43d9a5d7c3ea91a4c78d2e1b37bef8e4429df7bf Mon Sep 17 00:00:00 2001
From: Matias Vasquez <m.vasquezac@gmail.com>
Date: Fri, 8 Nov 2024 10:01:43 -0300
Subject: [PATCH 08/11] refactor: use snake case on action name

---
 plugin/js/admin.js     | 2 +-
 plugin/webpay-rest.php | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/plugin/js/admin.js b/plugin/js/admin.js
index 7ad78af..f769252 100644
--- a/plugin/js/admin.js
+++ b/plugin/js/admin.js
@@ -119,7 +119,7 @@ jQuery(function($) {
             url: ajaxurl,
             type: 'POST',
             data: {
-                action: 'checkCanDownloadLogFile',
+                action: 'check_can_download_file',
                 file: fileToDownload
             }
         }).then(function (response) {
diff --git a/plugin/webpay-rest.php b/plugin/webpay-rest.php
index 8705d9a..2e58e1c 100644
--- a/plugin/webpay-rest.php
+++ b/plugin/webpay-rest.php
@@ -43,7 +43,7 @@
 
 add_action('wp_ajax_check_connection', ConnectionCheck::class . '::check');
 add_action('wp_ajax_check_exist_tables', TableCheck::class . '::check');
-add_action('wp_ajax_checkCanDownloadLogFile', 'checkCanDownloadLogFile');
+add_action('wp_ajax_check_can_download_file', 'checkCanDownloadLogFile');
 add_action('wp_ajax_get_transaction_status', [new TransactionStatusController(), 'getStatus']);
 add_action('woocommerce_before_cart', 'transbank_rest_before_cart');
 

From abf33776193b29e7358616eb49238f107a843563 Mon Sep 17 00:00:00 2001
From: Matias Vasquez <m.vasquezac@gmail.com>
Date: Fri, 8 Nov 2024 10:03:48 -0300
Subject: [PATCH 09/11] refactor: use css colors as custom properties

---
 plugin/css/tbk.css | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/plugin/css/tbk.css b/plugin/css/tbk.css
index 1b7673f..002ae3f 100644
--- a/plugin/css/tbk.css
+++ b/plugin/css/tbk.css
@@ -221,6 +221,8 @@
 }
 
 .tbk-box {
+    --tbk-red: #D5006C;
+    --tbk-red-2: #C00063;
     background: #fff;
     border-radius: 10px;
     padding: 15px;
@@ -316,32 +318,32 @@
 }
 
 .button.tbk-button-primary {
-    background: #D5006C;
-    border-color: #D5006C;
+    background: var(--tbk-red);
+    border-color: var(--tbk-red);
     color: white;
     transition: background-color 0.3s ease;
 }
 
 .button.tbk-button-primary:hover,
 .button.tbk-button-primary:focus {
-    background: #C00063;
-    border-color: #C00063;
+    background: var(--tbk-red-2);
+    border-color: var(--tbk-red-2);
     color: white;
     box-shadow: 0 2px 4px rgba(0, 0, 0, 0.1);
 }
 
 .button.tbk-button-secondary {
     background: white;
-    border-color: #D5006C;
-    color: #D5006C;
+    border-color: var(--tbk-red);
+    color: var(--tbk-red);
     transition: background-color 0.3s ease;
 }
 
 .button.tbk-button-secondary:hover,
 .button.tbk-button-secondary:focus {
     background: white;
-    border-color: #D5006C;
-    color: #D5006C;
+    border-color: var(--tbk-red-2);
+    color: var(--tbk-red-2);
     box-shadow: 0 2px 4px rgba(0, 0, 0, 0.1);
 }
 

From 836dcd2e4c421ea9e8930f62c49dbc41df27852d Mon Sep 17 00:00:00 2001
From: Matias Vasquez <m.vasquezac@gmail.com>
Date: Fri, 8 Nov 2024 11:55:34 -0300
Subject: [PATCH 10/11] refactor: standardize responses from check permission

---
 plugin/js/admin.js     | 18 ++++++++----------
 plugin/webpay-rest.php | 21 +++++----------------
 2 files changed, 13 insertions(+), 26 deletions(-)

diff --git a/plugin/js/admin.js b/plugin/js/admin.js
index f769252..287557d 100644
--- a/plugin/js/admin.js
+++ b/plugin/js/admin.js
@@ -123,9 +123,11 @@ jQuery(function($) {
                 file: fileToDownload
             }
         }).then(function (response) {
-            return response.data;
-        }).catch(function () {
-            return false;
+            return response;
+        }).catch(function (error) {
+            return { success: false, data: {
+                error: error.message || "Error en la solicitud de descarga"
+            } };
         });
     }
 
@@ -159,15 +161,11 @@ jQuery(function($) {
         }
 
         checkPermission(logFileSelected).then(function (checkResponse) {
-            if (!checkResponse) {
-                showNotice('Error en la descarga', 'No es posible descargar el archivo', 'error');
-                return;
-            }
-            if (checkResponse?.canDownload) {
-                window.location.href = checkResponse.downloadUrl;
+            if (checkResponse.success) {
+                window.location.href = checkResponse.data.downloadUrl;
                 return;
             }
-            showNotice('Error en la descarga', checkResponse.error, 'error');
+            showNotice('Error en la descarga', checkResponse.data.error, 'error');
 
         });
     });
diff --git a/plugin/webpay-rest.php b/plugin/webpay-rest.php
index 2e58e1c..e01cd4d 100644
--- a/plugin/webpay-rest.php
+++ b/plugin/webpay-rest.php
@@ -287,21 +287,14 @@ function fileExistsInFolder($fileName, $folderPath)
 
 function checkCanDownloadLogFile()
 {
-    $response = [
-        'canDownload' => false,
-        'downloadUrl' => '',
-        'error' => ''
-    ];
-
     if (!is_user_logged_in()) {
-        $response['error'] = 'Debes iniciar sesión para poder descargar';
-        wp_send_json_error($response);
+        wp_send_json_error(['error' => 'Debes iniciar sesión para poder descargar']);
     }
 
     if (!current_user_can('manage_options')) {
-        $response['error'] = 'No tienes permisos para descargar';
-        wp_send_json_error($response);
+        wp_send_json_error(['error' => 'No tienes permisos para descargar']);
     }
+    wp_send_json_error(['error' => 'No tienes permisos para descargar']);
 
     $baseUploadDir = wp_upload_dir();
     $tbkLogsFolder = '/transbank_webpay_plus_rest/logs/';
@@ -310,12 +303,8 @@ function checkCanDownloadLogFile()
     $fileExists = fileExistsInFolder($logName, $folderPath);
 
     if (!$fileExists) {
-        $response['error'] = 'No existe el archivo solicitado';
-        wp_send_json_error($response);
+        wp_send_json_error(['error' => 'No existe el archivo solicitado']);
     }
 
-    $response['canDownload'] = true;
-    $response['downloadUrl'] = $baseUploadDir['baseurl'] . $tbkLogsFolder . $logName;
-
-    wp_send_json_success($response);
+    wp_send_json_success(['downloadUrl' => $baseUploadDir['baseurl'] . $tbkLogsFolder . $logName]);
 }

From 1b77e0c2e9fcf7c2a9327420897cf68e4ac17540 Mon Sep 17 00:00:00 2001
From: Matias Vasquez <m.vasquezac@gmail.com>
Date: Fri, 8 Nov 2024 15:24:15 -0300
Subject: [PATCH 11/11] refactor: move functions to plugin logger class methods

---
 plugin/shared/Helpers/PluginLogger.php | 32 +++++++++++++++++++++++
 plugin/webpay-rest.php                 | 36 ++------------------------
 2 files changed, 34 insertions(+), 34 deletions(-)

diff --git a/plugin/shared/Helpers/PluginLogger.php b/plugin/shared/Helpers/PluginLogger.php
index 1796f66..dba4940 100644
--- a/plugin/shared/Helpers/PluginLogger.php
+++ b/plugin/shared/Helpers/PluginLogger.php
@@ -171,4 +171,36 @@ private function saveLogFileNameInCache(string $logFileName, int $expireTime)
     {
         set_transient(self::CACHE_LOG_NAME, $logFileName, $expireTime);
     }
+
+    private static function fileExistsInFolder($fileName, $folderPath)
+    {
+        $filesInFolder = array_filter(scandir($folderPath), function ($file) use ($folderPath) {
+            return is_file($folderPath . '/' . $file);
+        });
+
+        return in_array($fileName, array_values($filesInFolder));
+    }
+
+    public static function checkCanDownloadLogFile()
+    {
+        if (!is_user_logged_in()) {
+            wp_send_json_error(['error' => 'Debes iniciar sesión para poder descargar']);
+        }
+
+        if (!current_user_can('manage_options')) {
+            wp_send_json_error(['error' => 'No tienes permisos para descargar']);
+        }
+
+        $baseUploadDir = wp_upload_dir();
+        $tbkLogsFolder = '/transbank_webpay_plus_rest/logs/';
+        $logName = sanitize_text_field($_POST['file']);
+        $folderPath = $baseUploadDir['basedir'] . $tbkLogsFolder;
+        $fileExists = self::fileExistsInFolder($logName, $folderPath);
+
+        if (!$fileExists) {
+            wp_send_json_error(['error' => 'No existe el archivo solicitado']);
+        }
+
+        wp_send_json_success(['downloadUrl' => $baseUploadDir['baseurl'] . $tbkLogsFolder . $logName]);
+    }
 }
diff --git a/plugin/webpay-rest.php b/plugin/webpay-rest.php
index e01cd4d..0154259 100644
--- a/plugin/webpay-rest.php
+++ b/plugin/webpay-rest.php
@@ -11,6 +11,7 @@
 use Transbank\WooCommerce\WebpayRest\Blocks\WCGatewayTransbankOneclickBlocks;
 use Transbank\WooCommerce\WebpayRest\Utils\ConnectionCheck;
 use Transbank\WooCommerce\WebpayRest\Utils\TableCheck;
+use Transbank\Plugin\Helpers\PluginLogger;
 
 if (!defined('ABSPATH')) {
     return;
@@ -43,7 +44,7 @@
 
 add_action('wp_ajax_check_connection', ConnectionCheck::class . '::check');
 add_action('wp_ajax_check_exist_tables', TableCheck::class . '::check');
-add_action('wp_ajax_check_can_download_file', 'checkCanDownloadLogFile');
+add_action('wp_ajax_check_can_download_file', PluginLogger::class . '::checkCanDownloadLogFile');
 add_action('wp_ajax_get_transaction_status', [new TransactionStatusController(), 'getStatus']);
 add_action('woocommerce_before_cart', 'transbank_rest_before_cart');
 
@@ -275,36 +276,3 @@ function () {
         }
     );
 }
-
-function fileExistsInFolder($fileName, $folderPath)
-{
-    $filesInFolder = array_filter(scandir($folderPath), function ($file) use ($folderPath) {
-        return is_file($folderPath . '/' . $file);
-    });
-
-    return in_array($fileName, array_values($filesInFolder));
-}
-
-function checkCanDownloadLogFile()
-{
-    if (!is_user_logged_in()) {
-        wp_send_json_error(['error' => 'Debes iniciar sesión para poder descargar']);
-    }
-
-    if (!current_user_can('manage_options')) {
-        wp_send_json_error(['error' => 'No tienes permisos para descargar']);
-    }
-    wp_send_json_error(['error' => 'No tienes permisos para descargar']);
-
-    $baseUploadDir = wp_upload_dir();
-    $tbkLogsFolder = '/transbank_webpay_plus_rest/logs/';
-    $logName = sanitize_text_field($_POST['file']);
-    $folderPath = $baseUploadDir['basedir'] . $tbkLogsFolder;
-    $fileExists = fileExistsInFolder($logName, $folderPath);
-
-    if (!$fileExists) {
-        wp_send_json_error(['error' => 'No existe el archivo solicitado']);
-    }
-
-    wp_send_json_success(['downloadUrl' => $baseUploadDir['baseurl'] . $tbkLogsFolder . $logName]);
-}