Merge branch 'main' into feat/ot-pdf-zip

ci4/app/Controllers/Importadores/ImportadorCatalogo.php

@@ -3,6 +3,7 @@ namespace App\Controllers\Importadores;
 
 use App\Controllers\BaseResourceController;
 use App\Entities\Catalogo\CatalogoLibroEntity;
+use App\Models\Presupuestos\PresupuestoModel;
 use App\Models\Catalogo\CatalogoLibroModel;
 use App\Controllers\Presupuestos\Presupuestocliente;
 use App\Services\PresupuestoService;
@@ -45,7 +46,7 @@ class ImportadorCatalogo extends BaseResourceController
 
         $viewData = [
             'pageSubTitle' => lang('Basic.global.ManageAllRecords', [lang('Importador.importadorCatalogoTitle')]),
-            
+
         ];
 
         $viewData = array_merge($this->viewData, $viewData); // merge any possible values from the parent controller class
@@ -58,37 +59,51 @@ class ImportadorCatalogo extends BaseResourceController
     {
         $json = $this->request->getJSON();
 
-        if (!$json || !isset($json->fila[0])) {
+        // Validación inicial del JSON y del ISBN
+        if (!$json || !isset($json->fila[0]) || empty(trim($json->fila[0]))) {
             return $this->response->setJSON([
                 'apto' => false,
-                'reason' => 'Datos inválidos'
+                'reason' => 'ISBN no proporcionado o datos inválidos'
             ]);
         }
 
-        $input = trim($json->fila[0]); // Asumimos que 'input' es el primer campo de la fila
+        $input = trim($json->fila[0]); // ISBN
+        $refCliente = isset($json->fila[1]) ? trim($json->fila[1]) : null;
 
-        if (empty($input)) {
+        // Validar formato del refCliente (esperado: idpedido-idlinea)
+        if (empty($refCliente) || strpos($refCliente, '-') === false) {
             return $this->response->setJSON([
                 'apto' => false,
-                'reason' => 'ISBN no proporiconado'
+                'reason' => 'Ref. cliente inválido'
+            ]);
+        }
+
+        // 1. Comprobar duplicado en tabla de presupuestos
+        $presupuestoModel = new PresupuestoModel(); // Usa el modelo real que corresponda
+        $yaExiste = $presupuestoModel->where('referencia_cliente', $refCliente)->first();
+
+        if ($yaExiste) {
+            return $this->response->setJSON([
+                'apto' => false,
+                'reason' => 'Referencia ya importada'
             ]);
         }
 
         $catalogoModel = new CatalogoLibroModel();
 
-        // 1. Buscar por ISBN exacto
+        // 2. Buscar por ISBN exacto
         $libroPorIsbn = $catalogoModel->where('isbn', $input)->first();
-
         if ($libroPorIsbn) {
             return $this->response->setJSON([
                 'apto' => true
             ]);
         }
 
-        // 2. Buscar por EAN sin guiones
+        // 3. Buscar por EAN sin guiones
         $eanLimpio = str_replace('-', '', $input);
-
-        $libroPorEan = $catalogoModel->where('REPLACE(ean, "-", "")', $eanLimpio)->first();
+        $libroPorEan = $catalogoModel
+            ->where('REPLACE(ean, "-", "")', $eanLimpio, false) // false para evitar escapado automático
+            ->first();
 
         if ($libroPorEan) {
             return $this->response->setJSON([
@@ -96,7 +111,7 @@ class ImportadorCatalogo extends BaseResourceController
             ]);
         }
 
-        // No encontrado
+        // 4. No encontrado
         return $this->response->setJSON([
             'apto' => false,
             'reason' => 'No encontrado en catálogo'
@@ -105,6 +120,7 @@ class ImportadorCatalogo extends BaseResourceController
 
 
 
+
     public function importarFila()
     {
         $json = $this->request->getJSON();
@@ -287,7 +303,7 @@ class ImportadorCatalogo extends BaseResourceController
                 $response['data']['sk_id'],
                 $precio_compra,
                 $tirada,
-                null, 
+                null,
                 true
             );
             if ($respuesta_ajuste['warning'] == true) {

ci4/app/Controllers/Logistica/EtiquetasTitulosController.php

@@ -166,7 +166,7 @@ class EtiquetasTitulosController extends BaseController
         $impresoras = $modelImpresora->select('id, name')
             ->where('deleted_at', null)
             ->where('tipo', 1)
-            ->orderBy('name', 'asc')
+            ->orderBy('name', 'desc')
             ->findAll();
         $etiquetaEntity->impresoras = $impresoras;
 

ci4/app/Controllers/Logistica/LogisticaController.php

@@ -345,7 +345,7 @@ class LogisticaController extends BaseController
         $impresoras = $modelImpresora->select('id, name')
             ->where('deleted_at', null)
             ->where('tipo', 1)
-            ->orderBy('name', 'asc')
+            ->orderBy('name', 'desc')
             ->findAll();
         $envioEntity->impresoras = $impresoras;
 

ci4/app/Controllers/Presupuestos/Presupuestocliente.php

@@ -2272,7 +2272,7 @@ class Presupuestocliente extends \App\Controllers\BaseResourceController
                     return $return_data;
                 }
 
-                $cantidad_total = intval($datosPedido->tirada) + intval($datosPedido->merma);
+                $cantidad_total = intval($datosPedido->tirada);// + intval($datosPedido->merma);
 
                 // Acabado Cubierta
                 if (intval($datos_entrada['cubierta']['acabado']) != 0) {