From e475b7f48cbb163eb16cdd67b0d65dbfe7d44dcb Mon Sep 17 00:00:00 2001
From: George <george@MacBook-Air-de-George.local>
Date: Wed, 14 Oct 2026 08:47:44 -0500
Subject: [PATCH] =?UTF-8?q?Picking:=20Notificacion=3Dcierre=20solo=20se=20?=
 =?UTF-8?q?exige=20con=20Proceso=20ULTIMA=20MILLA.=20Versi=C3=B3n=202.1.2.?=
 =?UTF-8?q?0?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

DESPACHO y PICKING vuelven a generar nota de crédito sin depender de
Notificacion. Documentada la regla Proceso + Notificacion.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
---
 CLAUDE.md                                     | 11 ++++-
 .../Net.Business.Services.csproj              |  6 +--
 Net.Data/WMS/Picking/PickingRepository.cs     |  7 +--
 docs/PICKING.md                               | 48 +++++++++++++------
 4 files changed, 51 insertions(+), 21 deletions(-)

diff --git a/CLAUDE.md b/CLAUDE.md
index 0670445..3189518 100644
--- a/CLAUDE.md
+++ b/CLAUDE.md
@@ -16,11 +16,20 @@ Detalle técnico en `docs/PORTE_FUNCIONALIDAD_NUEVA.md`,
 1. **SolicitudTransferencia**: `PATCH /api/SolicitudTransferencia/{docEntry}` y
    `POST /api/SolicitudTransferencia/{docEntry}/cerrar`.
 2. **Picking diurno/nocturno**: campo `Notificacion` (early-return en
-   `PickingRepository.Create` tras `HelperProcesoPiciking.Set`) y
+   `PickingRepository.Create` tras `HelperProcesoPiciking.Set`, **solo para
+   `Proceso = "ULTIMA MILLA"`**; DESPACHO/PICKING generan NC sin depender de
+   él) y
    `POST /api/Picking/HojaRuta/{docEntry}/Cerrar`.
 3. **`ConnectionServiceLayer`**: `PatchNoContentAsync` y `PostActionAsync`
    (métodos nuevos; los existentes no se tocaron).
 
+## Regla Proceso + Notificacion (v2.1.2.0)
+
+`Notificacion = "cierre"` se exige **solo si `Proceso = "ULTIMA MILLA"`**
+(si no, solo actualiza estado en HANA). `DESPACHO` y `PICKING` generan NC y
+transferencias siempre, sin depender de `Notificacion`. Detalle y tabla en
+`docs/PICKING.md`.
+
 ## Cambios de configuración y correcciones hechas en esta carpeta
 
 - **Credenciales y esquema HANA en `appsettings.json`**, sección `Hana`
diff --git a/Net.Business.Services/Net.Business.Services.csproj b/Net.Business.Services/Net.Business.Services.csproj
index 9c0ee30..1bc096a 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.1</Version>
-    <AssemblyVersion>2.1.1.0</AssemblyVersion>
-    <FileVersion>2.1.1.0</FileVersion>
+    <Version>2.1.2</Version>
+    <AssemblyVersion>2.1.2.0</AssemblyVersion>
+    <FileVersion>2.1.2.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 93982c3..ffe3818 100644
--- a/Net.Data/WMS/Picking/PickingRepository.cs
+++ b/Net.Data/WMS/Picking/PickingRepository.cs
@@ -120,11 +120,12 @@ namespace Net.Data.WMS.Picking
                     //=== REALIZA PROCESO COMPLETO DE ACTUALIZACION
                     HelperProcesoPiciking.Set(entidad.Proceso, entidad.IdEstadoCubico, entidad.DocEntry, entidad.CodMotivo, Proceso.Code);
 
