From 03ad6d198913c80079862fb4bc91f3f162725007 Mon Sep 17 00:00:00 2001
From: Adrian <adrian.hermida@informax.es>
Date: Mon, 25 Nov 2024 11:35:35 +0100
Subject: [PATCH] Mejoras estéticas en la intefaz del módulo en la ficha del producto. Implementación con prestashop 8.

---
 config.xml                              |   2 +-
 functionsForm.php                       |   2 +-
 imaxAcordeon.php                        |   1 +
 imaxmultiean.php                        |   4 ++--
 templates/hook/admin_products_extra.tpl | 155 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-------------------------------------------------------------------------
 5 files changed, 87 insertions(+), 77 deletions(-)

diff --git a/config.xml b/config.xml
index 1189044..319cfaf 100644
--- a/config.xml
+++ b/config.xml
@@ -2,7 +2,7 @@
         <module>
             <name>imaxmultiean</name>
             <displayName><![CDATA[Gestor de EANs Múltiples]]></displayName>
-            <version><![CDATA[1.6]]></version>
+            <version><![CDATA[1.7]]></version>
             <description><![CDATA[Permite añadir y gestionar múltiples códigos EAN para cada producto y combinación.]]></description>
             <author><![CDATA[Informax]]></author>
             <tab><![CDATA[administration]]></tab>
diff --git a/functionsForm.php b/functionsForm.php
index 50efe5d..c3007a0 100644
--- a/functionsForm.php
+++ b/functionsForm.php
@@ -5,7 +5,7 @@
  */
 class imaxForm {
 
-    private $sinFormulario, $formulario = '', $modulo;
+    private $sinFormulario, $formulario = '', $modulo, $context, $_path, $_modulePath, $ajax, $idLang, $idShop;
 
     function __construct($modulo, $modulePath = '', $action = '', $method = 'post', $name = '', $enctype = '', $extras = false, $sinFormulario = false, $pathToModule = '', $ajax = false) {
         if ($action == '') {
diff --git a/imaxAcordeon.php b/imaxAcordeon.php
index d3f0472..0ad8941 100644
--- a/imaxAcordeon.php
+++ b/imaxAcordeon.php
@@ -3,6 +3,7 @@
  * @version 1.3
  */
 class imaxAcordeon {
+    private $context, $_path;
 
     function __construct($modulePath) {
         $this->context = Context::getContext();
diff --git a/imaxmultiean.php b/imaxmultiean.php
index 85fac73..6ab2811 100644
--- a/imaxmultiean.php
+++ b/imaxmultiean.php
@@ -11,7 +11,7 @@ class ImaxMultiEAN extends Module {
     var $idShop;
     var $idLang;
     var $idTab;
-    private $_html = '';
+    private $_html = '', $path, $idManual, $forceCheck, $sufijo, $prefijo;
 	
 	private static $funciones;
     
@@ -22,7 +22,7 @@ class ImaxMultiEAN extends Module {
         $this->name = 'imaxmultiean';
         $this->tab = 'administration';
         $this->path = _PS_MODULE_DIR_ . $this->name . '/';
-        $this->version = '1.6';
+        $this->version = '1.7';
         $this->author = 'Informax';
         $this->need_instance = 0;
         $this->idManual = '';
diff --git a/templates/hook/admin_products_extra.tpl b/templates/hook/admin_products_extra.tpl
index 6dbe926..df57ea7 100644
--- a/templates/hook/admin_products_extra.tpl
+++ b/templates/hook/admin_products_extra.tpl
@@ -3,24 +3,30 @@
         font-family: Arial, sans-serif;
         width: 50%;
     }
-    
-    #multiEAN h3, #multiEAN h4 {
+
+    #multiEAN h3{
+        padding-top: 25px;
+    }
+
+    #multiEAN h3,
+    #multiEAN h4 {
         color: #333;
+    }
+
+    #multiEAN hr {
         border-bottom: 2px solid #ddd;
-        padding-bottom: 10px;
-        margin-bottom: 15px;
     }
-    
+
     #multiEAN p {
         color: #555;
         margin-bottom: 10px;
     }
-    
+
     #multiEAN ul {
         list-style-type: none;
         padding: 0;
     }
-    
+
     #multiEAN li {
         margin: 10px 0;
         background: #f9f9f9;
@@ -28,7 +34,7 @@
         border-radius: 5px;
         position: relative;
     }
-    
+
     #multiEAN .borrarEAN {
         color: #f44336;
         cursor: pointer;
@@ -39,7 +45,7 @@
         text-decoration: none;
         font-size: 0.9em;
     }
-    
+
     #multiEAN button {
         background-color: #4CAF50;
         color: white;
@@ -48,14 +54,14 @@
         border-radius: 5px;
         cursor: pointer;
         transition: background-color 0.2s;
-        display: block; 
+        display: block;
         margin: 10px 0;
     }
-    
+
     #multiEAN button:hover {
         background-color: #45a049;
     }
-    
+
     #multiEAN input[type="text"] {
         padding: 10px;
         border: 1px solid #ddd;
@@ -64,7 +70,6 @@
         box-sizing: border-box;
         margin-bottom: 10px;
     }
-
 </style>
 
 <div id="multiEAN">
@@ -77,50 +82,56 @@
     </ul>
     <input type="text" id="new_product_ean" placeholder="Introduce el nuevo EAN" />
     <button class="guardarEAN">Agregar EAN al producto</button>
