BaseMetals icon indicating copy to clipboard operation
BaseMetals copied to clipboard

Star steel buff only works when at full armor durability

Open DwarvenSteel25 opened this issue 6 years ago • 10 comments

I'm not sure if this is a bug or an intended feature but the star steel armor buff of the speed and jump buff, only works when at full armor durability. If it is intended seems harsh to me given that, this means even 1 damage makes the effect stop working.

This is in the 1.12 update using what is (at the time of posting) the latest release.

DwarvenSteel25 avatar Jun 25 '19 03:06 DwarvenSteel25

Not to sound anal but you should try, if possible, to post the specific versions of each mod used :) Also indicate the forge version and if applicable, what mod-pack and version this happens in. That's pretty much standard practice and will save everyone a lot of time 99% of the time. As for the armor effect, quite likely a bug on our side, I don't think even our team has that much of a sadist xD

YaibaToKen avatar Jun 25 '19 07:06 YaibaToKen

it is most likely this code: https://github.com/MinecraftModDevelopmentMods/BaseMetals/blob/3947842e12e5c649d7440b831a0ab3b757d34de1/src/main/java/com/mcmoddev/basemetals/properties/BMEPropertyBase.java#L17-L22 that is failing... because its comparing itemstacks and not doing it in a manner that is blind to item stack damage. A fix will be in -rc3... which will come when RL stops interfering :)

dshadowwolf avatar Jun 25 '19 10:06 dshadowwolf

Cold Iron act like this too(and I suppose every armor) Can't wait for RC3 then :D

drakray avatar Jul 30 '19 02:07 drakray

Can confirm for adamantite as well. Full durability gets the effect, any damage and it poofs away!

Akitori avatar Dec 13 '19 04:12 Akitori

Any ETA on this? will it be fixed in 1.12.2? thanks :)

drakray avatar Jan 15 '20 07:01 drakray

will be. Sorry - my paycheck keeps getting in the way of my hobbies, sadly

DRH

On Wed, Jan 15, 2020 at 2:28 AM Raymond Tremblay [email protected] wrote:

Any ETA on this? will it be fixed in 1.12.2? thanks :)

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/MinecraftModDevelopmentMods/BaseMetals/issues/450?email_source=notifications&email_token=AACL7IQCRRPMOAR5ZJT2ESTQ523JLA5CNFSM4H3D6SEKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEI7KTSY#issuecomment-574532043, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACL7IWZMJNRBC7W2MYV5ITQ523JLANCNFSM4H3D6SEA .

dshadowwolf avatar Jan 15 '20 15:01 dshadowwolf

As hard as it is to say... Meeting requirements and expectations of paying clients is getting in the way of me working on my hobbies. Sorry.

dshadowwolf avatar Jan 15 '20 15:01 dshadowwolf

I feel you XD, just wanted to make sure everything was OK and not abandoned lol RL always come first :)

drakray avatar Jan 15 '20 16:01 drakray

I just saw, right now (Basemetals-1.12-2.5.0-Rc2.332) if 3 pieces of armor are damaged, but 1 at full durability, the full armor effect is active

drakray avatar May 10 '20 00:05 drakray

Hrm... I don't recall pushing the fix for this, perhaps Jas did it...

dshadowwolf avatar May 10 '20 16:05 dshadowwolf