search

PucelaBits / websegura

Analizamos y mostramos seguridad HTTPS de sitios web públicos, como medida para visualizar aquellos que pueden suponer un riesgo para sus usuarios.
https://websegura.pucelabits.org/
GNU Affero General Public License v3.0
29 stars 34 forks source link

Ordenación en el Ranking #189

Open XtoreX opened 3 years ago

XtoreX commented 3 years ago

Describe el problema El orden de la visualización del Ranking por provincias y CCAA no es claro.

Para reproducirlo Para para reproducir el problema:

  1. Ve a https://websegura.pucelabits.org/ranking/#ranking-autonomico
  2. Haz scroll hasta el gráfico de barras y verás que no se ordena ni alfabéticamente, ni por número de webs seguras, quizás es por porcentaje, pero al usuario no le quedará claro.
XtoreX commented 3 years ago

Screenshot_2021-03-16-05-35-39-510_com android chrome

palmerabollo commented 2 years ago

Sí parece algún tipo de bug, porque se está haciendo un sort(reverse = true, attribute = 'safeScore') pero no parece estar funcionando bien, ya que salen territorios desordenados (ej. Valladolid por delante de Cádiz cuando tiene peor safeScore).

albfernandez commented 2 years ago

El problema viene porque safeScore es un tipo string, y lo compara como tal Ej: '8' es mayor que '50'

He subido una PullRequest que se asegura que el safeScore generado en territories.json sea numérico. El caso de ordenación lo resuelve, y supongo que puede que algún posible error de comparaciones tambien.

De todas formas revisadla bien porque no controlo mucho de la estructura de la aplicación y puede ser que rompa otras cosas.