WittleWolfie / WW-Blueprint-Core

A library to simplify modifying Pathfinder: Wrath of the Righteous.
MIT License
14 stars 5 forks source link

ActionsBuilder OnPets() will always apply to all pets no matter what the type is. #87

Closed Balkoth-dev closed 1 year ago

Balkoth-dev commented 1 year ago

Example

Kingmaker.ElementsSystem.ActionList applyLifeLink = ActionsBuilder.New().OnPets(ActionsBuilder.New()
.ApplyBuffPermanent(GetGUID.SummonerLifeLinkBuff, isNotDispelable: true)
.Build(),
PetTypeExtensions.Eidolon).Build();

Expected behavior is that it will only apply to pets of a certain type. This will affect all pets no matter the type because AllPets is defaulted to true. Fix I had to do was just to set the AllPets to false afterwards and it worked.

((ContextActionsOnPet)applyLifeLink.Actions[0]).AllPets = false;
WittleWolfie commented 1 year ago

Yeah I fucked the logic on this, need to swap the setting for AllPets... right now it accidentally sets to true if you do specify so it's broken both ways.

WittleWolfie commented 1 year ago

Fix in next release