up

app/Models/Confeccion.php

@@ -118,7 +118,8 @@ class Confeccion extends Model
     }
 
     /**
-     * Registrar un arreglo como ajuste histórico y crear traslado de reparaciones
+     * Registrar un arreglo: prendas que llegaron mal y salen a reparación
+     * Permiten re-recepción cuando regresen arregladas
      */
     public function registerArreglo(int $cantidad, ?string $notas = null, ?int $userId = null)
     {
@@ -126,9 +127,25 @@ class Confeccion extends Model
             throw \Illuminate\Validation\ValidationException::withMessages(['cantidad' => 'La cantidad debe ser mayor que 0.']);
         }
 
-        $recibido = $this->recibido_total;
-        if ($cantidad > $recibido) {
-            throw \Illuminate\Validation\ValidationException::withMessages(['cantidad' => 'La cantidad supera lo recibido.']);
+        // Validar que no exceda las recepciones efectivas (solo puedes marcar como arreglo lo que YA recibiste)
+        $recepcionesEfectivas = (int) $this->recepciones()->sum(\DB::raw('cantidad - COALESCE(prendas_defectuosas, 0)'));
+        $arreglosActuales = (int) $this->ajustes()->where('tipo', 'arreglo')->sum('cantidad');
+        
+        $disponibleParaArreglos = $recepcionesEfectivas - $arreglosActuales;
+
+        if ($cantidad > $disponibleParaArreglos) {
+            throw \Illuminate\Validation\ValidationException::withMessages([
+                'cantidad' => "No puedes registrar {$cantidad} prendas como arreglo.
+                
+                📊 Resumen:
+                • Recepciones efectivas: {$recepcionesEfectivas}
+                • Arreglos ya registrados: {$arreglosActuales}
+                • Disponible para marcar como arreglo: {$disponibleParaArreglos}
+                
+                ➡️ Solo puedes registrar máximo {$disponibleParaArreglos} prendas como arreglo.
+                
+                💡 Los arreglos deben ser de prendas YA recibidas que necesitan reparación."
+            ]);
         }
 
         // Crear ajuste
@@ -141,31 +158,12 @@ class Confeccion extends Model
             'notas' => $notas,
         ]);
 
-        // Crear traslado de reparaciones para reflejar la salida
-        \App\Models\TrasladoPrenda::create([
-            'orden_produccion_id' => $this->orden_produccion_id,
-            'referencia_type' => self::class,
-            'referencia_id' => $this->id,
-            'origen' => 'confeccion',
-            'destino' => 'reparaciones',
-            'cantidad_enviada' => $cantidad,
-            'cantidad_recibida' => 0,
-            'prendas_defectuosas' => 0,
-            'reparaciones' => $cantidad,
-            'fecha_envio' => now(),
-            'fecha_recepcion' => now(),
-            'estado' => 'recibido',
-        ]);
-
-        // Recalcular cantidad_recibida almacenada para compatibilidad con UI
-        $this->cantidad_recibida = $this->recibido_total;
-        $this->save();
-
         return $aj;
     }
 
     /**
-     * Registrar un cobro: descontar inventario y crear registro de cobro
+     * Registrar un cobro: solo registra el cobro sin descontar inventario
+     * Los cobros son prendas que NO llegaron - se cobran al confeccionista
      */
     public function registerCobro(int $cantidad, float $valor, ?string $notas = null, ?int $userId = null)
     {
@@ -173,46 +171,36 @@ class Confeccion extends Model
             throw \Illuminate\Validation\ValidationException::withMessages(['cantidad' => 'La cantidad debe ser mayor que 0.']);
         }
 
-        if (!$this->orden_produccion_id) {
-            throw \Illuminate\Validation\ValidationException::withMessages(['orden_produccion_id' => 'La confección debe tener una orden de producción asociada.']);
+        // Validar que no se exceda lo enviado
+        $enviada = (int) ($this->cantidad_enviada ?? 0);
+        if ($enviada <= 0) {
+            throw \Illuminate\Validation\ValidationException::withMessages(['cantidad' => 'La confección no tiene cantidad_enviada registrada.']);
         }
 
-        // Buscar inventarios disponibles de esta orden de producción
-        $inventarios = \App\Models\InventarioPrenda::where('orden_produccion_id', $this->orden_produccion_id)
-            ->where('cantidad_disponible', '>', 0)
-            ->orderBy('cantidad_disponible', 'desc')
-            ->get();
-
-        if ($inventarios->isEmpty()) {
-            throw \Illuminate\Validation\ValidationException::withMessages(['cantidad' => 'No hay inventario disponible para la orden de producción #' . $this->orden_produccion_id . '. Asegúrate de que exista inventario con cantidad_disponible > 0.']);
-        }
-
-        $totalDisponible = $inventarios->sum('cantidad_disponible');
+        // Calcular lo que ya se ha contabilizado
+        $recepcionesEfectivas = (int) $this->recepciones()->sum(\DB::raw('cantidad - COALESCE(prendas_defectuosas, 0)'));
+        $arreglos = (int) $this->ajustes()->where('tipo', 'arreglo')->sum('cantidad');
+        $recibidoNeto = $recepcionesEfectivas - $arreglos; // Los arreglos permiten re-recepción
         
-        if ($totalDisponible < $cantidad) {
-            throw \Illuminate\Validation\ValidationException::withMessages(['cantidad' => "Solo hay {$totalDisponible} unidades disponibles en inventario. No se puede descontar {$cantidad}."]);
-        }
+        $cobrosActuales = (int) $this->cobros()->sum('cantidad');
+        
+        $totalContabilizado = $recibidoNeto + $cobrosActuales;
+        $disponible = $enviada - $totalContabilizado;
 
-        // Descontar del inventario de forma distribuida si es necesario
-        $cantidadRestante = $cantidad;
-        foreach ($inventarios as $inventario) {
-            if ($cantidadRestante <= 0) {
-                break;
-            }
-
-            $aDescontar = min($cantidadRestante, $inventario->cantidad_disponible);
-            $inventario->cantidad_disponible -= $aDescontar;
-            $inventario->save();
-
-            // Si está asociado a producto, decrementar stock
-            if ($inventario->producto_id) {
-                $producto = \App\Models\Producto::find($inventario->producto_id);
-                if ($producto) {
-                    $producto->decrement('stock', $aDescontar);
-                }
-            }
-
-            $cantidadRestante -= $aDescontar;
+        if ($cantidad > $disponible) {
+            throw \Illuminate\Validation\ValidationException::withMessages([
+                'cantidad' => "No puedes registrar {$cantidad} prendas como cobro.
+                
+                📊 Resumen:
+                • Enviadas: {$enviada}
+                • Recepciones efectivas: {$recepcionesEfectivas}
+                • Arreglos (permiten re-recepción): {$arreglos}
+                • Recibido neto: {$recibidoNeto}
+                • Cobros registrados: {$cobrosActuales}
+                • Disponible para cobros: {$disponible}
+                
+                ➡️ Solo puedes registrar máximo {$disponible} prendas más como cobros."
+            ]);
         }
 
         // Crear registro de cobro