Skip to content

fix(api): handle null kill counts with COALESCE in ranking query (#84) - #101

Open
angelTomo9 wants to merge 1 commit into
Bitcoindefi:mainfrom
angelTomo9:fix/ranking-500-clean-db-84
Open

fix(api): handle null kill counts with COALESCE in ranking query (#84)#101
angelTomo9 wants to merge 1 commit into
Bitcoindefi:mainfrom
angelTomo9:fix/ranking-500-clean-db-84

Conversation

@angelTomo9

Copy link
Copy Markdown

Resumen de cambios

Resuelve #84 envolviendo las columnas ciudadanos_matados y criminales_matados en COALESCE(..., 0) dentro de la query SQL de listCharacterRanking para evitar que valores NULL propaguen un error o cálculo inválido en bases de datos recién inicializadas o limpias.

Puntos clave

  • Causa raíz: En bases limpias o con personajes sin inicializar en contadores de muertes, (c.ciudadanos_matados + c.criminales_matados) evaluaba a NULL generando fallos de ordenación o respuesta inconsistente.
  • Solución: Usar (COALESCE(c.ciudadanos_matados, 0) + COALESCE(c.criminales_matados, 0)) AS count_killed.
  • Trazabilidad y Fail-Safe: Añadido console.error con stack trace completo en el bloque catch de GET /ranking.

Closes #84

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

El endpoint de ranking devuelve 500 en un entorno limpio

1 participant