SpongeForge icon indicating copy to clipboard operation
SpongeForge copied to clipboard

Mod First Aid Disables shields when ran on a ForgeSponge Server

Open T0ED opened this issue 6 years ago • 14 comments

I am currently running

SpongeForge version: 7.1.7 Forge version: 14.23.5.2838 Java version: Java 8 Update 191 (64 bit) Operating System: Windows 10

Minecraft Version 1.12.2

Mods: Sponge 1.12.2-7.1.7 firstaid 1.6.12

Plugins: No additional plugins

Issue: All vanilla/modded Shields do not negate any damage. This issue is only present when running on a Sponge Forge server. Here is shield function with Sponge: https://youtu.be/icNkMsqpPRg Here is shield function on a Non-Sponge Forge server: https://youtu.be/icNkMsqpPRg

First Aid is not supposed to affect Shield functionality in any way shape or form. First Aid merely changes the the way Player Health works by adding locational hearts (hearts to body/head/legs etc).

In the video with Sponge, The shield did not negate any damage whatsoever and allowed the skeleton to land shots to the head.

Logs: There is no applicable log as this misbehavior does not produce any logs. Here is the issue I submitted to the developer of the Mod.

T0ED avatar Dec 05 '19 23:12 T0ED

Can you explain how to get a shield and how it's supposed to work, as if I've never heard or used the mod before?

I've a very good idea where the issue lies, but I need to know basically what to look for to step through debugging.

gabizou avatar Dec 05 '19 23:12 gabizou

Understood, I've updated my post.

T0ED avatar Dec 05 '19 23:12 T0ED

And can you just double check that shields work without first aid in general (Haven't tested shields in a while, so that'd be a good test case).

gabizou avatar Dec 05 '19 23:12 gabizou

Yep, I've tested several times and have included videos of the tests in my post. I actually found this bug whilst playing RLcraft and was able to Isolate it down to First Aid's incompatibility with Sponge Forge.

T0ED avatar Dec 06 '19 00:12 T0ED

So it's not just SpongeForge without any other mods, it's when it's in combination with First Aid only do shields not work, right?

gabizou avatar Dec 06 '19 01:12 gabizou

Yep, that's correct.

T0ED avatar Dec 06 '19 01:12 T0ED

Did anyone find a fix?

Gremzy avatar Feb 09 '20 05:02 Gremzy

Still looking for a fix https://github.com/ichttt/FirstAid/issues/73

blairwilson avatar Mar 07 '20 16:03 blairwilson

Is there any fix now on 2021, i hace the same issue with all updated:(

Dragonslayer02zx avatar Jun 27 '21 23:06 Dragonslayer02zx

this is still an issue, and pretty much everyone knows its an issue with having spongeforge and the first aid mod together. I dont know which one of the mods need to be fixed, but its been 2 years since this bug has been noticed, and its honestly kind of a big deal to not have shields work in a hardcore modpack like rlcraft.

bradleywoodrs avatar Aug 02 '21 15:08 bradleywoodrs

Please we need a fix

dDevTech avatar Aug 09 '21 15:08 dDevTech

Will take a look soon

clienthax avatar Oct 10 '21 23:10 clienthax

Sponge is calling the damage event (causing the bypass) before doing the shield negate check

Sponge shield:
EntityLivingBaseMixin.attackEntityFrom
  hookModAttack:
    ForgeHooks.onLivingAttack
    canBlockDamageSource
      true
      flag = true; (shield block)
    this.bridge$damageEntityHook:
      bridge$applyModDamage <-- shield negate not applied!
        onLivingHurt

Forge shield:
EntityLivingBase.attackEntityFrom
  ForgeHooks.onLivingAttack
  canBlockDamageSource
    true
      damageShield
      blockUsingShield
      amount = 0;
      flag = true; (shield block)
  damageEntity (0)
    onLivingHurt

clienthax avatar Oct 12 '21 13:10 clienthax

bump, this is till an issue

redalpha123123 avatar Nov 24 '21 00:11 redalpha123123