Merge pull request #44 from chechojgb/buttonsLovers

refactor: correccion modelo guardar pedidos

Author: chechojgb

app/Http/Controllers/BLPedidosController.php

@@ -17,6 +17,7 @@
 use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Str;
 use Illuminate\Support\Facades\Auth;
+use Illuminate\Support\Facades\Log;
 
 class BLPedidosController extends Controller
 {
@@ -51,7 +52,9 @@ public function index()
     public function store(Request $request)
     {
         $user = Auth::user();
+        logger()->info('Iniciando creación de pedido', ['user_id' => $user->id]);
 
+        // Validación de datos
         $validated = $request->validate([
             'cliente_id' => 'required|numeric|exists:bl_clientes,id',
             'fecha_acordada' => 'required|date',
@@ -64,7 +67,7 @@ public function store(Request $request)
         DB::beginTransaction();
 
         try {
-            // Crear el pedido principal
+            // Crear pedido principal
             $pedido = BLPedido::create([
                 'cliente_id' => $validated['cliente_id'],
                 'fecha_pedido' => $validated['fecha_acordada'],
@@ -73,46 +76,77 @@ public function store(Request $request)
                 'usuario_id' => $user->id,
             ]);
 
+            logger()->info('Pedido principal creado', ['pedido_id' => $pedido->id]);
+
+            // Procesar cada producto
             foreach ($validated['productos'] as $producto) {
+                $productoId = $producto['producto_id'];
                 $cantidadSolicitada = $producto['cantidad'];
 
-                // Buscar empaques disponibles para ese producto (ordenados por fecha ascendente)
-                $empaquesDisponibles = BLEmpaque::where('producto_id', $producto['producto_id'])
-                    ->where('estado', 'disponible')
-                    ->orderBy('created_at')
-                    ->get();
+                $estadoBuscado = 'disponible';
+                $query = BLEmpaque::where('producto_id', $productoId)
+                    ->whereRaw('LOWER(estado) = ?', [$estadoBuscado])
+                    ->orderBy('created_at');
+
+                if (DB::getDriverName() !== 'sqlite') {
+                    $query->lockForUpdate();
+                }
+
+                $empaquesDisponibles = $query->get();
+
+                logger()->info('Empaques disponibles', [
+                    'producto_id' => $productoId,
+                    'count' => $empaquesDisponibles->count(),
+                    'ids' => $empaquesDisponibles->pluck('id')
+                ]);
 
                 $itemsPedido = [];
                 $cantidadAsignada = 0;
 
                 foreach ($empaquesDisponibles as $empaque) {
-                    if ($cantidadAsignada >= $cantidadSolicitada) {
-                        break;
-                    }
+                    if ($cantidadAsignada >= $cantidadSolicitada) break;
+
+                    $cantidadPorEmpaque = $empaque->cantidad_por_empaque ?? $empaque->cantidad ?? 0;
+                    $cantidadRestante = $cantidadSolicitada - $cantidadAsignada;
+
+                    // Por defecto se toma el empaque completo
+                    $cantidadParaEsteItem = $cantidadPorEmpaque;
+                    $notaItem = null;
 
-                    $cantidadAsignada += $empaque->cantidad;
+                    // Si solo necesitamos una parte de este empaque
+                    if ($cantidadRestante < $cantidadPorEmpaque) {
+                        $cantidadParaEsteItem = $cantidadRestante;
+                        $notaItem = "Bolsa de {$cantidadPorEmpaque}, se toman {$cantidadRestante}, sobran " . ($cantidadPorEmpaque - $cantidadRestante);
+                    }
 
                     $itemsPedido[] = [
-                        'pedido_id' => $pedido->id,
-                        'cantidad_empaques' => $empaque->cantidad,
+                        'cantidad_empaques' => $cantidadParaEsteItem,
                         'empaque_id' => $empaque->id,
-                        'nota' => null,
+                        'nota' => $notaItem,
                     ];
 
-                    // Marcar empaque como "asignado"
+                    $cantidadAsignada += $cantidadParaEsteItem;
+
+                    // Marcar empaque como asignado
                     $empaque->estado = 'asignado';
                     $empaque->save();
-                }
 
-                // Si no se alcanzó la cantidad, se anota en el último ítem
-                if ($cantidadAsignada < $cantidadSolicitada && count($itemsPedido) > 0) {
-                    $faltante = $cantidadSolicitada - $cantidadAsignada;
-                    $itemsPedido[count($itemsPedido) - 1]['nota'] = "Faltan $faltante unidades para completar este pedido";
+                    logger()->info('Empaque asignado', [
+                        'empaque_id' => $empaque->id,
+                        'cantidad_asignada' => $cantidadParaEsteItem,
+                        'nota' => $notaItem
+                    ]);
                 }
 
-                // Insertar ítems del producto (si hay alguno)
-                if (count($itemsPedido) > 0) {
-                    BLPedidoItem::create($itemsPedido);
+                // Si no hay empaques asignados
+                if (empty($itemsPedido)) {
+                    logger()->warning('No se asignaron empaques', ['producto_id' => $productoId]);
+                } else {
+                    $pedido->items()->createMany($itemsPedido);
+                    logger()->info('Items creados', [
+                        'producto_id' => $productoId,
+                        'count' => count($itemsPedido)
+                    ]);
                 }
             }
 
@@ -126,6 +160,10 @@ public function store(Request $request)
 
         } catch (\Exception $e) {
             DB::rollBack();
+            logger()->error('Error al crear pedido', [
+                'error' => $e->getMessage(),
+                'trace' => $e->getTraceAsString()
+            ]);
 
             return response()->json([
                 'success' => false,
@@ -137,4 +175,7 @@ public function store(Request $request)
 
 
 
+
+
+
 }

app/Http/Controllers/BlProductoController.php

@@ -148,7 +148,7 @@ public function store(Request $request)
             'cantidad_por_empaque' => $validated['cantidad_por_empaque'],
             'codigo_barras' => $validated['codigo_barras'],
             'codigo_unico' => $validated['codigo_unico'],
-            'estado' => 'Disponible'
+            'estado' => 'disponible'
         ]);
 
 

database/migrations/2025_08_04_001906_create_bl_pedido_items.php

@@ -16,7 +16,7 @@ public function up(): void
             $table->foreignId('pedido_id')->constrained('bl_pedidos');
             $table->foreignId('empaque_id')->constrained('bl_empaques');
             $table->integer('cantidad_empaques'); // Ej: 2 empaques de 500 unidades = 1000 botones
-            $table->decimal('precio_unitario', 10, 2); // Precio por empaque
+            // $table->decimal('precio_unitario', 10, 2); // Precio por empaque
             $table->timestamps();
         });
     }

database/migrations/2025_08_06_161016_add_disponibilidad_to_bl_empaques_table.php

@@ -22,7 +22,7 @@ public function up(): void
     public function down(): void
     {
         Schema::table('bl_empaques', function (Blueprint $table) {
-            $table->string('estado');
+            $table->dropColumn('estado');
         });
     }
 

database/migrations/2025_08_06_161217_add_nota_paquete_to_bl_pedido_items_table.php

@@ -22,7 +22,7 @@ public function up(): void
     public function down(): void
     {
         Schema::table('bl_pedido_items', function (Blueprint $table) {
-            $table->string('nota');
+            $table->dropColumn('nota');
         });
     }
 };

resources/js/pages/BLClientes.jsx

@@ -127,17 +127,17 @@ export default function Clientes({clientes}) {
         setToast({
           show: true,
           success: true,
-          message: "Producto guardado correctamente"
+          message: "cliente guardado correctamente"
         });
-        // Refrescar la lista de productos
+        // Refrescar la lista de clientes
         // router.visit(route('clientes.index'));
       },
       onError: (errors) => {
       const primerError = Object.values(errors)[0];
       setToast({
         show: true,
         success: false,
-        message: primerError || "Error al guardar el producto"
+        message: primerError || "Error al guardar el cliente"
       });
       },
       onFinish: (visit) => {