From f886370d5cefe0f05904cad64e5cdeb91fcaa2b4 Mon Sep 17 00:00:00 2001 From: Administrator Date: Sun, 28 Jun 2026 23:03:21 -0500 Subject: [PATCH] Actualizar backend/services.py --- backend/services.py | 142 ++++++++++++++++---------------------------- 1 file changed, 52 insertions(+), 90 deletions(-) diff --git a/backend/services.py b/backend/services.py index 11702b5..bc34d4d 100644 --- a/backend/services.py +++ b/backend/services.py @@ -1,4 +1,3 @@ -# backend/services.py """ Capa de servicios: envuelve la lógica existente (modules/) con caché. NO modifica la lógica de negocio — solo la llama y serializa el resultado. @@ -14,14 +13,12 @@ from cache_manager import cache_get_or_set # Instancia única del DataManager (como @st.cache_resource) _DM = None - def get_dm() -> DataManager: global _DM if _DM is None: _DM = DataManager() return _DM - def _serializar_dicts(datos): out = [] for d in (datos or []): @@ -31,7 +28,6 @@ def _serializar_dicts(datos): out.append(fila) return out - def _serializar_filas(filas, limite=None): out = [] for f in (filas or []): @@ -39,21 +35,30 @@ def _serializar_filas(filas, limite=None): out.append([str(v) if v is not None else "" for v in vals]) return out - # ── OCUPABILIDAD ─────────────────────────────────────────────────────────── def ocupabilidad(ano, mes, sede="TODOS", programa="TODOS"): def _load(): dm = get_dm() analizador = AnalizadorCursos(dm) + print(f"[DEBUG Ocupabilidad] Buscando ano={ano}, mes={mes}, sede={sede}, programa={programa}") + + # Intento 1: Formato original (ej: "6") datos = analizador.obtener_datos_procesados(str(ano), str(mes), sede, programa) + + # Intento 2: Si viene vacío, probar con formato de dos dígitos (ej: "06") + if not datos and len(str(mes)) == 1: + mes_padded = str(mes).zfill(2) + print(f"[DEBUG Ocupabilidad] Sin resultados. Reintentando con mes acolchado: '{mes_padded}'") + datos = analizador.obtener_datos_procesados(str(ano), mes_padded, sede, programa) + filas = analizador.formatear_datos_para_tabla(datos) + print(f"[DEBUG Ocupabilidad] Proceso completado. Registros encontrados: {len(datos) if datos else 0}") return { "datos": _serializar_dicts(datos), "filas": _serializar_filas(filas), } return cache_get_or_set("ocupabilidad", (ano, mes, sede, programa), _load) - # ── VENTAS ───────────────────────────────────────────────────────────────── def ventas(ano, mes, sede="TODOS", programa="TODOS"): def _load(): @@ -62,17 +67,28 @@ def ventas(ano, mes, sede="TODOS", programa="TODOS"): dm._fecha_canc_overrides = comisiones_overrides_fecha() # {mat: fecha_canc override} dm._inv_neta_overrides = comisiones_overrides_inv_neta() # {mat: inversion_neta override} logic = VentasLogic(dm) - datos = logic.obtener_datos_brutos_filtrado(str(ano), str(mes), sede, programa) \ - if hasattr(logic, "obtener_datos_brutos_filtrado") else logic.obtener_datos_brutos(str(ano), str(mes)) - datos = [d for d in datos if d.get('VENDEDOR', 'SIN VENDEDOR') != 'SIN VENDEDOR'] + print(f"[DEBUG Ventas] Buscando ano={ano}, mes={mes}, sede={sede}, programa={programa}") + + mes_str = str(mes) + datos = logic.obtener_datos_brutos_filtrado(str(ano), mes_str, sede, programa) \ + if hasattr(logic, "obtener_datos_brutos_filtrado") else logic.obtener_datos_brutos(str(ano), mes_str) + + # Intento 2: Mes de dos dígitos si el original no trajo nada + if not datos and len(mes_str) == 1: + mes_padded = mes_str.zfill(2) + print(f"[DEBUG Ventas] Sin resultados. Reintentando con mes acolchado: '{mes_padded}'") + datos = logic.obtener_datos_brutos_filtrado(str(ano), mes_padded, sede, programa) \ + if hasattr(logic, "obtener_datos_brutos_filtrado") else logic.obtener_datos_brutos(str(ano), mes_padded) + + datos = [d for d in (datos or []) if d.get('VENDEDOR', 'SIN VENDEDOR') != 'SIN VENDEDOR'] filas = logic.formatear_datos_para_tabla(datos) + print(f"[DEBUG Ventas] Proceso completado. Registros encontrados: {len(datos)}") return { "filas": _serializar_filas(filas), "datos": _serializar_dicts(datos), } return cache_get_or_set("ventas", (ano, mes, sede, programa), _load) - def _clasif_filtro_programa(dm, dscp): """Misma clasificación de programa que ejecutar_consulta_ventas (SEMINARIOS/TEAC/TERC/OTROS).""" up = str(dscp or "").upper() @@ -83,7 +99,6 @@ def _clasif_filtro_programa(dm, dscp): return cat return fdata.get("DEFAULT", "OTROS") if isinstance(fdata.get("DEFAULT"), str) else "OTROS" - def ventas_detalle(vendedor, ano, mes, tipo_lista, sede="TODOS", programa="TODOS"): def _load(): dm = get_dm() @@ -93,8 +108,7 @@ def ventas_detalle(vendedor, ano, mes, tipo_lista, sede="TODOS", programa="TODOS resultado = logic.obtener_detalle_vendedor(vendedor, str(ano), str(mes)) filas = resultado.get(tipo_lista, []) if isinstance(resultado, dict) else [] filas = _aplicar_overrides_comisiones(filas) - # Filtro por SEDE [14] y PROGRAMA (clasificado desde el nombre [16]), - # mismas reglas que la tabla principal. Omite la fila TOTAL GENERAL. + _sede = str(sede or "TODOS").upper() _prog = str(programa or "TODOS").upper() if _sede != "TODOS" or _prog != "TODOS": @@ -106,8 +120,6 @@ def ventas_detalle(vendedor, ano, mes, tipo_lista, sede="TODOS", programa="TODOS out.append(f); continue if _sede != "TODOS" and str(f[14]).strip().upper() != _sede: continue - # Clasificar programa por el nombre de CLASIFICACIÓN [16] (prioriza dsc_programa); - # si no existe, usar el mostrado [8]. _prog_nombre = f[16] if len(f) > 16 else f[8] if _prog != "TODOS" and _clasif_filtro_programa(dm, _prog_nombre) != _prog: continue @@ -116,10 +128,7 @@ def ventas_detalle(vendedor, ano, mes, tipo_lista, sede="TODOS", programa="TODOS return {"filas": _serializar_filas(filas)} return cache_get_or_set("ventas_det", (vendedor, ano, mes, tipo_lista, sede, programa), _load) - def comisiones_detalle_todos(ano, mes): - """Devuelve TODOS los matriculados (todos los años/meses) para el buscador. - Aplica overrides de Supabase. Cacheado globalmente (no depende del mes).""" def _load(): dm = get_dm() logic = VentasLogic(dm) @@ -128,9 +137,7 @@ def comisiones_detalle_todos(ano, mes): return {"filas": _serializar_filas(filas)} return cache_get_or_set("comisiones_det_todos", ("GLOBAL",), _load) - def comisiones_overrides_fecha(): - """Devuelve dict {num_matricula: fecha_cancelacion1_override} para clasificación.""" try: sb = _sb() res = sb.table("comisiones_overrides").select("num_matricula,fecha_cancelacion1").execute() @@ -139,8 +146,6 @@ def comisiones_overrides_fecha(): mat = str(r["num_matricula"]) v = r.get("fecha_cancelacion1") sv = "" if v is None else str(v).strip() - # Solo "__VACIO__" = fecha borrada a propósito → anula la del SQL. - # None/"" = la fila tiene override de OTROS campos pero no toco la fecha -> usar SQL. if sv == "__VACIO__": out[mat] = "__VACIO__" elif sv != "": @@ -149,10 +154,7 @@ def comisiones_overrides_fecha(): except Exception: return {} - def comisiones_overrides_inv_neta(): - """Devuelve dict {num_matricula: inversion_neta(float)} para la tabla principal. - El valor es el monto final en soles (no se re-convierte por TC).""" try: sb = _sb() res = sb.table("comisiones_overrides").select("num_matricula,inversion_neta").execute() @@ -164,19 +166,14 @@ def comisiones_overrides_inv_neta(): s = str(v).replace("S/", "").replace(",", "").strip() if s == "": continue - try: - out[str(r["num_matricula"])] = float(s) + try: out[str(r["num_matricula"])] = float(s) except Exception: pass return out except Exception: return {} - def _aplicar_overrides_comisiones(filas): - """Sobreescribe valores de cada fila con los guardados en Supabase (por num_matricula). - Índices de fila: [2]F.MAT [7]INV.NETA [9]F.INI [10]SALDO MAT [11]SALDO C1 - [12]DÍAS [13]VALOR ADIC [14]TIPO PROG [15]num_matricula. (5=PROMEDIO CUOTA)""" try: dm = get_dm() if not getattr(dm, "supabase_client", None): @@ -185,7 +182,6 @@ def _aplicar_overrides_comisiones(filas): ov = {str(r["num_matricula"]): r for r in (res.data or [])} if not ov: return filas - # columna override -> índice en la fila mapa = { "fch_matricula": 2, "promedio_cuota": 5, "inversion_neta": 7, "fecha_inicio": 9, "saldo_matricula": 10, "saldo_cuota1": 11, @@ -196,10 +192,8 @@ def _aplicar_overrides_comisiones(filas): if len(f) < 16: continue mat = str(f[15]) - # [16] = nombre para clasificar (si no vino, usar el mostrado [8]) - # [17] = flag "tiene override" (0 por defecto) if len(f) == 16: - f.append(f[8]) # buscador: no trae [16] → reutiliza el nombre mostrado + f.append(f[8]) if len(f) == 17: f.append("0") if mat in ov: @@ -209,30 +203,38 @@ def _aplicar_overrides_comisiones(filas): if val is None: continue if str(val) == "__VACIO__": - f[idx] = "" # forzar vacío explícito (override "borrar") + f[idx] = "" elif str(val) != "": f[idx] = val - f[17] = "1" # marcar fila como editada + f[17] = "1" return filas except Exception as e: print(f"[overrides comisiones] {e}") return filas - # ── COBRANZA ─────────────────────────────────────────────────────────────── def cobranza(ano, mes, sectorista="TODOS", agrupacion="SEDE"): def _load(): dm = get_dm() logic = CobranzaLogic(dm) - filas = logic.obtener_datos_tabla(str(ano), str(mes), sectorista, agrupacion) - sectoristas = logic.obtener_lista_sectoristas(str(ano), str(mes)) + print(f"[DEBUG Cobranza] Buscando ano={ano}, mes={mes}, sectorista={sectorista}, agrupacion={agrupacion}") + + mes_str = str(mes) + filas = logic.obtener_datos_tabla(str(ano), mes_str, sectorista, agrupacion) + + if not filas and len(mes_str) == 1: + mes_padded = mes_str.zfill(2) + print(f"[DEBUG Cobranza] Sin resultados. Reintentando con mes acolchado: '{mes_padded}'") + filas = logic.obtener_datos_tabla(str(ano), mes_padded, sectorista, agrupacion) + + sectoristas = logic.obtener_lista_sectoristas(str(ano), mes_str) + print(f"[DEBUG Cobranza] Proceso completado. Filas encontradas: {len(filas) if filas else 0}") return { "filas": _serializar_filas(filas), "sectoristas": sectoristas or ["TODOS"], } return cache_get_or_set("cobranza", (ano, mes, sectorista, agrupacion), _load) - def cobranza_detalle(grupo, ano, mes, sectorista, agrupacion="PROGRAMA"): def _load(): dm = get_dm() @@ -240,7 +242,6 @@ def cobranza_detalle(grupo, ano, mes, sectorista, agrupacion="PROGRAMA"): if agrupacion == "ASESOR": datos = logic.obtener_detalle_programa_formateado(str(ano), str(mes), grupo, "TODOS") elif agrupacion == "SEDE": - # Filtrar por sede usando el MISMO clasificador del processor (sede.json) crudos = logic.processor.obtener_detalle_programa(str(ano), str(mes), sectorista, "TODOS") grupo_up = str(grupo).upper() mats_sede = set() @@ -250,14 +251,12 @@ def cobranza_detalle(grupo, ano, mes, sectorista, agrupacion="PROGRAMA"): if sede_det == grupo_up: mats_sede.add(str(d.get("MATRICULA", ""))) todas = logic.obtener_detalle_programa_formateado(str(ano), str(mes), sectorista, "TODOS") - # Solo alumnos de la sede (sin la fila TOTAL global; el frontend recalcula el total) datos = [f for f in (todas or []) if str(f[0]) in mats_sede] - else: # PROGRAMA + else: datos = logic.obtener_detalle_programa_formateado(str(ano), str(mes), sectorista, grupo) return {"filas": _serializar_filas(datos)} return cache_get_or_set("cobranza_det", (grupo, ano, mes, sectorista, agrupacion), _load) - # ── RENTABILIDAD ─────────────────────────────────────────────────────────── def rentabilidad(ano, mes, sede="TODOS", filtro_prog="TODOS"): def _load(): @@ -270,7 +269,6 @@ def rentabilidad(ano, mes, sede="TODOS", filtro_prog="TODOS"): return {"filas": filas_s, "programas": nombres} return cache_get_or_set("rentabilidad", (ano, mes, sede, filtro_prog), _load) - def rentabilidad_detalle(programa, ano, mes): def _load(): dm = get_dm() @@ -279,7 +277,6 @@ def rentabilidad_detalle(programa, ano, mes): return {"filas": _serializar_filas(resultado)} return cache_get_or_set("rentabilidad_det", (programa, ano, mes), _load) - def rentabilidad_costos(programa, ano, mes): def _load(): dm = get_dm() @@ -287,7 +284,6 @@ def rentabilidad_costos(programa, ano, mes): return {"costos": logic.obtener_datos_costos_programa(programa, str(ano), str(mes))} return cache_get_or_set("rentabilidad_costos", (programa, ano, mes), _load) - # ── SALDO PENDIENTE ──────────────────────────────────────────────────────── def saldo_pendiente(tipo_cuota): def _load(): @@ -297,8 +293,7 @@ def saldo_pendiente(tipo_cuota): return {"datos": _serializar_dicts(datos)} return cache_get_or_set("saldo", (tipo_cuota,), _load) - -# ── PRECARGA GLOBAL (calienta todo el caché) ─────────────────────────────── +# ── PRECARGA GLOBAL ──────────────────────────────────────────────────────── def precargar_todo(): from datetime import datetime ahora = datetime.now() @@ -318,16 +313,13 @@ def precargar_todo(): except Exception as e: print(f"[precarga] error: {e}") - _refresh_count = 0 def _recalcular_clave(key): - """Dada una clave 'prefijo:arg1|arg2|...', recalcula su valor sin usar el caché viejo.""" from cache_manager import cache_invalidate try: prefijo, _, resto = key.partition(":") args = resto.split("|") if resto else [] - # Invalidar SOLO esta entrada para forzar su recálculo cache_invalidate(prefijo + ":" + resto if resto else prefijo) if prefijo == "ocupabilidad" and len(args) == 4: return ocupabilidad(args[0], args[1], args[2], args[3]) @@ -343,31 +335,24 @@ def _recalcular_clave(key): return cobranza_detalle_todos(args[0], args[1], args[2]) except Exception as e: print(f"[recalcular] {key}: {e}") - return None # los detalles puntuales se recalculan al pedirse - + return None def refrescar_todo(): - """Refresca SOLO las entradas ya cacheadas (no vacía el caché), para que los - meses ya visitados se mantengan rápidos. Cada ~1h recarga config de GitHub.""" global _refresh_count from cache_manager import cache_refresh_existing, cache_keys _refresh_count += 1 - # El hilo corre cada 900s (15 min); 4 ciclos ≈ 60 min → recargar config GitHub if _refresh_count % 4 == 0: try: get_dm().cargar_toda_configuracion() print("[config] Recarga de queries/JSON de GitHub completada") except Exception as e: print(f"[config] error al recargar GitHub: {e}") - - # Si el caché está vacío (primer arranque), hacer precarga normal if not cache_keys(): precargar_todo() else: cache_refresh_existing(_recalcular_clave) - -# ── GUARDAR COSTOS (override Supabase) ───────────────────────────────────── +# ── OPERACIONES EXTRA ────────────────────────────────────────────────────── def guardar_costos(num_indice, costos_inicial, costos_actual): dm = get_dm() ok1 = dm.guardar_override_costo(num_indice, "inicial", costos_inicial) @@ -377,17 +362,12 @@ def guardar_costos(num_indice, costos_inicial, costos_actual): cache_invalidate("rentabilidad_costos") return bool(ok1 and ok2) - -# ── CLASIFICACIÓN DE SEDE (sede.json) ────────────────────────────────────── def clasificar_programas(nombres): - """Devuelve {programa: sede} usando el clasificador del processor (sede.json).""" dm = get_dm() logic = CobranzaLogic(dm) return {n: str(logic.processor.clasificar_sede(n)) for n in nombres} - def cobranza_detalle_todos(ano, mes, sectorista="TODOS"): - """Trae TODOS los alumnos en UNA sola consulta (para export y buscador).""" def _load(): dm = get_dm() logic = CobranzaLogic(dm) @@ -415,8 +395,7 @@ def cobranza_detalle_todos(ano, mes, sectorista="TODOS"): return {"alumnos": alumnos} return cache_get_or_set("cobranza_det_todos", (ano, mes, sectorista), _load) - -# ── GESTIÓN DE USUARIOS (Supabase Auth + tabla perfiles) ─────────────────── +# ── GESTIÓN DE USUARIOS ──────────────────────────────────────────────────── def _sb(): dm = get_dm() if not dm.supabase_client: @@ -465,17 +444,13 @@ def usuarios_eliminar(user_id): sb.auth.admin.delete_user(user_id) return {"status": "ok"} - -# ── VENDEDORES MANUALES (Comisiones, en Supabase) ────────────────────────── -# ── CONFIG MENSUAL COMISIONES (IMP_TC, META, COMISIÓN por tipo de programa) ── +# ── CONFIG MENSUAL COMISIONES ────────────────────────────────────────────── def comisiones_config_listar(ano, mes): - """Lista la config de un mes (todas las filas por tipo_programa).""" sb = _sb() res = sb.table("comisiones_config_mensual").select("*").eq("ano", int(ano)).eq("mes", int(mes)).execute() return {"config": res.data or []} def comisiones_config_guardar(ano, mes, filas): - """Reemplaza la config del mes. filas = [{tipo_programa, imp_tc, meta, comision}].""" sb = _sb() sb.table("comisiones_config_mensual").delete().eq("ano", int(ano)).eq("mes", int(mes)).execute() payload = [] @@ -495,7 +470,6 @@ def comisiones_config_guardar(ano, mes, filas): return {"status": "ok"} def comisiones_tc_del_mes(ano, mes): - """Devuelve el IMP_TC configurado para el mes (toma el primero que tenga TC > 0), o None.""" try: sb = _sb() res = sb.table("comisiones_config_mensual").select("imp_tc").eq("ano", int(ano)).eq("mes", int(mes)).execute() @@ -506,11 +480,10 @@ def comisiones_tc_del_mes(ano, mes): pass return None - def vendedores_manuales_listar(ano, mes): sb = _sb() query = sb.table("vendedores_manuales").select("*").eq("ano", int(ano)) - if int(mes) != 0: # mes=0 → traer todo el año (para cache en frontend) + if int(mes) != 0: query = query.eq("mes", int(mes)) res = query.execute() return {"vendedores": res.data or []} @@ -530,7 +503,6 @@ def vendedores_manuales_eliminar(id): sb.table("vendedores_manuales").delete().eq("id", id).execute() return {"status": "ok"} - def vendedores_manuales_actualizar(id, descripcion, fch_emision, monto): sb = _sb() sb.table("vendedores_manuales").update({ @@ -539,14 +511,9 @@ def vendedores_manuales_actualizar(id, descripcion, fch_emision, monto): }).eq("id", id).execute() return {"status": "ok"} - def vendedores_manuales_guardar_lote(nombre, ano, mes, filas): - """Reemplaza TODAS las filas de un vendedor manual en una sola operación. - filas = lista de dicts {descripcion, fch_emision, monto}. Si filas vacía → elimina el vendedor.""" sb = _sb() - # Borrar las existentes de ese vendedor/mes sb.table("vendedores_manuales").delete().eq("nombre", nombre).eq("ano", int(ano)).eq("mes", int(mes)).execute() - # Insertar las nuevas en un solo batch payload = [] for f in filas: payload.append({ @@ -560,12 +527,8 @@ def vendedores_manuales_guardar_lote(nombre, ano, mes, filas): sb.table("vendedores_manuales").insert(payload).execute() return {"status": "ok"} - - - -# ── OVERRIDES DE COMISIONES (editar valores por num_matricula) ───────────── +# ── OVERRIDES DE COMISIONES ──────────────────────────────────────────────── def comisiones_override_guardar(registros): - """Guarda/actualiza overrides en UNA sola operación (batch upsert).""" sb = _sb() from cache_manager import cache_invalidate validos = [r for r in registros if r.get("num_matricula")] @@ -577,11 +540,10 @@ def comisiones_override_guardar(registros): return {"status": "ok"} def comisiones_override_restaurar(num_matricula): - """Borra el override de un alumno (vuelve a los valores de SQL).""" sb = _sb() from cache_manager import cache_invalidate sb.table("comisiones_overrides").delete().eq("num_matricula", str(num_matricula)).execute() cache_invalidate("ventas_det") cache_invalidate("ventas") cache_invalidate("comisiones_det_todos") - return {"status": "ok"} + return {"status": "ok"} \ No newline at end of file