Skip to content
This repository was archived by the owner on Dec 30, 2022. It is now read-only.
This repository was archived by the owner on Dec 30, 2022. It is now read-only.

[BUG] Mods flagged outdated while they're actually up to date #52

Closed

Description

@FPSUsername

Describe the bug
Whenever I manually update a mod that's available on modrinth and curseforge (downloaded the mod from curseforge because it's not updated on modrinth yet), the modmanager flags the mod as outdated.
In addition, for Mousewheelie the modmanager downloads an outdated version (even though it's up to date on modrinth)

To Reproduce
Steps to reproduce the behavior:

  1. Install the latest mousewheelie mod (currently version mousewheelie-1.7.3-newconfig.3+mc1.17.1-pre1
  2. Install the mod
  3. Launch Minecraft and the mod manager
  4. Mod is flagged outdated
  5. Update the mod through the mod manager and see that it downloads version mousewheelie-1.7.3+mc1.17.1-pre1

Expected behavior
The mod manager should know that it's up to date

Screenshots
Screenshot_20210912_140322
Screenshot_20210912_141121

Please complete the following information:

  • OS: Arch Linux
  • Minecraft version: 1.17.1
  • Version 1.0.2-alpha

Logs
Latest log: https://pastebin.com/T9rLHVTi
(it also contains an error where it can't find a jar for LibZoomer, I think that's part of the OkZoomer mod)

Activity

  1. changed the title [-][BUG] Mods flagged outdated while they're actually newer[/-] [+][BUG] Mods flagged outdated while they're actually up to date[/+] on Sep 12, 2021
  2. DeathsGun commented on Sep 12, 2021

    @DeathsGun
    Member

    The log isn't available anymore on pastebin. Can you upload it here directly via drag n drop?

  3. FPSUsername commented on Sep 12, 2021

    @FPSUsername
    Author
  4. DeathsGun commented on Sep 13, 2021

    @DeathsGun
    Member

    Looks like mousewheelie isn't following the SemVer versioning scheme which fabric has implemented here.
    I could fix it so that when two versions are the same the release date is compared and the newer gets selected

  5. FPSUsername commented on Sep 13, 2021

    @FPSUsername
    Author

    Oh okay, that makes sense, it's what I was thinking of as well. Not every plugin/mod follows the version scheme of spigot/fabric, especially with beta/alpha releases.
    The solution you mention seems good to me (of course it needs to be checked for the MC version, but I bet that's already happening).

  6. DeathsGun commented on Sep 13, 2021

    @DeathsGun
    Member

    Yeah it filters automatically for the fabric loader and the correct mc version

  7. added this to the 1.1.0+1.17-alpha milestone on Sep 15, 2021
  8. DeathsGun commented on Sep 27, 2021

    @DeathsGun
    Member
  9. FPSUsername commented on Sep 28, 2021

    @FPSUsername
    Author

    Tried it, but unfortunately, mousewheelie isn't showing up anymore as outdated.
    However, I found new issues:
    Lithium's latest release is 0.7.4:
    image

    Heads down display is version 0.1.1, which I also have installed, yet the mod manager thinks it's version 0.0.0.
    image

    Dynamic FPS has a different version number on modrinth compared to what's in the jar file.
    image

    Last but not least, a miscellaneious:
    I use a fork of the Iris shader which also has its own version of sodium. Sodium is build in into the new Iris shader, but not yet on the forked version.
    image

    Using the official Iris release (Modrinth):
    image

  10. K0-RR commented on Sep 28, 2021

    @K0-RR

    Heads down display is version 0.1.1, which I also have installed, yet the mod manager thinks it's version 0.0.0.

    This issue is caused by the Heads down display itself - https://github.com/auoeke/heads-down-display/blob/1.17/src/main/resources/fabric.mod.json

  11. DeathsGun commented on Sep 28, 2021

    @DeathsGun
    Member

    I would suggest the following:

    • Moving the version comparing logic in an own class so it can be used in tests
    • Writing tests that will specifically target such invalid version numbers
    • Most important Testing other SemVer libraries which maybe are more complete than fabric's implementation
  12. DeathsGun commented on Sep 29, 2021

    @DeathsGun
    Member

    This version improves it a bit, but for mods who are declaring different version id's on modrinth and in their metadata is something I can't fix. You could only ask them nicely if they would follow the SemVer spec and only use the name field for differentiating between forge and fabric versions.

    modmanager-1.1.0+1.17-alpha.jar.zip

  13. DeathsGun commented on Oct 1, 2021

    @DeathsGun
    Member

    If the anymore issues with versions not following SemVer then please consider asking the mod author to change their versioning

  14. added a commit that references this issue on Dec 3, 2021
    c6d3b86
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions