diff --git a/CLAUDE.md b/CLAUDE.md index a8a578a..a3c14fa 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -10,7 +10,7 @@ comprimido (versión antigua)**. ## Versión y control de versiones -- **Versión actual: 2.1.7.0** (`Net.Business.Services.csproj`: `Version`, +- **Versión actual: 2.1.8.0** (rama `limpieza-2.1.8.0`; `main` = 2.1.7.0) (`Net.Business.Services.csproj`: `Version`, `AssemblyVersion`, `FileVersion`). Cada versión tiene su commit; al publicar se sube la versión (ver `publicar.ps1`). - **Git**: rama `main`, remoto `origin` = @@ -86,6 +86,15 @@ transferencias siempre, sin depender de `Notificacion`. Detalle y tabla en comparaba la serie origen (p. ej. `B005`) con `"03"` y siempre salía `FC10`. Pendiente: validar con contabilidad la serie de boletas, y que Recibo no genera transferencias por almacén destino (nunca llena `U_ITC_ALMDES`). +- **v2.1.8.0 — limpieza y comentarios** (rama `limpieza-2.1.8.0`, sin cambios de + comportamiento): se quitó código sin uso (`ValidarCertificado`, `Logout`, + `CancelAsyncSBA`, `DeleteAsyncSBA`, el `throw` inalcanzable de `PatchAsyncSBA`, + `HelperDB`, 11 propiedades de `RepositoryWrapper` sin uso, variables + `TotalDetalle_Parcial`/`_Invoice`/`SerieOrigen` y un `if` vacío en + `CreateHojaRuta`) y se agregaron comentarios en Picking, Recibo, + `ConnectionServiceLayer`, `CreditNotesRepository`, `InvoiceRepository` y + `RepositoryWrapper`. Sigue pendiente: código comentado, `Login`/`LoginPost` + duplicados y `PatchAsyncSBA`. - 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. @@ -139,11 +148,9 @@ Picking, columna `HORA` en `B_Log`. Lo que **no** se trajo: `Chatbots` y - `BE_ResultadoTransaccion.NombreEstacion`: setter con recursión infinita (bomba de tiempo; hoy nada lo asigna). - `ConnectionServiceLayer.PatchAsyncSBA`: con un `204` real devuelve `null` - sin marcar éxito. `Login()`/`LoginPost()` duplicados; `throw` inalcanzable - tras `return` en varios `catch`. -- `ValidarCertificado` acepta cualquier certificado TLS (siempre `true`). -- `HelperDB.SET(string query)` ejecuta SQL arbitrario; hoy sin llamadores - (candidato a borrar). + sin marcar éxito. `Login()`/`LoginPost()` duplicados. +- El cliente HTTP `bypass-ssl-validation` acepta cualquier certificado TLS hacia + el Service Layer (se quitó el método `ValidarCertificado`, que no se usaba). - `appsettings.json` (fuera de git, en `.gitignore`) contiene credenciales reales en texto plano (SAP Service Layer, **HANA `B1ADMIN`**, semilla JWT). Rotar la contraseña de `B1ADMIN` (estuvo en el código fuente) y evaluar un @@ -152,13 +159,9 @@ Picking, columna `HORA` en `B_Log`. Lo que **no** se trajo: `Chatbots` y `ITC_WMS_SAP` (`ITCA_LOG_INTEGRACION`), no en `LocalData.Schema`; si la tabla está en otro esquema el `INSERT` falla. `B_Log.Add` no tiene `try/catch`, así que un fallo del log puede devolver 500 tras haber creado el documento en SAP. -- Código sin uso (limpieza prevista en 2.1.8.0): `CancelAsyncSBA`, - `DeleteAsyncSBA`, `Logout`, `ValidarCertificado`, `HelperDB.SET`; 11 - propiedades de `RepositoryWrapper` que ningún controller usa - (`CreditNotes`, `InvoiceRepository`, `ReturnRequest`, `StockTransfer`, etc.); - `TotalDetalle_Parcial` y `var _Invoice` (bloque PICKING) en - `PickingRepository`; `SerieOrigen` en `ReciboRepository`; el `if` vacío tras - `StockTransfer` en Picking (un fallo de la transferencia se ignora). +- En Picking y Recibo, si la transferencia automática de stock falla, el `if` + de error está vacío: el fallo se ignora (la NC ya está creada) y no se informa + al consumidor. ## Despliegue diff --git a/Net.Business.Services/Net.Business.Services.csproj b/Net.Business.Services/Net.Business.Services.csproj index 7671394..5fa5f8d 100644 --- a/Net.Business.Services/Net.Business.Services.csproj +++ b/Net.Business.Services/Net.Business.Services.csproj @@ -3,9 +3,9 @@ netcoreapp3.1 ITCA_API_SAP_WMS - 2.1.7 - 2.1.7.0 - 2.1.7.0 + 2.1.8 + 2.1.8.0 + 2.1.8.0 24763424-3300-4c85-87e3-503f8fdebd14