From 3bce3ed1177ce5a254aa294f9c38041a4783d0f6 Mon Sep 17 00:00:00 2001 From: George Date: Wed, 14 Oct 2026 19:51:11 -0500 Subject: [PATCH] Quitar dos copias exactas sin llamadores: HelperHojaRuta.GetConsolidado y HRRepository.UpdateCreateCreditNotes Quedan HelperStock.GetConsolidado y CreditNotesRepository.UpdateCreateCreditNotes, que son los que usan Picking, Inventario y Recibo. Sin cambios de comportamiento. Co-Authored-By: Claude Sonnet 5.5 --- CLAUDE.md | 7 +++- .../HelperData/HojaRuta/HelperHojaRuta.cs | 37 ------------------- Net.Data/ServicesLayer/HR/HRRepository.cs | 24 ------------ 3 files changed, 5 insertions(+), 63 deletions(-) diff --git a/CLAUDE.md b/CLAUDE.md index 1467d1b..6f0c095 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -94,8 +94,11 @@ transferencias siempre, sin depender de `Notificacion`. Detalle y tabla en `CreateHojaRuta`) y se agregaron comentarios en Picking, Recibo, `ConnectionServiceLayer`, `CreditNotesRepository`, `InvoiceRepository` y `RepositoryWrapper`. Además se unificó `LoginPost()` en `Login()` (eran idénticos; - las 4 llamadas a `LoginPost` ahora llaman a `Login`). Sigue pendiente: código - comentado, `PatchAsyncSBA` y la redundancia de las transferencias automáticas + las 4 llamadas a `LoginPost` ahora llaman a `Login`) y se quitaron dos copias + exactas sin llamadores: `HelperHojaRuta.GetConsolidado` (queda `HelperStock.GetConsolidado`) + y `HRRepository.UpdateCreateCreditNotes` (queda `CreditNotesRepository`). Sigue pendiente: código + comentado, `PatchAsyncSBA`, repositorios de compras sin uso + (`SetCreatePurchaseInvoices` duplicado en `PurchaseOrdersRepository`) y la redundancia de las transferencias automáticas (3 copias en Picking/Recibo) y de los métodos POST/PATCH del cliente. - 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.Data/HelperData/HojaRuta/HelperHojaRuta.cs b/Net.Data/HelperData/HojaRuta/HelperHojaRuta.cs index 67a7c09..6d91843 100644 --- a/Net.Data/HelperData/HojaRuta/HelperHojaRuta.cs +++ b/Net.Data/HelperData/HojaRuta/HelperHojaRuta.cs @@ -56,42 +56,5 @@ namespace Net.Data.HelperData.HojaRuta return ListaStock; } - - public static List GetConsolidado(string ItemCode, string FromWarehouseCode) - { - List ListaStock = new List(); - DataTable dt = new DataTable(); - using (HanaConnection con = new HanaConnection(LocalData.ConnectionString)) - { - con.Open(); - HanaCommand cmd = new HanaCommand($"CALL\"{LocalData.Schema}\".\"ITCA_WMS_SAP_GetStockConsolidado\"(?, ?)", con); - cmd.Parameters.Add("ItemCode", HanaDbType.VarChar).Value = ItemCode ?? ""; - cmd.Parameters.Add("FromWarehouseCode", HanaDbType.VarChar).Value = FromWarehouseCode ?? ""; - HanaDataAdapter da = new HanaDataAdapter(cmd); - if (da != null) - { - da.Fill(dt); - } - con.Close(); - } - - for (int x = 0; x < dt.Rows.Count; x++) - { - ListaStock.Add(new BE_ItemStock() - { - BinCode = (dt.Rows[x]["BinCode"] != DBNull.Value) ? Convert.ToString(dt.Rows[x]["BinCode"]) : string.Empty, - DistNumber = (dt.Rows[x]["DistNumber"] != DBNull.Value) ? Convert.ToString(dt.Rows[x]["DistNumber"]) : string.Empty, - ExpDate = (dt.Rows[x]["ExpDate"] != DBNull.Value) ? Convert.ToDateTime(dt.Rows[x]["ExpDate"]) : DateTime.Now, - ItemCode = (dt.Rows[x]["ItemCode"] != DBNull.Value) ? Convert.ToString(dt.Rows[x]["ItemCode"]) : string.Empty, - ItemName = (dt.Rows[x]["ItemName"] != DBNull.Value) ? Convert.ToString(dt.Rows[x]["ItemName"]) : string.Empty, - OnHandQty = (dt.Rows[x]["OnHandQty"] != DBNull.Value) ? Convert.ToDouble(dt.Rows[x]["OnHandQty"]) : 0, - WhsCode = (dt.Rows[x]["WhsCode"] != DBNull.Value) ? Convert.ToString(dt.Rows[x]["WhsCode"]) : string.Empty, - WhsName = (dt.Rows[x]["WhsName"] != DBNull.Value) ? Convert.ToString(dt.Rows[x]["WhsName"]) : string.Empty, - AvgPrice = (dt.Rows[x]["AvgPrice"] != DBNull.Value) ? Convert.ToDouble(dt.Rows[x]["AvgPrice"]) : 0, - }); - } - - return ListaStock; - } } } diff --git a/Net.Data/ServicesLayer/HR/HRRepository.cs b/Net.Data/ServicesLayer/HR/HRRepository.cs index 4b2d599..246e4ad 100644 --- a/Net.Data/ServicesLayer/HR/HRRepository.cs +++ b/Net.Data/ServicesLayer/HR/HRRepository.cs @@ -116,30 +116,6 @@ namespace Net.Data.ServicesLayer.HR } - public async Task> UpdateCreateCreditNotes(dynamic entidad, int DocEntry) - { - _metodoName = regex.Match(System.Reflection.MethodBase.GetCurrentMethod().DeclaringType.Name).Groups[1].Value.ToString(); - - ResultadoTransaccion resultadoTransaccion = new ResultadoTransaccion(_metodoName, _aplicacionName); - - try - { - //var cadena = "Drafts"; - var cadena = $"CreditNotes({DocEntry})"; - var data = await _connectServiceLayer.PatchAsyncSBA(cadena, entidad); - if (!data.Exito) - { - return resultadoTransaccion.GetInformacionResultado(-1, -1, data.Mensaje); - } - - return resultadoTransaccion.GetInformacionResultado(0, "", "Se Listo correctamento...!!!", data); - } - catch (Exception ex) - { - return resultadoTransaccion.GetInformacionResultado(-1, -1, ex.Message.ToString()); - } - } - }