NachoSpigot icon indicating copy to clipboard operation
NachoSpigot copied to clipboard

Infinite stepping sound

Open andreasdc opened this issue 4 years ago • 7 comments

Observed Behavior

Sometimes players that are standing still are sending stepping sound infinitely.

Expected Behavior

Stepping sound should be played only when moving.

Steps To Reproduce

Join the server with lags? Can't tell exactly in what situations.

Plugin List

No response

Server Version

[10:16:30 INFO]: This server is running NachoSpigot version git-NachoSpigot-"393f913" (MC: 1.8.8) (Implementing API version 1.8.8-R0.2-SNAPSHOT) [10:16:30 INFO]: You are running the latest version

Other

Pretty old bug as I believe.

Agreements

  • [X] You were able to find this issue on the latest version of NachoSpigot.
  • [X] You have confirmed that there aren’t any issues open regarding this bug.
  • [X] You have confirmed that you are NOT using a fork of NachoSpigot in any way. YOUR changes are not OUR faults.

andreasdc avatar Nov 16 '21 16:11 andreasdc

https://github.com/CobbleSword/NachoSpigot/blob/98268b2bfbae71bb686a70db796940640540d277/NachoSpigot-Server/src/main/java/net/minecraft/server/Entity.java#L815 We could add check has player block changed but I don't think it would make sense since you can make a step sound on a block without changing position. Anyway, I decided to test will this check pass and won't break anything, surprisingly it doesn't break the steps sound - Should we make a config option for this?

sadcenter avatar Nov 16 '21 19:11 sadcenter

https://github.com/CobbleSword/NachoSpigot/blob/98268b2bfbae71bb686a70db796940640540d277/NachoSpigot-Server/src/main/java/net/minecraft/server/Entity.java#L815

We could add check has player position changed but I don't think it would make sense since you can make a step sound on a block without changing position. Anyway, I decided to test will this check pass and won't break anything, surprisingly it doesn't break the steps sound - Should we make a config option for this?

How do you want to make a step sound without moving?

andreasdc avatar Nov 16 '21 19:11 andreasdc

I meant to check if the block the player is standing on has changed if(positionChanged) {

sadcenter avatar Nov 16 '21 19:11 sadcenter

I meant to check if the block the player is standing on has changed if(positionChanged) {

"since you can make a step sound on a block without changing position"? https://github.com/CobbleSword/NachoSpigot/blob/98268b2bfbae71bb686a70db796940640540d277/NachoSpigot-Server/src/main/java/net/minecraft/server/Entity.java#L802 Isn't this line just broken?

andreasdc avatar Nov 16 '21 19:11 andreasdc

https://streamable.com/ntpdov "since you can make a step sound on a block without changing position"? I meant x, y, z as an integer. (Without fraction) I haven't changed the block but still made a sound.

sadcenter avatar Nov 16 '21 19:11 sadcenter

https://streamable.com/ntpdov "since you can make a step sound on a block without changing position"? I meant x, y, z as an integer. (Without fraction) I haven't changed the block but still made a sound.

Oh yea but you moved

andreasdc avatar Nov 16 '21 19:11 andreasdc

It's when two or more players are standing on the same block as far as I know.

RobinRMC avatar Dec 09 '21 21:12 RobinRMC