Merge branch 'giane' of https://github.com/BryamE/ProyectoLauck into bryam

app/Http/Controllers/SaleController.php

@@ -3,11 +3,11 @@
 namespace App\Http\Controllers;
 
 use Illuminate\Http\Request;
-use App\Models\Product; 
+use App\Models\Product;
+use App\Models\Client;
 use App\Models\Sale;
 use App\Models\SaleDetail;
 use Illuminate\Support\Facades\DB;
-use App\Models\Client;
 
 class SaleController extends Controller
 {
@@ -22,22 +22,22 @@ class SaleController extends Controller
 
     public function create()
     {
-        // Buscamos productos que tengan stock mayor a 0 para mostrar en el selector
+        // Traemos productos con stock y clientes para los selectores
         $products = Product::where('stock_quantity', '>', 0)->get();
-
-        $clients = Client::orderBy('name')->get(); 
+        $clients = Client::orderBy('name')->get();
 
         return view('sales.create', compact('products', 'clients'));
     }
 
     public function store(Request $request)
     {
+        // Validación estricta
         $request->validate([
+            'client_id'      => 'nullable|exists:clients,id',
             'payment_method' => 'required|string',
-            'client_id' => 'nullable|exists:clients,id',
-            'items' => 'required|array',
+            'items'          => 'required|array',
             'items.*.product_id' => 'required|exists:products,id',
-            'items.*.quantity' => 'required|integer|min:1',
+            'items.*.quantity'   => 'required|integer|min:1',
         ]);
 
         try {
@@ -52,6 +52,7 @@ class SaleController extends Controller
                     if ($product->stock_quantity < $item['quantity']) {
                         throw new \Exception("No hay suficiente stock de " . $product->name);
                     }
+                    
                 }
 
                 //cabecera
@@ -62,29 +63,27 @@ class SaleController extends Controller
                     'payment_method' => $request->payment_method,
                 ]);
 
-                foreach ($request->items as $item) {
-                    $product = Product::find($item['product_id']);
+                // 3. Guardar Detalle y Restar Stock
+                foreach ($request->items as $itemData) {
+                    $product = Product::findOrFail($itemData['product_id']);
 
                     SaleDetail::create([
                         'sale_id' => $newSale->id,
                         'product_id' => $product->id,
-                        'quantity' => $item['quantity'],
-                        'price' => $product->price,
+                        'quantity'   => $itemData['quantity'],
+                        'price'      => $product->price,
                     ]);
-                    $nuevoStock = $product->stock_quantity - $item['quantity'];
-                    $product->stock_quantity = $nuevoStock;
-                    $product->save();
+
+                    $product->decrement('stock_quantity', $itemData['quantity']);
                 }
                 return $newSale; // Retornamos la venta creada fuera de la transacción
             });
 
-             // 4. REDIRECCIÓN AL DETALLE (FACTURA)
-            // Usamos la variable $sale que nos devolvió la transacción
+             // 4. REDIRECCIÓN AL DETALLE
             return redirect()->route('sales.show', $sale)->with('success', '¡Venta registrada exitosamente!');
 
         } catch (\Exception $e) {
-            // Si algo falló (ej: falta stock), volvemos atrás con el error
-            return back()->with('error', 'Error en la venta: ' . $e->getMessage());
+            return back()->with('error', $e->getMessage())->withInput();
         }
     }