Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ordenación en el Ranking #189

Open
XtoreX opened this issue Mar 7, 2021 · 3 comments · May be fixed by #275
Open

Ordenación en el Ranking #189

XtoreX opened this issue Mar 7, 2021 · 3 comments · May be fixed by #275
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@XtoreX
Copy link
Contributor

XtoreX commented Mar 7, 2021

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 XtoreX added the bug Something isn't working label Mar 7, 2021
@XtoreX
Copy link
Contributor Author

XtoreX commented Mar 16, 2021

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

@nukeador nukeador added the help wanted Extra attention is needed label Jul 27, 2021
@palmerabollo
Copy link
Collaborator

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 added a commit to albfernandez/websegura that referenced this issue Oct 6, 2022
Corregimos la ordenacion en el ranking y posiblemente otros errores que ocurran al comparar como cadenas pensando que son numeros

Fix PucelaBits#189
@albfernandez
Copy link
Contributor

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.

@albfernandez albfernandez linked a pull request Oct 6, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants