Documentación del Sistema

Appearance

Preguntas sobre Referencias Contables ​

INFORMACION REQUERIDA - Preguntas generadas durante analisis retrospectivo el 2026-02-09

Preguntas Criticas (Afectan la documentacion de negocio) ​

1. Campos descri y marca en la tabla ref_con sin uso ​

Observacion: La migracion declara los campos descri (string, limit 1) y marca (string, limit 1) con comentario "Sin uso". Estos campos no se leen ni se escriben desde ningun endpoint.

Pregunta: Estos campos tenian algun proposito historico? Se pueden considerar obsoletos o hay planes de usarlos en el futuro?

Impacto: Afecta la completitud del schema documentado y si deben mencionarse como campos deprecados.

2. Campo impcom marcado como "Sin uso" en migracion pero si se usa en codigo ​

Observacion: El comentario de la migracion dice impcom - Sin uso, pero el modelo, el DTO y el frontend SI usan este campo como "Imputacion de Compras". Incluso MovimientoGananciaService y ComprobanteController lo utilizan para obtener la cuenta contable de compras del producto.

Pregunta: El comentario de la migracion esta desactualizado? Es impcom un campo activamente usado para la imputacion contable de compras?

Impacto: Critico - determina si la imputacion de compras es una funcionalidad oficial del recurso o una funcionalidad parcial/experimental.

3. Condicionalidad del campo Imputacion de Compras basada en permisos ​

Observacion: En el frontend (ref-contables.js, linea 112-116), el campo inputImpCompras solo se habilita si el usuario tiene el permiso modulo_compras. Si no tiene permiso, se deshabilita. Esto significa que la imputacion de compras es condicional.

Pregunta: Es correcto que la imputacion de compras solo se requiere cuando el modulo de compras esta habilitado? Que valor toma impcom si el modulo de compras no esta habilitado - queda null?

Impacto: Afecta la documentacion de reglas de negocio y validaciones condicionales.

4. Tabla compartida entre niveles EMPRESA y SUCURSAL ​

Observacion: La migracion define getDefaultLevels() como [LEVEL_EMPRESA, LEVEL_SUCURSAL]. Esto significa que la tabla ref_con existe tanto a nivel empresa como a nivel sucursal. Ademas, shouldExecute() depende de la habilitacion de modulos de ventas + (contabilidad o tesoreria) o compras.

Pregunta: Cada sucursal puede tener sus propias referencias contables independientes de las de la empresa? O es una configuracion dinamica donde algunos clientes usan nivel empresa (compartido) y otros nivel sucursal (independiente)?

Impacto: Afecta fundamentalmente la documentacion de multi-tenancy y como se describen las entidades de negocio.

5. Seed por defecto: cuenta 41150 ​

Observacion: El seed RefCon.php inserta por defecto una referencia "GEN" (General) con imputa = 41150 e impcom = 41150. Si ya existen registros sin imputacion, los actualiza con 41150. Ademas, asigna automaticamente a productos sin referencia.

Pregunta: La cuenta 41150 es una cuenta estandar del plan de cuentas del sistema? Que representa (ventas generales, ingresos por ventas)?

Impacto: Afecta la documentacion de valores por defecto y la comprension del proposito contable.

6. Relacion producto-referencia por codigo (string) en vez de ID (integer) ​

Observacion: La tabla producto referencia a ref_con mediante el campo refcon (VARCHAR 3) que corresponde a ref_con.codigo, NO a ref_con.id. No hay foreign key declarada en la migracion. La busqueda se hace via getByCodigo().

Pregunta: Es una decision de diseno legacy deliberada? Hay riesgo de inconsistencia si se modifica el codigo de una referencia contable que ya esta asignada a productos?

Respuesta: No posee FK de momento, sólo FK lógica como muchas de las FK del sistema

Impacto: Afecta la documentacion de relaciones entre entidades y los riesgos de integridad referencial.

Preguntas Tecnicas (Afectan la documentacion tecnica) ​

7. Casting imputa::bigint en queries ​

Observacion: Las queries del modelo castean imputa e impcom a bigint (imputa::bigint as "impVentas"), pero en la migracion estos campos son definidos como decimal(10). El DTO los declara como array|null (esperando un objeto cuenta contable con numero y nombre).

Pregunta: El tipo decimal(10) es correcto para almacenar numeros de cuenta contable? El casting a bigint pierde precision o es irrelevante dado que las cuentas son numeros enteros?

Impacto: Afecta la precision del schema documentado y los tipos de datos.

Como Usar Este Documento ​

Este documento contiene preguntas que surgieron durante el analisis del codigo. Cada pregunta incluye espacio para agregar la respuesta:

Formato de respuesta:

Respuesta:

Una vez respondidas, estas respuestas se incorporaran a la documentacion final.

Estado de Validacion ​

  • [ ] Preguntas criticas respondidas (6)
  • [ ] Preguntas tecnicas respondidas (1)
  • [ ] Respuestas validadas con stakeholders
  • [ ] Documentacion actualizada con respuestas