#482 LordPazuzu: mages, priests and crusaders should not able to equip weapons in the offhand

Closed
opened 1 year ago by neatz09 · 2 comments
neatz09 commented 1 year ago

i also looked into this is it looks like the skill check works correctly as we have 1852383242(dualwield) in the starting_skills with only existing entrys being class 2 and 31

i also looked into this is it looks like the skill check works correctly as we have 1852383242(dualwield) in the starting_skills with only existing entrys being class 2 and 31
devn00b commented 1 year ago
Owner

Took a quick look, and probably should dig deeper before posting this but whatever.

in EQ2Packet* Player::SwapEquippedItems(int8 slot1, int8 slot2, int16 version, int16 equip_type)There is a check for can equip in slot, but not a check for skill/class (I guess it should be Player::CanEquipItem for the check). I'll dig a bit deeper later tonight.

Took a quick look, and probably should dig deeper before posting this but whatever. in EQ2Packet* Player::SwapEquippedItems(int8 slot1, int8 slot2, int16 version, int16 equip_type)There is a check for can equip in slot, but not a check for skill/class (I guess it should be Player::CanEquipItem for the check). I'll dig a bit deeper later tonight.
devn00b commented 1 year ago
Owner

Added a check to canequipitem to check if the class can equip swords (1h, and a DW check should there be any of those items out there).

This issue should be done, leaving this open for a couple of days should I have missed something.

Added a check to canequipitem to check if the class can equip swords (1h, and a DW check should there be any of those items out there). This issue should be done, leaving this open for a couple of days should I have missed something.
Sign in to join this conversation.
Loading...
Cancel
Save
There is no content yet.