-                    // === NUEVO (Picking diurno/nocturno): "notificacion" distinta de "cierre"
+                    // === Solo para el proceso "ULTIMA MILLA": "notificacion" distinta de "cierre"
                     // (o ausente) es la notificacion preliminar en tiempo real - solo actualiza
                     // estado en HANA (ya hecho arriba), NUNCA debe crear documentos en SAP.
-                    // Default seguro: cualquier valor que no sea "cierre" exacto cae aca.
-                    if (!string.Equals(entidad.Notificacion, "cierre", StringComparison.OrdinalIgnoreCase))
+                    // DESPACHO, PICKING y demas procesos no dependen de este campo.
+                    if (string.Equals(entidad.Proceso, "ULTIMA MILLA", StringComparison.OrdinalIgnoreCase)
+                        && !string.Equals(entidad.Notificacion, "cierre", StringComparison.OrdinalIgnoreCase))
                     {
                         return resultadoTransaccion.GetInformacionResultado(entidad.DocEntry, entidad.NumOrden, "Estado actualizado correctamente", entidad);
                     }
diff --git a/docs/PICKING.md b/docs/PICKING.md
index 9680129..33d9480 100644
--- a/docs/PICKING.md
+++ b/docs/PICKING.md
@@ -22,11 +22,27 @@ recorre esas mismas facturas con un `FOR` de su lado llamando otra vez al
 mismo endpoint pero en modo **cierre** (ahí sí se crean los documentos SAP),
 y al final cierra la hoja de ruta completa **una sola vez**.
 
+## Regla Proceso + Notificacion (versión 2.1.2.0)
+
+`Notificacion` **solo se valida cuando `Proceso = "ULTIMA MILLA"`**. En los
+demás procesos el campo se ignora.
+
+| `Proceso` | `Notificacion` | Resultado |
+|---|---|---|
+| `ULTIMA MILLA` | `"cierre"` | Lógica completa: crea documentos en SAP (solicitud de devolución, estado 23). |
+| `ULTIMA MILLA` | ausente u otro valor | Solo actualiza estado en HANA; **no** crea nada en SAP. |
+| `DESPACHO` | cualquiera / ausente | Lógica completa siempre: NC y transferencias automáticas (estados 5 y 6). |
+| `PICKING` | cualquiera / ausente | Lógica completa siempre: NC y transferencias automáticas (estado 13). |
+
+Historial: en 2.1.1.0 el filtro se aplicaba a todos los procesos, por lo que
+DESPACHO y PICKING no generaban la NC si no llegaba `"cierre"`. Corregido en
+2.1.2.0.
+
 ## Resumen del contrato
 
 | Método | Ruta | Cambia hoy | Qué hace |
 |---|---|---|---|
-| `POST` | `/api/picking` | **Sí** — agrega campo `Notificacion` | `"cierre"` → lógica actual completa (crea documento SAP); cualquier otro valor/ausente → solo actualiza estado en HANA, nunca toca SAP. |
+| `POST` | `/api/picking` | **Sí** — agrega campo `Notificacion` | **Solo con `Proceso = "ULTIMA MILLA"`**: `"cierre"` → lógica completa (crea documento SAP); cualquier otro valor/ausente → solo actualiza estado en HANA, nunca toca SAP. Con `DESPACHO`, `PICKING` y demás procesos el campo se ignora y la lógica corre siempre (NC y transferencias). |
 | `POST` | `/api/picking/ruta` | No | Sin cambios — `CreateHojaRuta` ya existe y sigue igual. |
 | `POST` | `/api/picking/hojaruta/{docEntry}/cerrar` | **Nuevo** | Cierra el UDO `ARD_PHR` completo en SAP. Sin body. |
 
@@ -36,7 +52,7 @@ y al final cierra la hoja de ruta completa **una sola vez**.
 
 **Archivo**: `Net.Business.Entities/WMS/Picking/BE_PickingRequest.cs` — agregar:
 ```csharp
-public string Notificacion { get; set; }  // "preliminar" | "cierre"; cualquier otro valor o ausente = "preliminar"
+public string Notificacion { get; set; }  // solo se evalua con Proceso "ULTIMA MILLA": "cierre" | otro/ausente = preliminar
 ```
 
 **Archivo**: `Net.Business.DTO/WMS/Picking/Dto_PickingRequest.cs` — agregar la
