uku3lig / bettershields

fixes missing shield sounds
Mozilla Public License 2.0
3 stars 0 forks source link

The shield break sound fix still plays even when it shouldn't. #1

Closed realFusionFusion closed 1 year ago

realFusionFusion commented 1 year ago

Let me explain. It isn't actually possible to break someone's shield if they are in their damage tick. You would have to wait for their damage tick to end before breaking their shield. But despite that the mod still plays a break shield sound if I hit a shielded player with an axe even if they are in their damage tick.

Damage tick = The red phase that the player goes through when they are hit.

uku3lig commented 1 year ago

i tried this in 1.19.4 and the shield breaks while the player is in the damage tick

edit: shield breaks on 1.18.2 too

realFusionFusion commented 1 year ago

Try doing the damage tick with something that does more damage than an axe spam hit like a fully charged sword hit or a crystal hit. Doing like this will not let you stack multiple hits like what happened when you tried testing it.

uku3lig commented 1 year ago

no matter what i try, the shield breaks every single time

edit: here's a video

https://github.com/uku3lig/bettershields/assets/61147779/8ef4a271-75a2-4cf3-98d2-7c424135244d

realFusionFusion commented 1 year ago

What?????????? That is very, very weird... I demonstrated this bug in a video and in it everything behaves exactly how I described to you (you can't disable shield while they are in their damage tick + it plays a break shield sound if you hit them with an axe anyways BUT only if you hit them with a sword first, not a crystal). Here's the video:

https://youtu.be/O7C4qZTMMGg (notice how I didn't even disable their shield, all my axe hits were within their damage tick)

What version did you play on in the video you showed? Maybe not being able disable a shield in the circumstance I showed you is actually a bug within the game itself?

Edit: I tested on LAN, on MCPvP and EUPvP, all while being on 1.20.1.

uku3lig commented 1 year ago

as far as i know, the axe needs to be fully loaded for the shield to be broken, so a sound is indeed played incorrectly, but for a different reason

also, i did my testing on 1.19.4 and 1.18.2, i'll test again in 1.20.1 just to make sure

realFusionFusion commented 1 year ago

Are there any updates?

realFusionFusion commented 1 year ago

Hi Uku, it has been 2 weeks since the bug report was posted, I am just wondering how things are going? I hope this message is not bothersome to you

uku3lig commented 1 year ago

even while being on a damage tick, hitting with a non fully charged axe breaks the shield

(sorry for the wait, i was busy and demotivated)

realFusionFusion commented 1 year ago

Right, if that is true then why is the video I sent saying otherwise? I am guessing that the video was simply forgotten, so here it is again: https://youtu.be/O7C4qZTMMGg (again look at all my axe hits, they all happen during their damage tick so notice how the second hit doesn't even disable the shield).

Maybe it happens because I hit them first with a sword and this is on a normal server and not a LAN server? If you want to see the bug for yourself then just try playing against someone who uses a shield on a normal server, it makes things so much simpler. It will take some time for the bug to happen in that case so keep that in mind.

uku3lig commented 1 year ago

i didn't forget your video, but i have no clue why the shield didn't break there

i'll try to do some testing when i can

realFusionFusion commented 1 year ago

Hey Uku, did you find anything yet? I hope this is not bothering you too much

uku3lig commented 1 year ago

im on vacation until the 15th :)

realFusionFusion commented 1 year ago

Oh, I see, enjoy your holiday! :D