Required weapon categories for buffs & techniques

[dseguin]

Summary

None

Purpose of change

Fixes #52783. Very simple addition.
For martial arts, add the ability to specify one or more weapon categories required to perform a technique or benefit from a buff.

Describe the solution

Weapon category requirements can be defined using "weapon_categories_allowed". This field can accept a single string or an array of strings:

  "weapon_categories_allowed": "POLEARMS",

  "weapon_categories_allowed": [ "POLEARMS", "BATONS" ]

Example of defining weapon category requirements for a buff:

"onmiss_buffs": [
  {
    "id": "buff_fencing_onmiss",
    "name": "Remise",
    "description": "Your feint is the perfect setup for a devastating followup attack!\n\n+1 Accuracy.\nEnables \"Compound Attack\" technique.\nLasts 1 turn.",
    "skill_requirements": [ { "name": "melee", "level": 3 } ],
    "melee_allowed": true,
    "buff_duration": 1,
    "weapon_categories_allowed": [ "POLEARMS", "BATONS" ],
    "flat_bonuses": [ { "stat": "hit", "scale": 1.0 } ]
  }
]

As displayed on the info screen:

In this case, the player would only benefit from the buff when wielding either a polearm or baton. (This is not accurate to this specific martial art, it's just used as an example)

Describe alternatives you've considered

I think originally this was meant to address unarmed type weapons being uncategorized. For this, we would probably need to treat weapons with "UNARMED_WEAPON" as a special case.

Testing

Load game with defined weapon category restrictions for techniques and buffs.

Also added a unit test to check that ma_technique::is_valid_character() and ma_buff::is_valid_character() properly evaluate the weapon categories.

Additional context

Updated to use the new translated weapon category ids from #52639.

[Termineitor244]

Wow, that was fast! Thank you so much!! I'm basically jumping of happiness! Hahaha.