Vagabond Armor Perk

Started by LoveLess, May 27, 2022, 10:44:37 PM

Previous topic - Next topic

LoveLess

So the current iteration of the Vagabond perk probably uses something like GetItemACValue(); from what I have seen, but this calculates the existing bonus armor, or lack of, on an item. It would be better to use either the Appearance 2da or gold value of them to determine what AC it should be giving, along with the dexterity check, instead. As any time that someone uses Magic Vestment, Mage Armor, anything like that, it bugs the perk completely for the remainder of the reset for some reason. Something like the below...

    object oArmor = GetItemInSlot(INVENTORY_SLOT_CARMOUR, oTarget);

    int nAppr= GetItemAppearance(oArmor, ITEM_APPR_TYPE_ARMOR_MODEL, ITEM_APPR_ARMOR_MODEL_TORSO);

    int nAC = StringToInt(Get2DAString("parts_chest", "ACBONUS", nAppr));


Would fix it. As AC Bonus for armor doesn't stack, there's no harm in letting them coexist either. This wouldn't buff them in any way, just a bug fix.

Paha

It seems to me this should make sense. As I don't have time to do further tests, we can give it a go and see if any unexpected issues arise. From short local testing I at least had it working.

That said- CARMOUR would be inventory slot for creature armor and hides, but I get the gist and that's probably just simple mistake there. Point still very valid.

LoveLess

Oh great, thanks! And yeah as you are well aware, that was meant to be INVENTORY_SLOT_CHEST. Either just misclicked it as they are right next to each other... Or just had a brain fart when I saw *ARMOUR.