@@ -73,8 +89,10 @@ alcanza con un `if` adentro del mismo método).
 HelperProcesoPiciking.Set(entidad.Proceso, entidad.IdEstadoCubico, entidad.DocEntry, entidad.CodMotivo, Proceso.Code);
 // <-- linea 96 actual, sin tocar
 
-// === NUEVO: notificacion preliminar (diurna) - nunca toca SAP
-if (!string.Equals(entidad.Notificacion, "cierre", StringComparison.OrdinalIgnoreCase))
+// === Solo ULTIMA MILLA: notificacion preliminar (diurna) - nunca toca SAP.
+// DESPACHO / PICKING no dependen de Notificacion.
+if (string.Equals(entidad.Proceso, "ULTIMA MILLA", StringComparison.OrdinalIgnoreCase)
+    && !string.Equals(entidad.Notificacion, "cierre", StringComparison.OrdinalIgnoreCase))
 {
     return resultadoTransaccion.GetInformacionResultado(entidad.DocEntry, entidad.NumOrden, "Estado actualizado correctamente", entidad);
 }
@@ -85,8 +103,10 @@ if (!string.Equals(entidad.Notificacion, "cierre", StringComparison.OrdinalIgnor
 ```
 
 Es decir: **todo lo que hoy hace `Create()` (líneas 45-341 completas)** sigue
-siendo exactamente la lógica de `notificacion == "cierre"` — no se toca nada
-de ahí. Solo se agrega el `if` de corte después de la línea 96.
+siendo exactamente la lógica de cierre — no se toca nada de ahí. Solo se
+agrega el `if` de corte después de la línea 96, y **únicamente para
+`ULTIMA MILLA`**. Para `DESPACHO` y `PICKING` la NC se genera sin importar
+`Notificacion`.
 
 > **Nota de contrato**: `detalle` ya es obligatorio hoy en este endpoint (la
 > validación de arriba lo necesita para desambiguar `Existe > 1`), así que no
@@ -96,15 +116,15 @@ de ahí. Solo se agrega el `if` de corte después de la línea 96.
 
 ### Bodies de ejemplo
 
-**Notificación preliminar** (reparto en curso — `Notificacion` ausente o
-distinto de `"cierre"`):
+**Notificación preliminar** (solo `ULTIMA MILLA`, reparto en curso —
+`Notificacion` ausente o distinto de `"cierre"`):
 ```json
 {
   "numOrden": "OT-2026-001",
   "docEntry": 500,
-  "proceso": "DESPACHO",
+  "proceso": "ULTIMA MILLA",
   "codMotivo": "M01",
-  "idEstadoCubico": 6,
+  "idEstadoCubico": 23,
   "codigoAlmacen": "ALM01",
   "detalle": [
     { "item": 0, "codigoArticulo": "ART-001", "cantidadPedida": 3, "estado": "C" }
@@ -112,15 +132,15 @@ distinto de `"cierre"`):
 }
 ```
 
-**Cierre** (mismo shape, agrega `notificacion: "cierre"` — crea el documento
-SAP real, misma respuesta que hoy):
+**Cierre** (`ULTIMA MILLA`, mismo shape, agrega `notificacion: "cierre"` —
+crea el documento SAP real, misma respuesta que hoy):
 ```json
 {
   "numOrden": "OT-2026-001",
   "docEntry": 500,
-  "proceso": "DESPACHO",
+  "proceso": "ULTIMA MILLA",
   "codMotivo": "M01",
-  "idEstadoCubico": 6,
+  "idEstadoCubico": 23,
   "codigoAlmacen": "ALM01",
   "notificacion": "cierre",
   "detalle": [