-
-{foreach $combinations_details as $combination}
-    <h4>Combinación: {$combination.attribute_name}</h4>
-    <p>EANs:</p>
-    <ul id="combination_{$combination.id}_eans_list">
-        {foreach $combination.eans as $ean}
-            <li>{$ean.ean13} <span class="borrarEAN" data-ean13='{$ean.ean13}'  data-id_combinacion='{$combination.id}'>[Eliminar]</span></li>
-        {/foreach}
-    </ul>
-    <input type="text" id="new_combination_{$combination.id}_ean" placeholder="Introduce el nuevo EAN para esta combinación" />
-    <button class="guardarEAN" data-id_combinacion='{$combination.id}'>Agregar EAN a la combinación</button>
-{/foreach}
+    <hr>
+    {foreach $combinations_details as $combination}
+        <h4>Combinación: {$combination.attribute_name}</h4>
+        <p>EANs:</p>
+        <ul id="combination_{$combination.id}_eans_list">
+            {foreach $combination.eans as $ean}
+                <li>{$ean.ean13} <span class="borrarEAN" data-ean13='{$ean.ean13}'
+                        data-id_combinacion='{$combination.id}'>[Eliminar]</span></li>
+            {/foreach}
+        </ul>
+        <input type="text" id="new_combination_{$combination.id}_ean"
+            placeholder="Introduce el nuevo EAN para esta combinación" />
+        <button class="guardarEAN" data-id_combinacion='{$combination.id}'>Agregar EAN a la combinación</button>
+        <hr>
+    {/foreach}
 </div>
 
 <script>
-$(function(){
-    $('.guardarEAN').click(function(ev){
-        ev.preventDefault();
-        let data = {
-            accion: 'storeEAN',
-            productId: "{$product_id}",
-        };
-
-        let combinationId = $(this).data('id_combinacion');
-        let eanInput = combinationId ? $('#new_combination_' + combinationId + '_ean') : $('#new_product_ean');
-        data.ean = eanInput.val().trim();
-
-        if (!data.ean) {
-            showErrorMessage('El campo EAN no puede estar vacío.');
-            return;
-        }
-
-        if (combinationId) {
-            data.combinationId = combinationId;
-        }
-
-        $.getJSON('//{$urlBase}/imaxmultiean_ajax.php?callback=?', data, function (respuesta) {
+    $(function() {
+        $('.guardarEAN').click(function(ev) {
+            ev.preventDefault();
+            let data = {
+                accion: 'storeEAN',
+                productId: "{$product_id}",
+            };
+
+            let combinationId = $(this).data('id_combinacion');
+            let eanInput = combinationId ? $('#new_combination_' + combinationId + '_ean') : $(
+                '#new_product_ean');
+            data.ean = eanInput.val().trim();
+
+            if (!data.ean) {
+                showErrorMessage('El campo EAN no puede estar vacío.');
+                return;
+            }
+
+            if (combinationId) {
+                data.combinationId = combinationId;
+            }
+
+            $.getJSON('//{$urlBase}/imaxmultiean_ajax.php?callback=?', data, function (respuesta) {
             if (respuesta.ok) {
                 showSuccessMessage('EAN almacenado con éxito.');
 
-                let eanList = combinationId ? $('#combination_' + combinationId + '_eans_list') : $('#product_eans_list');
-                let eanItem = $('<li>').text(data.ean).append($('<span>').addClass('borrarEAN').attr('data-ean13', data.ean).text('[Eliminar]'));
+                let eanList = combinationId ? $('#combination_' + combinationId + '_eans_list') : $(
+                    '#product_eans_list');
+                let eanItem = $('<li>').text(data.ean).append($('<span>').addClass('borrarEAN').attr(
+                    'data-ean13', data.ean).text('[Eliminar]'));
                 eanList.append(eanItem);
-                
+
                 eanInput.val(''); // Limpiar el campo de entrada
 
             } else {
@@ -130,32 +141,30 @@ $(function(){
     });
 
     // En este bloque, usaremos la delegación de eventos para asegurarnos de que también funcione para los elementos EAN recién agregados.
-    $(document).on('click', '.borrarEAN', function(){
-        let data = {
-            accion: 'deleteEAN',
-            productId: "{$product_id}",
-            ean: $(this).data('ean13'),
-        };
+    $(document).on('click', '.borrarEAN', function() {
+    let data = {
+    accion: 'deleteEAN',
+    productId: "{$product_id}",
+    ean: $(this).data('ean13'),
+    };
+
+    let combinationId = $(this).data('id_combinacion');
+    if (combinationId) {
+    data.combinationId = combinationId;
+    }
 
-        let combinationId = $(this).data('id_combinacion');
-        if (combinationId) {
-            data.combinationId = combinationId;
-        }
+    let eanItem = $(this).closest('li'); // Referencia al elemento <li> que contiene el EAN
 
-        let eanItem = $(this).closest('li'); // Referencia al elemento <li> que contiene el EAN
+    $.getJSON('//{$urlBase}/imaxmultiean_ajax.php?callback=?', data, function (respuesta) {
+    if (respuesta.ok) {
+    showSuccessMessage('EAN eliminado con éxito.');
 
-        $.getJSON('//{$urlBase}/imaxmultiean_ajax.php?callback=?', data, function (respuesta) {
-            if (respuesta.ok) {
-                showSuccessMessage('EAN eliminado con éxito.');
-                
-                eanItem.remove(); // Remover el elemento <li> del DOM
+    eanItem.remove(); // Remover el elemento <li> del DOM
 
-            } else {
-                showErrorMessage('Hubo un error al eliminar el EAN.');
-            }
-        });
+    } else {
+    showErrorMessage('Hubo un error al eliminar el EAN.');
+    }
+    });
+    });
     });
-});
-
-    
 </script>
\ No newline at end of file
--
libgit2 0.21.4