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
Incompatible with Quilt modloader #21
Comments
I just ran my mod through the quilt mod loader and it worked fine(but I was using the fabric api not quilt). What files did you have in the mods folder at the time of running it? |
I believe I was only using Combat Enchantments and Quilted Fabric API at the time. I usually make fresh instances for debugging crashes like this (and have since deleted that one) but the logs don't seem to show any other mods being present. |
Yep, because it's incompatible with quilt api, not quilt itself.
|
Well it wasn't @ModifyVariable(at = @At("STORE"), method = "onStoppedUsing", ordinal = 0 )
public float onStoppedUsing(float value, ItemStack stack, World world, LivingEntity user)
{
StatusEffectInstance barrageEffectInstance = user.getStatusEffect(CombatEnchants.BARRAGE_EFFECT);
if(barrageEffectInstance != null)
{
if(barrageEffectInstance.getDuration() >= 20)
{
user.removeStatusEffect(CombatEnchants.BARRAGE_EFFECT);
user.addStatusEffect(new StatusEffectInstance(CombatEnchants.BARRAGE_EFFECT, 20));
}
return 1;
}
return value;
} the result is this same, but doesn't crash with quilt api. |
oh shoot didnt realize people were talking here. @SzczurekYT can you make a pull request with the new code? |
I was too lazy to do that as it's just a simple copy paste, but if you want then sure. |
Minecraft version:
1.19.2
Quilt version:
0.18.1-beta52
Quilted Fabric API version:
4.0.0-beta26
Combat Enchantments version:
2.14.2
Crash Report
Latest Log
Quilt is a fork of the Fabric modloader which is automatically compatible with most mods (out of the 300 I use, only 1 other is incompatible due to a mixin.) Unfortunately, it seems Combat Enchantments has a similar issue, and results in a startup crash. A possible fix would be much appreciated.
The text was updated successfully, but these errors were encountered: