arnonram / swade-npc-importer

Savage Worlds Stat-block importer for FoundryVTT
7 stars 8 forks source link

`equipped` changed to `equipStatus` in SWADE v 1.2.x #124

Closed SalieriC closed 1 year ago

SalieriC commented 1 year ago

SWADE v. 1.2.x changed the equipped property on items of type gear, shield, weapon and armor to equipStatus. Four lines in itemBuilder.js need to be changed to respect these changes.

equipStatus uses numbers to identify the status, these are as follows: Weapons: Stored = 0; Carried = 1; Off-Hand = 2; Main Hand = 4; Two Hands = 5 All other: Stored = 0; Carried = 1; Equipped = 3

I suggest changing

* This part is tricky as not all stat blocks - especially the ones from PEG - specifically state if a weapon is two-handed. This makes it a guessing game. But since weapons which are carried in both hands usually incur a parry penalty it is fair to assume that weapons which state that are used with two hands. It's not so easy with things like Staves or Spears but should work in many cases.

arnonram commented 1 year ago

for weapons: two hands can also be in the description but not mandatory, as for example: bastard sword, falchion, etc..

SalieriC commented 1 year ago

Maybe default to 2-H in these cases unless another weapon or shield is on the stat-block?

arnonram commented 1 year ago

if we start going down that road, it'll be overly complicated i think...

so i think the options are:

keeps it clean and simple

arnonram commented 1 year ago

Instead of trying to come up with every possibility of how it can be written, i went with the user config route

image