From 4ad0754ba09e23b7bffcffd7892f4e4137c42cc5 Mon Sep 17 00:00:00 2001
From: George <george@MacBook-Air-de-George.local>
Date: Wed, 14 Oct 2026 17:27:29 -0500
Subject: [PATCH] =?UTF-8?q?Cantidades:=20PICKING=20filtra=20lineas=20sin?=
 =?UTF-8?q?=20diferencia;=20transferencias=20juntan=20lotes=20de=20todas?=
 =?UTF-8?q?=20las=20lineas=20del=20articulo.=20Versi=C3=B3n=202.1.6.0?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
---
 CLAUDE.md                                       |  7 +++++++
 .../Net.Business.Services.csproj                |  6 +++---
 Net.Data/WMS/Picking/PickingRepository.cs       | 17 ++++++++++-------
 Net.Data/WMS/Recibo/ReciboRepository.cs         |  7 ++++---
 docs/PICKING.md                                 |  9 +++++++++
 5 files changed, 33 insertions(+), 13 deletions(-)

diff --git a/CLAUDE.md b/CLAUDE.md
index 8dece2e..c9a033e 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -53,6 +53,13 @@ transferencias siempre, sin depender de `Notificacion`. Detalle y tabla en
   consumidor no cambia y si el log falla no rompe el flujo. No cubre
   `PostActionAsync` (cuerpo `{}`) ni `PostReOpen`/`PostClose`. Los tamaños de las
   columnas del log no se verificaron.
+- **v2.1.6.0 — cantidades**: (1) la NC de PICKING (estado 13) solo incluye líneas
+  con `cantidad facturada − CantidadPedida > 0`, igual que DESPACHO (antes una
+  línea completa generaba una línea de NC con cantidad 0); (2) la transferencia
+  automática de stock (Picking y Recibo) junta los lotes de **todas** las líneas
+  del mismo artículo para el par origen/destino (antes solo tomaba los de la
+  primera línea y la suma no cuadraba con la cantidad). El almacén de la línea de
+  la NC sigue saliendo de la factura, no del `codigoSubAlmacen` del request.
 - Sin probar contra SAP: confirmar con la factura 292410 que SAP devuelve
   `BaseType`/`BaseEntry` en las líneas y `BatchNumbers` en las de la Entrega.
 
diff --git a/Net.Business.Services/Net.Business.Services.csproj b/Net.Business.Services/Net.Business.Services.csproj
index 6c5ed7c..975cdc7 100644
--- a/Net.Business.Services/Net.Business.Services.csproj
+++ b/Net.Business.Services/Net.Business.Services.csproj
@@ -3,9 +3,9 @@
   <PropertyGroup>
     <TargetFramework>netcoreapp3.1</TargetFramework>
     <AssemblyName>ITCA_API_SAP_WMS</AssemblyName>
-    <Version>2.1.5</Version>
-    <AssemblyVersion>2.1.5.0</AssemblyVersion>
-    <FileVersion>2.1.5.0</FileVersion>
+    <Version>2.1.6</Version>
+    <AssemblyVersion>2.1.6.0</AssemblyVersion>
+    <FileVersion>2.1.6.0</FileVersion>
     <UserSecretsId>24763424-3300-4c85-87e3-503f8fdebd14</UserSecretsId>
     <!-- Antes solo estaba condicionado a Debug|AnyCPU - un "dotnet publish -c Release"
          (lo normal para IIS) se quedaba sin este archivo, y Startup.cs revienta con
