Validamos, que el ean13 no esté vacío antes de ser almacenado. El ean13 entre co…

…mbinaciones de un mismo producto no puede ser repetido.

Validamos, que el ean13 no esté vacío antes de ser almacenado. El ean13 entre co…
…mbinaciones de un mismo producto no puede ser repetido.
Adrian
1 parent f9175a0c
Showing 6 changed files with 36 additions and 6 deletions
FuncionesImaxMultiEAN.php
config.xml
imaxmultiean.php
sql-install.php
templates/hook/admin_products_extra.tpl
upgrade/upgrade-1.4.php
@@ -14,12 +14,20 @@ class FuncionesImaxMultiEAN {
 	public function storeEAN($productId, $ean, $combinationId = 0) {
 		$sql = 'INSERT INTO ' . _DB_PREFIX_.$this->modulo->prefijo . "multiean (id_product, id_product_attribute, ean13) VALUES ('$productId', '$combinationId', '$ean')";
-		return Db::getInstance()->execute($sql);
+		try {
+			return Db::getInstance()->execute($sql);
+		} catch (Exception $e) {
+			return false;
+		}
 	}
 	public function deleteEAN($productId, $ean, $combinationId = 0) {
 		$sql = 'DELETE FROM ' . _DB_PREFIX_ . $this->modulo->prefijo . "multiean WHERE id_product = '$productId' AND id_product_attribute = '$combinationId' AND ean13 = '$ean'";
-		return Db::getInstance()->execute($sql);
+		try {
+			return Db::getInstance()->execute($sql);
+		} catch (Exception $e) {
+			return false;
+		}
 	}
 	public function getProductByEan($ean) {
@@ -2,7 +2,7 @@
         <module>
             <name>imaxmultiean</name>
             <displayName><![CDATA[Gestor de EANs Múltiples]]></displayName>
-            <version><![CDATA[1.3]]></version>
+            <version><![CDATA[1.4]]></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>
@@ -22,7 +22,7 @@ class ImaxMultiEAN extends Module {
         $this->name = 'imaxmultiean';
         $this->tab = 'administration';
         $this->path = _PS_MODULE_DIR_ . $this->name . '/';
-        $this->version = '1.3';
+        $this->version = '1.4';
         $this->author = 'Informax';
         $this->need_instance = 0;
         $this->idManual = '';
@@ -6,7 +6,8 @@ $sql[] = &quot;CREATE TABLE IF NOT EXISTS `&quot;._DB_PREFIX_.self::prefijo.&quot;multiean` (
 	`id_product` INT(10) NOT NULL,
 	`id_product_attribute` INT(10) NOT NULL,
 	`ean13` VARCHAR(255) NOT NULL,
-	PRIMARY KEY (`id`)
+	PRIMARY KEY (`id`),
+	UNIQUE KEY `unique_ean_per_product_combination` (`id_product`, `ean13`)
         )
         COLLATE='utf8_general_ci'
         ENGINE=MyIsam;";
@@ -101,7 +101,12 @@ $(function(){
         let combinationId = $(this).data('id_combinacion');
         let eanInput = combinationId ? $('#new_combination_' + combinationId + '_ean') : $('#new_product_ean');
-        data.ean = eanInput.val();
+        data.ean = eanInput.val().trim();
+
+        if (!data.ean) {
+            showErrorMessage('El campo EAN no puede estar vacío.');
+            return;
+        }
         if (combinationId) {
             data.combinationId = combinationId;
+<?php
+
+function upgrade_module_1_4($module) {
+    $sql = array();
+
+    $sql[] = "ALTER TABLE `"._DB_PREFIX_.$module->prefijo."multiean`
+              ADD UNIQUE KEY `unique_ean_per_product_combination` (`id_product`, `ean13`);";
+
+    foreach ($sql as $query) {
+        if (Db::getInstance()->execute($query) == false) {
+            return false;
+        }
+    }
+
+    return true;
+}