SuperMartijn642 / SimpleMagnets

3 stars 12 forks source link

[Bug] Magnet not disabled by Botania’s solegnolia flower #36

Closed Nevrai closed 2 years ago

Nevrai commented 2 years ago

The mod Botania has a flower called the solegnolia, which disables magnetization rings in a small radius, but it doesn’t disable Simple Magnets’ magnet.

I realized whilst writing this that Simple Magnets does have a demagnetization coil, but it would be nice if Botania’s solegnolia also worked.

Versions

SuperMartijn642 commented 2 years ago

The magnets will ignore any item entities which have the PreventRemoteMovement nbt tag. This a community standard which pretty much all magnet mods respect. This includes larger mods such as Thermal Innovation. This is also how other mods' anti-magnet blocks work, for example Demagnetize.

I will not be adding specific compatibility for Botania. Doing so means I would have to add specific compatibility for every mod which adds some sort of anti-magnet block. Instead, I suggest Botania also uses the PreventRemoteMovement nbt tag. This would also make botania compatible with other magnet mods (such as Thermal Innovation, Tiered Magnets, Ring of Attraction, Ender Magnet, etc.).