diff --git a/Net.Data/WMS/Picking/PickingRepository.cs b/Net.Data/WMS/Picking/PickingRepository.cs
index 166cb0a..81242e4 100644
--- a/Net.Data/WMS/Picking/PickingRepository.cs
+++ b/Net.Data/WMS/Picking/PickingRepository.cs
@@ -318,12 +318,13 @@ namespace Net.Data.WMS.Picking
                                                 ListaUnicaArticulo.ForEach(x =>
                                                 {
                                                     var ListLote = new List<BE_StockTransferLineBatchNumber>();
-                                                    ListaDetalle_PorAlmacen.Where(t => t.ItemCode == x).FirstOrDefault().BatchNumbers.ForEach(r =>
+                                                    // Lotes de todas las lineas del articulo (si el mismo lote esta en varias lineas se suma)
+                                                    ListaDetalle_PorAlmacen.Where(t => t.ItemCode == x).SelectMany(t => t.BatchNumbers).GroupBy(r => r.BatchNumber).ToList().ForEach(g =>
                                                     {
                                                         ListLote.Add(new BE_StockTransferLineBatchNumber()
                                                         {
-                                                            BatchNumber = r.BatchNumber,
-                                                            Quantity = r.Quantity
+                                                            BatchNumber = g.Key,
+                                                            Quantity = g.Sum(r => r.Quantity)
                                                         });
                                                     });
                                                     Transfer.StockTransferLines.Add(new BE_StockTransferLine()
@@ -549,7 +550,8 @@ namespace Net.Data.WMS.Picking
                         entidad.Detalle.ForEach(x =>
                         {
                             var Item = _invoices.DocumentLines.ToList().FirstOrDefault(t => t.LineNum == x.Item);
-                            if (Item != null)
+                            // Solo lineas con diferencia a devolver (igual que DESPACHO): una linea completa daria cantidad 0
+                            if (Item != null && Item.Quantity - x.CantidadPedida > 0)
                             {
                                 creditNotesNotes.DocumentLines.Add(new BE_CreditNotesLine()
                                 {
@@ -613,12 +615,13 @@ namespace Net.Data.WMS.Picking
                                             ListaUnicaArticulo.ForEach(x =>
                                             {
                                                 var ListLote = new List<BE_StockTransferLineBatchNumber>();
-                                                ListaDetalle_PorAlmacen.Where(t => t.ItemCode == x).FirstOrDefault().BatchNumbers.ForEach(r =>
+                                                // Lotes de todas las lineas del articulo (si el mismo lote esta en varias lineas se suma)
+                                                ListaDetalle_PorAlmacen.Where(t => t.ItemCode == x).SelectMany(t => t.BatchNumbers).GroupBy(r => r.BatchNumber).ToList().ForEach(g =>
                                                 {
                                                     ListLote.Add(new BE_StockTransferLineBatchNumber()
                                                     {
-                                                        BatchNumber = r.BatchNumber,
-                                                        Quantity = r.Quantity
+                                                        BatchNumber = g.Key,
+                                                        Quantity = g.Sum(r => r.Quantity)
                                                     });
                                                 });
                                                 Transfer.StockTransferLines.Add(new BE_StockTransferLine()
diff --git a/Net.Data/WMS/Recibo/ReciboRepository.cs b/Net.Data/WMS/Recibo/ReciboRepository.cs
index 8b45435..02892e4 100644
--- a/Net.Data/WMS/Recibo/ReciboRepository.cs
+++ b/Net.Data/WMS/Recibo/ReciboRepository.cs
@@ -556,12 +556,13 @@ namespace Net.Data.WMS.Recibo
                                         ListaUnicaArticulo.ForEach(x =>
                                         {
                                             var ListLote = new List<BE_StockTransferLineBatchNumber>();
-                                            ListaDetalle_PorAlmacen.Where(t => t.ItemCode == x).FirstOrDefault().BatchNumbers.ForEach(r =>
+                                            // Lotes de todas las lineas del articulo (si el mismo lote esta en varias lineas se suma)
+                                            ListaDetalle_PorAlmacen.Where(t => t.ItemCode == x).SelectMany(t => t.BatchNumbers).GroupBy(r => r.BatchNumber).ToList().ForEach(g =>
                                             {
                                                 ListLote.Add(new BE_StockTransferLineBatchNumber()
                                                 {
-                                                    BatchNumber = r.BatchNumber,
-                                                    Quantity = r.Quantity
+                                                    BatchNumber = g.Key,
+                                                    Quantity = g.Sum(r => r.Quantity)
                                                 });
                                             });
                                             Transfer.StockTransferLines.Add(new BE_StockTransferLine()
diff --git a/docs/PICKING.md b/docs/PICKING.md
index 36f7796..336eecc 100644
--- a/docs/PICKING.md
+++ b/docs/PICKING.md
@@ -53,6 +53,15 @@ enviar lotes incompletos a SAP. Si no hay lotes de origen (artículo sin
 control por lote) la línea se envía sin lotes. PICKING (estado 13) no cambia:
 genera un lote nuevo por fecha y hora.
 
+## Cantidades de la NC y de las transferencias (versión 2.1.6.0)
+
+- **Cantidad de la línea de NC** (DESPACHO y PICKING estado 13): `cantidad
+  facturada − CantidadPedida`; solo entran las líneas con resultado mayor a 0.
+- **Transferencia automática** (cuando `U_ITC_ALMDES` difiere del almacén de la
+  línea): una línea por artículo, con la cantidad sumada de las líneas de la NC
+  de ese par origen/destino y los lotes de todas esas líneas (el mismo lote en
+  varias líneas se suma).
+
 ## Resumen del contrato
 
 | Método | Ruta | Cambia hoy | Qué hace |