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

Updater doesn't find update on newer version because it only considers the build number #36

Open
asier-izquierdo opened this issue Jun 21, 2024 · 2 comments
Labels
bug Something isn't working Updater Related to the 'Updater' script

Comments

@asier-izquierdo
Copy link
Owner

No description provided.

@asier-izquierdo asier-izquierdo added bug Something isn't working Updater Related to the 'Updater' script labels Jun 21, 2024
@asier-izquierdo
Copy link
Owner Author

Con get() se busca el número de BUILD, y después en la ejecución se comparan. La situación cuando el server estaba en la 1.20.4 y la última update ha sido la 1.20.6 o 1.21 es que, como son actualizaciones más nuevas, la build era menor, por lo tanto, 1.20.4-555 es mayor que 1.21-30, así que técnicamente no encuentra una actualización.

Gracias a este fallo, se ha evitado que se actualice el servidor a la 1.21 que, a fecha de escritura, sigue en beta, sin embargo, es un error y para determinar si hay una actualización se deben comparar tanto las versiones como las builds, en ese orden:

1.20.6-555 vs 1.21-30 -> (suponiendo que el 1 es estático) ¿Es 20 mayor que 21?

  • Si no lo es, hay una actualización disponible
  • Si lo es (situación imposible) o son iguales, se compara la build, misma lógica.

@asier-izquierdo asier-izquierdo pinned this issue Jul 2, 2024
@asier-izquierdo
Copy link
Owner Author

Además, se debería comparar la actualización, es decir, el 6 en 1.20.6, de manera que si se comparan la 1.20.4 con la 1.20.6, a pesar de ser ambas versiones la 1.20, la actualización 6 es mayor (lógicamente).

Solo se deberían comparar las builds en caso de que tanto la versión y la actualización sean las mismas.

En caso de que una versión no tenga actualización (1.21 vs 1.21.1), se debería incluir un 0 por defecto (1.21.0 vs 1.21.1), por ejemplo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Updater Related to the 'Updater' script
Projects
None yet
Development

No branches or pull requests

1 participant