Debugify icon indicating copy to clipboard operation
Debugify copied to clipboard
verified publisher icon isXander

[Patch Request] MC-279003

Open fishstiz opened this issue 11 months ago • 1 comments

Mojang Bug Report ID

MC-279003

Bug adhere to the guidelines.

  • [x] I made sure that this bug adheres to the guidelines.

Has not already been requested.

  • [x] I am certain this patch has not already been requested.

Has not already been implemented.

  • [x] I know that this patch isn't already in the latest version of the mod.

Existing implementation

No response

fishstiz avatar Mar 25 '25 03:03 fishstiz

In 1.21.3, ServerExplosion#hurtEntities directly used Entity#setDeltaMovement(Vec3) to move entities. In 1.21.4, the wrapper method Entity#push(Vec3) is used instead. The issue is that AbstractWindCharge overrides Entity#push(double, double, double) but does not do anything with it, so wind charges are not propelled by explosions.

Calling on the super method of Entity#push(double, double, double) in AbstractWindCharge fixes it.

I've looked at the usages of AbstractWindCharge#push(double, double, double) for side effects and tested it in game, and so far it doesn't look like are any.

I could make a PR for this but dunno what counts as a "Basic" and "Gameplay" fix

fishstiz avatar Mar 25 '25 03:03 fishstiz