From 6386070f4eb607e9326f505a6193b6cb63c950bf Mon Sep 17 00:00:00 2001
From: George <george@MacBook-Air-de-George.local>
Date: Wed, 14 Oct 2026 10:15:40 -0500
Subject: [PATCH] =?UTF-8?q?Reopen=20de=20factura:=20tratar=20204=20sin=20c?=
 =?UTF-8?q?uerpo=20como=20exito=20y=20mostrar=20el=20motivo=20de=20SAP.=20?=
 =?UTF-8?q?Versi=C3=B3n=202.1.3.0?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

PostReOpen deserializaba el cuerpo vacio del 204 a null y fallaba al marcar
Exito, devolviendo "La Factura no se puede reabrir" aunque SAP la reabria.
Picking ahora agrega el mensaje real de SAP a ese error.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
---
 Net.Business.Services/Net.Business.Services.csproj    | 6 +++---
 Net.Connection.ServiceLayer/ConnectionServiceLayer.cs | 8 ++++++--
 Net.Data/WMS/Picking/PickingRepository.cs             | 4 ++--
 3 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/Net.Business.Services/Net.Business.Services.csproj b/Net.Business.Services/Net.Business.Services.csproj
index 1bc096a..ab7efce 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.2</Version>
-    <AssemblyVersion>2.1.2.0</AssemblyVersion>
-    <FileVersion>2.1.2.0</FileVersion>
+    <Version>2.1.3</Version>
+    <AssemblyVersion>2.1.3.0</AssemblyVersion>
+    <FileVersion>2.1.3.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.Connection.ServiceLayer/ConnectionServiceLayer.cs b/Net.Connection.ServiceLayer/ConnectionServiceLayer.cs
index bd73740..0ca46ac 100644
--- a/Net.Connection.ServiceLayer/ConnectionServiceLayer.cs
+++ b/Net.Connection.ServiceLayer/ConnectionServiceLayer.cs
@@ -568,7 +568,11 @@ namespace Net.Connection.ServiceLayer
                     if (response_.StatusCode == System.Net.HttpStatusCode.OK || response_.StatusCode == System.Net.HttpStatusCode.Created || response_.StatusCode ==  HttpStatusCode.NoContent) // 200 or Created
                     {
                         // DESERIALIZAMOS Content DEL RESPONSE
-                        dynamic responseBody = JsonConvert.DeserializeObject<T>(responseText_);
+                        // Reopen responde 204 sin cuerpo: DeserializeObject devuelve null, asi que
+                        // se crea una instancia vacia para poder marcar el exito.
+                        dynamic responseBody = string.IsNullOrWhiteSpace(responseText_)
+                            ? Activator.CreateInstance<T>()
+                            : JsonConvert.DeserializeObject<T>(responseText_);
                         responseBody.Exito = true;
                         var estado = (int)response_.StatusCode;
                         responseBody.CodigoHttp = estado.ToString();
@@ -638,7 +642,7 @@ namespace Net.Connection.ServiceLayer
             }
             finally
             {
-                // NO UTILIZAMOS CATCH, 
+                // NO UTILIZAMOS CATCH,
                 // PASAMOS LA EXCEPCIÓN A LA APP.
             }
         }
diff --git a/Net.Data/WMS/Picking/PickingRepository.cs b/Net.Data/WMS/Picking/PickingRepository.cs
index ffe3818..2e9658c 100644
--- a/Net.Data/WMS/Picking/PickingRepository.cs
+++ b/Net.Data/WMS/Picking/PickingRepository.cs
@@ -163,7 +163,7 @@ namespace Net.Data.WMS.Picking
                                 var Response_Reopen = invoicestRepository.ReOpen(_invoices.DocEntry).Result;
                                 if (Response_Reopen.DocEntry == -1)
                                 {
-                                    return resultadoTransaccion.GetInformacionResultado("SAP: La Factura no se puede reabrir", entidad);
+                                    return resultadoTransaccion.GetInformacionResultado("SAP: La Factura no se puede reabrir. " + Response_Reopen.Message, entidad);
                                 }
                             }
 
@@ -482,7 +482,7 @@ namespace Net.Data.WMS.Picking
                             var Response_Reopen = invoicestRepository.ReOpen(_invoices.DocEntry).Result;
                             if (Response_Reopen.DocEntry == -1)
                             {
-                                return resultadoTransaccion.GetInformacionResultado("SAP: La Factura no se puede reabrir", entidad);
+                                return resultadoTransaccion.GetInformacionResultado("SAP: La Factura no se puede reabrir. " + Response_Reopen.Message, entidad);
                             }
                         }