AgriCraft

AgriCraft

30M Downloads

Betterweather plugin appears to be using a function in the BETA BW release

waidesoper opened this issue · 3 comments

commented

Quick description

Crash on world load when Agricraft 3.0.1 is installed with Better Weather 1.2.9. The error is No Class Defined corgitaco/betterweather/api/Climate. I've checked the source for that release of Better Weather (which is the only non-beta release on curseforge) and it is indeed missing. The function is available in the Beta release 2.0.

Related issues and searches

  • Better Weather
  • Climate

Mods and versions used

  • Minecraft:
  • AgriCraft:
  • InfinityLib:
  • Better Weather 1.2.9

Steps to replicate the issue

  1. Load minecraft with the above mods
  2. Witness the fabulous crash

Expectations and explanations

I don't think it is meant to crash, especially when it is listed as one of the compatible mods on the Curseforge page. If it is going to continue to rely on beta software, perhaps listing the version required could point people in the right direction.

Your environment

Links to crash logs, stack traces, etc

Screenshots (or videos)

commented

I'm confused, is the version I'm depending on not available on CF, or do you want me to define a minimum version for the dependency?

commented

The release version on curseforge, and the main version on GitHub both lack the climate class that you are importing. The version you are using is on cf, but it's listed as a beta, and not obvious that it should be used. I think the simplest fix is to just define the minimum version

commented

Yes I need those classes for compat, it was me who requested them ages ago.
I wasn't aware it was still in beta.
I'll define a minimum version.