Bawnorton / BetterTrims

Add new armour trims and give each trim a unique feature.
12 stars 3 forks source link

[Bug Report] list of bugs I found #29

Closed peow closed 8 months ago

peow commented 10 months ago

What version of the mod are you using BetterTrims v2.0.5 (Fabric 1.20.1) AllTheTrims v3.3.7

I liked your mod, I tested it to put it in my modpack and ended up finding some problems:

/summon zombie ~ ~ ~ {HandItems:[{id:"minecraft:iron_sword",Count:1b,tag:{Enchantments:[{id:"minecraft:knockback",lvl:2s}]}},{}]}

I'm also experiencing an incompatibility problem with Slime Material, as it hasn't removed my fall damage, I don't know which other mod is causing this

Fire Charge Material - when a zombie attacks me and it starts to catch fire, my game crashes crash-2023-12-05_13.33.38-server.txt


When using BetterTrims together with AllTheTrims (your other mod that allows all armor to receive Trims and all items to be used as Trims) some bugs happen

Elytra + Netherite Shield (from Guarding mod) count as armor just when you hold them, for example if I'm using Redstone Material on Elytra and Netherite Shield, I gain speed just by leaving them in my hand, without even equipping Elytra, and that It's a problem, because I can put the 4 pieces of armor with Redstone Material + Elytra with Redstone Material in my hand and the Netherite Shield with Redstone Material in the off-hand, and I'll have a much higher speed than with just the armor

In these images I'm using armor with Netherite Material and an Elytra with Netherite Material, I configured it so that each Trim gives me 25% immunity to fire, so 4 pieces give me complete immunity, notice that with 3 armor equipped and nothing selected in the Hotbar I still catch fire, but when selecting Elytra I become immune to lava

2023-11-30_21 19 19 2023-11-30_21 19 25

*this problem also happens with Custom Elytras from Clutter mod

my suggestion is that only what is equipped in the armor slots gives you effects, I believe that the Netherite Shield should not give you effects either, as it just stays in your hand and is not equipped on your body, but if you want to let it the Netherite Shield still gives effects to the Player, I would recommend putting a limit on how many pieces you can receive each effect, for example you can only receive speed effects from 4 pieces, so if I have full armor (4 pieces) + the Netherite Shield on hand, I will only receive the x4 effect, not x5, so if someone wants to use the Netherite Shield, they will have to put another Trim Material on it, such as Iron or Silver


*The "Revert" option in the mod settings is not returning the numbers to the default, but to the previous settings

image

*I would like to ask if it was possible to deactivate the effects of all Materials, currently only Coal and some others that can be deactivated

I found some bugs in your other mod AllTheTrims too and I will post it on GitHub, I ask you to check it out too

and i beg you, do an update to 1.20.1 in Fabric Loader 0.14.22 , I would really like to be able to use BetterTrims in my modpack

Bawnorton commented 10 months ago

Thank you for the comprehensive list of bugs. I'll look into addressing these after exam season.

Bawnorton commented 9 months ago

For the sake of tracking:

peow commented 8 months ago

i dont know if this is relevant for u but i discovered that the Depth Strider enchantment causes problems already in Vanilla Minecraft, https://bugs.mojang.com/browse/MC-136249

Bawnorton commented 8 months ago

Fixed in 2.1.0

peow commented 8 months ago

hello, using the newest version (v2.2.3) I saw that Slime Trim is not removing fall damage, even when setting it to the highest possible setting and using a full set

my log: latest.log